双向引用对象（如父子关系）的setter方法设置有哪些最佳实践？

你当前的实现存在无限递归的核心问题：调用Parent.setChild()时会触发Child.setParent()，后者又会反向调用Parent.setChild()，会直接导致栈溢出，无法正常运行。

• 确定唯一控制方：只能选择其中一方作为关联关系的管理者，另一方的关联赋值方法不对外部暴露，也不要反向触发控制方的setter，从根源避免递归问题。通常选择聚合根（比如Parent作为包含Child的根对象）作为控制方。
• 优先使用不可变关联：如果业务允许父子关系绑定后不修改，直接在构造方法中完成双向关联，不对外暴露setter，是成本最低、一致性最高的方案。
• 先解旧关联再建新关联：更新关联时先清空原有双向绑定，再设置新的关联，避免出现旧对象残留引用的问题。
• 重复赋值提前拦截：setter入口先判断新值和旧值是否相同，相同则直接返回，避免无意义的关联更新操作。
• 运行时校验不要依赖assert：assert默认在生产环境会被关闭，仅适合开发测试阶段做一致性检查，不能作为业务逻辑的校验手段。如果要做运行时一致性保障，可以单独提供checkConsistency()方法主动校验，或者在非控制方的setter中抛出异常禁止直接调用。

选择Parent作为关联控制方，Child的内部关联赋值方法仅对Parent开放：

class Parent {
    private Child child;

    public Child getChild() { return child; } 

    public void setChild(Child newChild) { 
        // 相同引用直接返回，避免重复操作
        if (this.child == newChild) {
            return;
        }
        // 解除和旧Child的双向关联
        if (this.child != null) {
            this.child.setParentInternal(null);
        }
        // 绑定新的双向关联
        this.child = newChild;
        if (newChild != null) {
            newChild.setParentInternal(this);
        }
    }
}

class Child {
    private Parent parent;

    public Parent getParent() { return parent; } 

    // 包私有方法，仅允许Parent调用，不对外暴露
    void setParentInternal(Parent parent) {
        this.parent = parent;
    }

    // 若业务需要对外暴露Child.setParent，转发给Parent的控制逻辑即可
    public void setParent(Parent newParent) {
        if (this.parent == newParent) {
            return;
        }
        // 先解除旧关联
        if (this.parent != null) {
            this.parent.setChild(null);
        }
        // 建立新关联
        if (newParent != null) {
            newParent.setChild(this);
        }
    }
}

• 父类setter调用子类的关联赋值方法不是冗余设计，是保证双向引用一致性的必要操作，只要做好唯一控制方的划分，不会出现迂回问题。
• assert仅适合开发阶段做一致性校验，比如你可以在Child的getParent()方法中加入assert parent == null || parent.getChild() == this;，开发阶段快速定位关联不一致的问题，但不能替代正常的关联维护逻辑。

内容的提问来源于stack exchange，提问作者1288Meow