如何检查方法中的多个条件，并根据其返回值执行操作

Question

我目前正在尝试一个项目，包括创建银行账户系统。该系统允许您创建一个新的银行账户，您可以在其中说明该账户的归属、余额等。

其中一种方法是能够从账户中提取资金。除此之外，您还可以检查帐户是否透支。

对于撤回方法，我必须满足几个条件。这些是以下：

• 该值必须高于0.0，这意味着低于0的提款不会发生。
• 余额不得低于0.0，除非账户透支

目前，我已经能够做到这一点，所以如果金额低于0.0，该方法将返回false。如下所示：

public boolean withdraw (double amount) {
    if (hasOverdraft) {
        this.balance -= amount;
        return true;
    } else if (amount > 0.0) {
        this.balance -= amount;
        return true;
    } else {
        return false;
    }
}

但是，我已经尝试了几种不同的方法来使其与透支一起使用，但似乎无法做到。

请记住，最后，该方法需要能够：

< li >如果金额大于0.0，则允许用户取款 < li >允许用户透支(例如-100)，前提是他们确实有透支。

透支可以通过以下代码进行检查：

if (hasOverdraft)

为了检查代码是否工作，我们使用以下代码：

public static void main (String[] args) {
    BankAccount lewis = new BankAccount("1234", "Lewis", false);
    System.out.println(lewis);

    lewis.setBalance(200);
    System.out.println(lewis);
    lewis.withdraw(250);
    System.out.println(lewis);
}

BankAccount的“false”显示该账户没有透支。我们将使用false和true对其进行测试。

Answer 1

软件中如此多的复杂性来自于试图让一件事做两件事。-Ryan Singer

您的方法主要做两件事:

• 检查您是否可以从账户中提取资金（先决条件），这是允许透支的情况，否则余额不会变成负数。提取的金额也应该是正的，并且可能不是零（参见@forpas的评论）-这可以是先决条件的一部分，或者只是之前的参数验证步骤。
• 在可以的时候实际取钱（行动）。

分离这两个问题会使代码更加简洁，避免复制实际操作，并允许将这两个内容提取到单独的可重用方法中（如果以后需要）：

public boolean withdraw (double amount) {
    boolean canWithdraw = amount> 0 && (hasOverdraft || balance >= amount);

    if (canWithdraw ) {
        balance -= amount;
    }
    return canWithdraw;
}

正如这个问题的评论者已经指出的那样，你在先决条件中犯了一个错误（撤回金额后检查金额而不是余额）。