Button Action

[Titanpharao]

Hallo,

also irgendwie komme ich mit denn Button da gerade nicht klar.

		for(int i=0;i<5;++i){
			for (int j = 0; j < 5; j++) {		
				for(int k=0;k<f.i.itemlist.size();++k){
					if(f.spieler.bag[nr][0]==f.i.itemlist.get(k).id){
						items[nr]=new JButton(new ImageIcon(f.i.itemlist.get(k).pfad));
						System.out.println(f.i.itemlist.get(k).name);
						setsizelocation(items[nr], x+i*50, y+j*50, 50, 50);
						itemid=k;
						items[nr].addActionListener(new ActionListener() {
							public void actionPerformed(ActionEvent evt) {
								System.out.println(f.i.itemlist.get(itemid).name);
						}});	
					}					
				}				
				nr++;
			}
		}

Wenn ich auf denn button klicke, komt nicht der name. Er läst das "itemid" immer auf 1. Dabei wird es vor dem Button richtig angezeigt.

Nochwas...geht das nicht irgendwie, das ich nicht alle Variablen in der Klasse schreiben muss, welche ich in dem Button benötige?
Teills sinnlos abfragen und die dazu gehörtigen Variablen alles oben deklarieren...sieht doof aus.

 

[Marco13]

itemid ist wohl eine Variable in der Klasse. Und die ist am Ende (nachdem die Schleife durchgelaufen ist, bzw. wenn später die actionPerformed aufgerufen wird) immer gleich.

Also nicht

class XXX 
{
    int itemid;

    void foo()
    {
         itemid = k;

         items[nr].addActionListener(new ActionListener() {
                     public void actionPerformed(ActionEvent evt) {
                        System.out.println(f.i.itemlist.get(itemid).name);
                  }});    
}

sondern

class XXX 
{
    //int itemid; //--------- WEG

    void foo()
    {
         final int itemid = k; //---------- lokal und final !!!

         items[nr].addActionListener(new ActionListener() {
                     public void actionPerformed(ActionEvent evt) {
                        System.out.println(f.i.itemlist.get(itemid).name);
                  }});    
}

Schau vielleicht auch nochmal hier
http://www.java-forum.org/de/viewtopic.php?p=310844



Btw:
f.i.itemlist.get(itemid).name
:autsch: sieht häßlich aus :autsch:

 

[Titanpharao]

Für das hässlich gern verbesserungsvorschläge^.^

 

[The_S]

Getter und Setter verwenden, kein Static wo es nicht unbedingt sein muss.

 

[Marco13]

static? Wenn 'f' ein Klassenname ist, liegt NOCH mehr im Argen, als wenn es ein Variablenname ist :wink:

 

[Titanpharao]

f ist mein Fenster ;-)

Engine (spieler, gebiete, items)

aus engine wird menu aufgerufen new Menu(this

f.spieler....

Weis das man Getters und Setters benutzen soll. Aber was habe ich davon, abgesehen von ~10zeichen mehr Quelltext?

 

[The_S]

Müssen wir jetzt hier wirklich nochmal den Sinn von Getter und Setter durchkauen? Dazu gibt es unzählich Beispiele im Forum und einen super Beitrag in den FAQ.

Aber EIN simples Beispiel:

Du hast eine Klasse, die z. B. Noten von Schülern speichert. Erlaubst du von außen direkten Zugriff auf die Variable "note", könnte man z. B. dem Schüler auch die Note 0 oder 7 geben. Das könnte dann in der weiteren Programmverarbeitung zu fehlern führen, mal ganz davon abgesehen, dass es extrem unlogisch ist. Ist "note" aber private und du verwendest Getter und Setter, könntest du z. B. sowas machen

public void setNote(int note) {
   if (note > 6) {
      this.note = 6;
   }
   else if (note < 1) {
      this.note = 1;
   }
   else {
      this.note=note;
   }
}

 

[Leroy42]

Ist "note" aber private und du verwendest Getter und Setter, könntest du z. B. sowas machen

public void setNote(int note) {
   if (note > 6) {
      this.note = 6;
   }
   else if (note < 1) {
      this.note = 1;
   }
   else {
      this.note=note;
   }
}

Finde ich aber nicht so toll!
Eine Exception zu werfen, oder einfach System.err.println(...) aufzurufen,
ermöglicht immerhin die Erkennung, daß irgendwer setNote mit
unerlaubtem Argument aufgerufen hat. Bei soetwas wie oben, wird ein möglicher
Fehler einfach nur verschleiert.

 

[Marco13]

public void setNote(int note) 
{
   if (note >= 5) 
   {
      flenn();
   }
   else if (note <= 2) 
   {
      freu();
   }
   else 
   {
      ignorier();
   }
}

Besser? :wink: (Es war ja nur ein Beispiel - vielleicht nicht das beste, aber die allgemeine Aussage, dass es SEHR oft vorkommt, dass beim Setzen einer Variablen "irgendwas" passieren soll (Gültigkeitsabfrage, Event werfen, Aktualisierungen etc) sollte damit schon angekommen sein)

 

[Leroy42]

else if (note <= 2) 
   { 
      freu(); 
   }

Toll! Ich habe als Zensur eine -42 bekommen!

 

[The_S]

man kann auch kleinlich werden :autsch: :roll: