yhx07 / python Goto Github PK

https://github.com/DmitriiP1/python/blob/56846ed6b88f1d20bed0d518e1844ba2e921fd39/train.py#L56-L60

Повторение двух строчек кода, содержащих определенную и одинаковую логику - это плохо. Нужно избавиться от него

Нужно добавить main, чтобы можно было использовать функции из твоего скрипта

https://github.com/DmitriiP1/python/blob/56846ed6b88f1d20bed0d518e1844ba2e921fd39/generate.py#L75-L80

Такого варианта не было в условии задачи (и это не даст тебе настоящий рандом, основанный на частотах). Используй один из вариантов из условия задачи

https://github.com/DmitriiP1/python/blob/56846ed6b88f1d20bed0d518e1844ba2e921fd39/train.py#L32-L34

Комментарий с описанием функции лучше писать после объявления в стиле:

def foo():
    """My best function"""
    pass

Потому что потом такие комментарии всплывают как подсказки в текстовых редакторах

https://github.com/DmitriiP1/python/blob/56846ed6b88f1d20bed0d518e1844ba2e921fd39/generate.py#L38-L40

В словаре можно хранить tuple, не нужно все время разделять строку и объединять обратно. И если ты так делаешь, то как минимум используй format

https://github.com/DmitriiP1/python/blob/56846ed6b88f1d20bed0d518e1844ba2e921fd39/train.py#L43

Зачем тебе тут новая переменная, да еще и с таким названием?

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

https://github.com/DmitriiP1/python/blob/56846ed6b88f1d20bed0d518e1844ba2e921fd39/generate.py#L63

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

https://github.com/DmitriiP1/python/blob/56846ed6b88f1d20bed0d518e1844ba2e921fd39/train.py#L56-L57

Почитай про модуль и в идеале надо избавиться от цикла

https://github.com/DmitriiP1/python/blob/56846ed6b88f1d20bed0d518e1844ba2e921fd39/train.py#L70-L72

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

https://github.com/DmitriiP1/python/blob/56846ed6b88f1d20bed0d518e1844ba2e921fd39/generate.py#L143-L144

Это делается без циклов!

https://github.com/DmitriiP1/python/blob/d5c94b2b961e2b52b807fb86f66bd6e7ca73e823/train.py#L118
Это что за артефакт?

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

https://github.com/DmitriiP1/python/blob/56846ed6b88f1d20bed0d518e1844ba2e921fd39/train.py#L45

На самом деле данная переменная не является какой-то действительно глобальной. По смыслу она просто аргумент твоей функции

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

https://github.com/DmitriiP1/python/blob/56846ed6b88f1d20bed0d518e1844ba2e921fd39/train.py#L26

Нужно добавить try/except вокруг всего остального кода, либо (что лучше) во время записи модели в файл использовать with

https://github.com/DmitriiP1/python/blob/56846ed6b88f1d20bed0d518e1844ba2e921fd39/train.py#L1

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

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

https://github.com/DmitriiP1/python/blob/56846ed6b88f1d20bed0d518e1844ba2e921fd39/train.py#L62-L63

Циклы это медленно + ты сам реализуешь логику записи + она плохо реализована (как минимум надо было использовать format). Для сохранения модели есть стандартные методы (pickle, json, csv), используй их