Conversation
For Pull Request
| #include <cmath> | ||
| #include <iomanip> | ||
|
|
||
| std::vector<int> absMaxElement(std::vector<std::vector<double>> matrix) { |
There was a problem hiding this comment.
Ты передаешь вектор по значению, происходит лишнее копирование -> теряешь впроизводительности.
Ты никак не меняешь matrix, поэтому передавай ее по константной ссылке:
std::vector<int> absMacElement(const std::vector<std::vector<double>>& matrix)
| int x, y; | ||
| for (size_t i = 0; i < matrix.size(); ++i) { | ||
| for (size_t j = i; j < matrix.size(); ++j) { | ||
| if (i != j && fabs(matrix[i][j]) > maximum) { |
There was a problem hiding this comment.
Используй std::abs, это более адаптированная версия для C++
| } | ||
| } | ||
| } | ||
| std::vector<int> ans; |
There was a problem hiding this comment.
Ты используешь вектор, чтобы хранить два числа? Создай отдельную структуру для этого, с ней будет легче работать:
struct MaxElementPos {
size_t x;
size_t y;
}
| return ans; | ||
| } | ||
|
|
||
| double findFi (int i, int j, std::vector<std::vector<double>> matrix) { |
There was a problem hiding this comment.
Что такое фи? Добавь комменты, которые объясняют этот кусок кода
| std::cout << std::endl; | ||
| } | ||
|
|
||
| std::vector<std::vector<double>> multiplie(std::vector<std::vector<double>> a, std::vector<std::vector<double>> b) { |
There was a problem hiding this comment.
Сделать отдельный тип:
using matrix_t = std::vector<std::vector<double>>Тогда объявление станет проще:
matrix_t multiply(const matrix_t& first, const matrix_t& second)И multiply пишется по другому)
| return ans; | ||
| } | ||
|
|
||
| std::vector<std::vector<double>> returnv(double fi, int i, int j, int n) { |
There was a problem hiding this comment.
returnv -- мне название не понятно, добавь комменты
2 participants
For Pull Request