Skip to content

Alexander_homework #9

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

LightAboveFighter
Copy link
Contributor

No description provided.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

clang-tidy made some suggestions

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

clang-tidy made some suggestions

@LightAboveFighter LightAboveFighter changed the title add task_01 tests Alexander_homework Sep 21, 2024
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

clang-tidy made some suggestions

@@ -0,0 +1,49 @@
#ifndef __TOPOLOG_SORT_H__
#define __TOPOLOG_SORT_H__

Choose a reason for hiding this comment

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

warning: declaration uses identifier 'TOPOLOG_SORT_H', which is a reserved identifier [bugprone-reserved-identifier]

Suggested change
#define __TOPOLOG_SORT_H__
#define TOPOLOG_SORT_H_

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

clang-tidy made some suggestions

@@ -0,0 +1,49 @@
#ifndef __TOPOLOG_SORT_H__
#define __TOPOLOG_SORT_H__
Copy link

Choose a reason for hiding this comment

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

warning: declaration uses identifier 'TOPOLOG_SORT_H', which is a reserved identifier [bugprone-reserved-identifier]

Suggested change
#define __TOPOLOG_SORT_H__
#define TOPOLOG_SORT_H_

add_paths(links);
}

Node<T, NameType>* find_via_name(NameType name) {
Copy link
Contributor

Choose a reason for hiding this comment

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

FindByName по кодстайлу

template <typename T>
Graph<T, int>& topological_sort(Graph<T, int>& g) {
std::vector<Node<T, int>*> marked_points;
for (auto i : g.nodes) {
Copy link
Contributor

Choose a reason for hiding this comment

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

(auto node : g.nodes)

а ещё лучше node_ptr так понятнее будет что там указатель)

Copy link
Contributor Author

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
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants