LINUX.ORG.RU

Если бегло пройтись:

  • new/delete, сишные массивы без особых причин
  • функции вроде Draw_Table и SetCoordinats - это пипец
  • довольно много уровней вложенности, весь код зачем-то в одном классе Game

А так не знаю даже, работает или нет, ты на лоре запостил проект для винды

Gary ★★★★★
()

Много копипасты, например Game::Move_*() отличаются парой параметров. Надо вынести в отдельный метод общий код.

В commit message лучше всё-таки писать осмысленный код.

Имена переменных с большой буквы - плохо.

Draw_Table() - дичь, можно переписать в несколько строчек. У тебя же по сути известен шаг клетки.

Всё в одном классе - https://ru.wikipedia.org/wiki/Божественный_объект

P.S. Если это тестовое задание при устройстве на работу, то это уровень стажера, не выше.

hippi90 ★★★★★
()
Последнее исправление: hippi90 (всего исправлений: 1)
Ответ на: комментарий от bad_master

Вероятно он не делал силами ЛОР-а и не написал там дичь. Сорри, но за такое даже из шараги выгоняли.

peregrine ★★★★★
()
Вы не можете добавлять комментарии в эту тему. Тема перемещена в архив.