Fehler bei Button Array

[dahaack]

Hallo Leute!

Ich habe hier einen Button-Array...

import java.util.ArrayList;
import org.eclipse.swt.SWT;
import org.eclipse.swt.events.SelectionAdapter;
import org.eclipse.swt.events.SelectionEvent;
import org.eclipse.swt.widgets.Button;
import org.eclipse.swt.widgets.Display;
import org.eclipse.swt.widgets.Shell;


public class Pokercheat {
	static int farbe, wert;
	public static void main(String[] args) {
		
		Karte karte = new Karte(0,0);
		
    	// Display wird erstellt
        Display display = new Display();
        final Shell shell = new Shell(display);
        shell.setText("Kartenspiel");
        shell.setBounds(0, 0, 900, 600);
        
        final Button[][] buttonArray = new Button[4][13];
        ArrayList<Karte> gesamtKarten = new ArrayList<Karte>();
        
        
        for(farbe=0;farbe<=3;farbe++) {
        	for(wert=0;wert<=12;wert++) {
        		karte.setFarbe(farbe);
        		karte.setWert(wert+2);
        		gesamtKarten.add(karte);
        		buttonArray[farbe][wert] = new Button(shell, SWT.PUSH);
        		buttonArray[farbe][wert].setText(karte.toString());
        		buttonArray[farbe][wert].setBounds(20+100*farbe, 20+40*wert, 80, 30);
        	}
        }
        
        for(farbe=0;farbe<=3;farbe++) {
        	for(wert=0;wert<=12;wert++) {
		        buttonArray[farbe][wert].addSelectionListener(new SelectionAdapter() {
		        	public void widgetSelected(SelectionEvent e) {	
		        		System.out.println("Button gedrückt");
					System.out.println(buttonArray[farbe][wert].getText());   <<=== Hier spackt er ab.
		        	}
		        });
        	}
        }
        
        shell.open();
        while (!shell.isDisposed())
            if (!display.readAndDispatch())
                display.sleep();
	}
}

public class Karte {
	private int farbe;	// 0: Karo, 1: Hertz, 2: Kreuz,, 3: Piek
	private int wert;	// 2 = Zwei, 11 = Bubbe, 14 = Ass
		
	public Karte(int farbe, int wert) {
		super();
		this.farbe = farbe;
		this.wert = wert;
	}

	public String toString() {
		StringBuffer sb = new StringBuffer();
		// Farbe im StringBuffer speichern
		if(farbe==0) {
			sb.append("Karo").append(" ");
		} else if(farbe==1) {
			sb.append("Hertz").append(" ");
		} else if(farbe==2) {
			sb.append("Kreuz").append(" ");
		} else if(farbe==3) {
			sb.append("Piek").append(" ");
		}
		// Wert im StringBuffer speichern
		if(wert<=10) {
			sb.append(wert);
		} else if(wert==11) {
			sb.append("B");
		} else if(wert==12) {
			sb.append("D");
		} else if(wert==13) {
			sb.append("K");
		} else if(wert==14) {
			sb.append("A");
		}
		return sb.toString();
	}

	// Setters & Getters
	public int getFarbe() {
		return farbe;
	}
	public void setFarbe(int farbe) {
		this.farbe = farbe;
	}
	public int getWert() {
		return wert;
	}
	public void setWert(int wert) {
		this.wert = wert;
	}	
}

Der Fehler:
Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: 4

Wenn ich die Zeile 42 rausnehme gibt es keinen Fehler.


Wäre sehr nett, wenn mir jemand helfen könnte!

 

[SlaterB]

in dem SelectionAdapter ist farbe nicht etwa der Wert bei der Erzeugung des Objektes,
sondern der Inhalt der statischen Variable zum Zeitpunkt des Ausführens der Operation,

egal von welchem Button der SelectionAdapter aufgerufen wird, da die statische Variable 4 enthält wird immer an Position 4 geschaut

nun hast du ein oder vielleicht zwei Probleme:
1.) wieso ist farbe zum Zeitpunkt der Exception 4? an deinem Code kann man es nicht erkennen,
irgendwann wird diese statische Variable auf 4 gesetzt, was wahrscheinlich falsch ist
2.) deine SelectionAdapter sollen sich wahrscheinlich auf den jeweilen Button im Array beziehen, nicht von der statischen Variable abhängen,
das geht entweder mit

for(farbe=0;farbe<=3;farbe++) {
for(wert=0;wert<=12;wert++) {

final int farbeFinal = farbe;
final int wertFinal = wert;

buttonArray[farbe][wert].addSelectionListener(new SelectionAdapter() {
public void widgetSelected(SelectionEvent e) {
System.out.println("Button gedrückt");
System.out.println(buttonArray[farbeFinal ][wertFinal ].getText());
}
});
}
}


oder etwas sinnvoller mit

for(farbe=0;farbe<=3;farbe++) {
for(wert=0;wert<=12;wert++) {

final Button b = buttonArray[farbe][wert];

b.addSelectionListener(new SelectionAdapter() {
public void widgetSelected(SelectionEvent e) {
System.out.println("Button gedrückt");
System.out.println(b.getText());
}
});
}
}


wichtig ist das final

 

[dahaack]

Es funktioniert! Vielen Dank!


Aber so ganz hab ich nicht gecheckt, warum die Farbe immer den Wert 4 hatte.


Das liegt vielleicht daran dass ich nicht so genau weiß wann etwas static / final gehört und wann nicht.


Bei dem ButtonArray wird ein final int benötig, dass kann man ja nochmal so hinnehmen, aber wann mach ich etwas static und wann nicht?


Vielen Dank nochmal für deine schnelle Hilfe!

 

[SlaterB]

wenn du keinen Sinn für static sieht, dann verwende gar kein static,
so einfach ist das,

static nutzt mn z.B. für Konstanten,
darauf wollte ich dich vorhin schon hinweisen:
in deinem Code steht 3x 12 bzw. dreizehn,

da definierst du
static int NUMBER_CARDS = 4;
und verwendest überall nur diese Variable,

hier wird sich der Wert vielleicht nie mehr ändern, an anderen Stellen ist es leichter, so den Wert zu ändern/ zu verfolgen, wo er verwendet wird

----------


> final Button[][] buttonArray = new Button[4][13];
> for(farbe=0;farbe<=3;farbe++) {
> for(wert=0;wert<=12;wert++) {


wenn überhaupt ohne statische Konstante, dann schreibt man sowas übrigens

final Button[][] buttonArray = new Button[4][13];
for(farbe=0;farbe<4;farbe++) {
for(wert=0;wert<13;wert++) {

mit gleichen Zahlen

 

[dahaack]

Danke nochmal! :toll:


Aber wo ist dann der Unterschied zwischen static und final.

Ich benutzer static oder final immer nur, wenn mir das von eclipse empfohlen wird um einen Fehler zu berichtigen.

da definierst du
static int NUMBER_CARDS = 4;

Kannst du mir sagen in welcher Zeile? Ich blick es nicht.

 

[SlaterB]

na ich empfehle langsam in einem Buch nachzuschlagen, das sind ja Fachbegriffe

> Kannst du mir sagen in welcher Zeile?

das war nur ein Beispiel, kannst dir selber aussuchen

 

[dahaack]

In dem Buch, das ich habe steht leider nichts genaues darüber, wann man es anwenden muss und wann nicht.

Hier hätte ich noch eine kleine die glaub ich damit auch zu tun hat:

public class Test {
	final ArrayList<String> al = new ArrayList<String>();
	public Test() {
		for(int i=0;i<5;i++) {;
			al.add(String.valueOf(i));
			System.out.println(al.get(al.size()-1));
		}
		for(int i=0;i<5;i++) {;
		System.out.println(al.get(al.size()-1));
	}
	}
	
}

Es wird ausgegeben:

0
1
2
3
4
4
4
4
4


Warum gibt er bei der 2ten Schleife immer eine 4 aus?

 

[dahaack]

War schon spät gestern, ich zieh die Frage zurück.

Nochmal danke!

 

[SlaterB]

hier noch ein Buch:
http://www.galileocomputing.de/openbook/javainsel7/