修改克隆对象

Question

我正在尝试修改帐户对象，但更改似乎随后没有出现在原始列表中。也许有人可以指出错误。

请参阅下面的代码：

if(aBank.getAccount(number)!=null){
                        System.out.println("Account information is listed below");
                        System.out.println(aBank.getAccount(number).toString());
                        System.out.println("Modify first name y or n");
                         answer=keyboard.nextLine();
                            if(answer.equals("Y")||answer.equals("y")){
                                System.out.println("Enter first name:");
                                firstName=keyboard.nextLine();
                                aBank.getAccount(number).getCustomer().setFirstName(firstName);

                            }
                        System.out.println("Modify last name y or n");
                        answer=keyboard.nextLine();
                            if(answer.equals("Y")|| answer.equals("y")){
                                System.out.println("Enter last name:");
                                lastName=keyboard.nextLine();
                                aBank.getAccount(number).getCustomer().setLastName(lastName);
                            }

                    }

                else{
                    System.out.println("Account not found");
                }

注意：getAccount(number) 返回帐户的克隆，该克隆是深层复制，getCustomer 也返回一个克隆，该克隆是深层复制

getAccount 的内容

public Account getAccount(long accountNumber ) throws Exception { 
    boolean found=false; 
    for(int i=0;i<accounts.size();i++){ 
        if(accounts.get(i).getAccountNumber().compareTo(accountNumber)==0){ 
            found=true; 
            return accounts.get(i).clone(); 
        } 
    } 
    if (!found){ 
        return null; 
    } 
    return null; 
} 

Answer 1

简单地调用clone()将不会返回对象的深层副本。它将返回一个浅副本。覆盖克隆是很棘手的。遵循 Effective Java 中 Joshua Bloch 的建议，避免使用 clone( ) 支持复制构造函数。

private Account(Account account) { 
   this.name = account.getName();
   //etc
}

public void createCopy(Account account) { 
    return new Account(account);
}

另外，为什么不将 Accounts 集合存储在 Map 中，这样在复制之前就不需要遍历 N 个 Accounts 了？您还需要仔细阅读布莱恩的回答。

Answer 2

在这种情况下，您应该获取帐户的副本（通过 getAccount），对其进行修改，然后将其重新插入列表。

正如您所指出的，您正在修改副本。该副本本身并未连接到您的集合中，因此当您退出范围时您将丢失它。

修改副本并重新插入它是否是最佳解决方案是另一回事。您可能想要就地修改，但这会让您面临各种问题（例如，线程 - 如果您修改帐户的过程中，另一个客户端遍历列表并读取帐户详细信息，会发生什么？）

Answer 3

如果 getAccount() 返回您想要稍后修改的深层克隆，则应将其存储在变量中。如果不这样做，每次调用 getAccount() 时，您都会获得一个新对象。