Исключение броска в Java

Question

У меня возникла проблема, когда я прошу пользователя высвободить сумму из своего баланса. У меня есть метод, называемый «убрать», и я передаю их баланс. Затем я хочу проверить, меньше ли сумма, которую они хотят снять, меньше их баланса. Если да, я хотел бы заставить пользователя повторить попытку.

Пока он проверяет ввод, но я продолжаю получать выходные данные для каждой попытки.

public void withdraw (double balance) 
{ 
    System.out.println("How much would you like to withdraw?"); 
    double amount = keyboard.nextDouble(); 

    try 
    { 
     if(amount > balance) 
     { 
      throw new IncorrectWithdrawException(); 
     } 
    } 
    catch(IncorrectWithdrawException e) 
    { 
     System.out.println(e.getMessage()); 
     withdraw(balance);// keeps calling the method for a loop if they keep entering incorrect amount 
    } 
    balance = balance-amount; 
    System.out.println("You have withdrawn "+amount+ " and your new balance is " +balance); 
} 

Выход:

Что такое баланс? 100 Сколько вы хотите снять? 200 ------ ОШИБКА ------ Это не действительная сумма для снятия средств. Сколько вы хотели бы снять? 500 ------ ОШИБКА ------ Это не действительная сумма для снятия. Сколько вы хотели бы снять? 50

Вы отозвали 50,0 и ваш новый баланс 50,0 Вы отозвали 500,0 и ваш новый баланс -400,0 Вы отозвали 200,0 и ваш новый баланс -100,0

Я не хочу, последние два выходы ...

Answer 1

когда сумма> следующий входной баланс, вы перешли пользователя как новый баланс, который очень проблематично ... пользователь может обмануть баланс

Answer 2

Одно важное замечание:

вы должны проверить свой баланс, чтобы убедиться, что он больше или равен 0.

(Предположим, что вы не позволите отрицательный б alance случается.)

Answer 3

Не используйте исключения внутри методов как скрытый способ ниндзя, если/или условный код: это не то, для чего они предназначены. Исключения являются последней защитой от ненормальных условий, на которые нельзя иначе проверить. Ваш код имеет совершенно нормальное описание и совершенно нормальное поведение. Палка с нормальной if/then:

public void withdraw (double balance) 
{ 
    double amount; 
    do 
    { 
    System.out.println("How much would you like to withdraw?"); 
    amount = keyboard.nextDouble(); 
    if (amount > balance) 
    { 
     System.out.println("I'm sorry, you cannot overdraw, please pick a lower withdrawal amount."); 
    } 
    } 
    while(amount > balance); 

    // Although you probably want to say something like 
    // "or type 0 not to withdraw anything", to be nice. 

    balance = balance-amount; 
    System.out.println("You have withdrawn "+amount+ " and your new balance is " +balance); 
}