hjælp til lidt kode

hej allesammen, jeg er i gang med at lave et simpelt regnemaskine program, jeg bruger Netbeans IDE 5,5 men når jeg skriver dette:

Kode 

class Regnemaskine {
        public static void main(String[] arg)
{ 
             java.util.Scanner tastatur = new java.util.Scanner(System.in);
             
             System.out.println("vaelg regnemetode");
             System.out.println("1 = +");
             System.out.println("2 = -");
             System.out.println("3 = *");
             System.out.println("4 = /");
             int valg, tal1, tal2, ialt;
             valg = tastatur.nextInt();
             
             if (valg == 1) 
             {   
                 System.out.println("du valgte plus-regnemaskinen!");
             System.out.println("tast foerste tal");
             tal1 = tastatur.nextInt();
             System.out.println("tast andet tal");
             tal2 = tastatur.nextInt();
             ialt = tal1 + tal2;
             System.out.println("tal1 + tal2 = " + ialt);
             }
             else if (valg == 2) System.out.println("du valgte minus-regnemaskinen!");
                          {   
             System.out.println("tast foerste tal");
             tal1 = tastatur.nextInt();
             System.out.println("tast andet tal");
             tal2 = tastatur.nextInt();
             ialt = tal1 - tal2;
             System.out.println("tal1 - tal2 = " + ialt);
             }
             else if (valg == 3) System.out.println("du valgte gange-regnemaskinen!");
                          {   
             System.out.println("tast foerste tal");
             tal1 = tastatur.nextInt();
             System.out.println("tast andet tal");
             tal2 = tastatur.nextInt();
             ialt = tal1 * tal2;
             System.out.println("tal1 * tal2 = " + ialt);
             }
             else if (valg == 4) System.out.println("du valgte divider-regnemaskinen!");
                          {   
             System.out.println("tast foerste tal");
             tal1 = tastatur.nextInt();
             System.out.println("tast andet tal");
             tal2 = tastatur.nextInt();
             ialt = tal1 / tal2;
             System.out.println("tal1 / tal2 = " + ialt);
             }
             else (valg >= 4) System.out.println("du skal vælge et tal mellem 1-4!");

}


}

siger den "else without if", og understreger
else if valg==3
else if valg==4
else if valg>=4
hvad er der galt?

syntaxen er "altid".

if(valg==3)
{

}
else if(valg==4)
{

}
else if(valg>=4)
{

}

Altså skal du starte med en if statement, og tilføje else if.

Det er lidt nemmere at læse koden hvis du laver indryk:

class Regnemaskine
{
        public static void main(String[] arg)
        { 
             java.util.Scanner tastatur = new java.util.Scanner(System.in);
             
             System.out.println("vaelg regnemetode");
             System.out.println("1 = +");
             System.out.println("2 = -");
             System.out.println("3 = *");
             System.out.println("4 = /");
             int valg, tal1, tal2, ialt;
             valg = tastatur.nextInt();
             
             if (valg == 1) 
             {   
                 System.out.println("du valgte plus-regnemaskinen!");
                 System.out.println("tast foerste tal");
                 tal1 = tastatur.nextInt();
                 System.out.println("tast andet tal");
                 tal2 = tastatur.nextInt();
                 ialt = tal1 + tal2;
                 System.out.println("tal1 + tal2 = " + ialt);
             }
             else if (valg == 2)
                 System.out.println("du valgte minus-regnemaskinen!");
             {   
                 System.out.println("tast foerste tal");
                 tal1 = tastatur.nextInt();
                 System.out.println("tast andet tal");
                 tal2 = tastatur.nextInt();
                 ialt = tal1 - tal2;
                 System.out.println("tal1 - tal2 = " + ialt);
             }
             else if (valg == 3)
                 System.out.println("du valgte gange-regnemaskinen!");
             {
                 System.out.println("tast foerste tal");
                 tal1 = tastatur.nextInt();
                 System.out.println("tast andet tal");
                 tal2 = tastatur.nextInt();
                 ialt = tal1 * tal2;
                 System.out.println("tal1 * tal2 = " + ialt);
             }
             else if (valg == 4)
                 System.out.println("du valgte divider-regnemaskinen!");
             {
                 System.out.println("tast foerste tal");
                 tal1 = tastatur.nextInt();
                 System.out.println("tast andet tal");
                 tal2 = tastatur.nextInt();
                 ialt = tal1 / tal2;
                 System.out.println("tal1 / tal2 = " + ialt);
             }
             else (valg >= 4)
                 System.out.println("du skal vælge et tal mellem 1-4!");

        }

}

Du har gjort den fejl at du har skrevet:

             else if (valg == 2)
                 System.out.println("du valgte minus-regnemaskinen!");
             {   
                 ...
             }

det betyder det samme som:
HVIS valg == 2
SÅ skriv "bla bla"
Lav noget andet

             else if (valg == 2)
             {   
                 System.out.println("du valgte minus-regnemaskinen!");
                 ...
             }

Betyder derimod:
HVIS valg == 2
SÅ skriv "bla bla"
OG Lav noget andet

Håber du forstår forskellen.

Den endelige kode skal være:

class Regnemaskine
{
        public static void main(String[] arg)
        { 
             java.util.Scanner tastatur = new java.util.Scanner(System.in);
             
             System.out.println("vaelg regnemetode");
             System.out.println("1 = +");
             System.out.println("2 = -");
             System.out.println("3 = *");
             System.out.println("4 = /");
             int valg, tal1, tal2, ialt;
             valg = tastatur.nextInt();
             
             if (valg == 1) 
             {   
                 System.out.println("du valgte plus-regnemaskinen!");
                 System.out.println("tast foerste tal");
                 tal1 = tastatur.nextInt();
                 System.out.println("tast andet tal");
                 tal2 = tastatur.nextInt();
                 ialt = tal1 + tal2;
                 System.out.println("tal1 + tal2 = " + ialt);
             }
             else if (valg == 2)
             {   
                 System.out.println("du valgte minus-regnemaskinen!");
                 System.out.println("tast foerste tal");
                 tal1 = tastatur.nextInt();
                 System.out.println("tast andet tal");
                 tal2 = tastatur.nextInt();
                 ialt = tal1 - tal2;
                 System.out.println("tal1 - tal2 = " + ialt);
             }
             else if (valg == 3)
             {
                 System.out.println("du valgte gange-regnemaskinen!");
                 System.out.println("tast foerste tal");
                 tal1 = tastatur.nextInt();
                 System.out.println("tast andet tal");
                 tal2 = tastatur.nextInt();
                 ialt = tal1 * tal2;
                 System.out.println("tal1 * tal2 = " + ialt);
             }
             else if (valg == 4)
             {
                 System.out.println("du valgte divider-regnemaskinen!");
                 System.out.println("tast foerste tal");
                 tal1 = tastatur.nextInt();
                 System.out.println("tast andet tal");
                 tal2 = tastatur.nextInt();
                 ialt = tal1 / tal2;
                 System.out.println("tal1 / tal2 = " + ialt);
             }
             else (valg >= 4)
                 System.out.println("du skal vælge et tal mellem 1-4!");

        }

}