WW_exceptions Kozyrev Andrei lab_12

[Антон Филатов]

$ make
mkdir -p obj
g++ -O2 -Wall -Werror -std=c++11 -Iinclude -c -MMD -o obj/matrix.o src/matrix.cpp
src/matrix.cpp: In constructor ‘Matrix::Matrix(std::size_t, std::size_t)’:
src/matrix.cpp:8:19: error: ‘buffer’ may be used uninitialized in this function [-Werror=maybe-uninitialized]
         delete [] buffer;
                   ^~~~~~
src/main.cpp: In function ‘Matrix* load(std::ifstream&)’:
src/main.cpp:94:35: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare]
         for (std::size_t i = 0; i < rows; ++i)
                                 ~~^~~~~~
src/main.cpp:95:39: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare]
             for (std::size_t j = 0; j < cols; ++j) {
cc1plus: all warnings being treated as errors
Makefile:17: recipe for target 'obj/matrix.o' failed
make: *** [obj/matrix.o] Error 1

Тесты пройдены (правда иногда лишние endl-ы печатате)

Стиль:

• если new не сработал, то удалять память не надо
• в init вы почему-то не ловите bad_alloc во время new
• Сишный принт в файл надо бы переписать
• у вас же есть чудесный метод init, почему же вы не сделали конструктор копирования, чтобы в операторе= сделать своп-трик за две строчки?
• кое-где не хватает noexcept

5.5/10

[Andrei Kozyrev]

Я ведь правильно понимаю, что первый дедлайн по exceptions лабе был в четверг, а не в среду? Просто так на гитхабе сказано. И я в четверг сдал, а вы как то не проверили совсем.

[Антон Филатов]

10/10