Да я первый раз все делаю, где это надо сделать?

|
ne1ega1 2022-06-26 18:25:52
мелочь, а выглядит приятней)
ne1ega1 2022-06-26 19:11:05
https://github.com/ne1ega1/sitepolls.git

GitHub — ne1ega1/sitepollsGitHub
Contribute to ne1ega1/sitepolls development by creating an account on GitHub.
ne1ega1 2022-06-26 19:12:02
зацените пожалуйста ребят, буду признателен любой критике! это мой первый проект, в принципе финальная версия
Lutiy_Pes 2022-06-26 19:17:43
Господа, кто-нибудь знает , где можно проверить документацию на ошибки?
2022-06-26 19:35:02
ne1ega1 2022-06-26 19:12:02
зацените пожалуйста ребят, буду признателен любой критике! это мой первый проект, в принципе финальная версия

базу данных и миграции не нужно заливать в гит

ne1ega1 2022-06-26 19:39:13
2022-06-26 19:35:02
базу данных и миграции не нужно заливать в гит

уже поправил, спасибо! гитигнор тупит

Ph0eN1 2022-06-26 20:40:00
ne1ega1 2022-06-26 19:12:02
зацените пожалуйста ребят, буду признателен любой критике! это мой первый проект, в принципе финальная версия

Проект довольно маленький, много не скажешь, но всё же.
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, добавить какие-то кастомные обработчики ошибок, добавить логгер…

ne1ega1 2022-06-26 20:44:51
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, добавить какие-то кастомные обработчики ошибок, добавить логгер…

Спасибо! сегодня разберу все по пунктам

ranwise 2022-06-26 23:15:03
ne1ega1 2022-06-26 19:11:05
https://github.com/ne1ega1/sitepolls.git

а где описание о чём проект?

ne1ega1 2022-06-26 23:32:08
ranwise 2022-06-26 23:15:03
а где описание о чём проект?

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

LUBASKIN_CODE 2022-06-26 23:33:55
ne1ega1 2022-06-26 19:12:02
зацените пожалуйста ребят, буду признателен любой критике! это мой первый проект, в принципе финальная версия

добавь описание репо

LUBASKIN_CODE 2022-06-26 23:34:06
а то я вообще не понял что за проект
LUBASKIN_CODE 2022-06-26 23:34:17
и жди пуллреквесты
ne1ega1 2022-06-26 23:34:40
LUBASKIN_CODE 2022-06-26 23:33:55
добавь описание репо

куда добавить?

LUBASKIN_CODE 2022-06-26 23:34:52
в реадме
alexintp 2022-06-26 23:35:37
Ребят, привет! Кто может подсказать, что я делаю не так.
У меня есть вот такая структура:
|support
│ tree.txt
│ __init__.py
│ old_queries.py

├───utils
│ │ utilsFuncs.py
│ │ __init__.py

sys.path в utilsFuncs.py вот такой:
D:\Users\user1\Documents\WRK\gitlab\support\utils
C:\Program Files\Python39\python39.zip
C:\Program Files\Python39\DLLs
C:\Program Files\Python39\lib
C:\Program Files\Python39
D:\Users\user1\Documents\WRK\gitlab\support\venv
D:\Users\user1\Documents\WRK\gitlab\support\venv\lib\site-packages
D:\Users\user1\Documents\WRK\gitlab\support

В начале файла utilsFuncs.py прописано:
from pathlib import Path
import sys
path= Path(__file__)
sys.path.append(path.parent.parent)
from old_queries import *

При запуске utilsFuncs.py пишет:
(venv) PS D:\Users\user1\Documents\WRK\gitlab\support> python .\utils\utilsFuncs.py
Traceback (most recent call last):
File «D:\Users\user1\Documents\WRK\gitlab\support\utils\utilsFuncs.py», line 5, in <module>
from old_queries import *
ModuleNotFoundError: No module named ‘old_queries’

В чем дело?

LUBASKIN_CODE 2022-06-26 23:36:26
alexintp 2022-06-26 23:35:37
Ребят, привет! Кто может подсказать, что я делаю не так.
У меня есть вот такая структура:
|support
│ tree.txt
│ __init__.py
│ old_queries.py

├───utils
│ │ utilsFuncs.py
│ │ __init__.py

sys.path в utilsFuncs.py вот такой:
D:\Users\user1\Documents\WRK\gitlab\support\utils
C:\Program Files\Python39\python39.zip
C:\Program Files\Python39\DLLs
C:\Program Files\Python39\lib
C:\Program Files\Python39
D:\Users\user1\Documents\WRK\gitlab\support\venv
D:\Users\user1\Documents\WRK\gitlab\support\venv\lib\site-packages
D:\Users\user1\Documents\WRK\gitlab\support

В начале файла utilsFuncs.py прописано:
from pathlib import Path
import sys
path= Path(__file__)
sys.path.append(path.parent.parent)
from old_queries import *

При запуске utilsFuncs.py пишет:
(venv) PS D:\Users\user1\Documents\WRK\gitlab\support> python .\utils\utilsFuncs.py
Traceback (most recent call last):
File «D:\Users\user1\Documents\WRK\gitlab\support\utils\utilsFuncs.py», line 5, in <module>
from old_queries import *
ModuleNotFoundError: No module named ‘old_queries’

В чем дело?

прочитай ошибку

MSD_Junior 2022-06-26 23:36:44
Доброго времени суток. Я навичок в джанго пилю свой блог, сталкнулся с такой проблеммой по созданию добавлению товара в корзину и оформления. Есть у кого нибудь доки на эту тему или на водка?
ne1ega1 2022-06-26 23:39:12
LUBASKIN_CODE 2022-06-26 23:34:52
в реадме

прям подробное описание или как? извини, это мой первый проект, только учусь. по сути проект просто голосовалка

LUBASKIN_CODE 2022-06-26 23:39:29
MSD_Junior 2022-06-26 23:36:44
Доброго времени суток. Я навичок в джанго пилю свой блог, сталкнулся с такой проблеммой по созданию добавлению товара в корзину и оформления. Есть у кого нибудь доки на эту тему или на водка?

водка есть приезжай раскумаримся

https://github.com/bmentges/django-cart

GitHub — bmentges/django-cart: Django simple shopping cart solution, with tests 🙂GitHub
Django simple shopping cart solution, with tests 🙂 — GitHub — bmentges/django-cart: Django simple shopping cart solution, with tests 🙂
alexintp 2022-06-26 23:39:31
LUBASKIN_CODE 2022-06-26 23:36:26
прочитай ошибку

не может найти модуль))

LUBASKIN_CODE 2022-06-26 23:39:51
ne1ega1 2022-06-26 23:39:12
прям подробное описание или как? извини, это мой первый проект, только учусь. по сути проект просто голосовалка

поищи видосики «оформление репозитория github»

MSD_Junior 2022-06-26 23:40:44
LUBASKIN_CODE 2022-06-26 23:39:29
водка есть приезжай раскумаримся

https://github.com/bmentges/django-cart

Cпасибо)

ne1ega1 2022-06-26 23:42:07
LUBASKIN_CODE 2022-06-26 23:39:51
поищи видосики «оформление репозитория github»

окей, спасибо, щас гляну

ne1ega1 2022-06-27 01:30:22
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, добавить какие-то кастомные обработчики ошибок, добавить логгер…

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

|