AleksandrSlakvych
commented
5 years ago Thank you Andrei. I corrected mistakes and, please, look at my project again.
Open podkolzzzin opened 5 years ago
AleksandrSlakvych
commented
5 years ago Thank you Andrei. I corrected mistakes and, please, look at my project again.
podkolzzzin
commented
5 years ago https://github.com/AleksandrSlakvych/Project2GameSnake/blob/master/Project2GameSnake/Project2GameSnake.csproj#L36 - dll лежит за пределами репозитория, из-за этого проект неовозможно собрать. -0.5 - не исправлено
https://github.com/AleksandrSlakvych/Project2GameSnake/blob/master/Project2GameSnake/SnakeF.cs#L133 - куда лучше вынести этот иф за цикл. Он все равно выполнится только один раз. -0.5 - не исправлено
= 89
Работа принята.
13 дней опоздания -6.5 Игровое меню +5
https://github.com/AleksandrSlakvych/Project2GameSnake/blob/master/Project2GameSnake/Project2GameSnake.csproj#L36 - dll лежит за пределами репозитория, из-за этого проект неовозможно собрать. -0.5
https://github.com/AleksandrSlakvych/Project2GameSnake/commit/fcf84c9b85a581ed6ae1d410b865104567d903a6 - давай более внятные названия комитам -0.5
https://github.com/AleksandrSlakvych/Project2GameSnake/blob/master/Project2GameSnake/GameEngine.cs#L12 - публичное поле плохо само по себе, к тому же нигде не используется за пределами класса -1
https://github.com/AleksandrSlakvych/Project2GameSnake/blob/master/Project2GameSnake/GameEngine.cs#L14 - остальные приватные поля именованы с маленькой буквы -0.5
https://github.com/AleksandrSlakvych/Project2GameSnake/blob/master/Project2GameSnake/GameEngine.cs#L47 - публичное состояние в статике плохая штука. Когда проект растет все сложнее отследить кто меняет. -1
https://github.com/AleksandrSlakvych/Project2GameSnake/blob/master/Project2GameSnake/GameSettings.cs#L20 - инстанционный конструктор сбрасывает значение статических полей... К тому же вообще нет ничего инстанционного... Кому может быть нужен такой объект? -1
https://github.com/AleksandrSlakvych/Project2GameSnake/blob/master/Project2GameSnake/Button.cs#L15 - зачем кому-то может быть нужна кнопка без координат и текста?.. -1
https://github.com/AleksandrSlakvych/Project2GameSnake/blob/master/Project2GameSnake/Part.cs#L10 - везде пробелы ставишь по обе стороны от оператора, а здесь нет -0.5
https://github.com/AleksandrSlakvych/Project2GameSnake/blob/master/Project2GameSnake/Part.cs#L22 - уже или пиши this везде или не пиши... -0.5
https://github.com/AleksandrSlakvych/Project2GameSnake/blob/master/Project2GameSnake/SnakeF.cs#L14 - обычно не разделяешь поля переносами строк, а здесь почему решил? -0.5
https://github.com/AleksandrSlakvych/Project2GameSnake/blob/master/Project2GameSnake/SnakeF.cs#L27 - обычно пишешь модификатор доступа, а здесь почему забыл? -0.5
https://github.com/AleksandrSlakvych/Project2GameSnake/blob/master/Project2GameSnake/SnakeF.cs#L32 - а ну вот и это создание не нужного объекта. А GC его потом побирать...
https://github.com/AleksandrSlakvych/Project2GameSnake/blob/master/Project2GameSnake/SnakeF.cs#L56 - можно просто написать !GameSettings.GameOver -0.5
https://github.com/AleksandrSlakvych/Project2GameSnake/blob/master/Project2GameSnake/SnakeF.cs#L94 - в переводе этот метод назвается СоздатьЕсть. Думаю имелось ввиду CreateFood -0.5
https://github.com/AleksandrSlakvych/Project2GameSnake/blob/master/Project2GameSnake/SnakeF.cs#L133 - куда лучше вынести этот иф за цикл. Он все равно выполнится только один раз. -0.5
Сделано на самом деле вполне хорошо и серьезных проблем в плане почти и нет. Разве что крайне необходимо класс GameSettings выглядит крайне неправильным.
Промежуточный итог: = 89.5
Работа пока не принята.