Ghennadi-Berezovschi - Roadmap.md

https://github.com/Ghennadi-Berezovschi/Roadmap
Ревью от Алексея

Хорошая реализация игры в процедурном стиле. Упрощенный функционал.

Недостатки реализации:

1. Нет списка правильных и неправильных букв, которые вводили
2. Можно ввести одну и ту же неправильную букву несколько раз и каждый раз это будет считаться новой ошибкой
3. В игре всего 4 слова, что хорошо для отладки, но не очень, как игра

Хорошо:

1. Нейминг
2. Вся игра в одном классе, но разбита на маленькие методы
3. Простой и понятный алгоритм
4. Достаточное количество констант

Замечания:

1. Нейминг

Убирай товтологию

HangmanGame.playGame()

//ПРАВИЛЬНО:
HangmanGame.play()

2. Создавай вспомогательные методы, делай программу более простой и понятной:

if (userInput.isEmpty() || userInput.length() != 1 || !Character.isLetter(userInput.charAt(0))) {...}

//ПРАВИЛЬНО:
if (isМетодСназваниемКотороеВсеОбьясняет()) {...}

private static isМетодСназваниемКотороеВсеОбьясняет() {
  return userInput.isEmpty() || userInput.length() != 1 || !Character.isLetter(userInput.charAt(0));
}

3. Всегда используй классы через их интерфейсы

ArrayList<String> words = createWords();

//ПРАВИЛЬНО:
List<String> words = createWords();

4. Нарушение инкапсуляции- все методы публичные.

Вывод: в процедурщине ты неплох. Переходи к ООП.