Opened 8 years ago

Closed 7 years ago

Last modified 7 years ago

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

Домашнее задание №1

Reported by: Darya Rodionova Owned by: Vladimir Rutsky
Priority: проверка Milestone:
Component: HA#1 matrices Version:
Keywords: Cc: darya.rodionova@…

Description

Ожидается code-review

Change History (8)

comment:1 Changed 8 years ago by Vladimir Rutsky

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

Замечания:

  1. Файлы с функциями для работы с матрицами должны называться matrices.cpp и matrices.hpp.
  1. Используйте тип size_t для размеров и индексов.
  1. Используйте nullptr вместо NULL.
  1. Вынесите код выделения памяти для матрицы в отдельную функцию.

comment:2 Changed 8 years ago by Darya Rodionova

Cc: darya.rodionova@… added
Type: ожидаются исправления → ожидается проверка

comment:3 Changed 8 years ago by Vladimir Rutsky

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

Замечения:

  1. Вы переименовали matrix.h в matrices.h, а требовалось в matrices.hpp.
  1. Вместо (*M).print() используйте M->print().
  1. При запуске ./matrices 2.txt --add xxx.txt память от Matrix с ненайденной xxx.txt не освобождается.
Last edited 7 years ago by Vladimir Rutsky (previous) (diff)

comment:4 Changed 8 years ago by Darya Rodionova

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

comment:5 Changed 7 years ago by Vladimir Rutsky

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

Замечания:

  1. Замечание 3 из предыдущей проверки не исправлено. Судя по логу изменений, вы его исправили, а потом обратно вернули.
  1. Если не удалось открыть первую матрицу вы не выводите сообщение об ошибке.

Исправьте, пожалуйста, эти замечания в ближайшее время, так как у вас в какой-то момент уже было практически рабочее решение.

comment:6 Changed 7 years ago by Darya Rodionova

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

comment:7 Changed 7 years ago by Vladimir Rutsky

Resolution: → задача сдана
Status: new → closed

Решение зачтено.

comment:8 Changed 7 years ago by Vladimir Rutsky

Milestone: ha1-deadline

Milestone ha1-deadline deleted

Note: See TracTickets for help on using tickets.