Skip to content

lesson 8 completed finally#1278

Open
Karadesh wants to merge 1 commit intoDmitryChitalov:masterfrom
Karadesh:lesson_8
Open

lesson 8 completed finally#1278
Karadesh wants to merge 1 commit intoDmitryChitalov:masterfrom
Karadesh:lesson_8

Conversation

@Karadesh
Copy link

Сделано: алгоритм Хаффмана через очереди (В том числе декодер). На выходе получаем закодированную строку.

Copy link
Owner

Choose a reason for hiding this comment

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

допустим ли такой стиль по пеп-8??

Copy link
Owner

Choose a reason for hiding this comment

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

он недопустим

Copy link
Author

Choose a reason for hiding this comment

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

Спасибо! Да, нужно было через нижнее подчеркивание (а лучше - максимально упрощенно, одним словом), а не CamelCase. Теперь не перепутаю.

Copy link
Owner

Choose a reason for hiding this comment

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

такие имена ф-ций - шифрокоддругие разработчики не поймут что делает ф-ция
нужны информативные имена

Copy link
Author

Choose a reason for hiding this comment

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

такие имена ф-ций - шифрокоддругие разработчики не поймут что делает ф-ция
нужны информативные имена

Спасибо! Согласен, нужно было это учитывать.

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

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants