chigan0
@chigan0
back end developer

Оцените тестовое задание?

Всем доброго дня, на днях делал тестовое задания, но его по какой то причини не приняли без указании на ошибки или проблемы. Если у кого есть лишние время можете глянуть и конструктивно указать на мои ошибки

ссылка на проект
  • Вопрос задан
  • 230 просмотров
Пригласить эксперта
Ответы на вопрос 2
SoreMix
@SoreMix Куратор тега Python
yellow
Самого задания не будет? Оценивать то нечего толком. В описании куча грамматических ошибок. На этом можно закончить, это уже говорит о плохом подходе. Вот это вообще без комментариев.
61eff88f8b71b313754399.png

Много коммитов с изменением ридми, можно ведь один раз написать грамотно и не редачить по 10 (буквально) раз. Честно говоря, уже на этом бы моменте даже не стал дальше смотреть. Сама "документация" написана тожн абсолютно никак.

SQL запросы составлены неверно. Нейминги переменных разные. Некоторые переменные вообще непонятно зачем существуют, как, например, error в методе авторизации.
https://github.com/chigan0/apptrix.test/blob/main/...

`utils` откуда-то слизаны, там везде почему-то табы, когда в остальном коде пробелы.
Красиво
61effb278a845424859968.png

Английские слова точно так же написаны с ошибками.

Дока не соответствует коду.
?lastname фильтрация по фамилии

В коде:
"last_name" in request.args

JS разработкой не занимаюсь, но первый раз вижу, чтобы форму формировали не через тег form и/или с использованием ajax, а просто пложив инпуты на страницу, без атрибута name и потом собрав их руками в formdata, и через fetch бросить на сервер.

Лишние комментарии в коде, остатки от каких-то тестов. Отступы скачут. И так далее И так далее И так далее. И всё это до теста главного функционала приложения
Ответ написан
Комментировать
kshnkvn
@kshnkvn
yay ✌️ t.me/kshnkvn
Потому что все очень плохо, буквально в каждой строчке кода. Полное несоблюдение PEP8 везде, где только можно.
main.py и settings.py принято держать в корне проекта.
Хранить какие либо ключи и/или пароли прямо в .py файле и тем более заливать это в github - отвратительная практика.

Структура проекта отсутствует в принципе:
- Почему у тебя в папке models конфигурация БД?
- Почему у тебя параметр подключения к БД прямо в файле, где происходит подключение, а не в конфиге?
- Зачем вообще файл create_database.py?

Пробежавшись по остальным файлам разбирать их чисто для комментария нет никакого желания - там полный мрак.
Ответ написан
Ваш ответ на вопрос

Войдите, чтобы написать ответ

Войти через центр авторизации
Похожие вопросы