Java command prompt loop

Question

I have this simple code that works like a small command prompt. User is asked to enter a command and is displayed a message according to what he entered. I just want to make it loop so that the user is asked to enter a command again and again and again until he types 'exit'. How do I do that? Here is the code:

public static void main(String[] args) {

    Scanner command = new Scanner(System.in);

    System.out.println("Enter command: ");

    String text = command.nextLine();

        switch(text){

        case "start":
            System.out.println("Machine started!");
            break;

        case "stop":
            System.out.println("Machine stopped.");
            break;

        case "exit":
            System.out.println("Application Closed");
            break;

        default:
            System.out.println("Command not recognized!");
            break;
        }

    command.close();
}

Thank you

while ( not condition met ){ /* your code */ reset value to test } — Stultuske
– Stultuske, Commented Apr 27, 2015 at 13:09
Try to use chain if-else and use text.equals("xxxxxx") for the condition. — omerfarukdogan
– omerfarukdogan, Commented Apr 27, 2015 at 13:16
@farukdgn: you are testing for the wrong value. "xxxxxxxx" and "exit" will never be equal. I was, however, referring to your if-else proposal. — Stultuske
– Stultuske, Commented Apr 27, 2015 at 13:23

Praeterii · Accepted Answer · 2015-04-27 13:39:17Z

5

I prefer:

public static void main(String[] args) {

    Scanner command = new Scanner(System.in);

    System.out.println("Enter command: ");
    boolean running = true;

    while(running){

        switch(command.nextLine()){

        case "start":
            System.out.println("Machine started!");
            break;

        case "stop":
            System.out.println("Machine stopped.");
            break;

        case "exit":
            System.out.println("Application Closed");
            running = false;
            break;

        default:
            System.out.println("Command not recognized!");
            break;
        }
    }
    command.close();
}

edited Apr 27, 2015 at 13:39

answered Apr 27, 2015 at 13:15

Praeterii

3406 silver badges16 bronze badges

Sign up to request clarification or add additional context in comments.

4 Comments

Stultuske Over a year ago

Constantly re-declaring text? Why not just once?

Bubletan Over a year ago

@Stultuske How's that bad? Keeping references within their own scope.

Stultuske Over a year ago

@Bubletan: it was more a question than a remark. considering the scope here: String text = ""; while ( .... ){ /* code */ } Yes, limiting to the scope is not bad, but re-creating the instance variable each time, there are those who don't like to see that either.

Bubletan Over a year ago

@Stultuske Yah, best way would be just switch (command.nextLine()).

brso05 · Accepted Answer · 2015-04-27 13:10:59Z

3

You can put it in a while loop like this:

String text = "";

while(!text.equalsIgnoreCase("exit"))
{
    System.out.println("Enter command: ");

    text = command.nextLine();

        switch(text){

        case "start":
            System.out.println("Machine started!");
            break;

        case "stop":
            System.out.println("Machine stopped.");
            break;

        case "exit":
            System.out.println("Application Closed");
            break;

        default:
            System.out.println("Command not recognized!");
            break;
        }
}

Just let the condition for your while loop be while text <> "exit".

answered Apr 27, 2015 at 13:10

brso05

13.3k2 gold badges25 silver badges41 bronze badges

Comments

Frigo · Accepted Answer · 2015-04-27 13:16:06Z

Something like this? I prefer do-while for this type of situation because you probably want to give at least one command.

public static void main(String[] args) {
    boolean running = true;

    Scanner command = new Scanner(System.in);
    do {
        System.out.println("Enter command: ");
        String text = command.nextLine();
        switch(text){
            case "start":
                System.out.println("Machine started!");
                break;
            case "stop":
                System.out.println("Machine stopped.");
                running = false;  //here?
                break;
            case "exit":
                System.out.println("Application Closed");
                running = false; //..or here?
                break;
            default:
                System.out.println("Command not recognized!");
                break;
        }
    } while(running);
    command.close();
}

Bubletan · Accepted Answer · 2015-04-27 13:15:30Z

0

If you want to use a switch, easiest way would probably be a simple infinite while-loop with a label. When the input matches "exit", we just break the loop.

loop:
while (true) {
    String text = command.nextLine();
    switch (text) {
    // ...
    case "exit":
        break loop;
    }
}

answered Apr 27, 2015 at 13:15

Bubletan

3,8636 gold badges27 silver badges33 bronze badges

Comments

ioseb · Accepted Answer · 2015-04-27 16:13:46Z

0

while loop is missing in the initial example:

    public static void main(String args[]) {
    BufferedReader br = new BufferedReader(new InputStreamReader(System.in));

    while (true) {
        String command = br.readLine().toLowerCase();

        switch (command) {
            case "exit": {
                // exit here
            } break;

            default: {
                // unrecognised command
            }
        }
    }
}

edited Apr 27, 2015 at 16:13

answered Apr 27, 2015 at 13:13

ioseb

17k3 gold badges37 silver badges31 bronze badges

1 Comment

Bubletan Over a year ago

You misspelled siwtch. Also, you're never leaving the while-loop.

Collectives™ on Stack Overflow

Java command prompt loop

5 Answers 5

4 Comments

Comments

Comments

Comments

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

5 Answers 5

4 Comments

Comments

Comments

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related