Есть спецы в nginx?

ne1ega1 2022-06-27 01:31:04

LUBASKIN_CODE 2022-06-26 23:34:52

в реадме

может и дичь какую-то сделал, но краткое описание дал)

LUBASKIN_CODE 2022-06-27 01:32:38

ne1ega1 2022-06-27 01:31:04

может и дичь какую-то сделал, но краткое описание дал)

вау, так в разы лучше, молодец, завтра клонирую твой репо и может пул реквесты будут, помогу рефакторить

ne1ega1 2022-06-27 01:32:42

Ph0eN1 2022-06-26 20:40:00

Проект довольно маленький, много не скажешь, но всё же.
1. Используешь timezone, используй её везде, не миксуй с datetime (модель Question, default)
2. Где-то всё же забыл verbose_name прописать (Choice.question)
3. Нейминг, тут много всего. Модели, переменные, функции:
Модель Choice — что это за выбор, без понятия, нет конкретики
«Ответы», а не «Ответ» (мб, конечно у тебя там есть multiple choice), но в любом случае, это были бы answer(s), а не choice_text
«Выбранный ответ» в UserVote, ты называешь choice, что по сути выбор
Функция в сервисах «get_list_of_question», она могла бы называться get_first_5_questions. Ибо название твоей функции вообще никак не говорит о том, что она по сути делает.
4. Переносы в тех же моделях не в стиле Django (можешь открыть стандартный AbstractUser в django, посмотреть как он написан)

Скорее всего это не всё, но это в общем. Также можешь добавить комментарии, что для моделей, что для функций, что для вьюх, добавить type hinting, добавить какие-то кастомные обработчики ошибок, добавить логгер…

но я еще не пушил, щас с аннотациями подумаю

Ph0eN1 2022-06-27 01:33:18

ne1ega1 2022-06-27 01:30:22

1. Поправил
2. Дописал
3. С неймингами может ты и прав, но переделывать уже не буду, а название функции поменял
4. в PyCharm просто жмякал автокорректировку, но сделал ваще без переносов
комментарии это сильно важно?

Когда проекты будут больше, поймёшь, что да. Порой возвращаясь к коду, который писал пол года назад, довольно тяжко разбираться что к чему

ne1ega1 2022-06-27 01:33:24

LUBASKIN_CODE 2022-06-27 01:32:38

вау, так в разы лучше, молодец, завтра клонирую твой репо и может пул реквесты будут, помогу рефакторить

клево) спасибо! пока щас по мелочам доработаю еще

Ph0eN1 2022-06-27 01:34:56

А эти маленькие подсказки по типу правильных названий, комментов сделают жизнь проще. Ну и есть же шанс, что ты будешь работать в команде, читать комментарии легче, чем чужой код

ne1ega1 2022-06-27 01:35:41

Ph0eN1 2022-06-27 01:33:18

Когда проекты будут больше, поймёшь, что да. Порой возвращаясь к коду, который писал пол года назад, довольно тяжко разбираться что к чему

Да я это понимаю, сейчас просто преследую цель устроиться на работу, а это как мое портфолио, которое буду пополнять, вот и ломаюсь — толи оставить чистый красивый код, толи написать везде комментарии, а в команде и на работе безусловно всегда надо писать комменты

Ph0eN1 2022-06-27 01:36:36

ne1ega1 2022-06-27 01:35:41

Да я это понимаю, сейчас просто преследую цель устроиться на работу, а это как мое портфолио, которое буду пополнять, вот и ломаюсь — толи оставить чистый красивый код, толи написать везде комментарии, а в команде и на работе безусловно всегда надо писать комменты

По твоему мнению код с комментариями не красивый?

ne1ega1 2022-06-27 01:37:18

Ph0eN1 2022-06-27 01:36:36

По твоему мнению код с комментариями не красивый?

Я минималист, поэтому чисто субъективно да, объективно не знаю, наверное просто так и должно быть

Ph0eN1 2022-06-27 01:37:50

ne1ega1 2022-06-27 01:37:18

Я минималист, поэтому чисто субъективно да, объективно не знаю, наверное просто так и должно быть

Со временем придет) удачи

ne1ega1 2022-06-27 01:38:32

Ph0eN1 2022-06-27 01:37:50

Со временем придет) удачи

Многое приходит со временем) спасибо, и тебе удачи, а так же большое спасибо за уделенное время, для меня это очень важно

whoamins 2022-06-27 12:41:03

У кого-нибудь было такое? Просто не отображаются данные в стандартной админке

The_unname 2022-06-27 12:42:56

whoamins 2022-06-27 12:41:03

У кого-нибудь было такое? Просто не отображаются данные в стандартной админке

Попробуй колект статик сделать

whoamins 2022-06-27 12:43:59

Попробую, но думаю дело не в этом, для других моделей таблицы нормально отображаются

Дима Коваль 2022-06-27 12:45:36

Метод модели str — есть? Что то возвращает? Да и колонки в админке можно настроить.

whoamins 2022-06-27 13:11:38

Дима Коваль 2022-06-27 12:45:36

Метод модели str — есть? Что то возвращает? Да и колонки в админке можно настроить.

Да, все есть.

Дима Коваль 2022-06-27 13:13:21

whoamins 2022-06-27 13:11:38

Да, все есть.

Обычно такое бывает когда str возвращает например self.name — и в то же время поля name в твоих объектах не заполнены