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

Question

[Максим Макаров]

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

ссылка на проект

Comments

[Алексей Уколов]

Чтобы оценивать тестовое задание нужно знать тестовое задание.
"Не принять" могли не задание, а ваше резюме, например.
Тестовое задание призвано показать ваш уровень, а вы там сразу пишете "писал я его на скорую руку левой пяткой".

[mkone112]

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

[Влад Григорьев]

все плохо, на кой-то на дробили код где не надо, например с create_database, да и app, main туда же

`except:` - без ошибки и без логирования, так делать не надо

большая лесенка плохо читается, и её надо выровнять

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

импорты в функциях ...

это тоже выглядит плохо читаемо и понятно, точнее понятно, но надо делать иначе

for i,p in zip(res,['public_id','email','first_name','last_name','s','avatar']):
            data[p] = i

эт ваще стыд, читается плохо, и выглядит ещё хуже, где вы такое придумали-то

if ("email" in data
                and "password" in data
                    and "password1" in data
                        and "first_name" in data
                            and "last_name" in data
                                and "s" in data
                                    and "avatar" in avatar):

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

[pcdesign]

res = session.execute(f"""
                                SELECT public_id,first_name,last_name,s,avatar 
                                    FROM user 
                                        WHERE first_name = '{str(request.args.get('first_name')).lower()}' 
                                            OR last_name = '{str(request.args.get('last_name')).lower()}' 
                                                {f"OR s = {gender}" if gender != None else ''}
                                                    LIMIT {limit}""")

Это ведь без проблем можно сделать sql инъекцию.

[Максим Макаров]

Хорошо я всё понял

Answer 1

[soremix]

Самого задания не будет? Оценивать то нечего толком. В описании куча грамматических ошибок. На этом можно закончить, это уже говорит о плохом подходе. Вот это вообще без комментариев.


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

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

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


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

Дока не соответствует коду.

?lastname фильтрация по фамилии

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

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

Лишние комментарии в коде, остатки от каких-то тестов. Отступы скачут. И так далее И так далее И так далее. И всё это до теста главного функционала приложения

Answer 2

[Ivan Yakushenko]

Потому что все очень плохо, буквально в каждой строчке кода. Полное несоблюдение PEP8 везде, где только можно.
main.py и settings.py принято держать в корне проекта.
Хранить какие либо ключи и/или пароли прямо в .py файле и тем более заливать это в github - отвратительная практика.

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

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

Comments

[Максим Макаров]

Ok понял