break-Anweisung in der "if else" - java

Ich immer eine Fehlermeldung, if ohne else.

Versuchte ich else if sowie

for (;;){
        System.out.println("---> Your choice: ");
        choice = input.nextInt();
        if (choice==1)
            playGame();
        if (choice==2)
            loadGame();
        if (choice==3)
            options();
        if (choice==4)
            credits();
        if (choice==5)
            System.out.println("End of Game\n Thank you for playing with us!");
            break;
        else
            System.out.println("Not a valid choice!\n Please try again...\n");=[;'mm
    }

auch wenn Sie eine bessere Idee, wie dieser code bitte zögern Sie nicht 🙂

Java ist nicht Python. Sie können nicht erwarten, die zwei Linien in der gleichen block, nur weil Sie die gleiche Einrückung haben...
Ich sehe xD danke!
Auch das wird das problem nicht lösen vollständig. Alle if Aussagen müssen verbunden werden mit else.
Sollte es nicht: "else ohne if"?

InformationsquelleAutor John | 2013-12-18

  1. 12

    Weil Ihre else ist nicht angebracht, zu nichts. Die if ohne Klammern umfasst nur die einzige Anweisung, die unmittelbar folgt. 

    if (choice==5)
{
    System.out.println("End of Game\n Thank you for playing with us!");
    break;
}
else
{
   System.out.println("Not a valid choice!\n Please try again...\n");
}

    Nicht mit geschweiften Klammern ist im Allgemeinen angesehen als eine schlechte Praxis, weil es kann dazu führen, die genauen Probleme, denen Sie begegnen.

    Zusätzlich mit einem switch hier mehr Sinn machen würde.

    int choice;
boolean keepGoing = true;
while(keepGoing)
{
    System.out.println("---> Your choice: ");
    choice = input.nextInt();
    switch(choice)
    {
        case 1: 
            playGame();
            break;
        case 2: 
            loadGame();
            break;
        //your other cases
        //...
        case 5: 
            System.out.println("End of Game\n Thank you for playing with us!");
            keepGoing = false;
            break;
        default:
            System.out.println("Not a valid choice!\n Please try again...\n");
     }
 }

    Beachten Sie, dass anstelle einer unendlichen for Schleife habe ich eine while(boolean), so dass es einfach zu beenden die Schleife. Ein weiterer Ansatz wäre die Verwendung von break mit Etiketten.

    Sie möchten notieren Sie, wie Sie brechen aus der Schleife, als break; in einem switch wird nur brechen aus dem Schalter.
    Huch - guter Punkt. Bearbeiten

    InformationsquelleAutor Brian Roach

  2. 13

    Den "Pause" - Befehl funktioniert nicht innerhalb einer "if" - Anweisung.

    Wenn Sie entfernen die "Pause" - Befehl aus dem code und dann den code zu testen, sollten Sie feststellen, dass der code funktioniert genau das gleiche, ohne eine "Pause" - Befehl wie mit einer.

    "Pause" ist speziell für den Einsatz in Schleifen (for, while, do-while, verstärkt für und switch).

    Das ist falsche Antwort. Der break-Befehl innerhalb der IF-Anweisung beendet die FOR-Schleife.
    Die 'Pause' - Befehl funktioniert nicht in einem 'if' - Anweisung, wenn, wenn-Anweisung nicht innerhalb einer Kontroll-Struktur-loop/switch. Ich denke, das ist, wo die Verwirrung liegt? Wenn die leichter zu verstehen, stellen Sie sich vor, ein 'continue' - Anweisung in einer if-Anweisung.

    InformationsquelleAutor Alan J. Robinson

  3. 5

    Das Problem ist, dass Sie versuchen, mehrere Anweisungen in eine if ohne Verwendung {}.
    Was Sie derzeit haben, wird interpretiert wie:

    if( choice==5 )
{
    System.out.println( ... );
}
break;
else
{
    //...
}

    Sie wirklich wollen:

    if( choice==5 )
{
    System.out.println( ... );
    break;
}
else
{
    //...
}

    Auch, als Farce hat erklärt, es wäre besser, verwenden else if für alle Bedingungen, statt if denn wenn choice==1 ist, wird es noch durch gehen, und prüfen Sie, ob choice==5, die fehlschlagen würde, und es wird immer noch gehen Sie in Ihrem else-block.

    if( choice==1 )
    //...
else if( choice==2 )
    //...
else if( choice==3 )
    //...
else if( choice==4 )
    //...
else if( choice==5 )
{
    //...
}
else
    //...

    Eine elegantere Lösung wäre die Verwendung eines switch - Anweisung. Allerdings break nur bricht aus dem innersten "block", es sei denn, Sie verwenden die Etiketten. Sie wollen So zu Kennzeichnen, Schleife und brechen aus, wenn der Fall ist 5:

    LOOP:
for(;;)
{
    System.out.println("---> Your choice: ");
    choice = input.nextInt();
    switch( choice )
    {
        case 1:
            playGame();
            break;
        case 2:
            loadGame();
            break;
        case 2:
            options();
            break;
        case 4:
            credits();
            break;
        case 5:
            System.out.println("End of Game\n Thank you for playing with us!");
            break LOOP;
        default:
            System.out.println( ... );
    }
}

    Statt der Kennzeichnung der Schleife, könnte man auch ein flag zu sagen, die Schleife zu stoppen.

    bool finished = false;
while( !finished )
{
    switch( choice )
    {
        //...
        case 5:
            System.out.println( ... )
            finished = true;
            break;
        //...
    }
}

    InformationsquelleAutor clcto

Schreibe einen Kommentar