Ревью и рефакторинг проекта "Виселица" на Kotlin от студента @belgrano82

ТЗ проекта - zhukovsd.githu...
Автор реализации - github.com/bel...
Код проекта - github.com/bel...
Мой практический курс по Java - zhukovsd.githu...
Мой телеграм канал - t.me/zhukovsd_...
Поддержать - boosty.to/zhuk...

[@older_jackd6002]

на счет ввода пустой строки /enteredLatter и exception, там у человека все правильно и без exception работало, так что зря вы это записали как его косяк, но тут нужно было другой вывод сделать, почему при рефакторинге произошло такое, а все потому что очень много конструкций if () {} else {}, и это плохо читается, да вы сказали что мол большая вложеность это проблема, но тут больше про то что куда удобней читается код когда не используется else там где его можно избежать и это куда важней как мне кажется, если пишется код без лишних else то и вложеность не так страшна и при этом более прозрачно понимаешь как от нее можно избавиться , Если класс Game, почему gemeLoop() ? почему не назвать Start/NewGame? почему он возвращает какое-то значение если результат мы там же выводим? тогда и результат игры наверное выводить в gameLauncher слово можно генерить при вызове gemeLoop(), но по идее его наверное правильней генерить при старте игры, а не при инициализации класса , тогда в gameLauncher можно было бы избавиться от постоянной инициализации класса в цикле, а сделать это один раз а потом постоянно вызывать gemeLoop(), и счетчики wins/losse можно будет перенести в класс, и соответственно можно добавить функцию которая отдает результаты игры. второй момент который с этим связан вы такие говорите мол у тебя для того чтобы узнать значение слова тебе снова нужно читать Dictionary(), читать файл и все такое, да я согласен с тем что словарь по хорошему должен быть в game, вы так подсвитили проблему мол плохо много раз читать из файла, тут согласен, но по итогу от проблемы не избавились, просто из одного место в другое перенесли, как мне кажется куда удачней было создать один экземплар Dictionary(), и пусть его принимает класс при инициализации, то есть также как сделано было у человека только вместо слова экзепляр класса, соответственно в дальнейшем, можно будет подсовывать разные словари в класс не меняя сам класс и разруливать это на уровне gameLauncher ну и да было бы удобней смотреть если бы вы скрыли консоль при редактирование когда она не нужна

[@zhukovsd_it_mentor]

Спасибо за комментарии, со многим согласен

[@fordfalconxbgtcoupe1973]

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