Skip to content

Borisov Savelii Sorts#14

Open
Amazingkivas wants to merge 25 commits into
ValentinV95:mainfrom
Amazingkivas:main
Open

Borisov Savelii Sorts#14
Amazingkivas wants to merge 25 commits into
ValentinV95:mainfrom
Amazingkivas:main

Conversation

@Amazingkivas
Copy link
Copy Markdown

No description provided.

@Amazingkivas
Copy link
Copy Markdown
Author

готово к проверке

@ValentinV95
Copy link
Copy Markdown
Owner

Проект не настроен, код написан не на Си. Переменные объявлены в любых местах, а результат malloc не приводится к нужному типу

@Amazingkivas
Copy link
Copy Markdown
Author

исправил

@Amazingkivas
Copy link
Copy Markdown
Author

загрузил отчёт

@ValentinV95
Copy link
Copy Markdown
Owner

Код не компилируется. Название аргументов в Поразрядной сортировки подразумевает, что выходной массив второй, а не первый как у вас

@Amazingkivas
Copy link
Copy Markdown
Author

Amazingkivas commented Dec 27, 2021

Поправил. Также немного изменил отчёт в связи с изменениями кода

@ValentinV95
Copy link
Copy Markdown
Owner

Код принят

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

Добавил код для второй лабораторной

@Amazingkivas
Copy link
Copy Markdown
Author

Загрузил отчёт для второй лабораторной

@Amazingkivas
Copy link
Copy Markdown
Author

Код можно проверять

@ValentinV95
Copy link
Copy Markdown
Owner

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

@ValentinV95
Copy link
Copy Markdown
Owner

Код с математическими функциями принят

@Amazingkivas
Copy link
Copy Markdown
Author

Код по методу Гаусса можно проверять

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.

Код по методу Гаусса принят (+-), есть небольшие замечания, можно исправить

tmp = this->coordinates[tmp_column][column];
this->coordinates[tmp_column][column] = this->coordinates[tmp_column][max_row];
this->coordinates[tmp_column][max_row] = tmp;
}
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.

В данном месте можно делать перемещение вектора целиком, если сделать правильную последовательность строка-столбец

max_row = column;
for (int row = column + 1; row < this->size; row++) // finding maximum element in a column
{
if (fabs(this->coordinates[column][max_row]) < fabs(this->coordinates[column][row]))
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.

Обычно первая координата это номер строки, а вторая столбца

@Amazingkivas
Copy link
Copy Markdown
Author

Поправил код согласно замечаниям

@Amazingkivas
Copy link
Copy Markdown
Author

Загрузил отчёт для лабораторной по методу Гаусса

@ValentinV95
Copy link
Copy Markdown
Owner

Код метода Гаусса принят (+)

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