Skip to content

Neverov Lev PMI2#23

Open
lev-neverov wants to merge 9 commits into
ValentinV95:mainfrom
lev-neverov:main
Open

Neverov Lev PMI2#23
lev-neverov wants to merge 9 commits into
ValentinV95:mainfrom
lev-neverov:main

Conversation

@lev-neverov
Copy link
Copy Markdown

Ready for review.

@ValentinV95
Copy link
Copy Markdown
Owner

Программа написана не на Си. Падает при введение числа элементов, массив не выводит

@lev-neverov
Copy link
Copy Markdown
Author

lev-neverov commented Dec 27, 2021

Теперь всё должно работать. Проверял работоспособность на нескольких онлайн компиляторах.

@ValentinV95
Copy link
Copy Markdown
Owner

Код принят

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

@lev-neverov
Copy link
Copy Markdown
Author

Здравствуйте Валентин Дмитриевич. Выше прикрепил отчёт по лабораторной работе.

@lev-neverov
Copy link
Copy Markdown
Author

Здравствуйте, Валентин Дмитриевич. Отчёт по второй лабораторной работе отправил Вам на портале. Код готов к проверке.

@lev-neverov
Copy link
Copy Markdown
Author

otchet2.docx
Здравствуйте, Валентин Дмитриевич. Отчёт по второй лабораторной работе также прикрепляю и здесь.

@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.

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

Comment thread Neverov Lev Lab2/Lab2/Lab2/Lab2.cpp Outdated
#include <stdlib.h>
#include<math.h>

#define summType standartSumm(nums,n);
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.

В данном случае необходимо использовать define с параметрами. Так как текущий использует конкретные имена массива и размеров

printf("Enter x value:\n");
scanf_s("%f", &x);

switch (choice) {
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, чтобы не было магических констант в коде

@lev-neverov
Copy link
Copy Markdown
Author

ПМИ2НеверовОтчёт.docx
Здравствуйте. Перепутал файлы. Прикрепляю отчет по первой лабораторно работе. Также, я отправлял вам отчет на портале в срок сдачи.

@ValentinV95
Copy link
Copy Markdown
Owner

Код по рядам принят (+)

@lev-neverov
Copy link
Copy Markdown
Author

Здравствуйте, Валентин Дмитриевич. Отчёт по третьей лабораторной работе также прикрепляю и здесь
LabReport3_Neverov_PMI2.docx
.

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.

Код метода Гаусса принят (+-, ближе к -+). В случае ввода несовместной системы выдает неверное решение и при проверки говорит, что решение неправильное


public:
my_vector(int basic_size = 1) {
if (basic_size < 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.

Что будет означать значение равное 0?

b[row] /= this->xy[row][column];
}
else {
this->xy[row][column_tmp] /= this->xy[row][column];
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.

Тут и дальше надо было использовать оператор, но их не было реализовано

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