(Swing) IndexOutOfBoundsException, aber warum?!

[Developer_X]

Hi, wenn ich den Code starte, dann komtm da eine IndexOutOfBoundsException, aber warum?

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
package Main;
 
import java.awt.Color;
import java.awt.GridLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
 
import javax.swing.*;
 
@SuppressWarnings("serial")
public class Main extends JFrame
{
    Field[] f = new Field[36];
    int i = 0;
    
    public Main()
    {
        setLayout(new GridLayout(6,6));
 
        setGame();
        
        setSize(200,200);
        setVisible(true);
    }
    public void setGame()
    {
        i = 0;
        while(i<=f.length-1)
        {
            f[i] = new Field(""+i);
            add(f[i]);
            f[i].addActionListener(new ActionListener()
            {
                public void actionPerformed(ActionEvent arg0) 
                {       
                    f[i].set();
                    f[i].setBackground(Color.black);
                    f[i].setForeground(Color.cyan);
                }           
            });         
            i++;
        }
 
    }
    public static void main(String[]args)
    {
        new Main();
    }
}

PS: Field ist eine Instanz von JButton

[SlaterB]

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
 
public class Test
{
    public static void main(String[] args)
        throws Exception
    {
 
        new Main();
    }
 
}
class Field extends JButton {
 
    public Field(String string)
    {
    }
 
    public void set()
    {
    }
    
}
class Main extends JFrame
{
    Field[] f = new Field[36];
    int i = 0;
    
    public Main()
    {
        setLayout(new GridLayout(6,6));
 
        setGame();
        
        setSize(200,200);
        setVisible(true);
    }
    public void setGame()
    {
        i = 0;
        while(i<=f.length-1)
        {
            f[i] = new Field(""+i);
            add(f[i]);
            f[i].addActionListener(new ActionListener()
            {
                public void actionPerformed(ActionEvent arg0) 
                {       
                    f[i].set();
                    f[i].setBackground(Color.black);
                    f[i].setForeground(Color.cyan);
                }           
            });         
            i++;
        }
 
    }
}

keine Exception

-----

nebenbei:
while(i<=f.length-1)

->

while(i<f.length)

[Ebenius]

Wann genau kommt die Exception? Wenn Du auf den Button drückst, oder? Warum ist i ein Feld und keine lokale Variable in einer for -Schleife? Dann könnte das nämlich gar nicht passieren. Mal sehen, ob Du's rausfindest...

Ebenius

[Ebenius]

Zitat: SlaterB

	keine Exception

Lügner

Ebenius

[Developer_X]

ja so ist keine exception, wenn man aber auf die buttons klickt, dann fällt diese exception

[M4x0r]

Tipp:
In der while Schleife fügst du zwar einen ActionListener hinzu, aber die Schleife ist durchgelaufen wenn du per Klick ein ActionEvent auslöst

[Developer_X]

Zitat: Ebenius

	Wann genau kommt die Exception? Wenn Du auf den Button drückst, oder? Warum ist i ein Feld und keine lokale Variable in einer for -Schleife? Dann könnte das nämlich gar nicht passieren. Mal sehen, ob Du's rausfindest... Ebenius

die Klasse Field brauche ich public, weil ich mit diesem Array ein paar sachen wo anders, in ner anderen methode anstellen muss

[Ebenius]

Lies meine Frage nochmal! Deine Antwort hat mit meiner Frage nichts zu tun. Ggf. ersetze das Wort "Feld" durch "Member-Variable", wenn es sich dann besser verstehen lässt.

Ebenius

[Developer_X]

hey, wenn ich am ende so mache:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 
 public void setGame()
    {
        i = 0;
        while(i<=f.length-1)
        {
            f[i] = new Field(""+i);
            add(f[i]);
            f[i].addActionListener(new ActionListener()
            {
                public void actionPerformed(ActionEvent arg0) 
                {       
                    f[i].set();
                    f[i].setBackground(Color.black);
                    f[i].setForeground(Color.cyan);
                }           
            });         
            i++;
        }
        i= 0;
    }

dann funktionierts, ein problem dabei ist nur, dass wenn man auf irgendeinen button zmum ersten mal klickt, dass dann der button f[0] und der angeklickte button sich ändern, das ist ärgerlich.

[Ebenius]

Mach i aus der Klasse raus:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
        for(int i = 0; i < f.length; i ++
        {
            final int buttonIndex = i;
            f[buttonIndex] = new Field(""+buttonIndex);
            add(f[buttonIndex]);
            f[buttonIndex].addActionListener(new ActionListener()
            {
                public void actionPerformed(ActionEvent arg0) 
                {       
                    f[buttonIndex].set();
                    f[buttonIndex].setBackground(Color.black);
                    f[buttonIndex].setForeground(Color.cyan);
                }           
            });         
        }

Ebeniu

[Developer_X]

ich will ein vier gewinnt spiel machen,
so da hatte ich mir gedacht, bevor ich alle initialisiere und dann auch noch die actionlistener einzeln mache, mach ich das ganze in ner schleife, so, und nun funktioniert das ganze nicht richtig, weil die schleife beim klicken neu durchlaufen wird, wie max40 schon gesagt hat, da dachte ich mir, ich machs so, dass ich nach der schleife "i" wieder "0" setzte.

Jetzt gehts aber wieder net

@Edit: danke ebenius

[SlaterB]

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
for(int i = 0; i < f.length; i ++
        {
            final Field fi = new Field(""+buttonIndex);
            f[buttonIndex] = fi;
            add(fi);
            fi.addActionListener(new ActionListener()
            {
                public void actionPerformed(ActionEvent arg0) 
                {       
                    fi.set();
                    fi.setBackground(Color.black);
                    fi.setForeground(Color.cyan);
                }           
            });         
        }

die Source des ActionEvents wäre auch noch ne Variante

[M4x0r]

Hallo,
was hätte es denn bringen sollen i=0 zu setzen?
wenn du zum beispiel auf den 20. Button klickst, dann wird der ActionListener des 20.Buttons aktiviert.
in diesem steht dann sowas wie field[i].... aber i ist 0 und es müsste eigentlich 20 sein.
PS: wenn du im ActionListener nicht mehr machst als das was du bisher geschrieben hast, ist es denke ich für die Performance besser wenn du einfach einen einzigen ActionListener in die klasse einbindest. da könntest du dann mit if( e.getSource() instanceof Field) arbeiten. wird wahrscheinlich wenig, wenn überhaupt einen unterschied machen, aber who knows...

EDIT: Bei Vier Gewinnt gibts üblicherweise 7 Spalten...d.h. 42 "Fields"

[Developer_X]

ich weiß, aber 6 6 gefällt mir besser, weil das dann gleich quadratisch ist,

[roque]

die Frage ist nur, ob es sich auf nem 6x6 Feld auch so gut spielen lässt...

und für jeden einzelnen Button einen Actionlistener ist, denke ich, nicht nur für die Performance schlecht sondern auch überflüssig oder? Macht doch keinen Unterschied an welche Stelle ner Spalte du klickst? Man wirft ja immer oben rein..

[M4x0r]

ja, also ich hätte es vielleicht so gemacht:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
import java.awt.Color;
import java.awt.GridLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
 
import javax.swing.JFrame;
 
public class Main extends JFrame implements ActionListener {
    private static final long serialVersionUID = 1L;
    Field[] f = new Field[36];
 
    public Main() {
        setLayout(new GridLayout(6, 6));
        setGame();
        setSize(200, 200);
        setVisible(true);
    }
 
    public void setGame() {
 
        for (int i = 0; i < f.length; i++) {
            f[i] = new Field("" + i);
            add(f[i]);
            f[i].addActionListener(this);
 
        }
 
    }
 
    @Override
    public void actionPerformed(ActionEvent e) {
        if (e.getSource() instanceof Field) {
            Field current = (Field) e.getSource();
            current.set();
            current.setBackground(Color.black);
            current.setForeground(Color.cyan);
        }
 
    }
 
    public static void main(String[] args) {
        new Main();
    }
 
}