Итерация ревью

[akhtyamovpavel]

Приветствую!

Посмотрел код - увидел хорошие аспекты. Но есть те вещи, которые можно улучшить:

• [x] https://github.com/leramorozova/EDGrPol/blob/develop/WebApp/GrPolDict.db - лучше не заливать базу, если в этой базе есть какие-то секретные данные
• [x] https://github.com/leramorozova/EDGrPol/tree/develop/WebApp/mysite/__pycache__ - .pyc лучше добавить эти файлы в .gitignore и удалить из репозитория эти файлы
• [x] https://github.com/leramorozova/EDGrPol/blob/develop/WebApp/EDGrPol/views.py#L10 - чтобы не указывать папку, лучше настроить путь к template-ам в настройках Django
• [x] https://github.com/leramorozova/EDGrPol/blob/develop/WebApp/EDGrPol/views.py#L34-L85 - лучше разбить на несколько функций, где в каждой сделать отдельную обработку параметров
• [x] https://github.com/leramorozova/EDGrPol/blob/develop/WebApp/EDGrPol/views.py#L8 - чтобы понимать тип запроса, можно попробовать перейти на class-based views. Тогда в классе можно реализовать методы get и post
• [ ] https://github.com/leramorozova/EDGrPol/blob/develop/WebApp/EDGrPol/templates/EDGrPol/materials.html - мы не желаем сделать render этого из базы? (А то вдруг появятся еще библиотеки)
• [x] https://github.com/leramorozova/EDGrPol/blob/develop/WebApp/EDGrPol/SearchUtilities/lemma_search.py#L16-L27 - почему здесь не используется Django ORM?
• [x] https://github.com/leramorozova/EDGrPol/blob/develop/WebApp/EDGrPol/SearchUtilities/lemma_search.py#L101-L115 - если есть магическая константа 150, то ее лучше назвать, и дальше использовать
• [x] https://github.com/leramorozova/EDGrPol/blob/develop/WebApp/EDGrPol/SearchUtilities/articles.py#L41 - не много ли ответственностей у этого класса?

В остальном - все хорошо! Буду ждать обновленную версию!

[leramorozova]

https://github.com/leramorozova/EDGrPol/blob/develop/WebApp/EDGrPol/templates/EDGrPol/materials.html - мы не желаем сделать render этого из базы? (А то вдруг появятся еще библиотеки)

Надо бы, но это не так удобно сейчас будет, потому что сама по себе база данных обновляется не по-человечески, а путем автоматизированной компилляции из десятка гугл-документов по запросу составителей словаря (так получилось, потому что я так и не успела сделать интерфейс для редактирования словарных статей, а составители словаря, с которыми я работаю - люди крайне малограмотные технически)... В общем, good note, но я возьму это на заметку обязательно, пока что мартышкин труд немного.