Opened 4 years ago

Closed 4 years ago

#584 closed ожидаются исправления (задача сдана)

WW #10

Reported by: Maria Chizhova Owned by: Maria Chizhova
Component: WW figures Version: 1.0
Keywords: Cc:

Description


Change History (2)

comment:1 Changed 4 years ago by Дмитрий Свиридкин

Owner: changed from Дмитрий Свиридкин to Maria Chizhova
Type: ожидается проверкаожидаются исправления

При переопределении виртуальных методов лучше все-таки писать override.

Зачем get_id делать виртуальным?

Дефолтный конструктор Figure не инициализирует ни одного поля. int не имеет конструктора, так что поля останутся неинициализированными -> конструктор класса создал объект в неконсистентном состоянии, что совсем не хорошо.

Поля x,y лучше переименовать в соответсвии с тем, что эти координаты обозначают.

Дефолтные деструкторы можно попросить сгенерировать компилятор (= default)

Поиск по id повторяется в трех местах.


7.5

Last edited 4 years ago by Дмитрий Свиридкин (previous) (diff)

comment:2 Changed 4 years ago by Дмитрий Свиридкин

Resolution: задача сдана
Status: assignedclosed
Note: See TracTickets for help on using tickets.