WW #9

[Артур Гулецкий (huletski)]

Внутренние тесты решение проходит.

Замечания:

• не выполняется требование из задания "умный указатель, указывающий в nullptr, не должен требовать дополнительной памяти." -> -1;
• sh_ptr.cpp:7,8: очередную запятую в строке списка инициализации (СИ) обычно ставят под : (в начале строки); так проще воспринимать код, т.к. имена полей в начале строк CИ выровнены по вертикали;
• sh_ptr.cpp:36, 40. где-то используете списки инициализации, где-то нет -> -0.25;
• sh_ptr.cpp:54, 58, 66, 69. дублирование кода извлечения сырого указателя, это оптимизация такая?;
• sh_ptr.cpp:58. Лучше сравнивать с nullptr (C++11) или с NULL (C++11-).

Баллы: 8.75.

[Артур Гулецкий (huletski)]

Замечания (shared_ptr.cpp):
2: #include <cstdlib>
9: можно было вынести в список инициализации: _data(mtx != nullptr ? mtx : nullptr). В такой записи очевидно, что if лишний -> _data(mtx);
17: привычнее видеть delete без скобок;
50: лучше using std::swap; swap(…, …); . Про причины, вероятно, будет рассказано в след. семестре.

не выполняется требование из задания "умный указатель, указывающий в nullptr, не должен требовать дополнительной памяти." -> -1;

Все еще актуально (Storage создается для всех значений хранимого указателя).

Баллы: 9.