search

moevm / oop

Object-oriented programming course
30 stars 225 forks source link

[8303]Udod_Maxim_L5 #672

Closed TheShenk closed 4 years ago

TheShenk commented 4 years ago

Необходимо просматривать: UI/Commands/Save UI/Commands/Load UI/Commands/CommandMemento.h UI/Commands/Command.h (Функция getMemento у класса Command)

anyfilatov commented 4 years ago

Я не увидел у вас контроля, что загружаемое состояние валидно. В остальном ок 15 баллов (доделать 5)

TheShenk commented 4 years ago

Я не увидел у вас контроля, что загружаемое состояние валидно.

Это происходит при выполнении команды. Будут выполнены только те, которые соответсвуют требованиям одного из CommandHandler'ов.

anyfilatov commented 4 years ago

Не-не-не, речь о том, что структура файла совпадает. Что при чтении файла все поля gameField будут правильно заполняться, и не окажется, что на место "размер" придёт строковая переменная

TheShenk commented 4 years ago

Изменены: Utils/utils.h (Добавлена проверка на правильность преобразования к int) UI/Commands/Load и UI/Commands/Save (Проверка на корректность файла. Теперь большая часть изменени файла приведет к выводу предупреждения в логи)