Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

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

ДЗ №1

Reported by: Лейла Owned by: Vladimir Rutsky
Priority: проверка Milestone:
Component: HA#1 matrices Version: 1.0
Keywords: Cc:

Description

Проверьте, пожалуйста, задание с матрицами.
Прошу прощения, что в последний момент.
В случае чего, готова все быстренько исправить.
Надеюсь, что засчитаете, хотя бы со штрафом :)

Change History (9)

comment:1 Changed 7 years ago by Vladimir Rutsky

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

Замечания:

  1. Передавайте объекты, которые не собираетесь менять, по константной ссылке:
Matrix* read(std::string const & file) {
  1. В случае возникновения ошибок вы выводите сообщение об ошибке и выходите с ненулевым кодом, но не освобождаете выделенную память.
  1. Случай запуска программы без аргументов обрабатывается некорректно --- вы выводите в качестве результата матрицу размера 0x0, а должны вывести сообщение об ошибке.

К сожалению, я не могу зачесть ваше решение с данными ошибками.

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

comment:2 Changed 7 years ago by Vladimir Rutsky

Milestone: ha1-deadlineha1-deadline2
Resolution: задача НЕ сдана
Status: closedreopened
Type: ожидается проверкаожидаются исправления

comment:3 Changed 7 years ago by Vladimir Rutsky

В соответствии с решением кураторов вам дана ещё одна попытка сдачи данной задачи.

Настоятельно рекомендую прогнать ваше решение на smoke test-е со включенным valgrind (для этого вам понадобится Linux-based дистрибутив, если вы пользуетесь только Windows, можете попробовать поставить/загрузить любой дистрибутив Linux, например, Ubuntu, в виртуальной машине, например, в VirtualBox).

comment:4 Changed 7 years ago by Лейла

Пофиксила

comment:5 Changed 7 years ago by Vladimir Rutsky

Лейла, а все ли указанные замечания вы пофиксили?

comment:6 Changed 7 years ago by Vladimir Rutsky

Моя вина, пропустил ещё одно замечания:

  1. Не обрабатывается случай, когда файл с матрицей не удалось открыть.

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

comment:7 Changed 7 years ago by Лейла

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

Добавила обработку случая, когда файл с матрицей не удалось открыть.

comment:8 Changed 7 years ago by Vladimir Rutsky

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

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

comment:9 Changed 7 years ago by Vladimir Rutsky

Milestone: ha1-deadline2

Milestone ha1-deadline2 deleted

Note: See TracTickets for help on using tickets.