#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
Milestone: | ha1-milestone1 → ha1-milestone2 |
---|---|
Type: | ожидается проверка → ожидаются исправления |
comment:2 Changed 8 years ago by
Cc: | darya.rodionova@… added |
---|---|
Type: | ожидаются исправления → ожидается проверка |
comment:3 Changed 8 years ago by
Milestone: | ha1-milestone2 → ha1-deadline |
---|---|
Type: | ожидается проверка → ожидаются исправления |
Замечения:
- Вы переименовали
matrix.h
вmatrices.h
, а требовалось вmatrices.hpp
.
- Вместо
(*M).print()
используйтеM->print()
.
- При запуске
./matrices 2.txt --add xxx.txt
память отMatrix
с зенайденнойxxx.txt
не освобождается.
Version 0, edited 8 years ago
by
(next)
comment:4 Changed 8 years ago by
Type: | ожидаются исправления → ожидается проверка |
---|
comment:5 Changed 7 years ago by
Type: | ожидается проверка → ожидаются исправления |
---|
Замечания:
- Замечание 3 из предыдущей проверки не исправлено. Судя по логу изменений, вы его исправили, а потом обратно вернули.
- Если не удалось открыть первую матрицу вы не выводите сообщение об ошибке.
Исправьте, пожалуйста, эти замечания в ближайшее время, так как у вас в какой-то момент уже было практически рабочее решение.
comment:6 Changed 7 years ago by
Type: | ожидаются исправления → ожидается проверка |
---|
Note: See
TracTickets for help on using
tickets.
Замечания:
matrices.cpp
иmatrices.hpp
.size_t
для размеров и индексов.nullptr
вместоNULL
.