search

moevm / oop

Object-oriented programming course
29 stars 225 forks source link

[8303] Zhurbin Kirill lr5 #628

Closed kera0106 closed 4 years ago

kera0106 commented 4 years ago

lr5 done

kera0106 commented 4 years ago

Смотреть файлы SnaphotField.h/cpp, File.h, BaseHandler.h

anyfilatov commented 4 years ago

У вас тут какая-то беда с RAII. идея в том, чтобы файл захватывался в конструкторе (и не просто синглтона File, а класса, который этим файлом пользуется), а отпускался в деструкторе. (Лучше, чтобы это было в классе логов, а не классе поля) У вас же файл захватывается в определённом методе.

В остальном всё ок 19, доделать 3

kera0106 commented 4 years ago

Переделал. Теперь экземпляр файла File создается (захватывается) в конструкторе класса Snapshot (см. SnapshotField.cpp), а высвобождается в деструкторе Snapshot