2

这是情况。我有以下伪类：

class Agent:
    def __init__(self):
       self.blah = whatever
       self.boo = thingy

    def doA(self, transport):
       res = transport.doX(self.blah)

    def doB(self, transport):
       res = transport.doY(self.boo)

class Transport:
    def __init__(self):
        self.bah = weee

    def doX(self, item):
        #.. do some other stuff here

    def doY(self, item):
        #.. do some more stuff here

现在，我的问题是 - 我应该将我的传输对象传递给需要它的方法的代理方法，还是应该简单地将传输对象传递给代理构造函数并从方法中调用它？IE

transport = Transport()
agent = Agent(transport)
class Agent:
    def __init__(self, transport):
        self.blah = whatever
        self.transport = transport

    def doA(self):
        res = self.transport.doX(self.blah)

我将 Transport 移到它自己的班级的原因是：

1) 代理实例可以共享相同的传输对象 - 即：我只需要一个传输对象用于一组代理。我可以有多个传输（一个用于agentA..N，另一个用于agentO..Z）。

2) 传输包含将代理列表作为参数的方法。例如，给定代理对象列表，它可以对代理运行并发通信测试。

但是，代理本身有时需要在传输中设置的属性（SSL 证书的位置等），以便在代理类中有意义的一些方法（远程重新启动代理等）。

我是否应该采用需要代理列表的方法并将它们移动为 Transport 的类方法，然后在包含 Transport 实例的 Agent 对象中创建一个属性？我是否应该继续我正在做的事情，只需要将 Transport 对象传递给某些 Agent 对象方法？我应该以我没有看到的某种方式重构这一切吗？

任何有用的意见将不胜感激。

4

2 回答 2

3

由于您甚至考虑将传输作为参数添加到代理的构造函数，因此我假设一个代理永远不会与多个传输一起使用。

我假设一个传输实例可以在没有任何代理连接的情况下做一些有用的事情，反之亦然。

如果我的假设是正确的，我会提出这个建议：

class System:
  def associate_transport_and_agent(transport, *agents):
    for agent in agents:
      transport.add_agents(*agents)
      agent.set_transport(transport)

class Transport:
  def __init__(self):
    self.bah = weee
    self.agents = set()
  def add_agents(self, *agents):
    self.agents.update(agents)
  def remove_agent(self, agent):
    self.agents.remove(agent)
  def doX(self, item):
    # ...


class Agent:
  def __init__(self):
    self.blah = whatever
    self.transport = None
  def set_transport(self, transport):
    if self.transport == transport:
      return
    if self.transport is not None:
      self.transport.remove_agent(self)
    self.transport = transport

  def doA(self):
    res = self.transport.doX(self.blah)

如果您打算删除对象，您可能需要通过使用从任一代理到传输或另一个方向的弱引用来帮助垃圾收集器（或者实际上，在您的情况下，在两个方向上，因为您可能在其他地方包含对它们的强引用）。

编辑：

更新以反映代理可能会更改其传输：

  • 将代理列表更改为代理集
  • 添加了 Transport.remove_agent
  • 修改 Agent.set_transport 以检查之前是否已设置
于 2012-04-06T01:25:29.143 回答
0

鉴于“代理只能使用 1 个传输，尽管可以切换该传输”，我建议您使用（可选）参数来__init__设置传输，并将代理的传输属性设为公共。如果您需要在分配时运行代码，请创建一个属性。

于 2012-04-06T16:39:31.400 回答