Opened 4 years ago

Closed 4 years ago

#756 closed ожидается проверка (задача сдана)

WW #14

Reported by: Карнаухов Кирилл Owned by: Sokolov Viacheslav
Component: WW_exceptions Version: 2.0
Keywords: Cc:

Description


Change History (7)

comment:1 Changed 4 years ago by Egor Suvorov

Owner: changed from Sokolov Viacheslav to Карнаухов Кирилл
Type: ожидается проверка → ожидаются исправления

comment:2 Changed 4 years ago by Карнаухов Кирилл

Owner: changed from Карнаухов Кирилл to Sokolov Viacheslav
Type: ожидаются исправления → ожидается проверка

comment:3 Changed 4 years ago by Sokolov Viacheslav

Type: ожидается проверка → ожидаются исправления

25 size_t row_size = 0;
26 size_t col_size = 0;

какая мотивация делать инициализацию в .h?

2 #include <iostream>

в include/matrix.h достаточно #include <iosfwd>

(int)row_size
не используйте c-style cast в проектах. Кроме количества символов в нем нет ничего хорошего. В данном случае нужен static_cast

58 return (*this = Matrix(result));

во время исполнения этой строчки будет потребляться памяти больше, чем могло бы. Сейчас ничего делать не нужно, но в языке для решения этой проблемы придумали rvalue references.

98 MatrixException::MatrixException?(const std::string &_msg) :
99 std::logic_error{_msg} {}

этот конструктор не используется?

61 static int read_int_safely(std::istream &in) {
а зачем здесь static?

comment:4 Changed 4 years ago by Sokolov Viacheslav

стоит добавить noexcept туда, где это может быть важно

comment:5 Changed 4 years ago by Sokolov Viacheslav

по Стандарту int может быть 16-битным

comment:6 Changed 4 years ago by Карнаухов Кирилл

Type: ожидаются исправления → ожидается проверка
Version: 1.0 → 2.0

comment:7 Changed 4 years ago by Sokolov Viacheslav

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