python - 断言错误: This AttributeImpl is not configured to track parents

Question

我最近遇到了这个问题，并且已经被困了几天了。基本上，这里感兴趣的两个表是“Activity”和“Goal”，其中“Activity”是“Goal”的单父表；一个目标只有一个父级，而一个事件有一个或多个目标子级:

class Activity(Base, Table):
    __tablename__ = 'activity'

    pk_id = Column(Integer, primary_key=True)
    name = Column(String)
    wave = Column(Integer)
    enabled = Column(Boolean)

    goals = relationship("Goal", backref='activity', single_parent=True, cascade="all,delete-orphan")
    activity_tags = relationship("ActivityTags", backref='tag_activity', cascade="all,delete-orphan")

和

class Goal(Base, Table):
    __tablename__ = 'goal'

    pk_id                 = Column(Integer, primary_key=True)
    activity_id           = Column(Integer, ForeignKey('activity.pk_id'))
    category_id           = Column(Integer, ForeignKey('category.pk_id'))

    name                  = Column(String)
    minutes_expected      = Column(Integer)
    date_last_invoked     = Column(Date)
    date_last_invalidated = Column(Date)
    enabled               = Column(Boolean)

    milestones = relationship("Milestone", backref='goal', single_parent=True, cascade="all,delete-orphan")

现在，当我创建一个 Activity 对象(该对象又创建 1 个 Goal 子对象)时，我遇到了以下错误:

Traceback (most recent call last):
  File "unit.py", line 49, in wrapper
    r = func(*args, **kwargs)
  File "unit.py", line 158, in testActivityWaveOverdue
    a = Activity(c, 'Pull-Ups', wave)
  File "<string>", line 4, in __init__
  File "/home/me/dev/coach/.venv/lib/python2.7/site-packages/sqlalchemy/orm/state.py", line 98, in initialize_instance
    return manager.original_init(*mixed[1:], **kwargs)
  File "/home/me/dev/coach/coach/activity.py", line 64, in __init__
    self.add_goal(category, 'maintenance')
  File "/home/me/dev/coach/coach/activity.py", line 190, in add_goal
    return Goal(self, category, name, minutes_expected)
  File "<string>", line 4, in __init__
  File "/home/me/dev/coach/.venv/lib/python2.7/site-packages/sqlalchemy/orm/state.py", line 98, in initialize_instance
    return manager.original_init(*mixed[1:], **kwargs)
  File "/home/me/dev/coach/coach/goal.py", line 76, in __init__
    db.session.commit()
  File "/home/me/dev/coach/.venv/lib/python2.7/site-packages/sqlalchemy/orm/session.py", line 656, in commit
    self.transaction.commit()
  File "/home/me/dev/coach/.venv/lib/python2.7/site-packages/sqlalchemy/orm/session.py", line 314, in commit
    self._prepare_impl()
  File "/home/me/dev/coach/.venv/lib/python2.7/site-packages/sqlalchemy/orm/session.py", line 298, in _prepare_impl
    self.session.flush()
  File "/home/me/dev/coach/.venv/lib/python2.7/site-packages/sqlalchemy/orm/session.py", line 1583, in flush
    self._flush(objects)
  File "/home/me/dev/coach/.venv/lib/python2.7/site-packages/sqlalchemy/orm/session.py", line 1636, in _flush
    is_orphan = _state_mapper(state)._is_orphan(state) and state.has_identity
  File "/home/me/dev/coach/.venv/lib/python2.7/site-packages/sqlalchemy/orm/mapper.py", line 1237, in _is_orphan
    state, key, optimistic=bool(state.key)):
  File "/home/me/dev/coach/.venv/lib/python2.7/site-packages/sqlalchemy/orm/instrumentation.py", line 331, in has_parent
    return self.get_impl(key).hasparent(state, optimistic=optimistic)
  File "/home/me/dev/coach/.venv/lib/python2.7/site-packages/sqlalchemy/orm/attributes.py", line 349, in hasparent
    assert self.trackparent, "This AttributeImpl is not configured to track parents."
AssertionError: This AttributeImpl is not configured to track parents.

这是 Activity 初始化代码:

def __init__(self, category, name, wave):
    self.name = name
    self.wave = wave
    self.enabled = True
    db.session.add(self)
    db.session.commit()

    self.add_goal(category, 'maintenance')

def add_goal(self, category, name, minutes_expected=30):
    return Goal(self, category, name, minutes_expected)

目标:

def __init__(self, activity, category, name, minutes_expected):
    self.activity_id            = activity.pk_id
    self.category_id            = category.pk_id
    self.name                   = name
    self.minutes_expected       = minutes_expected

    self.date_last_invoked      = START_OF_TIME
    self.date_last_invalidated  = START_OF_TIME
    self.enabled                = True

    db.session.add(self)
    db.session.commit()

如果您希望我在上面包含任何其他内容，请告诉我 - 我不想让帖子变得比需要的更大，所以只包含我认为与问题相关的内容.

最后，Activity 和 Goal 继承的“Table”父类(super class)仅提供辅助/帮助方法。

Answer 1

这里有一些非常规的用法，并且考虑到 sqlalchemy 的 ORM 已经为您做了什么，并且您的 _init_ 中存在一些不必要的循环，并且这可能会导致问题。

ORM 已经能够将您的 Activity 和 Goal 实例逻辑链接在一起，而无需您在 __init__ 方法中强制执行此操作。

尝试以下操作:

1. 不要让Goal.__init__将事件作为参数。顺便说一句，您甚至不必为使用 sqlalchemy 声明性基础的模型类编写 __init__ 方法；你可以，但只有当你确实需要完成一些额外的事情时才有必要。那么，您的新 Goal 实例将如何链接到它的一个 Activity 实例呢？ Activity.goals 关系及其 backref 会为您处理这些问题。

2. 顺便说一句，由于每个目标仅与单个事件关联，因此您应该更改 backref 的形式。您当前的 backref='activity' 将生成与每个目标链接的事件集合(在本例中，默认情况下为列表)。根据您的模型描述，您想要的是 backref=backref('activity', uselist=False)。这将导致对 somegoal.activity 的引用返回单个对象(用 sqlalchemy 术语来说，是“标量”)，而不是列表。

3. 不要让 Activity.__init__ 添加目标；只需在调用 Activity() 来创建事件实例的代码部分中执行此操作即可。一般来说，设置像这里这样的初始化链是不必要的复杂，其中在内部实例化一个事物会导致另一个事物的实例化。最好将该事件提升一个级别，并将其放入想要执行这些操作的主程序代码中，并让每个实例化都用自己的代码行来明确。比将其隐藏在 __init__ 内部要好得多。这也更Pythonic(显式优于隐式，少用“魔法”)。

4. 请注意，在您的 Activity.add_goal() 方法中，返回值实际上并未完成。不要编写返回值被丢弃的方法，或者仅为了产生副作用而调用其他方法(在本例中为构造函数)的方法。上面的第 1 点和第 3 点可以概括为“不要编写有副作用的构造函数”。

我的观点不是教条主义，而是教条主义。我希望您能看到，这些建议中的每一个都是为了创建更清晰的代码，以更少的复杂性完成所需的事情，并且更容易阅读、遵循和调试。

至于对错误回溯中发生的情况进行实际分析:我们需要一个实际的工作代码示例才能真正做到这一点，但我的猜测是 sqlalchemy 会根据您的请求创建您的 Activity 对象，并尝试遵循 Activity 类映射器的 relationship 创建从新事件到目标的前向引用，以及从目标到事件的反向引用。但一路上，你试图让 __init__ 来做一些工作的方式让你感到困惑。我认为如果你稍微理清并简化一下，你很可能会得到你想要的。

最后说明:如果您还没有逐节阅读 sqlalchemy 的 ORM 教程并直接遵循其中的每一个部分，强烈建议您这样做。一路上(并遵循旁边的一些背景引用资料)，您会发现一些示例，这些示例显示了您想要在此处建模的情况，以及其他非常相似的情况，并且您会看到最常见的(也是最简单/最清晰的)/最简单)设置​​它们的方法。