Conversation
|
Забыла кнопку нажать) |
IlyaOrlov
left a comment
IlyaOrlov
left a comment
There was a problem hiding this comment.
Хорошо. Но нужно оптимизировать (и кое-что исправить).
|
Принт - это НЕ аналог возврата. Принт - это вывод на экран. |
IlyaOrlov
left a comment
IlyaOrlov
left a comment
There was a problem hiding this comment.
Осталось ещё по мелочи исправить.
IlyaOrlov
left a comment
IlyaOrlov
left a comment
There was a problem hiding this comment.
Хорошо. Осталось 3 неисправленных замечания.
IlyaOrlov
left a comment
IlyaOrlov
left a comment
There was a problem hiding this comment.
Всё исправлено!
Остались вопросы только по (внезапно) pep8tasksh.py
|
|
||
| class Shuffler: | ||
|
|
||
| def __init__(self, map): # Отсутствует параметр map |
There was a problem hiding this comment.
А почему этот параметр должен присутствовать?
| def __init__(self, map): # Отсутствует параметр map | ||
| self.map = {} | ||
|
|
||
| def rename(self, dirname, output): # Может быть статическим методом |
There was a problem hiding this comment.
Не может. Т.к. в его теле используется self.map
| for path, mp3 in mp3s: | ||
| hashname = self.generate_name() + '.mp3' | ||
| self.map[hashname] = mp3 | ||
| os.rename(path + '/' + mp3), path + '/' + hashname)) # Возможно ошибка в названии mp3 |
2 participants
No description provided.