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 #384

Closed

Conversation

alesyacurtis
Copy link
Contributor

Разработка консольного приложения для подсчета числа компонент связности графа

Copy link

codecov bot commented Jun 4, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.04%. Comparing base (0735d63) to head (ce4139a).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #384      +/-   ##
==========================================
- Coverage   94.09%   94.04%   -0.06%     
==========================================
  Files         186      187       +1     
  Lines        6100     6098       -2     
==========================================
- Hits         5740     5735       -5     
- Misses        360      363       +3     

@alesyacurtis
Copy link
Contributor Author

@Kuznetsov-Artyom

@Kuznetsov-Artyom Kuznetsov-Artyom self-requested a review June 5, 2024 02:00
Copy link
Collaborator

@Kuznetsov-Artyom Kuznetsov-Artyom left a comment

Choose a reason for hiding this comment

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

Все хорошо, поправьте один момент

}
if (b) {
std::stringstream str;
str << std::to_string(graph_components_numbers(n, v));
Copy link
Collaborator

Choose a reason for hiding this comment

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

В данном случае не обязательно приводить возвращаемое значение из функции graph_components_numbers к строке. sstream поддерживает все фундаментальные типы. (также будут поддерживаться пользовательские типы данных, для которых перегружен оператор вывода operator<<)

@Kuznetsov-Artyom
Copy link
Collaborator

@alesyacurtis у вас пропал весь код :_)

@alesyacurtis
Copy link
Contributor Author

@Kuznetsov-Artyom вроде вернула....Случился конфликт веток и напуталось что-то. Момент с ненужным приведением функции к строке поправила.

@Kuznetsov-Artyom
Copy link
Collaborator

@Kuznetsov-Artyom вроде вернула....Случился конфликт веток и напуталось что-то. Момент с ненужным приведением функции к строке поправила.

Дело в том, что над этим модулем уже работал другой человек. Создайте новую папку для 3 лр и перенесите туда свою реализацию

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.

2 participants