Zugriff auf Objekte aus doppelten Array

[Mr. Q]

Guten Tag,

ich arbeite an einem Programm, in dem zwei Listen auf dieselben Objekte zugreifen(sollen). Es handelt sich um ein Sudoku-Programm und die 81 Kästchen sollen sowohl zeilen- und spaltenweise über das Array Felder, als auch Gruppenweise (diese 9er-Blöcke) über das Array Gruppenkatalog, das die neun Blöcke (GruppeA- GruppeI) enthält, die wieder je 9 Kästchen enthalten.

Ich habe das Programm mal aufs nötigste zusammengefasst, es enthält jetzt nur noch 2 Blöcke und ergibt keinen Sinn mehr, aber die Fehlermeldung bleibt die gleiche.

Hier das Programm (die Kästchen heißen dort Zellen):

import java.awt.GridLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;

public class Main extends JFrame implements ActionListener{
	
	Zelle Felder[][] = new Zelle[9][9];
	
	JButton eins = new JButton("1");
	
	Zelle GruppeA[]=new Zelle[9];
	Zelle GruppeB[]=new Zelle[9];
	Zelle[][] Gruppenkatalog= new Zelle[9][9];

	JLabel zwei = new JLabel();
	JLabel drei = new JLabel();
	JLabel vier = new JLabel();
	JLabel fünf = new JLabel();
	JLabel sech = new JLabel();
	
	
	public Main()
	{
		setLayout(new GridLayout(4,6));
		for(int i=0;i<3;i++)
		{
			for(int j=0;j<6;j++)
			{
				Felder[i][j]=new Zelle(i,j);
				add(Felder[i][j]);
				
			}
			
		}
		add(eins);
		add(zwei);
		add(drei);
		add(vier);
		eins.addActionListener(this);
		
		Zelle GruppeA[]={Felder[0][0], Felder[0][1], Felder[0][2], 
						 Felder[1][0], Felder[1][1], Felder[1][2], 
						 Felder[2][0], Felder[2][1], Felder[2][2]};

		Zelle GruppeB[]={Felder[0][3], Felder[0][4], Felder[0][5], 
				 		 Felder[1][3], Felder[1][4], Felder[1][5], 
				 		 Felder[2][3], Felder[2][4], Felder[2][5]};

		
		Zelle[] Gruppenkatalog[]={GruppeA, GruppeB};
		
		setVisible(true);
		setResizable(true);
		pack();
	}

	
	
public static void main (String[]args){
		new Main();
}

	public void actionPerformed(ActionEvent arg0) {
		
		if(arg0.getSource()==eins){	
			
			if(Prüfung()==false){                            //(Das ist Zeile 70)
				System.out.println("Falsch");
			}else{
				System.out.println("Richtig");
			}
		}
	}
	
	public boolean Prüfung(){
		boolean fehlerfrei=true;
		int gruppe=1;
		int Ziffer=1;
		int element=2; 
		
		if(Gruppenkatalog[gruppe][element].Belegung==Ziffer){    //Das ist Zeile 84
			fehlerfrei=false;
		}
		return fehlerfrei;
	}
}

und die Klasse Zelle:

public class Zelle extends JButton{

	int spalte;
	int zeile;
	public Zelle(int a, int b){
		spalte=a;
		zeile=b;
	}
	
	public Dimension getPreferredSize(){
		return(new Dimension(70,70));    }
	
	int Belegung=0;
	
}

Und die Fehlermeldung:

Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
	at Main.Prüfung(Main.java:84)
	at Main.actionPerformed(Main.java:70)
	at javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:2012)
	at javax.swing.AbstractButton$Handler.actionPerformed(AbstractButton.java:2335)
	at javax.swing.DefaultButtonModel.fireActionPerformed(DefaultButtonModel.java:404)
	at javax.swing.DefaultButtonModel.setPressed(DefaultButtonModel.java:259)
	at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(BasicButtonListener.java:253)
	at java.awt.Component.processMouseEvent(Component.java:6108)
	at javax.swing.JComponent.processMouseEvent(JComponent.java:3267)
	at java.awt.Component.processEvent(Component.java:5873)
	at java.awt.Container.processEvent(Container.java:2105)
	at java.awt.Component.dispatchEventImpl(Component.java:4469)
	at java.awt.Container.dispatchEventImpl(Container.java:2163)
	at java.awt.Component.dispatchEvent(Component.java:4295)
	at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4461)
	at java.awt.LightweightDispatcher.processMouseEvent(Container.java:4125)
	at java.awt.LightweightDispatcher.dispatchEvent(Container.java:4055)
	at java.awt.Container.dispatchEventImpl(Container.java:2149)
	at java.awt.Window.dispatchEventImpl(Window.java:2478)
	at java.awt.Component.dispatchEvent(Component.java:4295)
	at java.awt.EventQueue.dispatchEvent(EventQueue.java:604)
	at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:275)
	at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:200)
	at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:190)
	at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:185)
	at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:177)
	at java.awt.EventDispatchThread.run(EventDispatchThread.java:138)

Ich nehme jetzt mal an das entweder Gruppenkatalog nicht richtig deklariert wird, oder 2 Arrays nicht das selbe (sondern nur das gleiche) Objekt enthalten können.
Nur wie geht das richtig?

 

[eRaaaa]

 int element=9;

Du greifst auf ein Element zu das es nicht gibt. Das Array fängt bei 0 an, demnach ist der größte Index bei dir 8 (0-8) und nicht 9 !

 

[Mr. Q]

ok, sorry mein fehler beim kürzen ich änder das mal. Das ändert aber nichts an der Fehlermeldung.

 

[XHelp]

int gruppe=4;

du hast doch nur 2?

 

[Mr. Q]

Ok, solangsam wirds peinlich...

Gruppe muss auch kleiner sein, das ist aber auch nicht der Hauptfehler, mit

int gruppe=1

verschwindet die Fehlermeldung noch nicht.

 

[XHelp]

Und in der Expection steht genau das gleiche, auch mit der 9?

 

[Michael...]

Hatte eigentlich eher eine NullPointerException erwartet, da die Felder von

Gruppenkatalog

auf die Du in der Methode Prüfung zugreifst, nicht initialisiert sind.
Du hast folgende Arrays sowohl als lokale Variablen im Konstruktor und Instanzvariablen der Klasse:
Gruppenkatalog, GruppeA, GruppeB

Und nur die im Konstruktor werden gefüllt --> keine lokalen Variablen im Konstruktor erzeugen und nur die Instanzvariablen initialisieren.

 

[Mr. Q]

Oh, nein überhaupt nicht. Jetzt ist es endlich wie schon im ungekürzten Programm eine NullPointerException.

 

[Mr. Q]

Momentmal, ich fülle doch "Felder" auch erst im Konstruktor und auf dessen Elemente kann ich zugreifen...

 

[XHelp]

Nö, tust du nicht. Du erstellst im Konstruktor Methodenvariablen, die du dann befüllst. Außerhalb der Methode existieren die nicht.
(Mach die Typendeklarationen weg, dann sollte es klappen)

 

[Michael...]

Momentmal, ich fülle doch "Felder" auch erst im Konstruktor und auf dessen Elemente kann ich zugreifen...

Ja Felder ist die einzige Instanzvariable deren Felder initialisiert werden. Bei den anderen dreien erzeugst Du im Konstruktor neue Variablen mit dem selben Namen wie die der Instanzvariablen
--> Entferne die Typdeklarationen vor den Variablen im Konstruktor

 

[Mr. Q]

So?

public class Main extends JFrame implements ActionListener{
    
    Zelle Felder[][] = new Zelle[9][9];
    
    [...]

    Zelle GruppeA[]=new Zelle[9];
    Zelle GruppeB[]=new Zelle[9];
    Zelle[][] Gruppenkatalog= new Zelle[9][9];
 
    [...]
    
    public Main()
    {
        setLayout(new GridLayout(4,6));
        for(int i=0;i<3;i++)
        {
            for(int j=0;j<6;j++)
            {
                Felder[i][j]=new Zelle(i,j);
                add(Felder[i][j]);
                
            }
            
        }

        [...] 

        GruppeA[]={Felder[0][0], Felder[0][1], Felder[0][2],    //Marke 1
                         Felder[1][0], Felder[1][1], Felder[1][2], 
                         Felder[2][0], Felder[2][1], Felder[2][2]};
 
        GruppeB[]={Felder[0][3], Felder[0][4], Felder[0][5],    //Marke 2:
                         Felder[1][3], Felder[1][4], Felder[1][5], 
                         Felder[2][3], Felder[2][4], Felder[2][5]};
 
        Gruppenkatalog[]={GruppeA, GruppeB};                   //Marke 3
        
        setVisible(true);
        setResizable(true);
        pack();
    }

liefert folgende Fehlermeldungen:
Marke 1:
Multiple markers at this line
- Syntax error on token "]", VariableDeclaratorId expected after this token
- Type mismatch: cannot convert from Zelle to GruppeA (9mal)
- GruppeA cannot be resolved to a type

Marke 2:
Multiple markers at this line
- Type mismatch: cannot convert from Zelle to GruppeB (9mal)
- Duplicate local variable $missing$
- GruppeB cannot be resolved to a type
- Syntax error on token "]", VariableDeclaratorId expected after this token

Marke 3:
Multiple markers at this line
- Type mismatch: cannot convert from Zelle[] to Gruppenkatalog
- Duplicate local variable $missing$
- Type mismatch: cannot convert from Zelle[] to Gruppenkatalog
- Syntax error on token "]", VariableDeclaratorId expected after this token
- Gruppenkatalog cannot be resolved to a type

Übrigens:
Mir fällt grad auf, das es funktioniert, wenn man die Zellen im Konstruktor einzeln initialisiert, also

GruppeA[0]=Felder[0][0];
GruppeA[1]=Felder[0][1];
...

 

[SlaterB]

bei Verwendung der Array-Variable keine []

deshalb ist es generell besser, auch die Variablen schon anders zu deklarieren, das [] beim Typ, nicht beim Variablennamen:

int[] ar = new int[5];
ar[4] = 45; // ok, hier doch mit []
sort(ar); // kein []

wenn du erst noch ein Objekt zuweist, dann musst du vorher kein leeres Array erstellen

Zelle[][] felder = null;
felder = getArray();
felder = new Zelle[][] {..., ..}; // das new Zelle[][] muss leider auch hin, bisschen komisch in Java

Variablen klein schreiben!

 

[Mr. Q]

durch weglassen der [] im Konstruktor ergibt sich ein neuer Fehler:

Array constants can only be used in initializers

 

[XHelp]

Zeig mal genau den Code

 

[Marco13]

Das erscheint AFAIK bei sowas wie

int array[];
array = {1,2,3}

Alternative vielleicht:

int array[];
array = new int[]{1,2,3};

 

[Wuella]

Mr. Q du sollst nicht so viel pumpen und dich auf bambamscorner aufhalten, dann klappts vielleicht auch besser mit Java