Skip to content

add homework8#8

Open
Galukhin wants to merge 1 commit intomainfrom
lesson8
Open

add homework8#8
Galukhin wants to merge 1 commit intomainfrom
lesson8

Conversation

@Galukhin
Copy link
Copy Markdown
Owner

Источник Интсаграмм

На вход программе подается 2 имени пользователя
Задача программы найти самую короткую цепочку рукопожатий между этими пользователями

рукопожатием считаем только взаимоподписанных пользователей

Comment thread les8.py
import dotenv


class InstaChain:
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

все клево, но 1 проблемма. Попробуйте найти цепочку между собой и Маском
Скорее всего бот схлопочет либо бан либо сожрет очень много памяти.
Не использовать в данной задаче БД самоубийство

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

если притянули сторонние библиотеки, то можно было бы и притянуть библиотеки для построения Графа. Было бы немного оптимизировнанее и код стал бы более компактным

Copy link
Copy Markdown
Owner 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