Opened 6 years ago

Closed 6 years ago

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

Проверка домашнего задания №1

Reported by: Andrey Chausov Owned by: rutsky,grabovoy.philipp
Priority: проверка Milestone: ha1-milestone2
Component: HA#1 matrices Version:
Keywords: Cc: dxahtepb@…

Description

Проверка домашнего задания №1 - Матрицы

Change History (4)

comment:1 Changed 6 years ago by Филипп

Привет!

Несколько вещей для исправления:

  1. Из списка замечаний: 6 (название бинаря), 24
  2. Файлы открываются для некорретного агрумента: при четном количестве, последний открытый файл указывает на байты, которые, возможно, не принадлежат процессу.

И еще несколько заметок:

  1. Неявное приведение std::ifstream к bool происходит, самостоятельно можно не приводить.
  2. При работе с кодами ошибок функции более привычно их видеть в качестве возвращаемого значения, а результат получать через изменяющиеся аргументы (которые традицинно размещают в конце списка).
  3. size_t & first_columns в multiply достаточно сильно выбивается из виду. Почему бы тогда полностью не менять матрицу in-place внутри функций (т.е. переделать семантику на +=, *=)? Или можно снаружи задавать размеры получившейся матрицы, проливая всю реализацию наружу.

comment:2 Changed 6 years ago by Филипп

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

comment:3 Changed 6 years ago by Andrey Chausov

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

Здравствуйте!
Поправил ошибки, проверьте пожалуйста.

comment:4 Changed 6 years ago by Филипп

Resolution: задача сдана
Status: newclosed
Note: See TracTickets for help on using tickets.