Opened 8 years ago

Closed 7 years ago

Last modified 7 years ago

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

ha1 - update 2

Reported by: gordeev.stanislav Owned by: Vladimir Rutsky
Priority: проверка Milestone:
Component: HA#1 matrices Version:
Keywords: Cc: gordeev.so@…

Description


Change History (6)

comment:1 Changed 7 years ago by Vladimir Rutsky

Milestone: ha1-milestone2ha1-deadline
Type: ожидается проверкаожидаются исправления

Замечания:

  1. Передавайте неизменяемые объекты явно константными:
void WriteMatrix(double const * const * m, size_t rows, size_t columns);
  1. Не сравнивайте с true/false в логических выражениях:
//if(read_matrix.is_open() != true)
if(!read_matrix.is_open())
  1. В данной задаче операции производятся только над матрицами, и матрица 1x1 это именно матрица, а не скаляр, её нельзя умножить, например, на матрицу 2x2.

comment:2 Changed 7 years ago by gordeev.stanislav

Cc: gordeev.so@… added
Summary: ha1ha1 - update 1
Type: ожидаются исправленияожидается проверка
  1. Изменены параметры функций, не модифицирующих объекты(матрицы).
  2. Исправлены сравнения с true/false.
  3. Удален функционал с умножением на скаляр. Теперь операции с матрицами 1x1 соответствуют операциям с матрицами.

comment:3 Changed 7 years ago by Vladimir Rutsky

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

Замечания:

  1. При запуске с недостающей последней матрицей ваша программа падает (./matrices 0.txt --add 1.txt --add).

Исправьте, пожалуйста, в ближайшее время.

comment:4 Changed 7 years ago by gordeev.stanislav

Summary: ha1 - update 1ha1 - update 2
Type: ожидаются исправленияожидается проверка

Исправлено.

comment:5 Changed 7 years ago by Vladimir Rutsky

Resolution: задача сдана
Status: newclosed

Решение зачтено (с минусом, т.к. вы допустили с одной стороны пустяковую ошибку, с другой стороны она приводила к критически некорректному поведению).

comment:6 Changed 7 years ago by Vladimir Rutsky

Milestone: ha1-deadline

Milestone ha1-deadline deleted

Note: See TracTickets for help on using tickets.