简体   繁体   中英

My Do-While loop doesn't check my condition?

I'm making the following method about making an order on a menu with the following code :

   public static double menu_received() {

        Scanner input = new Scanner(System.in);
        int userResponse, total = 0;

        do {

            System.out.println();
            System.out.println("Here is our menu:");
            System.out.println("1. Bulgogi -- $15.00");
            System.out.println("2. Kalbi -- $18.00");
            System.out.println("3. Kimchi Fried Rice -- $16.00");
            System.out.println("Please choose one option at one time using the number (0 to end the order):");

            userResponse = input.nextInt();
            System.out.println("User Menu choice is " + userResponse);

            if (userResponse == 1) {
                System.out.println("Thank you for ordering Bulgogi");
                total = total + 15;
                System.out.println("Your total is $" + total);
            }

            if (userResponse == 2) {
                System.out.println("Thank you for ordering Kalbi");
                total = total + 18;
                System.out.println("Your total is $" + total);
            }

            if (userResponse == 3) {
                System.out.println("Thank you for ordering Kimichi Fried Rice");
                total = total + 16;
                System.out.println("Your total is $" + total);
            }

        } while (userResponse != 0);

        double total2 = total + 2.55;
        // total = (double) total + 2.55;
        System.out.println("Your total after tax is $" + total2);

        return total2;
    }

However the issue I have is when I call it, it does not respond as I want. It doesn't stop to call the menu even after I enter 0. Is there an issue with my condition in the while(userResponse != 0) Can someone explain to me why and how to correct it? Thank you

Here my output :

>
Here is our menu:
1. Bulgogi -- $15.00
2. Kalbi -- $18.00
3. Kimchi Fried Rice -- $16.00
Please choose one option at one time using the number (0 to end the order):
1
User Menu choice is 1
Thank you for ordering Bulgogi
Your total is $15

Here is our menu:
1. Bulgogi -- $15.00
2. Kalbi -- $18.00
3. Kimchi Fried Rice -- $16.00
Please choose one option at one time using the number (0 to end the order):
1
User Menu choice is 1
Thank you for ordering Bulgogi
Your total is $30

Here is our menu:
1. Bulgogi -- $15.00
2. Kalbi -- $18.00
3. Kimchi Fried Rice -- $16.00
Please choose one option at one time using the number (0 to end the order):
0
User Menu choice is 0
Your total after tax is $32.55

Here is our menu:
1. Bulgogi -- $15.00
2. Kalbi -- $18.00
3. Kimchi Fried Rice -- $16.00
Please choose one option at one time using the number (0 to end the order):
0
User Menu choice is 0
Your total after tax is $2.55

PS: Don't mind the statement "Your total after tax is XXXX" - It comes from another method.

If I understand your comment above, change your main from :

menu_received(); // First call
System.out.println("Value of menu_received : " + menu_received()); // Second call
tip_calculator(menu_received()); // Third call

To :

double result = menu_received(); // Only one call
System.out.println("Value of menu_received : " + result);
tip_calculator(result);

The technical post webpages of this site follow the CC BY-SA 4.0 protocol. If you need to reprint, please indicate the site URL or the original address.Any question please contact:yoyou2525@163.com.

 
粤ICP备18138465号  © 2020-2024 STACKOOM.COM