1

我正在尝试执行一个对象的深层副本，然后在不更改原始对象的情况下修改新对象。为了测试功能，我使用 JUnit 来确保功能正常工作。

目前的测试基本上是抄袭《质量效应3》游戏并更改当前配音演员。

游戏课

@Override
protected Object clone(){
    Game obj = new Game(getTitle(), getLeadVoiceActor(), getRating());
    return obj;
}

JUnit 测试

@Before
public void setUp() {
    p1 = new Person("Mark", "Meer");
    g1 = new Game("Mass Effect 3", p1, 5);
}

@Test
public void testClone() throws CloneNotSupportedException {
    //This works
    Game g2 = (Game)g1.clone();
    assertEquals(g2, g1);
    assertNotSame(g2, g1);

    //This doesn't even though the lead voice actor is now different
    p1 = g1.getLeadVoiceActor();
    p1.setFirstName("Jennifer");
    p1.setLastName("Hale");
    assertFalse(g2.equals(g1));
    assertEquals("Jennifer Hale", g1.getLeadVoiceActor().toString());
    assertEquals("Mark Meer", g2.getLeadVoiceActor().toString());
}

测试因 junit.framework.assertionfailederror 而失败。

4

1 回答 1

3

如果你的问题是：为什么原版游戏中的人和克隆游戏中的人是一样的？，那么答案就是你不是深克隆而是浅克隆。

调用getLeadVoiceActor()并将结果分配给克隆游戏会将对该人的引用传递给克隆游戏。它不会复制任何内容。要制作副本，您需要

protected Object clone(){
    Game obj = new Game(getTitle(), (Person) getLeadVoiceActor().clone(), getRating());
    return obj;
}

或者

protected Object clone(){
    Game obj = new Game(getTitle(), 
                        new Person(getLeadVoiceActor().getFirstName(), getLeadVoiceActor().getLastName()), 
                        getRating());
    return obj;
}

请注意，clone()不鼓励使用 复制构造函数。如果你实现克隆，那么你应该使用super.clone().

另请注意，如果 Person 是不可变的，则不必克隆它。

于 2013-01-07T19:53:30.893 回答