Как реализовать функцию duplicate (поясните плиз)?

Question

[Дарья]

Как реализовать функцию duplicate, которая должна принимать в качестве аргумента список и удваивать этот список "по месту" (примечание: список передаётся по ссылке). Удваивание здесь означает, что на выходе список должен иметь копию всех элементов, добавленную в конец (см. пример ниже).
Пример:

>>> l = [1, 2]
>>> duplicate(l)
>>> l
[1, 2, 1, 2]

Моё решение:

def duplicate(value):
    if value != None:
        result = value * 2
        return result

Answer 1

[Александр Моисейкин]

Стоит воспользоваться функцией deepcopy из стандартной библиотеки для полиморфизма. И на тот случай, когда значение содержит вложенные объекты.

from copy import deepcopy

def duplicate(value):
    if value is not None:
        value += deepcopy(value)
        return value

Comments

[Roman K]

Чукча не читатель?

удваивать этот список "по месту" (примечание: список передаётся по ссылке)

[longclaps]

Александр Моисейкин, ты неправ. Сотри эту глупость.

[Andrey Dugin]

Александр Моисейкин всё правильно написал. Поддерживаю. Кто не понял фишку, см. коммент.

Answer 2

[Ivan Yakushenko]

def duplicate(value):
    if type(value) is list and value:
        value += value

Comments

[longclaps]

Ivan Yakushenko, традиционно проверка на None выглядит так:
value is not None:
Это связано с тем, что None - синглтон, и чтобы убедиться, что value не None, достаточно проверить, что id(value) != id(None) (эту проверку и осуществляет оператор is)
Здесь же это проверка вообще неуместна - она не защитит от ошибки при умножении объекта, который не None, но и умножаться не умеет (да хоть дикт).

[Ivan Yakushenko]

longclaps,

Здесь же это проврка вообще неуместна

Вы имеете ввиду проверка вообще, или конкретно !=?

[longclaps]

Ivan Yakushenko, вообщеЖ
duplicate({}) # и чо?

[Ivan Yakushenko]

longclaps, если так, то по хорошему нужно if type(value) is list and value:
Я посчитал, что оба варианта равноценно бессмысленны, т.к. оба, например, могут пропустить пустой список, по-этому и не стал менять.
Но да, согласен, это было ошибочное мнение.

[longclaps]

Ivan Yakushenko, ну если считать, что это осмысленное задание, проверка должна быть на наличие атрибута-функции __mul__

[Ivan Yakushenko]

longclaps, а смысл, если

>>> a = 2
>>> a.__mul__(a)
4

Answer 3

[Freddi Hich]

def duplicate(items):
items.extend(items)