tolmachevpk / textgenerator Goto Github PK

https://github.com/tolmachevpk/Review-1/blob/1e71a41b82b3b01e857efc3a587adace129702a5/textgenerator/train.py#L11-L29

Не надо использовать Counter как словарь) он на самом деле гораздо более умный и мощный

https://github.com/tolmachevpk/Review-1/blob/b3cc1fa53bae8659bffcc3d89f79c8a1ed05a560/textgenerator/generate.py#L86-L89

Когда у тебя есть условие по которому надо выйти из цикла, проверяй его. Код станет понятнее и короче. То есть тут лаконичнее писать:
if len(help_val) == 0: break

https://github.com/tolmachevpk/Review-1/blob/1e71a41b82b3b01e857efc3a587adace129702a5/textgenerator/train.py#L68

В питоне принято возвращать новые значения, а не пытаться изменить их внутри функции, потому что все переменные передаются по значению => это неинтуитивное поведение, когда содержимое переменной изменилось, хотя она передалась по значению

https://github.com/tolmachevpk/Review-1/blob/b3cc1fa53bae8659bffcc3d89f79c8a1ed05a560/textgenerator/generate.py#L60-L62

Ты обучаешь модель один раз, а используешь потом бесконечно. Поэтому приведи ее сразу в нормальный вид

https://github.com/tolmachevpk/Review-1/blob/1e71a41b82b3b01e857efc3a587adace129702a5/textgenerator/generate.py#L53-L64

Не нужно использовать исключение в этой части кода. Исключения - это дорогой инструмент

https://github.com/tolmachevpk/Review-1/blob/1e71a41b82b3b01e857efc3a587adace129702a5/textgenerator/generate.py#L49-L52

pass вообще не надо использовать в законченном коде

https://github.com/tolmachevpk/Review-1/blob/1e71a41b82b3b01e857efc3a587adace129702a5/textgenerator/train.py#L61

Это что за артефакт?

https://github.com/tolmachevpk/Review-1/blob/1e71a41b82b3b01e857efc3a587adace129702a5/textgenerator/generate.py#L34

Во-первых такие ситуации обрабатываются исключениями)
Во-вторых, текст странный какой-то

https://github.com/tolmachevpk/Review-1/blob/1e71a41b82b3b01e857efc3a587adace129702a5/textgenerator/train.py#L62-L65

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

https://github.com/tolmachevpk/Review-1/blob/b3cc1fa53bae8659bffcc3d89f79c8a1ed05a560/textgenerator/train.py#L83

Переменные в питоне называются не таким способом. Все должно быть маленькими буквами и через нижние подчеркивания (в общем чате кидала ссылку на хорошую статью об этом)

https://github.com/tolmachevpk/Review-1/blob/1e71a41b82b3b01e857efc3a587adace129702a5/textgenerator/train.py#L75

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

https://github.com/tolmachevpk/Review-1/blob/1e71a41b82b3b01e857efc3a587adace129702a5/textgenerator/train.py#L60-L73

При работе с файлами всегда используй with, потому что тогда даже при исключительной ситуации, все ресурсы освободятся

https://github.com/tolmachevpk/Review-1/blob/1e71a41b82b3b01e857efc3a587adace129702a5/textgenerator/train.py#L52

Эту строчку точно нужно перенести в main. Он же пишется, чтобы ты мог импортировать свой скрипт из другого скрипта. А момент парсинга аргументов тебе этого сделать уже не даст, потому что там есть обязательные аргументы

Разбей main в обоих скриптах на более маленькие логические части. Подумай, какие функции хотелось бы переиспользовать в будущем отдельно от твоего проекта

1. Нужно добавить общее описание скрипта в самое начало файла: что он делает, кто автор, какая версия и все такое
2. К каждой функции нужно добавить комментарии примерно в таком стиле: https://stackoverflow.com/questions/13850049/proper-way-to-comment-code-python

https://github.com/tolmachevpk/Review-1/blob/b3cc1fa53bae8659bffcc3d89f79c8a1ed05a560/textgenerator/train.py#L75-L82

Во-первых, НИКОГДА не используй бесконечные циклы, если можно обойтись без них. И вообще лучше использовать for, если можно его использовать, а тут очень даже можно

Во-вторых, НИКОГДА не используй try/catch, если без него можно обойтись. Это гораздо более дорогая операция, чем if

Исправляй этот кусок кода

https://github.com/tolmachevpk/Review-1/blob/1e71a41b82b3b01e857efc3a587adace129702a5/textgenerator/train.py#L85