CODEWITHSUNDEEP
javaloopswhile-loopdo-while
Nish Grewal
Nish Grewal

Reputation: 13

Do While Loop Skipping User Input

I want it to loop again when the user enters "Y" or "y" and quit when they enter "N" or "n". The quitting option works, however, when they enter Y/y, it shows the first system out, but does not let the user pick which operation they wish to do. Instead the option to continue pops up again and inhibits the user from making any choice.

Here is the code:

import java.util.Scanner;

public class Calc2 {

    public static void main(String[] args) {

        Scanner input = new Scanner(System.in);

        double numOne, numTwo, ans;
        String option;

        do {

            System.out.println(
                    "For addition press '1', for subtraction press '2', for division press '3', for multiplication press '4'");
            String choice = input.nextLine();

            if (choice.contains("1")) {
                System.out.println("Enter the first number : ");
                numOne = input.nextDouble();
                System.out.println("Enter the second number : ");
                numTwo = input.nextDouble();

                ans = numOne + numTwo;
                System.out.println("The answer is: " + ans + " ya bish.");
            }

            else if (choice.contains("2")) {
                System.out.println("Enter the first number : ");
                numOne = input.nextDouble();
                System.out.println("Enter the second number : ");
                numTwo = input.nextDouble();

                ans = numOne - numTwo;
                System.out.println("The answer is: " + ans + " ya bish.");
            } else if (choice.contains("4")) {
                System.out.println("Enter the first number : ");
                numOne = input.nextDouble();
                System.out.println("Enter the second number : ");
                numTwo = input.nextDouble();

                ans = numOne * numTwo;
                System.out.println("The answer is: " + ans + " ya bish.");
            } else if (choice.contains("3")) {
                System.out.println("Enter the first number : ");
                numOne = input.nextDouble();
                System.out.println("Enter the second number : ");
                numTwo = input.nextDouble();

                ans = numOne / numTwo;
                System.out.println("The answer is: " + ans + " ya bish.");

            }
            System.out.println("Press 'Y' to continue or 'N' to quit.");

            option = input.next();
        } while (option.equals("Y") || option.equals("y"));

        if (option.equals("N") || option.equals("n")) {
            System.out.println("Thank you. ");
        }

    }
}

If anyone can help me, it'd be greatly appreciated. Thanks!

Upvotes: 0

Views: 2267

Answers (3)

Subodh Joshi
Subodh Joshi

Reputation: 13482

Please change below line in your code 

    String choice = input.nextLine();

from this code 

    String choice = input.next();

Upvotes: 1

Ori Lentz
Ori Lentz

Reputation: 3688

Your problem appears to be at the beginning of your do-while loop as such:

System.out.println(
        "For addition press '1', for subtraction press '2', " +
         "for division press '3', for multiplication press '4'");
String choice = input.nextLine();

This is the only place where you use nextLine method (rahter than next or nextDouble and so on). This means that after you've read the option argument at the end of the iteration:

option = input.next();

there's still a new line character that hasn't been read by the scanner. When you do nextLine() in the next iteration it reads the new line character before the user has any chance to input something).

Either change that first line to input.next() as well, or make sure every time you read a value, you clear the new line character (for instance by reading nextLine and then casting the value - this would also allow you to do input validations).

Upvotes: 0

Kabilan Kumaravelu
Kabilan Kumaravelu

Reputation: 1

There trouble you see here is the use of nextLine after nextDouble. Check here [Scanner is skipping nextLine() after using next(), nextInt() or other nextFoo() methods

Upvotes: 0

Related Questions