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