Home > Back-end >  My Do-While loop doesn't check my condition?
My Do-While loop doesn't check my condition?

Time:10-29

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.

CodePudding user response:

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);
  • Related