JPanel-Array in GridLayout

[Chris22]

Hallo, ich habe ein Problem mit einer selbstgeschriebenen Klasse die ich aus JPanel abgeleitet habe
erstmal der Code :

Die selbstgeschriebene Klasse

import javax.swing.JLabel;
import javax.swing.JPanel;

public class LigaSchütze extends JPanel {
    
    static String name;
    static int[] serie = new int[4];
        
    public LigaSchütze () {
        
                add(new JLabel(name));                
                for (int x = 0; 4 > x; x++) {
                JLabel Serie = new JLabel(""+serie[x]);
                add(Serie);
                }                
                add(new JLabel(""+ getSumme()));
    }

    public void setSerie (int serie, int ringe) {
        this.serie[serie-1] = ringe;
    }
    
    public void setSchützenName (String name) {
        this.name = name;
    }
}

Die Hauptklasse:

import java.awt.*;
import javax.swing.*;

public class Liga extends JFrame{
    
    String[][] Name = { {"Arthur", "Marvin","Trillian","Zaphod","Ford"},
                                   {"Tim", "Tom","Theo","Tanja","Tamara"}};
    
            
    LigaSchütze[] Heim = new LigaSchütze[5];
    LigaSchütze[] Gast = new LigaSchütze[5];
    
    public static void main(String[] args)  {
    new Liga ();
    }
    
    public Liga () {
        super ("Bezirksliga");
        setSize(Toolkit.getDefaultToolkit().getScreenSize());
        setDefaultCloseOperation(EXIT_ON_CLOSE);
        
            JPanel Körperteil = new JPanel();
            Körperteil.setLayout(new GridLayout(5, 2));
            
                for (int x = 0; 5 > x; x++) {
//                        Heim[x] = new LigaSchütze();           
                        Heim[x].setSchützenName(Name[0][x]); 
                        Heim[x].setSerie(1, x*10);
                    Körperteil.add(Heim[x]);
                        
//                        Gast[x] = new LigaSchütze();          
                        Gast[x].setSchützenName(Name[1][x]);
                        Gast[x].setSerie(1, x*10+5);
                    Körperteil.add(Gast[x]);
                }
        
        add(Körperteil, BorderLayout.CENTER);   
        setVisible(true);        
    }
 }

Wenn ich das Kompiliere bricht er danach ab mit

run:
Exception in thread "main" java.lang.NullPointerException
at de.schafschütze.Liga.<init>(Liga.java:32)
at de.schafschütze.Liga.main(Liga.java:14)
Java Result: 1

Wenn ich mein LigaSchützen in der for-Schleife neu instanziere (Zeilen 26 & 31) bricht das Programm zwar nicht ab aber, die Panels scheinen "verschoben". d.h. alle Panels sind an der Stelle wo sie sein sollen, nur das erste Panel ist leer, im zweiten sind die Angaben die im ersten sein sollten, im dritten die die im zweiten sein sollten,.. usw die informationen die im zehnten (letzten) stehen sollten tauchen gar nicht auf.
Außerdem wenn ich die in der Schleife neu instanziere werden die Arrays aus Zeile 10&11 nicht überschrieben ??

Ich bedanke mich schon mal im Vorraus.

MfG Chris

 

[SlaterB]

> Wenn ich mein LigaSchützen in der for-Schleife neu instanziere (Zeilen 26 & 31)

ohne das gehts nun nicht, Exception, müssen wir nicht weiter drüber reden

> nur das erste Panel ist leer, im zweiten sind die Angaben die im ersten sein sollten

worauf beziehst du dich genau, den Namen?
kein Wunder, wenn du den Namen erst eine Zeile später setzt (Zeile 28 zu 27 z.B.)

> Außerdem wenn ich die in der Schleife neu instanziere werden die Arrays aus Zeile 10&11 nicht überschrieben ??

wie stellst du das fest?

bzw. falls das eine Frage war, ob da überschrieben wird:
nun, vorher war das Array leer, durch die Schleife wird etwas eingetragen

 

[Chris22]

Die Sache, mit dem verschobenen Inhalt habe ich gelöst, ich hatte ja die Variablen in "LigaSchütze" als static, aber in jedem Objekt sollten die ja anders sein. Also habe ich das static rausgenommen.

Jetzt zum Problem das die zehn erstelten Objekte aus LigaSchütze "leer" sind:

am Anfang der Klasser werden die Objekte deklariert:

LigaSchütze[] Heim = new LigaSchütze[5];
LigaSchütze[] Gast = new LigaSchütze[5];

und später mit einer for-Schleife in das GridLayout eingefügt:

for (int x = 0; 5 > x; x++) {
                    
       Heim[x] = new LigaSchütze();         
       Heim[x].setSchützenName(Name[0][x]); 
       Heim[x].setSerie(1, x*10);
       System.out.println(Heim[x].getSchützenName());
       add(Heim[x]);
                        
       Gast[x] = new LigaSchütze();          
       Gast[x].setSchützenName(Name[1][x]);
       Gast[x].setSerie(1, x*10+5);
       add(Gast[x]);
}

Wenn ich die Inhalte per System.out.println abfrage bekomme ich aus den Objekten die korrekten Werte (Zeile 6) jedoch im Fenster sind die Werte leer (kein Name wird angezeigt, alle Serien 0). Wenn ich die Werte z.B. den Namen über den Konstruktor, einsetze, sind diese im Fenster korrekt dargestellt. Aber auch in dem Fall werden die Nachträglich mit setSerie geänderten Werte nicht angezeigt. Aber auch die sind wieder korrekt mit System.out.println(getSerie) abfragbar. repaint und validate ändern beide nicht die Darstellung.

Es ist keine Lösung die einzelnen Serien über den Konstruktor einzusetzen, da die einzelnen Serien erst im laufe des Programmes gesetzt werde.

 

[SlaterB]

mit setSerie/ setSchützenName usw setzt du nur Attribute innerhalb von LigaSchütze,
die JLabel merken davon gar nix, bei deren Erstellung übergibst du einmalig den anzuzeigenden Wert, dieser wird aber nicht automatisch aktualisiert,

an Stelle von oder zusätzlich zum Setzen der Attribute in LigeSchütze musst du auch die JLabel mit einem aktiven Aufruf
jLabel.setText(neuerText) aktualisieren,
dazu musst du auch die JLabel in einem Array speichern usw.

 

[Chris22]

Vielen Dank jetzt funktioniert das so wie ich es mir gedacht habe.