Форум — Games Независимая оценка кода тестового задания? gamedev, игрострой 0 1 Задачка https://ibb.co/HPJPhNp И код https://github.com/Beginerok/DominiGames Ссылка
class Game { public: SDL_Window*window; <...> } Все плохо.trex6 ★★★★★ (23.05.20 00:23:04) Показать ответ Ссылка
Ответ на: комментарий от trex6 23.05.20 00:23:04Зато имитация неплохая bad_master (23.05.20 01:15:46) Ссылка
Если бегло пройтись: new/delete, сишные массивы без особых причин функции вроде Draw_Table и SetCoordinats - это пипец довольно много уровней вложенности, весь код зачем-то в одном классе Game А так не знаю даже, работает или нет, ты на лоре запостил проект для винды Gary ★★★★★ (23.05.20 12:04:59) Ссылка
Game.cpp#L607 //Координаты квадратов для мыши Напомнило это https://i.imgur.com/Ici1lvN.png dsxl ★ (18.06.20 18:23:49) Ссылка
Много копипасты, например Game::Move_*() отличаются парой параметров. Надо вынести в отдельный метод общий код.В commit message лучше всё-таки писать осмысленный код.Имена переменных с большой буквы - плохо.Draw_Table() - дичь, можно переписать в несколько строчек. У тебя же по сути известен шаг клетки.Всё в одном классе - https://ru.wikipedia.org/wiki/Божественный_объектP.S. Если это тестовое задание при устройстве на работу, то это уровень стажера, не выше.hippi90 ★★★★ (18.06.20 18:37:00)Последнее исправление: hippi90 18.06.20 18:38:02 (всего исправлений: 1) Ссылка
А в итоге 12 человек тоже сделало ТЗ и кого-то взяли одного bad_master (18.06.20 23:53:34) Показать ответ Ссылка
Ответ на: комментарий от bad_master 18.06.20 23:53:34Вероятно он не делал силами ЛОР-а и не написал там дичь. Сорри, но за такое даже из шараги выгоняли.peregrine ★★★★★ (19.06.20 01:26:06) Ссылка