ekolodin / mipt-python-2018 Goto Github PK

Если в параметр передается строчка 'stdin', обработать это как "считать текст из входного потока"

С консольным интерфейсом-то всё в порядке, только реализовывать его стоит не руками, а через библиотеку argparse

Сериализацию и десериализацию модели лучше делать встроенными средствами. Это менее error prone и меньше кода

Это же не очень правильно, одна и та же частота может быть у разных слов. Вообще лучше делать random.choices, передав туда слова и частоту

В defaultdict(int) не нужно проверять наличие элемента.Он хорош как раз тем, что по дефолту считается, что там лежит 0. Так что можно спокойно делать везде += 1 и это не упадет

В питоне принят snake_case для названия функций и переменных.

Ну, и подчеркивания в переменных вне классов смотрятся немного бессмысленно, их можно смело убирать

При генерации лучше не составлять строку сразу, а хранить list слов и потом вернуть ' '.join(generated_words)
Иначе происходит много лишнего копирования, так быстрее