Java if/else

ej ups, else if sætningen skulle hedde:

else if (janej == "Nej")
  JOptionPane.showMessageDialog(null, "Skod");

Men problemet har alligevel ikke noget med dette at gøre..

din janej variable er en String variable...

Derfor skal du teste med .equals("...");

if( janej.equalsIgnoreCase("ja") ) {
...
} else if( janej.equalsIgnoreCase("nej") ) {
...
} else {
...
}

/Søren Munk Skrøder

når du sammenligner String værdier skal du bruge
    janej.equals("Ja")
ig ikke ==.  == kan kun bruges til atomære variable.

equalsIgnoreCase() gør at du både kan skrive "JA", "ja", "jA" og "Ja" - de vil allesammen virke, da den ser bort fra store/små bogstaver...

/Søren

if(janej.equals("Ja"))

men husk på at java er case sens.

argh..sorry så ikke at der var nogen der havde svaret....Lyt til bearhugx

jacoba>> det er sgu' sjældent at jeg når at komme før dig :-) *LOL*

bearhugx >> bare rolig, det vil ikke ske igen :-))

*LOL*

Mange tak for hjælpen ;)

Jakoba...nu vi er ved det....hvorfor egentlig ? har altid undret mig over equals og == ved String

Takker for points :-)

En string er et objekt der indeholder et antal bogstaver
så når du siger
    streng1 == streng2
er det to pegepinde du sammenligner, ikke 2 bogstavsekvenser

ok tak jakoba

jakoba, du mener vel at man _burde_ bruge equals med Strings og ikke skal.

2 ens String bliver gjort til alias'er af compileren så med Strings virker == også (næsten altid)

public class StringEquals
{
    String instanceVar1 = "sammenlign", instanceVar2, instanceVar3;
    String instanceNewVar = new String("sammenlign");
   
    public StringEquals(String lokalStreng)
    {
        instanceVar2 = "sammenlign";
        instanceVar3 = lokalStreng;
       
        // nedenstående giver true fordi String objekterne er alias
        System.out.println ("instanceVar1 equals sammenlign: " +
            instanceVar1.equals("sammenlign"));
        System.out.println ("instanceVar1 == sammenlign: " +
            (instanceVar1 == "sammenlign"));
        System.out.println ("instanceVar1 == instanceVar2: " +
            (instanceVar1 == instanceVar2));
        System.out.println ("instanceVar1 == instanceVar3: " +
            (instanceVar1 == instanceVar3));
        System.out.println ("lokalStreng == instanceVar1: " +
            (lokalStreng==instanceVar1));
       
        // Hvis der bruges new String, giver kun equals true
        System.out.println ("\ninstanceVar1 equals NewVar: " +
            instanceVar1.equals(instanceNewVar));
        System.out.println ("instanceVar1 == NewVar: " +
            (instanceVar1 == instanceNewVar));
    }
   
    public static void main(String[] args)
    {
        new StringEquals("sammenlign");
    }
}

Giver som output

instanceVar1 equals sammenlign: true
instanceVar1 == sammenlign: true
instanceVar1 == instanceVar2: true
instanceVar1 == instanceVar3: true
lokalStreng == instanceVar1: true

instanceVar1 equals NewVar: true
instanceVar1 == NewVar: false

Derfor brug aldrig new String med en streng som parameter. Det bruger bare mere ram og performer langsommere.