LINUX.ORG.RU

class Game
{
public:
	SDL_Window*window;
<...>
}




Все плохо.

trex6 ★★★★★ ()

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

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

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

Gary ★★★★★ ()

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

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

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

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

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

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

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

А в итоге 12 человек тоже сделало ТЗ и кого-то взяли одного

bad_master ()
Ответ на: комментарий от bad_master

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

peregrine ★★★★★ ()
Ограничение на отправку комментариев: только для зарегистрированных пользователей