luckydev777 / fast_api_bank_app Goto Github PK

1. Нет reame.md (например там можно написать как всю эту штуку запускать)
2. Импорты не по пепке
   https://github.com/MansurMekin/fast_api_bank_app/blob/ba40c37ccffa1ffc0f3051de4ca403dd06113904/main.py#L1-L4
3. Запятую в конце надо
   https://github.com/MansurMekin/fast_api_bank_app/blob/ba40c37ccffa1ffc0f3051de4ca403dd06113904/main.py#L7
4. Не по алфавиту импорты
   https://github.com/MansurMekin/fast_api_bank_app/blob/ba40c37ccffa1ffc0f3051de4ca403dd06113904/models.py#L2
5. Хранить емейл в текст филде - не оч хорошая идея. Юзай специальное поле для емейла (с телефоном наверное чет такое же)
   https://sqlalchemy-utils.readthedocs.io/en/latest/data_types.html#module-sqlalchemy_utils.types.email
6. account_number - строка, а почему аккаунт число - это строка?
   https://github.com/MansurMekin/fast_api_bank_app/blob/ba40c37ccffa1ffc0f3051de4ca403dd06113904/models.py#L22
7. Лучше назвать поле is_blocked
   https://github.com/MansurMekin/fast_api_bank_app/blob/ba40c37ccffa1ffc0f3051de4ca403dd06113904/models.py#L24
8. Это надо из енва тянуть
   https://github.com/MansurMekin/fast_api_bank_app/blob/ba40c37ccffa1ffc0f3051de4ca403dd06113904/database.py#L6
9. Зачем тебе инит в основной директории?
   
10. А ты точно уверен, что это прям необходимо писать?
    
11. Почему у тебя в этих файлах намешано все что только можно? По хорему тут для каждой папка отдельная - и в каждой папке отдельная папка это модели, схемы, роуты
    
12. По какому принципу тут 2 пустые строки?
    https://github.com/MansurMekin/fast_api_bank_app/blob/ba40c37ccffa1ffc0f3051de4ca403dd06113904/routers/accounts.py#L5-L8
13. Почему ты эту функцию создаешь в каждом файле, а не импортишь?
    https://github.com/MansurMekin/fast_api_bank_app/blob/ba40c37ccffa1ffc0f3051de4ca403dd06113904/routers/accounts.py#L20
14. Ну эт еще раз напишу - у тебя не должно быть в файле схем и роутов - они должны быть разделены
15. А какой это банк дает создать аккаунт сразу с милионом баксов например? Мне кажется, что нельзя тут принимать баланс.
    https://github.com/MansurMekin/fast_api_bank_app/blob/ba40c37ccffa1ffc0f3051de4ca403dd06113904/routers/accounts.py#L32-L34
    Или что это
16. Эту штуку ты дублируешь кучу раз - стоит задуматься
    https://github.com/MansurMekin/fast_api_bank_app/blob/ba40c37ccffa1ffc0f3051de4ca403dd06113904/routers/accounts.py#L79-L81
17. Операции с балансом - это бизнес логика, ее не должно быть в вьюшке
18. Это тоже репитется
    https://github.com/MansurMekin/fast_api_bank_app/blob/ba40c37ccffa1ffc0f3051de4ca403dd06113904/routers/accounts.py#L109-L116
19. Это надо в транзакцию, вдруг ты у юзера деньги спишешь, а другому не начислишь
    https://github.com/MansurMekin/fast_api_bank_app/blob/ba40c37ccffa1ffc0f3051de4ca403dd06113904/routers/accounts.py#L107-L122
20. А если тебе это None вернет?
    https://github.com/MansurMekin/fast_api_bank_app/blob/ba40c37ccffa1ffc0f3051de4ca403dd06113904/routers/accounts.py#L119
21. Почему ты называешь аргумент from_account, а не from_account_id
    https://github.com/MansurMekin/fast_api_bank_app/blob/ba40c37ccffa1ffc0f3051de4ca403dd06113904/routers/accounts.py#L106
22. Почему ты в accounts.py ничего не ретурнил, когда фактически операции одинаковы?
    https://github.com/MansurMekin/fast_api_bank_app/blob/ba40c37ccffa1ffc0f3051de4ca403dd06113904/routers/admin.py#L42
23. Роутер анблок аккаунт - функция блок аккаунт
    https://github.com/MansurMekin/fast_api_bank_app/blob/ba40c37ccffa1ffc0f3051de4ca403dd06113904/routers/admin.py#L46-L47
24. У тебя 2 роутера, у который почти одинаковый код (фактически отличаются только True False) - это как минимум выглядить не очень, а как максимум очень странно. Подумай как убрать такой дубляж кода
    
25. Давать челу по дефолту емейл - какая-то плохая идея
    https://github.com/MansurMekin/fast_api_bank_app/blob/ba40c37ccffa1ffc0f3051de4ca403dd06113904/routers/customers.py#L30
26. Номер телефона тоже

Короче главная проблема - это куча дублированного кода