具有多个init方法的Python子类?
我在Python2.7中构建了一个应用程序,它按原样工作,但感觉不干净,在发生的事情中也不是很明确,因此如果我离开代码一段时间,我很难记住它实际上是如何工作的,这显然是不好的。我对代码进行了重构,代码看起来更加明确,但实际上并没有更清晰 我试图找出用两种不同的方式初始化这些类的最干净的方法——1)从用户生成的实例化(在程序执行期间从头开始添加新对象的情况下)或2)从JSON导入对象的历史记录(从以前的程序执行)。以下是我的最新做法:具有多个init方法的Python子类?,python,json,python-2.7,oop,subclass,Python,Json,Python 2.7,Oop,Subclass,我在Python2.7中构建了一个应用程序,它按原样工作,但感觉不干净,在发生的事情中也不是很明确,因此如果我离开代码一段时间,我很难记住它实际上是如何工作的,这显然是不好的。我对代码进行了重构,代码看起来更加明确,但实际上并没有更清晰 我试图找出用两种不同的方式初始化这些类的最干净的方法——1)从用户生成的实例化(在程序执行期间从头开始添加新对象的情况下)或2)从JSON导入对象的历史记录(从以前的程序执行)。以下是我的最新做法: class Device(object): def _
class Device(object):
def __init__(self, dev_type, preset_prefix, default_preset,
from_json=False, json_path=None, **device_attrs):
if not from_json: # otherwise set in child class __init__
self.name = device_attrs['name']
self.sn = device_attrs['sn']
self.mfg = device_attrs['mfg']
self.tech = device_attrs['tech']
self.model = device_attrs['model']
self.sw_ver = device_attrs['sw_ver']
self.hours = 0
else:
self.hours = device_attrs['hours']
self.type = dev_type
self.json = json_path
self.preset_prefix = preset_prefix
self.preset = default_preset
class Monitor(Device):
def __init__(self, name, sn, mfg, tech, model, sw_ver, from_json=False,
json_path=None, **monitor_dict):
if from_json:
self.__dict__ = monitor_dict
device_properties = {'name': name, 'sn': sn, 'mfg': mfg, 'tech': tech,
'model': model, 'sw_ver': sw_ver}
monitor_dict.update(device_properties)
super(Monitor, self).__init__('monitor', 'user', 1, from_json,
json_path, **monitor_dict)
if cals:
self._init_cal_from_json(monitor_dict['cals'])
现在,我可以从以前保存的JSON(从该对象生成)进行初始化,以便确保键/值对正确无误:
或作为从头开始的新对象:
my_monitor = Monitor('monitor01', '12345', 'HP', 'LCD',
'HP-27', 'v1.0')
这看起来有点混乱,但仍然比我的原始版本要好,因为我的原始版本没有任何子init的位置参数(这使得很难知道必须传入哪些数据),它只需要**monitor\u dict,希望它包含正确的键/值对。然而,这种将这些参数合并成一个dict的方法似乎很奇怪,但我已经重构了很多次,这似乎是最干净的方法
这是以多种方式初始化对象的最佳方法,还是我可以创建两个独立的init函数,一个用于从JSON加载,另一个用于新建全新对象?我更喜欢创建新的构造函数作为类方法,类似于这样,如果需要,您可以创建更多,或在必要时进行调整:
class Device(object):
def __init__(self, dev_type, preset_prefix, default_preset, json_path=None, **device_attrs):
self.name = device_attrs['name']
self.sn = device_attrs['sn']
self.mfg = device_attrs['mfg']
self.tech = device_attrs['tech']
self.model = device_attrs['model']
self.sw_ver = device_attrs['sw_ver']
self.hours = 0
self.type = dev_type
self.json = json_path
self.preset_prefix = preset_prefix
self.preset = default_preset
class Monitor(Device):
@classmethod
def new_from_json(self, name, sn, mfg, tech, model, sw_ver, json_path=None, **monitor_dict):
self.__dict__ = monitor_dict
device_properties = {'name': name, 'sn': sn, 'mfg': mfg, 'tech': tech,
'model': model, 'sw_ver': sw_ver}
monitor_dict.update(device_properties)
super(Monitor, self).__init__('monitor', 'user', 1,
json_path, **monitor_dict)
例如:
class Parent():
def __init__(self,some):
self.some = some
class Object(Parent):
@classmethod
def new_from_dict(self,some):
Parent.__init__(self,some)
self.adress = {"Me": 123}
return self
然后:
具有额外的构造函数方法(使用
@classmethod
decorator),如中所示,是一种非常典型的Python习惯用法。标准库中有许多示例(我想到的是dict.fromkeys()
方法)。使用\uuuu init\uuuu
作为最常用或最期望的构造函数,并根据需要创建其他构造函数。因此,我建议添加一个来自_json的
方法。嗨@ice,我已经检查了编辑过的问题,如果你可以提出建议,你可能应该问另一个问题,关于新问题,我的意思是,我的回答只解决你的主要和原始问题(对于python 3,我的错误),如果你问一个新问题,关于新问题(还有这个问题的链接)你会成功的,我可以向我帮助的其他人保证你会成功的。如果你问了一个新问题,请告诉我所有的细节:),我希望我能帮助更多谢谢,我以前读过关于类方法和工厂的文章,但我并没有把它们放在一起讨论如何在这里应用。在你的回答之后再多读一点,我认为这是一个干净的、合乎逻辑的解决方案。我已经更新了我的问题,加入了一个修订版。实际上,我正在尝试一些更接近你在这里为我所拥有的另一种类型的类所做的事情,而我的修订版代码并没有真正应用或工作。我无法理解的问题是——在您提供的代码中,new_from_json()中的自指是什么?在代码中的这一点上,没有任何东西被初始化,因此self没有引用任何特定的对象,self.\uu dict\uuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuu?我遗漏了什么吗?@iceblueorbitz我将在最后为您添加一个示例,这样您可能会看到它做得更好这仍然会导致相同的问题:TypeError:unbound method\uuuuu init\uuuuu()必须以父实例作为第一个参数进行调用(取而代之的是type instance)
。当使用类方法装饰器时,PyCharm建议使用cls而不是self,然后使用cls(一些)初始化,但这将调用子类“Object”init,在本例中它不存在。@iceblueorbitz尝试将类更改为super,我在python 3.6中这样做了,并且工作正常。这是一个奇怪的错误。。
class Parent():
def __init__(self,some):
self.some = some
class Object(Parent):
@classmethod
def new_from_dict(self,some):
Parent.__init__(self,some)
self.adress = {"Me": 123}
return self
obj = Object.new_from_dict("ME")
obj.adress
{"Me": 123}
obj.some
"ME"