[Løst]If setning oppfører seg snodig

[miXer]

Jeg har laget en egen funksjon som returnerer en boolean utifra om brukeren angir "yes" eller "no" i konsollet, men min if setning tolker dette helt tullete.

Det som skal skje:

- Brukeren skriver inn "yes", funksjonen tolker dette som true

- Brukeren skriver inn "no", funksjonen tolker dette som false

- Brukeren skriver inn noe helt annet, funksjonen tolker dette som false

 

Det som skjer:

- Brukeren skriver inn "yes", funksjonen tolker dette som noe helt annet, dermed false

- Brukeren skriver inn "no", funksjonen tolker dette som noe helt annet, dermed false

- Brukeren skriver inn noe helt annet, funksjonen tolker dette som noe helt annet, dermed false

 

Her er koden som kaller funksjonen:

boolean numsPresent = false;
numsPresent = readBool();

 

Her er funksjonen:

private static boolean readBool()
{
	BufferedReader br = new BufferedReader(new InputStreamReader(System.in));
	try 
	{
		String s = br.readLine();
		if (s == "yes")
			return true;
		else if (s == "no")
			return false;
		else
		{
			System.out.println("\nYou entered something weird, so i take that as a big fat NO!");
			return false;
		}
	}
	catch (Exception ex) { System.out.println(ex); return false; }
}

[steingrim]

if (s == "yes")
  return true;
else if (s == "no")
  return false;

== gjør ikke det du tror den gjør. Stringer er objekter og == sammenligner objekt-likhet, altså om referansene peker til samme objekt. Det gjør de ikke.

 

Det du vil er å bruke equals(), eller muligens equalsIgnoreCase()

 

if (s.equals("yes")) return true;
// osv...

[miXer]

Takk for svar og løsning. Er vant til C# som er noget mere logisk føler jeg

[MailMan13]

En som koder Java til vanlig vil nok si nøyaktig det samme om C#

Endret 3. desember 2009 av MailMan13

[Sokkalf™]

Hvilken versjon av java kompilerer du med? == for å sammenligne strings har vel vært gyldig en stund nå? (1.5, 1.6?).

 

Kompilerte et lite testprogram for å verifisere, og jo, det funket å bruke ==.

 

Edit: Jeg er forøvrig helt enig i at .equals() er det "rette" måten å gjøre det på!

Endret 3. desember 2009 av Sokkalf™

[miXer]

Jeg bruker 1.6.0_16

Har du det lille testprogrammet ditt, så kan jeg se om det fungerer her å.

 

Og om .equals() er den rette eller ikke, det er vel bare smak og behag tror jeg

[Sokkalf™]

Nja, == gir warnings i NetBeans ihvertfall.

 

Vel, testet måten du gjorde det på, og det funker ikke her heller.

 

String y = "yes";

 

Og deretter : (y == "yes") gir true, men når jeg prøvde å lese inn strengen med readLine, så fungerte det ikke.

 

Litt usikker på hvorfor, men kanskje noen som er litt klokere enn meg kan svare på det.

 

Edit: http://www.devdaily.com/java/edu/qanda/pjq...tml#comment-474

 

Ser ut som jeg har blitt lurt av java-compileren, .equals er definitivt og i alle tilfeller måten å gjøre det på.

Endret 3. desember 2009 av Sokkalf™

[steingrim]

Hvilken versjon av java kompilerer du med? == for å sammenligne strings har vel vært gyldig en stund nå? (1.5, 1.6?).

Tullball. == er ikke overloadet for String i Java, slik den er i C#.

Endret 3. desember 2009 av steingrim

[steingrim]

String y = "yes";

 

Og deretter : (y == "yes") gir true, men når jeg prøvde å lese inn strengen med readLine, så fungerte det ikke.

Grunnen til at det fungerer i det ene tilfellet og ikke det andre har med en liten detalj som heter string interning å gjøre. Alle literale strenger i .java-filene dine opprettes som statiske String-objekter i en String-pool. Dvs at om du har flere "yes"-literaler i .java-filene dine så vil alle disse referere til samme "yes"-objekt i intern-poolen.

 

Men når du kaller readLine() lages et nytt String-objekt. == vil derfor gi false, ettersom dette nye String-objektet ikke er det samme som det som ligger i intern-poolen.

[Sokkalf™]

Ja, det var det jeg svarte meg selv på i den redigerte posten min over, ref. url'en jeg postet der.

[steingrim]

Heh, gadd ikke klikke på den linken og så ikke at det var du var samme bruker