Question

我正在尝试创建代码，该代码将更改多个帐户引用的客户对象的firstname和lastname字段。即每个客户可以拥有更多的帐户。但是，似乎只为一个帐户更改了名称，并且更改未显示在连接到同一客户的其他帐户中。也许有人可以查明错误。

请参阅以下代码：

摘自主要方法

System.out.println("Enter the number of the account that you would like to modify:");
            number=keyboard.nextLong();
            keyboard.nextLine();
            firstName=null;
            lastName=null;

            try{
                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();


                            }
                        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.changeName(number,firstName,lastName);

                    }

                else{
                    System.out.println("Account not found");
                }
            }
            catch(Exception e){
                System.out.println("Unable to process request.\n" + e.getMessage());
            }

适用的银行类方法：

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();
        }



        }

public void changeName(Long accountNumber, String firstName, String lastName) throws Exception{


    if (getAccount(accountNumber)!=null){
        accounts.get(accounts.indexOf(getAccount(accountNumber))).getCustomer().modifyName(firstName, lastName);
    }
    else{
        throw new Exception("Account not found");

    }

适用的帐户类方法

private Account (Account a){
    //copy constructor
    this.accountNumber=a.accountNumber;
    this.startBalance=a.startBalance;
    this.customer=a.customer;
    this.trans=a.trans;

}


public Customer getCustomer() {

    return this.customer.clone();

}
public void modifyName(String firstName, String lastName){
    if(firstName!=null){
        customer.setFirstName(firstName);
    }

    if(lastName!=null){
        customer.setLastName(lastName);
    }
}

适用的客户类方法

private Customer(Customer c){
    //copy constructor
    this.customerNumber=c.customerNumber;
    this.socialSecurityNo=c.socialSecurityNo;
    this.firstName=c.firstName;
    this.lastName=c.lastName;
}

Answer 1

看起来您的代码根本不起作用，因为您克隆了Customer对象，然后修改了克隆。账目也是如此。

一些简化可能有助于调试。大部分代码可以使用标准集合进行简化，而不是迭代和使用compareTo。

逻辑也有点奇怪 - 如果任务是修改客户详细信息，那么为什么要从帐号开始然后转到客户？

if (getAccount(accountNumber)!=null){
        accounts.get(accounts.indexOf(getAccount(accountNumber))).getCustomer().modifyName(firstName, lastName);
    }
    else{
        throw new Exception("Account not found");
    }

可以简化为：

getAccount(accountNumber).getCustomer().setName(firstName, lastName);

如果找不到该项，则getAccount和getCustomer会抛出异常。

getAccount方法可以简化为：

public Account getAccount(long accountNumber )  {
        return accounts.get(accountNumber )
    }

（如果帐户是Map<Long,Account>

，则几乎不值得使用方法！）

Answer 2

由于您在帐户的getter中使用.clone而主要是客户，因此您只能获得客户的副本并更改其名称。由于String是一个不可变对象，因此您只能替换副本中的名称。

修改对象的字段

2 个答案: