Skip to content

FOMICHEVDMITRIY#7

Open
Termtre wants to merge 27 commits into
ValentinV95:mainfrom
Termtre:main
Open

FOMICHEVDMITRIY#7
Termtre wants to merge 27 commits into
ValentinV95:mainfrom
Termtre:main

Conversation

@Termtre
Copy link
Copy Markdown

@Termtre Termtre commented Nov 27, 2021

No description provided.

@Termtre Termtre changed the title test FOMICHEVDMITRIY Nov 27, 2021
@Termtre
Copy link
Copy Markdown
Author

Termtre commented Dec 21, 2021

Код готов.
Можно проверять.
Итоговый код пузырьковой в bubblesort, для слияния в MERGESORT(которое после radixsort), для быстрой в Квиксорт, для поразрядной в RADIXSORT
Или перейти по ссылке(около Merge branch 'main' of). Там все хорошо.

@Termtre
Copy link
Copy Markdown
Author

Termtre commented Dec 22, 2021

Отчет готов.
FDEReport - это отчет.

@ValentinV95
Copy link
Copy Markdown
Owner

Проекты с сортировкой слияния и поразрядной настроены не корректно. Также почему-то есть лишний проект. Надо исправить

@Termtre
Copy link
Copy Markdown
Author

Termtre commented Dec 24, 2021

Код готов.
Все сортировки в одном.

@ValentinV95
Copy link
Copy Markdown
Owner

Код принят

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

Termtre commented Mar 23, 2022

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

@ValentinV95
Copy link
Copy Markdown
Owner

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

@ValentinV95
Copy link
Copy Markdown
Owner

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

@Termtre
Copy link
Copy Markdown
Author

Termtre commented May 14, 2022

исправил

@Termtre
Copy link
Copy Markdown
Author

Termtre commented May 25, 2022

Код 3 лабораторной готов

@Termtre
Copy link
Copy Markdown
Author

Termtre commented May 26, 2022

Код второй лабораторной был изменен

@ValentinV95
Copy link
Copy Markdown
Owner

ValentinV95 commented May 29, 2022

Код по рядам принят (-+). Проверял последнюю версию 2.2. Отсутствует необходимый по заданию переход от одного слагаемого к другому (позволяет не считать степени и факториалы), а также в попарном суммирование написан частный случай только на 10 элементов

Также необходимо навести порядок в репозитории, ориентироваться по такому количеству папок сложно

{
return -one;
}
}
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.

Стандартная функция abs

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.

Есть замечания по коду Гаусса

private:
int size;
T* data = nullptr;
int count = 0;
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.

Зачем нужна переменная count? В данном классе она не используется

{
private:
int row;
T** matrix_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.

Матрица, как наследник vector<vector> должна хранить и уже хранит информацию там. Таким способом вы переиспользуете все возможности класса вектор, который написали ранее

matrix<T> A;
vector<T> b;
vector<T> x;
vector<T> error;
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.

В СЛАУ необходимо было либо сделать включение, либо наследование

int size = A.get_row();
vector<T>* vectors;

vectors = new vector<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.

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

@ValentinV95
Copy link
Copy Markdown
Owner

Код метода Гаусса принят (+). На будущие, у вас в зависимости от знака temp_matrix[i][j] написано 2 ветки с одинаковым кодом

@ValentinV95
Copy link
Copy Markdown
Owner

Не вижу отчета

@Termtre
Copy link
Copy Markdown
Author

Termtre commented Jun 7, 2022

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

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