Opened 6 years ago
Closed 6 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 6 years ago by
comment:2 Changed 6 years ago by
| Milestone: | ha1-milestone1 → ha1-milestone2 |
|---|---|
| Type: | ожидается проверка → ожидаются исправления |
comment:3 Changed 6 years ago by
| Summary: | ha1 Matrices Golovchanskaya Yuliya → Ha1 matrices [Golovchanskaya Yuliya] v.2 |
|---|---|
| Type: | ожидаются исправления → ожидается проверка |
Добрый день! Спасибо за ответ.
Учла все замечания, надеюсь правильно. Файлы обновила.
(1)
- Удалила std::ifstream::close();.
- В matrices.hpp мне все таки необходимо подключение <iostream> из-за моей функции show_matrix();
2
Исправила.
Для повышения эффективности считываю суммы для произведения матриц в локальную переменную.
comment:4 Changed 6 years ago by
| Milestone: | ha1-milestone2 → ha1-deadline |
|---|---|
| Type: | ожидается проверка → ожидаются исправления |
В matrices.hpp мне все таки необходимо подключение <iostream> из-за моей функции show_matrix();
Для объявления функции данный заголовочный файл не нужен -- в параметрах нет специфических типов. В реализации нужен, поэтому его стоит включить в matrices.cpp
У временной суммы очень нужно поменять тип -- иначе получается совсем не то.
И еще один важный момент: ловите исключения по константной ссылке -- чтобы избежать копирования.
comment:5 Changed 6 years ago by
| Summary: | Ha1 matrices [Golovchanskaya Yuliya] v.2 → Ha1 matrices [Golovchanskaya Yuliya] v.3 |
|---|---|
| Type: | ожидаются исправления → ожидается проверка |
Добрый день! Спасибо за ответ.
Исправила все 3 пункта:
1) В matrices.hpp оставила только <cstddef> для типа size_t.
comment:6 Changed 6 years ago by
| Resolution: | → задача сдана |
|---|---|
| Status: | new → closed |
Приветствую!
Несколько пунктов для исправления:
std::invalid_argument)И для повышения эффективности вычислений можно при умножении матриц считать сумму в локальную переменную, а потом записывать значение в ячейку -- так не будет происходить лишних разыменований указателя.