У меня возникают трудности при переходе на правильный участок цикла

Question

Я пытаюсь сделать так, чтобы пользователь ввел числитель и знаменатель и отобразил частное.Я использую метод для проверки и проверки правильности числа с помощью Integer.parseInt и попытки перехвата.Проблема в том, что если try catch перехватывает исключение и отображает ошибку, она переходит к следующей части кода, а не возвращается к началу цикла.

public static void main(String[] args) {
        String numerator = "";
        String denominator = "";
        String message = "";
        int num = 0;
        int den = 0;
        int quo = 0;
        int yesNo = 0;
        do
        {
            // Displays a JOptionPane asking the user to input a numerator.
            numerator = JOptionPane.showInputDialog(null, "Please enter a numerator.", "Enter a Numerator",JOptionPane.QUESTION_MESSAGE);
            if(numerator == null)// This breaks the loop if  the user clicks the "cancel" button.
                break;
            num = IsValid(numerator); // Calls the IsValid method, passing it the String entered by the user to validate if it is a proper integer.
            denominator = JOptionPane.showInputDialog(null, "Please enter a denominator.", "Enter a Denominator",JOptionPane.QUESTION_MESSAGE);
            if(denominator == null) // same as above but for the denominator
                break;
            den = IsValid(denominator);
            /*if(den != 0 && num<den) 
            { // This section is unrelated to my problem
                quo = num/den;
                message = "Numerator: " + num + "\n" +
                            "Denominator: " + den + "\n" +
                            "Quotient: ";
                JOptionPane.showMessageDialog(null, message,"Calculation",JOptionPane.INFORMATION_MESSAGE);
            }*/
            yesNo = JOptionPane.showConfirmDialog(null,"Would you like to make another calculation?","Continue?",JOptionPane.YES_NO_OPTION);
        }while(yesNo == JOptionPane.YES_OPTION);
    }
    public static int IsValid(String input)
    {
        int num = 0; // This method is passed the String that the JOptionPane receives and using Integer.parseInt to make sure it's an integer, and assigns it to int "num".
        try{num = Integer.parseInt(input);}
        catch(NumberFormatException ex)
        {
            JOptionPane.showMessageDialog(null, "Error: Invalid number. Please enter a valid number.","Invalid Input",JOptionPane.WARNING_MESSAGE);
        }
        return num;
        // My main issue is that it continues with the code instead of jumping back to the JOptionPane to ask the user for input a second time.
    }

По сути, я хочувызовите IsValid (числитель) и присвойте ему int "num", и, если выполняется оператор catch IsValid, вернитесь к началу цикла do, чтобы он снова отобразил этот JOptionPane.Если оператор catch выполняется для знаменателя, вернитесь к нему вместо самого начала.Я не могу поместить JOptionPane в метод, потому что метод не сможет сказать, отправлял ли я ему числитель или знаменатель.Я довольно новичок в Java, и я уверен, что есть более простые способы сделать это, поэтому любая помощь приветствуется.Спасибо!

Answer 1

Вызов isValid должен прервать вашу программу, если ввод неверен.Самый естественный способ - дать ему исключение.Затем ваша программа ловит исключение и завершает работу.

package so20190423;

import java.awt.HeadlessException;

import javax.swing.JOptionPane;

public class Snippet {
    public static void main(String[] args) {
            String numerator = "";
            String denominator = "";
            String message = "";
            int num = 0;
            int den = 0;
            int quo = 0;
            int yesNo = 0;
            try {
                do
                {
                    // Displays a JOptionPane asking the user to input a numerator.
                    numerator = JOptionPane.showInputDialog(null, "Please enter a numerator.", "Enter a Numerator",JOptionPane.QUESTION_MESSAGE);
                    if(numerator == null)// This breaks the loop if  the user clicks the "cancel" button.
                        break;
                    num = IsValid(numerator); // Calls the IsValid method, passing it the String entered by the user to validate if it is a proper integer.
                    denominator = JOptionPane.showInputDialog(null, "Please enter a denominator.", "Enter a Denominator",JOptionPane.QUESTION_MESSAGE);
                    if(denominator == null) // same as above but for the denominator
                        break;
                    den = IsValid(denominator);
                    /*if(den != 0 && num<den) 
                    { // This section is unrelated to my problem
                        quo = num/den;
                        message = "Numerator: " + num + "\n" +
                                    "Denominator: " + den + "\n" +
                                    "Quotient: ";
                        JOptionPane.showMessageDialog(null, message,"Calculation",JOptionPane.INFORMATION_MESSAGE);
                    }*/
                    yesNo = JOptionPane.showConfirmDialog(null,"Would you like to make another calculation?","Continue?",JOptionPane.YES_NO_OPTION);
                }while(yesNo == JOptionPane.YES_OPTION);
            }
            catch(NumberFormatException ex)
            {
                JOptionPane.showMessageDialog(null, "Error: Invalid number. Please enter a valid number.","Invalid Input",JOptionPane.WARNING_MESSAGE);
            }
        }
        public static int IsValid(String input) throws NumberFormatException
        {
            int num = 0; // This method is passed the String that the JOptionPane receives and using Integer.parseInt to make sure it's an integer, and assigns it to int "num".
            num = Integer.parseInt(input);
            return num;
            // My main issue is that it continues with the code instead of jumping back to the JOptionPane to ask the user for input a second time.
        }
}

Вы можете даже ограничить попытку / перехват в цикле while, чтобы восстановить после неудачного ввода и повторить попытку!

package so20190423;

import java.awt.HeadlessException;

import javax.swing.JOptionPane;

public class Snippet {
    public static void main(String[] args) {
            String numerator = "";
            String denominator = "";
            String message = "";
            int num = 0;
            int den = 0;
            int quo = 0;
            int yesNo = 0;
                do
                {
                    try {
                    // Displays a JOptionPane asking the user to input a numerator.
                    numerator = JOptionPane.showInputDialog(null, "Please enter a numerator.", "Enter a Numerator",JOptionPane.QUESTION_MESSAGE);
                    if(numerator == null)// This breaks the loop if  the user clicks the "cancel" button.
                        break;
                    num = IsValid(numerator); // Calls the IsValid method, passing it the String entered by the user to validate if it is a proper integer.
                    denominator = JOptionPane.showInputDialog(null, "Please enter a denominator.", "Enter a Denominator",JOptionPane.QUESTION_MESSAGE);
                    if(denominator == null) // same as above but for the denominator
                        break;
                    den = IsValid(denominator);
                    /*if(den != 0 && num<den) 
                    { // This section is unrelated to my problem
                        quo = num/den;
                        message = "Numerator: " + num + "\n" +
                                    "Denominator: " + den + "\n" +
                                    "Quotient: ";
                        JOptionPane.showMessageDialog(null, message,"Calculation",JOptionPane.INFORMATION_MESSAGE);
                    }*/
                    yesNo = JOptionPane.showConfirmDialog(null,"Would you like to make another calculation?","Continue?",JOptionPane.YES_NO_OPTION);
                }
                catch(NumberFormatException ex)
                {
                    JOptionPane.showMessageDialog(null, "Error: Invalid number. Please enter a valid number.","Invalid Input",JOptionPane.WARNING_MESSAGE);
                }
                }while(yesNo == JOptionPane.YES_OPTION);
        }
        public static int IsValid(String input) throws NumberFormatException
        {
            int num = 0; // This method is passed the String that the JOptionPane receives and using Integer.parseInt to make sure it's an integer, and assigns it to int "num".
            num = Integer.parseInt(input);
            return num;
            // My main issue is that it continues with the code instead of jumping back to the JOptionPane to ask the user for input a second time.
        }
}

Последний вариант, если вы хотите повторить каждую попыткуввод, пока он не будет действителен (или отменен пользователем):

package so20190423;

import javax.swing.JOptionPane;

public class Snippet {
    public static void main(String[] args) {
        String numerator = "";
        String denominator = "";
        String message = "";
        int num = 0;
        int den = 0;
        int quo = 0;
        int yesNo = 0;
        do {
            boolean userCancelled = false;
            boolean numeratorIsOk = false;
            do {
                try {
                    numerator = JOptionPane.showInputDialog(null, "Please enter a numerator.", "Enter a Numerator",
                            JOptionPane.QUESTION_MESSAGE);
                    if (numerator == null) {
                        userCancelled = true;
                    } else {
                        num = IsValid(numerator);
                        numeratorIsOk = true;
                    }
                } catch (NumberFormatException ex) {
                    JOptionPane.showMessageDialog(null, "Error: Invalid number. Please enter a valid number.",
                            "Invalid Input", JOptionPane.WARNING_MESSAGE);
                }
            } while (!numeratorIsOk && !userCancelled);
            if (userCancelled) {
                break;
            }

            boolean denominatorIsOk = false;
            do {
                try {
                    denominator = JOptionPane.showInputDialog(null, "Please enter a denominator.",
                            "Enter a Denominator", JOptionPane.QUESTION_MESSAGE);
                    if (denominator == null) {
                        userCancelled = true;
                    } else {
                        den = IsValid(denominator);
                        denominatorIsOk = true;
                    }
                    /*
                     * if(den != 0 && num<den) { // This section is unrelated to my problem quo =
                     * num/den; message = "Numerator: " + num + "\n" + "Denominator: " + den + "\n"
                     * + "Quotient: "; JOptionPane.showMessageDialog(null,
                     * message,"Calculation",JOptionPane.INFORMATION_MESSAGE); }
                     */
                    yesNo = JOptionPane.showConfirmDialog(null, "Would you like to make another calculation?",
                            "Continue?", JOptionPane.YES_NO_OPTION);
                } catch (NumberFormatException ex) {
                    JOptionPane.showMessageDialog(null, "Error: Invalid number. Please enter a valid number.",
                            "Invalid Input", JOptionPane.WARNING_MESSAGE);
                }

            } while (!denominatorIsOk && !userCancelled);
            if (userCancelled) {
                break;
            }
        } while (yesNo == JOptionPane.YES_OPTION);

    }

    public static int IsValid(String input) throws NumberFormatException {
        int num = 0; // This method is passed the String that the JOptionPane receives and using
                        // Integer.parseInt to make sure it's an integer, and assigns it to int "num".
        num = Integer.parseInt(input);
        return num;
        // My main issue is that it continues with the code instead of jumping back to
        // the JOptionPane to ask the user for input a second time.
    }
}

HTH!