AZ
вообще если тебе часто надо удалять элемент из середины по индексу, то стоит задуматься, может словарь тебе лучше подойдет
Size: a a a
2021 June 23
E
вообще-то обсуждалось pop(i)
E
pop(0) и pop(), которые выполняются за O(1) - это не обсуждалось. Понятно, что если ты используешь список как стек и снимаешь с него элементы, то ты используешь pop.
AZ
я бы вообще сказал, что удаление элемента из списка по индексу это антипаттерн, и вместо него на самом деле должен быть filter (когда мы проверяем условие для всех элементов), либо это должен быть словарь, если разные элементы имеют разное семантическое значение
E
Да, я согласен. Это как раз следующий вопрос, который надо задавать себе - а может тут нужна более подходящая структура данных?
ВТ
А что такого…
Б
do_something(..., on_success=collection.pop)Зачем что-то импортировать?
А вот это как с
del написать?collection.pop(key, None)Ифы добавлять? Нет уж, извольте. pop более универсальный и не имеет никаких недостатков по сравнению с del. Зачем нужен del?
Б
Зачем что-то присваивать переменной там, где присваивание не требуется?
ВТ
Это гошный подход. Метод же что-то возвращает, так что лучше явно это показать и не путать читающего код. Просто присваиваешь в переменную, которая не будет нигде использоваться, своего рода /dev/null
E
А может мне не надо "мягкое удаление" (вариант с None).
По аналогии в тех же множествах есть discard для "мягкого удаления". То есть сразу видно, что удаляемого элемента в множестве может не быть.
По аналогии в тех же множествах есть discard для "мягкого удаления". То есть сразу видно, что удаляемого элемента в множестве может не быть.
Б
Вот явно показываю:
bubbles.pop()Б
Нет присваивания == значение ненужно
E
del нужен за тем, когда тебе нужен del, а не pop. Что тут спорить? "Более универсально" - это не нужно если мне нужно просто удалить элемент по индексу
E
Я кстати не понял, какой ещё None в pop для списка? Там же индексы.
Б
Зачем может быть нужен del, а не pop?
E
Я уже писал выше, повторяться не буду.
ВТ
Вот читаю я такой код на ревью. Ты забыл значение присвоить или осознанно это сделал? С подходом _ = … видно что ты осознанно игнорируешь значение
AZ
если в других строчках ничего не используется похожее на выкидываемое значение, то оно явно не нужно
AZ
а если переменная присвоена, но не используется, то может это потому что ты в процессе дебага удалил что-то, и забыл назад добавить?