itemStateChanged

[p-flash]

Hi,

Eclipse zeigt mir bei dem Code eine Fehler-Meldung an, dabei ist String rot unterstrichen:

Syntax error, insert ";" to complete Statement

	public void itemStateChanged(ItemEvent e)
	{
		if(e.getSource() == colorChoice)
			String farbe = e.getItem().toString();
	}

Weiß jemand wo der Fehler ist?

/edit
Hmm...wenn ich Die Anweisung in {} packe klappt das. Aber das müsste doch auch so klappen, oder? Das ist ja nur eine Zeile.

Danke.

p-flash

 

[m@sQ]

bei der methode darfst du die klammern nicht weglassen, da ja nachher mehr als eine zeile code folgt (oder man darf die bei methoden so oder so nie weglassen, das weiss ich nicht genau). bei der if-abfrage kannst du sie weglassen, da ja nur eine zeile folgt.

Gruss m@sQ

 

[bambi]

Ich versteh den Sinn von Deiner if-Abfrage eh net. Du erzugst eine Variable in der Abfrage. Die ist dann doch aber nach
dem Block nicht mehr vorhanden. Mach's doch lieber so:

public void itemStateChanged(ItemEvent e)
   {
      String farbe = null;
      if(e.getSource() == colorChoice)
         farbe = e.getItem().toString();

      // tu was mit farbe ...
   }

Dann klappt's auch mit dem Eclipse :wink:

 

[Sky]

1.) Der Compiler weiss bei deinem Fall sofort, dass die Anweisung unsinnig ist, weil man "farbe" nicht mehr verwenden kann

2.) Es ist sowieso guter Programmierstil immer die Statements in einem if-Block zu Klammern. vgl. auch: http://java.sun.com/docs/codeconv/html/CodeConventions.doc6.html#449