[Iso]

Hi Leute,

habe ein kleines Problem und weiss einfach nicht, wo der Fehler liegt.

Mit der Methode readLine vom BufferedReader-Object stdin

	private InputStreamReader inStream = new InputStreamReader ( System.in );
	private BufferedReader stdin = new BufferedReader ( inStream );

lese ich ja einen Stream als String ein, später will ich diesen String mit einem Vorgegeben vergleichen in einer if-abfrage,
z.b.

if(glastype="speciaalglas"){
   kosten = 50;
   ...
}

doch leider ist die if-Anweisung immer False auch wenn der String glastype im Debugger genauso der selbe Wert ist wie "speciaalglas",
ich komme nicht dahinter, woran das liegt, warscheinlich irgendwie an der Verarbeitung des Strings vom BufferedReader, aber kp, was genau und wieso.

Danke im Voraus!

[Witi]

Zitat

if(glastype="speciaalglas")

*doppel-autsch* Zwei Fehler in einer Zeile.

Mal schauen, ob du selber drauf kommst...
1. Du verwendest einen Zuweisungs- und keinen Vergleichsoperator.
2. Vergleichsoperatoren nutzt man unter Java nur bei Datentypen. String ist aber eine Klasse. Und Objekte vergleicht man anders. Tipp: Klasse Object.

[G.I.Joe]

if(glastype.equals("speciaalglas"))

Dieser Beitrag wurde von deglerj bearbeitet: 02. November 2006 - 21:02

[MNG]

Und falls glastype irgendwann mal null sein kann

if("speciaalglas".equals(glastype))

[Iso]

Sorry, dass mit dem Vergleichsoperator weiss ich wohl auch, hab aber vergessen 2 = zu machen

Hmm bei dem equals mekkert der immer, dass eine ) fehlt, obwohl ich das genau so abgetippt habe, wie beschrieben ...

[MNG]

Hmm, also die if-Bedingung stimmt so. Hast du beim Einfügen vielleicht irgendwo eine Klammer gekillt?

[Iso]

Mal wieder mein Fehler, nicht mein Tag heute,
hab den Punkt vergessen zwischen "peciaalglas" und eguals, ist ja auch irgendwie logisch -.- ...

Jetzt funktioniert es auch,
danke!

Dieser Beitrag wurde von Iso bearbeitet: 03. November 2006 - 00:32

[Rika]

Trotzdem bin ich dafür, daß du Interfaces statt konkrete Klassen verwendest und am besten auch gleich schachtelst.

private Reader stdin = new BufferedReader(new InputStreamReader ( System.in));

[Iso]

Ja, da hast du recht, sieht auch gleich viel Übersichtlicher aus