Opened 4 years ago

Closed 4 years ago

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

Ha1 matrices [Golovchanskaya Yuliya] v.3

Reported by: golovchanskaya.yuliya Owned by: rutsky,grabovoy.philipp
Priority: проверка Milestone: ha1-deadline
Component: HA#1 matrices Version:
Keywords: Cc:

Description


Change History (6)

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

Приветствую!

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

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

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

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

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

comment:3 Changed 4 years ago by golovchanskaya.yuliya

Summary: ha1 Matrices Golovchanskaya YuliyaHa1 matrices [Golovchanskaya Yuliya] v.2
Type: ожидаются исправленияожидается проверка

Добрый день! Спасибо за ответ.

Учла все замечания, надеюсь правильно. Файлы обновила.

Version 0, edited 4 years ago by golovchanskaya.yuliya (next)

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

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

В matrices.hpp мне все таки необходимо подключение <iostream> из-за моей функции show_matrix();

Для объявления функции данный заголовочный файл не нужен -- в параметрах нет специфических типов. В реализации нужен, поэтому его стоит включить в matrices.cpp

У временной суммы очень нужно поменять тип -- иначе получается совсем не то.

И еще один важный момент: ловите исключения по константной ссылке -- чтобы избежать копирования.

comment:5 Changed 4 years ago by golovchanskaya.yuliya

Summary: Ha1 matrices [Golovchanskaya Yuliya] v.2Ha1 matrices [Golovchanskaya Yuliya] v.3
Type: ожидаются исправленияожидается проверка

Добрый день! Спасибо за ответ.

Исправила все 3 пункта:
1) В matrices.hpp оставила только <cstddef> для типа size_t.

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

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