Почему не работает deepcopy?

Question

[Max Payne]

command = tools.get_attr_recursive(scripts, command_class_name) 
# command.name по умолчанию пуст
# print(command.name)
# ""
self.commands[command_name] = copy.deepcopy(command)
self.commands[command_name].name = command_name
print(command.name, self.commands[command_name].name)
# command.name == self.commands[command_name].name
# True

Comments

[DDDsa]

Уточните, что именно не работает. Хотя бы результат принта покажите.

[Max Payne]

DDDsa, name присваивается не только объекту в словаре self.commands, но и классу command, глубокая копия которого присваивается в 5 строке.

[Stanislav Pugachev]

Max Payne,
толи вы нам не весь код показали
толи одно из двух

import copy


class Command(object):
    def __init__(self):
        self.name = ""


command = Command()

commands = {}
command_name = 'test'
commands[command_name] = copy.deepcopy(command)
commands[command_name].name = command_name

print(command.name, commands[command_name].name)  # ('', 'test')
print(command.name == commands[command_name].name)  # False

[DDDsa]

Max Payne, А! То есть, вы хотите сделать deepcopy класса? Обычно это делают с экземплярами класса и это работает. С классами действительно не работает.
Почему — не знаю, буду ждать ответа знатока вместе с вами

[Stanislav Pugachev]

DDDsa, а где видно что он хочет именно deepcopy класса, а не экземпляра?

[Max Payne]

Stanislav Pugachev, Класс в модуле и не инициализирован в момент присваивания. То есть, грубо говоря, в моём коде первую строку можно заменить на
command = scripts.weather.TodayWeather
Функция tools.get_attr_recursive возвращает этот класс по его имени.

[Max Payne]

DDDsa, Stanislav Pugachev, Да, deepcopy класса.

[Модератор]

Max Payne, дорогой пользователь, настоятельно рекомендуем еще раз обратить самое пристальное внимание на п. 3.1 регламента работы сервиса (и, в особенности, на его последний абзац). В противном случае, ваши вопросы будут удаляться по причине тег-спама, а систематические нарушения приведут к блокировке учетной записи.

Answer 1

[Stanislav Pugachev]

кароче тема такая

import copy


class Command(object):
    name = ""


cmd = Command
cmd2 = copy.deepcopy(Command)
assert cmd == cmd2

копирование действительно ничего не дает, потому что как написано здесь copy и deepcopy занимаются копированием объектов
в нашем случае экземпляров действительно нет

что вы хотите сделать - это создать класс с похожим поведением но немного отличный
есть вариант унаследоваться и переопределить нужные части
либо можно собрать класс вручную переиспользовав кишки от оригинального класса

class Command(object):
    name = ""

cmd = Command
cmd3 = type('NewCommand', Command.__bases__, dict(Command.__dict__))
assert cmd == cmd3  # AssertionError

Comments

[Max Payne]

Хорошо, но ещё один вопрос, зачем вы передаёте object классу?

[Stanislav Pugachev]

Max Payne,

ну если питон третий - то формально вы можете не наследоваться от object
если второй то желательно наследоваться дабы обойти new/old style classes проблему
так что у меня это скорее мышечная память, руки сами так пишут
плюс это сохраняет обратную совместимость кода с второй версией питона

[Roman K]

Stanislav Pugachev, вот бы в 2018 году писать обратно-совместимый код для python 2 :)

[Max Payne]

Stanislav Pugachev, Работало, но, как оказалось, не совсем.

self.commands = []

cmd = SomeClass

print(cmd.run_args)
# {"count": 1}

self.commands["гифка"] = type(cmd.__name__, cmd.__bases__, dict(cmd.__dict__))
self.commands["гифка"].name = "гифка"
self.commands["гифка"].run_args.update({"count": 1})
print(cmd.run_args, self.commands["гифка"].run_args)
# {"count": 1} {"count": 1} 
# пока всё идёт по плану

cmd = SomeClass

self.commands["гифки"] = type(cmd.__name__, cmd.__bases__, dict(cmd.__dict__))
self.commands["гифки"].name = "гифки"
self.commands["гифки"].run_args.update({"count": 6})

print(cmd.run_args, self.commands["гифки"].run_args, self.commands["гифка"].run_args)
# {"count": 6} {"count": 6} {"count": 6}

[Max Payne]

Попробовал наследовать

def copy_class(data):
    class Class(data):
        pass

    return Class

Результат тот же. Совсем ничего не понимаю теперь :(

[Stanislav Pugachev]

Max Payne, а почему

type(cmd.__name__, cmd.__bases__, dict(cmd.__dict__))

а не

type(SomeClass.__name__, SomeClass.__bases__, dict(SomeClass.__dict__))

[Max Payne]

Stanislav Pugachev, ну у меня функция возвращает этот класс, она ищет его по имени. Например

command_class_name = "weather.TodayWeather"

cmd = tools.get_attr_recursive(scripts, command_class_name)

Она и возвращает этот SomeClass. Тут разве может быть проблема?

[Stanislav Pugachev]

аааа. я понял
run_args это это ж словарь
поэтому при копировании __dict__
dict(SomeClass.__dict__)
вы получаете чото типа такого
{'__dict__': , '__module__': '__main__', '__weakref__': , 'run_args': {}, '__doc__': None}

где run_args ссылка на тот же словарь предка!
поэтмоу если мы сделаем deepcopy для __dict__ то пролема пропадет

пример 1

class SomeClass:
    run_args = {}

cmd = SomeClass

cmd1 = type(SomeClass.__name__, SomeClass.__bases__, dict(SomeClass.__dict__))
assert cmd != cmd1


cmd.run_args[1] = 2
print(cmd1.run_args)  #  {1: 2}

пример 2

from copy import deepcopy

class SomeClass:
    run_args = {}

cmd = SomeClass

cmd1 = type(SomeClass.__name__, SomeClass.__bases__, deepcopy(SomeClass.__dict__))
assert cmd != cmd1


cmd.run_args[1] = 2
print(cmd1.run_args)  #  {}

то есть классы реально разные у вас
но ссылаются сука на один и тот же словарик внутри
поэтому копировать сам __dict__ оригинала нужно через deepcopy

[Stanislav Pugachev]

Max Payne, не, мне просто интересно было

[Stanislav Pugachev]

Max Payne, ну шо получилось?

[Max Payne]

Stanislav Pugachev, Неа, can't pickle mappingproxy objects.

[Max Payne]

Stanislav Pugachev, Ещё я попробовал
cmd.__dict__.copy()
Но тоже никак.

[Max Payne]

Stanislav Pugachev, copy.deepcopy(dict(cmd.__dict__))
Сработало! Спасибо, очень дикая трабла попалась конечно.