ShubinGeorge / cplusplus-tasks

Represetory for C++ homework
2 stars 0 forks source link

homework3-forest #3

Closed gitluup closed 5 years ago

gitluup commented 5 years ago

https://github.com/ShubinGeorge/cplusplus-tasks/blob/719f1a74fc5394249adb780f109140a4d122963c/Forest/Forest/forest.cpp#L13 https://github.com/ShubinGeorge/cplusplus-tasks/blob/719f1a74fc5394249adb780f109140a4d122963c/Forest/Forest/forest.cpp#L15 Было бы здорово, если бы в этих двух строчках была применена move-семантика)


https://github.com/ShubinGeorge/cplusplus-tasks/blob/719f1a74fc5394249adb780f109140a4d122963c/Forest/Forest/forest.cpp#L20 Метод Update довольно большой по размеру. Есть такое неформальное правило: функции (или методу) желательно влезать в экран вашего компьютера) Из него можно было бы, например, вынести логику подсчёта числа горящих соседей в отдельный метод класса Forest, тем самым уменьшив размер метода Update

gitluup commented 5 years ago

Всё круто, задание засчитано)