Konstantin Novash

[knovash]

setup

[AlexeiBuzuma]

1. Сообщения коммитов в гите должны нести какой-то смысл изменения, которое было сделано в этом коммите (zz и dd однозначно очень плохие сообщения). Подряд не должны идти коммиты с одним и тем же сообщением, это выглядит странно.
2. В репозитории не должны лежить файлы, которые связаны с твоей средой разработки. Папка .idea никогда не должна попадать в репозиторий.
3. импорт через * -- это зло. Крайне не рекомендуется использовать такие импорты.
4. Очень не рекомендуется оставлять комментарии в коде на русском языке
5. i, 'k', 'j' -- все это магические переменные, которые не несут никакого смысла. Название переменной должно четко описывать ее задачу.
6. функция print не должна вызываться в тестах. Это засоряет вывод в консоль. Когда тестов действительно много, то такие принты только мешают искать полезную информацию в аутпуте.
7. В идеале один тест кейс (или группа тест кейсов очень связанных по смыслу) -- одна функция. В данном случае есть всего две функции и в каждой огромное количество тест кейсов. Если упадет какой-нибудь из них, то функция не продолжет исполняться далее и на текущем прогоне будет неизвестно, отработали ли другие тест кейсы или нет.
8. Зачастую функции очень тяжело читать из-за их размера и количества логики, которая содержиться в этих функциях. Советую в будущем больше внимание уделять на декомпозицию. Много маленьких функций намного лучше, чем несколько больших. Возможно использование ООП помогло бы сделать код более читаемым.
9. Советую много практиковаться и постоянно писать код. Есть много ресурсов, где можно найти большое количество небольших и интересных заданий. Например https://www.codewars.com/

[knovash]

ok)