我有一个我在Python2.7中构建的应用程序按原样工作,但是感觉不干净,也不是很清楚发生了什么,所以如果我离开代码一段时间我有很难记住它是如何在引擎盖下工作的,这显然不是很好。我重构了代码,看起来更明确,但不是更清晰。
我试图找出以两种不同方式初始化这些类的最简洁方法 - 1)来自用户生成的实例化(如果在程序执行期间从头开始添加新对象),或者2)从JSON导入对象的历史(来自先前的程序执行)。这是我最新的解决方法:
_.find()
现在我可以从之前保存的JSON初始化(从这个对象生成,所以我可以确定键/值对是正确的):
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'])
或者从头开始作为新对象:
my_monitor = Monitor(from_json=True, json_path=device_json_file, **device_json_dict))
这看起来有点混乱,但仍然比我原来的版本更好,因为孩子没有任何位置参数 init (很难知道必须传递什么数据),它只需要** monitor_dict,希望它包含正确的键/值对。然而,这种获取这些参数并将它们合并成一个字典的方法看起来很奇怪,但我已经多次重构,这似乎是最简洁的方法。
这是处理以多种方式初始化对象的最佳方式,还是可以以某种方式创建两个单独的 init 函数,一个用于从JSON加载,另一个用于新创建全新对象? / p>
答案 0 :(得分:1)
我更喜欢创建新的构造函数作为类方法,如果需要,可以根据需要创建更多,或者在需要时调整它:
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
然后:
obj = Object.new_from_dict("ME")
obj.adress
{"Me": 123}
obj.some
"ME"