哪个类应该存储查找表？ [关闭]

世界上有不同地点的代理，但任何地点都只有一个代理。每个特工都知道他在哪里，但我还需要快速检查给定位置是否有特工。因此，我还维护了一张从位置到代理的地图。我在决定这张地图属于哪里时遇到问题：class World, class Agent（作为类属性）或其他地方。

下面我放了查找表，agent_locations, in class World。但现在代理必须打电话world.update_agent_location每次他们搬家的时候。这很烦人；如果我稍后决定跟踪代理的其他信息（除了他们的位置）怎么办？我是否需要添加对整个世界对象的回调Agent code?

class World:
  def __init__(self, n_agents):
    # ...
    self.agents = []
    self.agent_locations = {}
    for id in range(n_agents):
      x, y = self.find_location()
      agent = Agent(self,x,y)
      self.agents.append(agent)
      self.agent_locations[x,y] = agent
  def update_agent_location(self, agent, x, y):
    del self.agent_locations[agent.x, agent.y]
    self.agent_locations[x, y] = agent
  def update(self): # next step in the simulation
    for agent in self.agents:
      agent.update() # next step for this agent
  # ...

class Agent:
  def __init__(self, world, x, y):
    self.world = world
    self.x, self.y = x, y
  def move(self, x1, y1):
    self.world.update_agent_location(self, x1, y1)
    self.x, self.y = x1, y1
  def update():
    # find a good location that is not occupied and move there
    for x, y in self.valid_locations():
      if not self.location_is_good(x, y):
        continue
      if self.world.agent_locations[x, y]: # location occupied
        continue
      self.move(x, y)

我可以改为agent_locations in class Agent作为类属性。但这仅在我有一个时才有效World目的。如果我稍后决定实例化多个World对象，查找表需要是世界特定的。

我确信有更好的解决方案...

编辑：我在代码中添加了几行来展示如何agent_locations用来。请注意，它只能从内部使用Agent对象，但我不知道这种情况是否会永远保持下去。

好吧，我想我可以提供我的答案，这可能更多的是一种意见，而不是明确的“这样做”（我没有接受过任何正式的编程培训）。

我觉得你agent_locations应该是每个的成员World实例。

我尝试主要从界面角度进行思考。在我看来，世界级应该负责管理你的世界的资源，在本例中是空间。自从World是空间经理，代理商应该询问他们的world如果空间可用（即未占用），则彼此不存在。因此，我认为你的self.location_is_good更合适的称呼是self.world.is_location_available(x, y) [1]

这使得世界很自然地负责查找给定空间的可用性。世界级还可能有其他变量来决定空间是否可用。如果那里有灌木丛怎么办？或者其他的东西。您可能已经有某种桌子可供您使用(x, y)每个世界的坐标。 “被占用”可以是这些对象的属性。

此外：你的世界已经知道每个代理的状态（通过[(agent.x, agent.y) for agent in self.agents][2]）。这agent_locationsdict 本质上是这些属性的索引或缓​​存，因此属于World.

关于将状态送回的痛苦World...好吧，你不会通过以下方式解决这个问题Agent而是这样做。但做update_agent_location(self, agent, x, y)是完全多余的，因为x == agent.x; y == agent.y（如果你颠倒你调用它的行）。你可以在 World 中简单地使用一种方法，update_agent_state(self, agent)，World 可以使用它来更新其索引。您甚至可以提交一个额外的参数来描述状态更改的类型（如果您不想每次都更新所有属性）。

class World(object):
  # ...
  def update_agent_state(self, agent, state_change=None):
    # Update properties based on what changed, or
    # drop state_change param and update everything everytime
    if state_change == Agent.LOCATION_CHANGE:
      self.agent_locations[agent.x, agent.y] = agent
    elif state_change == Agent.WHATEVER:
      pass

class Agent(object):
  LOCATION_CHANGE = 1

  def update(self):
    for x, y in self.valid_locations():
      if not self.can_move_to(x, y)
        continue

      self.move(x, y)

  def can_move_to(self, x, y):
    """Determines if x, y is a location where we can move."""
    if not self.world.is_location_available(x, y):
      return False
    if not self.has_money_to_travel_to(x, y):
      return False

    return True

  def move(self, x, y):
    """Moves to x, y and notifies world of state change."""
    self.x = x
    self.y = y

    self.world.update_agent_state(self, Agent.LOCATION_CHANGE)

类似的东西（阅读我的脚注）。

[1] 当然，除非空间是自由的，否则位置的“好坏”取决于其他变量。例如。如果您只应搬到 (x, y)，如果 1) 该位置可用并且 2) 代理有 1000 美元支付机票，那么您应该有Agent.can_move_to(x, y)它反过来调用世界的方法，并检查它的钱包。

[2] 我假设你的self.agents = {}是一个错字，因为你不能append在字典上。你的意思是一个列表（[]） 正确的？