moevm / oop

Object-oriented programming course
30 stars 225 forks source link

8383_kolmykov_vladislav_lab5 #630

Closed TSVlad closed 4 years ago

Ksenox commented 4 years ago

Отчет распишите подробнее. Не вижу UML диаграммы

TSVlad commented 4 years ago

UML приложены в jpeg файлах, как и написано в отчете.

TSVlad commented 4 years ago

Что конкретно дописать в отчете?

Ksenox commented 4 years ago

Что, как и для чего реализовывали. И в каких файлах находится.

TSVlad commented 4 years ago

Дополнил

Ksenox commented 4 years ago

1) в Unit, и других местах получились слишком большие конструкторы. Лучше сделать конструктор, который принимающий снимок, и восстанавливающий из него параметры. Либо создать структуру UnitTraits, ее заполнять и передавать в конструктор. 2) Как проводится контроль корректности данных?

P.S. Лучше не делать сразу всю UML диаграмму, так как в ней тяжело ориентироваться. Ее можно разбить на несколько более маленьких.

TSVlad commented 4 years ago

Он проверяет, что все поля заполняются (hp, damage и тд...) тем, что перед каждым юнитом вписывает в файл символ U, перед башней T и тд. Есть порядок считывания (например ландшафт, юнит, нейтральный объект...), соответственно считывая когда приходит пора считывать юнита проверяется, что первым идет символ U. Если это не так, значит U была раньше, и каких-то данных для записи не было (либо U не было вовсе и соответственно формат файла тоже неправильный)

TSVlad commented 4 years ago

В UML диаграмму для лабы записывать только классы, написанные в лабе и те, с которыми они связаны?

Ksenox commented 4 years ago

В UML диаграмму для лабы записывать только классы, написанные в лабе и те, с которыми они связаны?

Достаточно тех, что относятся к лаб. работе.

TSVlad commented 4 years ago

Выполнены все требования ЛР

Ksenox commented 4 years ago

20 баллов. Отчет в следующий раз расписывайте подробнее