Skip to content

Миронова Анастасия#28

Open
AMironova21 wants to merge 18 commits into
ValentinV95:mainfrom
AMironova21:main
Open

Миронова Анастасия#28
AMironova21 wants to merge 18 commits into
ValentinV95:mainfrom
AMironova21:main

Conversation

@AMironova21
Copy link
Copy Markdown

Сортировки готовы, можно проверять. Отчет в работе.

@ValentinV95
Copy link
Copy Markdown
Owner

Проект не открывается, надо вернуть исходную структуру

@AMironova21
Copy link
Copy Markdown
Author

Загрузила заново

@ValentinV95
Copy link
Copy Markdown
Owner

Код принят

@ValentinV95 ValentinV95 added the good first issue Good for newcomers label Dec 28, 2021
@AMironova21
Copy link
Copy Markdown
Author

Добавила отчет

@ValentinV95
Copy link
Copy Markdown
Owner

Отчет по сортировкам принят

Copy link
Copy Markdown
Owner

@ValentinV95 ValentinV95 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Есть замечания. Также необходимо удалить лишние файлы

array[1] = x;
for (int i = 2; i < n; i++) {
array[i] = sin_next(x, array[i - 2], i);

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Нулевые элементы обычно не считают. И когда говорят про 5 слагаемых ряда, имеют ввиду 5 не нулевых

for (int i = 0; i < n; i++) {
array[i] = float(0);
}
if (func == 1) {
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Во всех местах, где используется выбор функции, лучше сделать Enum

@ValentinV95
Copy link
Copy Markdown
Owner

Код по рядам принят (+-). Нулевые элементы в sin и cos не надо хранить. Еще можно было избежать дублирования при разных функциях на вызове суммирований

Copy link
Copy Markdown
Owner

@ValentinV95 ValentinV95 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Код метода Гаусса принят (+-). Из-за опечатки не определяются некоторые несовместные системы

Comment thread Laba_3/Laba_3/classes.h
T* data;
size_t size;
public:
vector() { size = 0; data = nullptr; }
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Что такое математический вектор размера 0?

Comment thread Laba_3/Laba_3/classes.h
vector() { size = 0; data = nullptr; }
vector(size_t _size) {
size = _size;
data = new T[size];
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Если размер будет отрицательный?

Comment thread Laba_3/Laba_3/classes.h
}
}

void triangle_lower(matrix<T>& a, size_t n, vector<T>& vec)
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Решение системы необходимо было реализовать в отдельном классе

Comment thread Laba_3/Laba_3/classes.h
for (size_t i = 0; i < n - 1; ++i) { // n - 1
size_t imax = max_col(i);
if (i != imax) {
swap(i, imax);
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

swap происходит с data, а остальные действия с a

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

good first issue Good for newcomers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants