Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Савчук Антон. Лабораторная работа 3 #366

Merged
merged 7 commits into from
May 30, 2024

Conversation

savchukPR2
Copy link
Contributor

Реализовал приложение для Fibonacci Heap!

Copy link

codecov bot commented May 5, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.44%. Comparing base (e31997f) to head (d078c97).
Report is 16 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #366      +/-   ##
==========================================
- Coverage   94.56%   94.44%   -0.12%     
==========================================
  Files         159      178      +19     
  Lines        5171     5816     +645     
==========================================
+ Hits         4890     5493     +603     
- Misses        281      323      +42     

@savchukPR2
Copy link
Contributor Author

@vadimbelan @overinvest прошу посмотрите 🙏

@overinvest
Copy link
Contributor

overinvest commented May 5, 2024

@vadimbelan @overinvest прошу посмотрите 🙏

Чтобы все тесты прошли, нужно доработать ( добавить или изменить функции ) основной модуль. При добавлении каких-то новых функций, сделать к ним тесты.

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

Как все тесты пройдут, тегни меня

Copy link
Contributor

@vadimbelan vadimbelan left a comment

Choose a reason for hiding this comment

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

а мне все нравится 🥰

@overinvest
Copy link
Contributor

а мне все нравится 🥰

Пока у него все чеки не пройдут. Лабу все равно не замержат, а чтобы чеки прошли, нужно внести изменения в основной модуль. Иначе никак

@overinvest
Copy link
Contributor

Тут к сожалению, мой голос не решит ничего, пока он не добавит изменения

@overinvest
Copy link
Contributor

Приложение сделано хорошо

@vadimbelan
Copy link
Contributor

а мне все нравится 🥰

Пока у него все чеки не пройдут. Лабу все равно не замержат, а чтобы чеки прошли, нужно внести изменения в основной модуль. Иначе никак

codecov/project не проходит во всех последних работах

Comment on lines 13 to 47
void Console::printMenu() {
std::cout << "Fibonacci Heap Operations:" << std::endl;
std::cout << "1. Insert Element" << std::endl;
std::cout << "2. Extract Minimum" << std::endl;
std::cout << "3. Decrease Key" << std::endl;
std::cout << "4. Delete Node" << std::endl;
std::cout << "5. Merge Heaps" << std::endl;
std::cout << "Enter your choice: ";
}

void Console::handleUserInput() {
int choice;
std::cin >> choice;

switch (choice) {
case 1:
insertElement();
break;
case 2:
extractMinimum();
break;
case 3:
decreaseKey();
break;
case 4:
deleteNode();
break;
case 5:
mergeHeaps();
break;
default:
std::cout << "Invalid choice. Please try again." << std::endl;
break;
}
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Используйте, пожалуйста, команды для выбора опции, а не числа
Пример: --insert; --extract и т.д

Copy link
Collaborator

Choose a reason for hiding this comment

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

@savchukPR2 доработайте пожалуйста свою работу

@Kuznetsov-Artyom
Copy link
Collaborator

@savchukPR2

@savchukPR2
Copy link
Contributor Author

@Kuznetsov-Artyom

@Kuznetsov-Artyom Kuznetsov-Artyom merged commit 107fea4 into UNN-ITMM-Software:main May 30, 2024
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants