这是一个组合关系的例子吗？

Question

这是一个组合关系的例子吗？

3

class Polygon {
    private final LinkedHashSet <Line> polygon;
    private int objID;
    private static int classID;
    Iterator <Line> it;

    public Polygon(LinkedHashSet<Line> polygon) {
        this.polygon = new LinkedHashSet<Line>();
        for(Line l: polygon)
            this.polygon.add (l);
        objID = ++ classID;
        it = this.polygon.iterator();
    }
}


class Line {
    Point start, end;

    public Line(Point a, Point b){
        start = a;
        end = b;
    }
}

在我的理解中，当我们在整体中构建“部分”对象时，关系是“组合”，这导致部分随整体一起被销毁。

在这种情况下，由于我们在构造函数中将新数据分配到多边形LinkedHashSet上并深度复制线对象，因此可以安全地假设这是多边形（作为整体）和线（作为部分）之间的“组合”关系吗？在这种情况下，当对象被销毁时，变量“polygon”中的数据肯定会被销毁。

- jdecer

1

这不是完全的深拷贝。你是逐行复制，但每一行都是一个对象，所以两个哈希集合中使用的是同一个对象。要进行深拷贝，你需要clone()这些行，即构造行的副本。 - Christophe

1个回答

网页内容由stack overflow 提供, 点击上面的

可以查看英文原文，
原文链接

- Christophe · Accepted Answer

组合聚合，更准确地说，是指对所组成的对象的生命周期具有独占的所有权和控制。

您最初的代码

由于 Java 的引用赋值原则，Polygon 的 Line 元素可能会在多个多边形之间共享。也许它们不是这样的，但是独占所有权并不能得到保证。

因此，在您的 UML 图中可以考虑共享聚合，它通常被理解为没有排他性要求的部分整体聚合。但当前的 UML 规范不再定义共享聚合的语义。因此，实际上，我建议使用简单关联，具有明确的语义。

您编辑后的问题

如果进行深拷贝，包括克隆每个输入的 Line，则可以合理地保证该线段未从其他位置重复使用：

class Polygon {
    ...
    public Polygon(Collection<Line> polygon) {
        this.polygon = new LinkedHashSet<Line>();
        for(Line l: polygon)
            this.polygon.add (l.clone());
        objID = ++ classID;
        it = this.polygon.iterator();
    }
}
class Line {
    ...
    public Line(Point a, Point b){
        ...
    }
    public Line clone() {
        return (new Line(start, end)); 
    }
}

你还应该通过确保从多边形中仅返回克隆对象而不是任何方法的线条来防止泄漏多边形的线条。在这种情况下，复合聚合将是正确的。

无关: 你的构造函数似乎泄漏了一些实现细节。是否有使用LinkedHashSet<Line>参数而不是更通用的Collection<Line>的原因？