Opened 4 years ago

Closed 4 years ago

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

Home assignment #1 Anastasia Lavrenko

Reported by: Anastasia Lavrenko Owned by: rutsky,grabovoy.philipp
Priority: проверка Milestone: ha1-deadline
Component: HA#1 matrices Version:
Keywords: Cc: lavrenko.a@…

Description

Check my solution, please.

Change History (6)

comment:1 Changed 4 years ago by Anastasia Lavrenko

Cc: lavrenko.a@… added

comment:2 Changed 4 years ago by Vladimir Rutsky

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

Замечания:

  1. Объявление create_matrix_array() можно убрать из matrices.hpp, это можно считать деталью реализации.
  1. В вашем решений присутствуют ошибки из списка из условия задачи "часто возникающие ошибки":
    • 9 (print()),
    • 12 (например, при ./matrices 5.txt --a 2.txt вы не освобождаете ).
  1. Вы не освобождаете память от выводимой на экран результирующей матрицы.
  1. *(data_[row] + column) эквивалентно data_[row][column], но последнее лучше читается.

comment:3 Changed 4 years ago by Anastasia Lavrenko

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

Спасибо, поправила, можно снова смотреть.

comment:4 Changed 4 years ago by Vladimir Rutsky

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

Замечания:

  1. Не стоит использовать альтернативные написания логических операторов (and): if (matrix_left != nullptr and matrix_right != nullptr), используйте &&: if (matrix_left != nullptr && matrix_right != nullptr) --- это общепринято в C++, тогда как альтернативные написания были введены для кодировок или клавиатур, в которых нет таких символов, как &.
  1. Память освобождается не во всех ошибочных ситуациях, например при вызове ./matrices 5.txt --a 2.txt происходит утечка памяти.

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

comment:5 Changed 4 years ago by Anastasia Lavrenko

Type: ожидаются исправленияожидается проверка
  1. Спасибо, это по инерции с питона =)

Поправила, можно проверять

comment:6 Changed 4 years ago by Vladimir Rutsky

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

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

Note: See TracTickets for help on using tickets.