40
Awesome Team
Alexander Lyabah
http://www.checkio.org/user/oduvan/
Last seen 21 hours ago
Member for 13 years, 11 months, 6 days
Difficulty Easy
love it!
лучше не менять входящие данные, т.е. лучше создавать отдельную переменную, в которой отсортированные данные, а не сортировать данные data
More
First-Julia_Makeyeva
везде после result = идет return
поэтому можно поменять на return
More
First-Valentyna_Zelik
если после break идет return, значит можно просто сразу сделать return
т.е. строчки 10, 11 и 12 можно просто заменить на return False
More
First-Valentyna_Zelik
строка 2
lambda a: abs(a)
это все равно что
abs
ну, по крайней мере работает одинаково.
More
строка 12: зачем проходится по индексам, если ты их используешь только для получения значений. Проходись по значениям тогда
More
Я правильно понимаю, ты раскладываешь интервалы, чтобы потом склеить обратно?
Ну, мне кажется, есть метод по лучше :)
More
First-Julia_Makeyeva
о.. не плохо, но есть моменты для оптимизации
Например - почему не проверять на max_len прям в 17ой строке. и тогда массив result не нужен
More
First-ukrainer.dp
если после break у тебя сразу идет return, то тогда в break особо смысле нет
More
First-ukrainer.dp
в строках 12, 14 и 16 не надо использовать функцию str потому что 'hi' - это уже строка.
А так молодец, продолжай в том же ритме
More
First-Valentyna_Zelik
Хорошо.
Единственное - тебе не нужна функция list в 3ей строке
More
First-Valentyna_Zelik
тут 2 момента.
1. не очень хорошо изменять входные данные, т.к. list - это изменяемый тип данных
1. попробуй решить эту же задачу, но используя for
More
First-NikitaYatsura
Тут в решении много лишнего кода. Подумай, что можно выкинуть.
More
First-NikitaYatsura
Молодец. Но скобки для return не нужны. Можно просто писать
return "Fizz"
More
ты знал :)
скажи, а решение работало бы если бы вторая строка была
x,list1='','()[]{}'
More
А еще, обрати внимание, что у тебя 3 повторяющихся блока кода (что не очень круто). Подумай как переписать решение так, чтобы этого не было
More
