如何检查方法transferMoney(User fromUser, User toUser

Question

@Override

public void withdraw(User user, int amount) {

    if (!checkWithdraw(user,amount))

        return;

    user.setBalance(user.getBalance() - amount - amount*user.getBank().getCommission(amount));

}

@Override

public void fund(User user, int amount) {

    if (!checkFund(user, amount))

        return;

    user.setBalance(user.getBalance() + amount);

}

@Override

public void transferMoney(User fromUser, User toUser, int amount) {

    if (!checkWithdraw(fromUser,amount))

        return;

    if (!checkFund(toUser, amount))

        return;

    fromUser.setBalance(fromUser.getBalance()-amount-amount*fromUser.getBank().getCommission(amount));

    toUser.setBalance(toUser.getBalance()+amount);

    **if (fromUser.getBank().getCurrency()==toUser.getBank().getCurrency()) return;**

}

我需要检查用户之间的货币是否不同，如果是，我需要返回 null。我觉得不对，帮我看看

Answer 1

要知道转账是否已完成，我建议返回一个布尔值，然后检查货币并false如果不同则返回。还可以使用您编写的方法来转账，这也是一个好主意@Overridepublic boolean transferMoney(User fromUser, User toUser, int amount) {    if (!checkWithdraw(fromUser, amount)) return false;    if (!checkFund(toUser, amount)) return false;    if (fromUser.getBank().getCurrency() != toUser.getBank().getCurrency()) return false;    withdraw(fromUser, amount);    fund(toUser, amount);    return true;}但是，随着方法withdraw和fund检查的进行，会调用冗余代码，您可以这样做：@Overridepublic boolean withdraw(User user, int amount) {    if (!checkWithdraw(user, amount)) return false;    user.setBalance(user.getBalance() - amount - amount * user.getBank().getCommission(amount));    return true;}@Overridepublic boolean fund(User user, int amount) {    if (!checkFund(user, amount)) return false;    user.setBalance(user.getBalance() + amount);    return true;}@Overridepublic boolean transferMoney(User fromUser, User toUser, int amount) {    if (fromUser.getBank().getCurrency() != toUser.getBank().getCurrency()) return false;    return withdraw(fromUser, amount) && fund(toUser, amount);}解释：你会打电话给withdraw如果有效，它将返回true，并且逻辑表达式的第二部分将被测试并执行如果它不起作用（checkWithdraw不好），它将返回false，并且逻辑表达式的第二部分将不会被执行，并且您将不会执行fund其他帐户如果withdraw返回，true它将调用该fund方法，并且全局将返回fund返回 true ：true && true所以返回truefund返回 false：true && false所以返回false