Compiler-Fehler NullPointerException lösen

[KandaYamamotoo]

Ich erhalte bei folgendem Code folgende Fehler:

mport javax.swing.*;
import java.awt.*;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

/**
* Created by kanda on 16.05.17.
*/
public class PlayGround {
    private JPanel jPanel2;
    private JButton button1;
    private Spielkarte[][] Spielfeld = new Spielkarte[3][3];


    public PlayGround() {
        button1.addActionListener(new ActionListener() {
            @Override
            public void actionPerformed(ActionEvent actionEvent) {
                generate();
            }
        });
    }

    public void generate() {
        for (int i = 0; i < 3; i++) {
            for (int j = 0; j < 3; j++) {
                Spielfeld[i][j] = new Spielkarte();
                Spielfeld[i][j].setBounds(50 + (i * 30), 80 + (j * 30), 30, 30);
                Spielfeld[i][j].setBackground(new Color(0, 0, 0));
                Spielfeld[i][j].setForeground(new Color(0, 0, 0));
                Spielfeld[i][j].setMargin(new Insets(2, 2, 2, 2));
                Spielfeld[i][j].setText("");
                jPanel2.add(Spielfeld[i][j]);
            }
        }
    }



    public static void main(String[] args) {
        JFrame frame = new JFrame("PlayGround");
        frame.setContentPane(new PlayGround().jPanel2);
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.pack();
        frame.setVisible(true);
        }
    }

Fehler:

Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
    at com.intellij.uiDesigner.core.GridLayoutManager.addLayoutComponent(GridLayoutManager.java:134)
    at java.awt.Container.addImpl(Container.java:1127)
    at java.awt.Container.add(Container.java:417)
    at PlayGround.generate(PlayGround.java:33)
    at PlayGround$1.actionPerformed(PlayGround.java:19)
    at javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:2022)
    at javax.swing.AbstractButton$Handler.actionPerformed(AbstractButton.java:2348)
    at javax.swing.DefaultButtonModel.fireActionPerformed(DefaultButtonModel.java:402)
    at javax.swing.DefaultButtonModel.setPressed(DefaultButtonModel.java:259)
    at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(BasicButtonListener.java:252)
    at java.awt.Component.processMouseEvent(Component.java:6533)
    at javax.swing.JComponent.processMouseEvent(JComponent.java:3324)
    at java.awt.Component.processEvent(Component.java:6298)
    at java.awt.Container.processEvent(Container.java:2236)
    at java.awt.Component.dispatchEventImpl(Component.java:4889)
    at java.awt.Container.dispatchEventImpl(Container.java:2294)
    at java.awt.Component.dispatchEvent(Component.java:4711)
    at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4888)
    at java.awt.LightweightDispatcher.processMouseEvent(Container.java:4525)
    at java.awt.LightweightDispatcher.dispatchEvent(Container.java:4466)
    at java.awt.Container.dispatchEventImpl(Container.java:2280)
    at java.awt.Window.dispatchEventImpl(Window.java:2746)
    at java.awt.Component.dispatchEvent(Component.java:4711)
    at java.awt.EventQueue.dispatchEventImpl(EventQueue.java:758)
    at java.awt.EventQueue.access$500(EventQueue.java:97)
    at java.awt.EventQueue$3.run(EventQueue.java:709)
    at java.awt.EventQueue$3.run(EventQueue.java:703)
    at java.security.AccessController.doPrivileged(Native Method)
    at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:80)
    at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:90)
    at java.awt.EventQueue$4.run(EventQueue.java:731)
    at java.awt.EventQueue$4.run(EventQueue.java:729)
    at java.security.AccessController.doPrivileged(Native Method)
    at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:80)
    at java.awt.EventQueue.dispatchEvent(EventQueue.java:728)
    at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:201)
    at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:116)
    at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:105)
    at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:101)
    at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:93)
    at java.awt.EventDispatchThread.run(EventDispatchThread.java:82)

Wie löse ich das Problem?

 

[mrBrown]

Wie löse ich das Problem?

In dem du uns mitteilst, an welcher Stelle im Code die Exception geworfen wird 😉


Meine Glaskugel sagt, jPanel2 ist null, weil du das nie initialisierst 😉

 

[JStein52]

Im gezeigten Code wird jPanel2 nirgends erzeugt. Das heisst dies ist vermutlich null

 

[Flown]

NPE fliegt schon im Konstruktor, da button1 null ist, aber ein ActionListener hinzugefügt wird.

 

[JStein52]

NPE fliegt schon im Konstruktor, da button1 null ist

Hätte ich auch gedacht. Aber der Stacktrace sagt ja sie fliegt in der Methode generate() ??!!

 

[KandaYamamotoo]

Folgender Stand: Wenn ich das Programm so ausführe öffnet sich ein Fenster und der Button1 ist zu sehen. Sobald dieser betätigt wird springt der Fehler auf, aber eine Spielkarte wird erstellt. (wie in create() beschrieben). Ab da ist dann Schluss.
Ich verwende die IntelliJ IDEA. Dafür, dass alle Elemente der GUI erstellt werden ist doch frame.pack() (Zeile 43) verantwortlich, oder?

 

[JStein52]

Dafür, dass alle Elemente der GUI erstellt werden ist doch frame.pack() (Zeile 43) verantwortlich, oder

Natürlich nicht. pack() ist dafür da die Elemente anzuordnen, aber doch nicht um sie zu erzeugen. Und wenn das da oben dein ganzer Code ist dann ist es mir schon ein Rätsel wo der Button1 herkommt.

 

[KandaYamamotoo]

Alles klar. Ich bin aus einer früheren Entwicklungsumgebung aus der Schulzeit gewohnt, dass GUI-Objekte, die man im Designer anordnet, automatisch im Quellcode erzeugt werden. Das heißt, dass ich das erst einmal von Hand nachholen muss.

 

[MBadzura]

pack() (aus der Klasse Windows vererbt) dient nur dazu, deinen Frame eine optimal Größe zu verpassen, d. h. die hinzugefügten Elemente so zusammenzustellen, dass diese am wenigstens Platz benötigen (theoretisch zumindest). Das hinzufügen der Elemente muss schon vorher passieren.

 

[JStein52]

GUI-Objekte, die man im Designer anordnet,

Das ist jetzt aber ein neuer Gesichtspunkt. Hast du das mit dem Designer gemacht ? Kann es dann sein dass es noch Code dazu gibt den du uns nicht gezeigt hast ? In NetBeans ist der Code der vom Designer erstellt wird versteckt und muss erst aufgeklappt werden damit man ihn sieht

Edit: das würde auch dazu passen dass der Button1 angezeigt wird. Der taucht ja nicht aus einem Wurmloch einfach so auf.

 

[KandaYamamotoo]

Ich habe im Folgenden neu angesetzt. Hier sind alle Klassen. Ich habe mit dem Gui-Designer lediglich ein JPanel in einem JFrame erstellt.
Mein Ziel ist es ein Feld aus Karten generieren zu lassen. Das Feld soll generiert werden, sobald das Fenster geöffnet wird. Ich generiere mit der Methode generate() das Feld. Wenn es beim Starten geöffnet werden soll müsste es ja in die main Methode. Dann bekomme ich aber den Fehler: non-static variable this cannot be referenced from a static context.
Wie muss ich vorgehen?

Klasse Spielfenster:

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

/**
* Created by kanda on 17.05.17.
*/
public class Spielfenster {
    //Anfang Attribute
    private Karte[][] Spielfeld = new Karte[3][3];
    private JPanel jPanel1;
    //Ende Attribute
    //Anfang Methoden
    public void create(){
        for(int zeile = 0; zeile < 3; zeile++){
            for(int spalte = 0; spalte < 3; spalte++){
                Spielfeld[zeile][spalte] = new Karte();
                Spielfeld[zeile][spalte].setBounds(100 + (zeile * 30), 70 + (zeile * 30), 30, 30);
                Spielfeld[zeile][spalte].setBackground(Color.BLACK);
                Spielfeld[zeile][spalte].setText(zeile + " " + spalte);
                jPanel1.add(Spielfeld[zeile][spalte]);
            }
        }
    }

    //Ende Methoden


    public static void main(String[] args) {
        JFrame frame = new JFrame("Spielfenster");
        frame.setContentPane(new Spielfenster().jPanel1);
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.pack();
        frame.setVisible(true);
        JPanel jPanel1 = new JPanel();
    }
}

Klasse Karte (hier die einzelnen Spielkarten aus denen das Feld erstellt werden soll):

import javax.swing.*;
import java.awt.*;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

/**
* Created by kanda on 17.05.17.
*/
public class Karte extends JButton{
    private boolean gedrueckt;
    private String inhalt;

    public boolean isGedrueckt() {
        return gedrueckt;
    }

    public void setGedrueckt(boolean gedrueckt) {
        this.gedrueckt = gedrueckt;
    }

    public String getInhalt() {
        return inhalt;
    }

    public void setInhalt(String inhalt) {
        this.inhalt = inhalt;
    }

    public Karte() {
        gedrueckt = false;
        addActionListener(new ActionListener() {
            @Override
            public void actionPerformed(ActionEvent actionEvent) {
                if(isGedrueckt() == true){
                    setBackground(new Color(255,255,255));
                    setForeground(new Color(0,0,0));
                    setGedrueckt(true);
                }
            }
        });
    }
}

 

[JStein52]

jPanel1 ist keine static-Variable. Du könntest das was du jetzt in main machst in einen Konstruktor von Spielfenster verlagern. Und in main legst du dann einfach eine neue Instanz von Spielfenster an.

 

[BadBat]

Ist swing nicht schon abgelöst von fx?
Da geht auch sowas mit Lambdas

eventButton.setOnMouseClicked(event -> System.out.println(event.getButton()));

 

[mrBrown]

Ist swing nicht schon abgelöst von fx?
Da geht auch sowas mit Lambdas

eventButton.setOnMouseClicked(event -> System.out.println(event.getButton()));

Das geht auch in Swing mit Lambdas, die haben nichts mit javaFx zu tun^^

 

[BadBat]

Das geht auch in Swing mit Lambdas, die haben nichts mit javaFx zu tun^^

Hat Swing sonst irgendwelche vorteile? Also lohnt das nochmal rückwärts zu lernen?

 

[JStein52]

Also lohnt das nochmal rückwärts zu lernen?

Drei viertel aller GUI's benutzen SWING. Und ob JavaFX der Weisheit letzter Schluss war, darüber streitet die Fachwelt noch. Da kannst du dir Artikel Pro und Contra reinziehen, je nach dem was du gerne hättest.

 

[Neumi5694]

Alles klar. Ich bin aus einer früheren Entwicklungsumgebung aus der Schulzeit gewohnt, dass GUI-Objekte, die man im Designer anordnet, automatisch im Quellcode erzeugt werden.

Das ist in Netbeans und Eclipse genau gleich.

...Das heißt, dass ich das erst einmal von Hand nachholen muss.

nö
Aber du kannst sie natürlich erst verwenden, NACHDEM der automatisch generierte Code auch ausgeführt wurde.
Netbeans erstellt für Swing eine initComponents-Methode, die muss im Konstruktor als aufgerufen werden (steht im Normalfall auch an erster Stelle, wenn man nicht reinpfuscht), bevor man die Elemente verwenden kann.