Критика кода возможна?

Question

[MuffinLover]

Ссылка удалена модератором.

Буду признателен за любую критику

Comments

[Dmitry Roo]

Это же задание, а не вопрос.
Код ревью - часть работы.
Любой труд должен быть оплачен.

[Алан Гибизов]

Есть специальный тэг code review. Поставьте его и, возможно, найдутся добрые самаритяне. А возможно и не найдутся.

[Модератор]

Важный момент! Запрещено давать ссылки на малоизвестные посторонние ресурсы, включая репозитории. П.5.7 Регламента.
Ссылка удалена модератором.

Фрагменты кода надо размещать в виде текста и оборачивать тэгом code для корректного отображения. Удобно делать кнопкой </>
Это обязательно, см.п.3.8 Регламента.

Сюда же относится traceback, ввод и вывод в консоли и другая структурированная текстовая инфа.

[Dmitry Roo]

Модератор, почему это не 5.12? "Пользу от достижения этой цели получает, как правило, лишь её автор."

[Сергей Горностаев]

Dmitry Roo, не видел что там за код, но возможно, что в его ревью могут быть даны общие рекомендации, которые пригодятся другим начинающим программистам в подобных случаях.

Answer 1

[mayton2019]

Тут вместо цепочки if можно switch-case использовать.

@Override
    public void notification(Event event) {
        if (event.type() == EventType.REDRAW_REQUEST) {
            printField(event.field());
            return;
        }

        if (event.type() == EventType.BOMB_OPENED) {
            System.out.println("You opened bomb. Game failed");
            printField(event.field());
        }

И в других местах - похожие косяки.