Problem mit Methodenauruf

[Nathy]

Guten Tag

Ich habe ein Problem mit einem Methodenaufruf und weiss nicht warum.

public class GUI_Tippabgabe extends Frame implements WindowListener, ActionListener {
    
    private DBConnector dbc;
    Tippabgabe myTipp;
    Label lblTitel = new Label("Tippabgabe");
    Button speichern = new Button("Speichern");
    
    /** Creates a new instance of GUI_Tippabgabe */
    public GUI_Tippabgabe(DBConnector dbconn) 
    {
        int anzahl = 0;
        this.dbc = dbconn;        
        setTitle("Tippabgabe");
        setSize(1000,300);
        
        Tippabgabe myTipp = new Tippabgabe(dbconn);
        anzahl = myTipp.fillArrayLists();
        System.out.println(anzahl);
        
        Panel mainPanel = new Panel();
	mainPanel.setLayout(new BorderLayout());
	
	Panel labelPanel = new Panel();
	labelPanel.setLayout(new BorderLayout());
	labelPanel.add(lblTitel, BorderLayout.CENTER);
	
        
        Panel bottomPanel = new Panel();
        bottomPanel.setLayout(new BorderLayout());
        speichern.addActionListener(this);
        bottomPanel.add(speichern, BorderLayout.EAST);
        
        mainPanel.add(labelPanel, BorderLayout.NORTH);
        mainPanel.add(bottomPanel, BorderLayout.SOUTH);
	add(mainPanel);
        this.addWindowListener(this);      
        setVisible(true);
    }
    
    public void actionPerformed(ActionEvent e)
    {
        if(e.getActionCommand()=="Speichern"){
          myTipp.test();
        }
    }

Das Problem liegt bei der Methode myTipp.test() (Macht nur ein System.out.println("test")
Wenn ich sie im Konstruktor aufrufe, gibt es keine Problem. Wenn ich sie aber im public void actionPerformed aufrufe kommt folgende Exception:

Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
        at tippspiel.GUI_Tippabgabe.actionPerformed(GUI_Tippabgabe.java:99)
        at java.awt.Button.processActionEvent(Button.java:388)
        at java.awt.Button.processEvent(Button.java:356)
        at java.awt.Component.dispatchEventImpl(Component.java:3955)
        at java.awt.Component.dispatchEvent(Component.java:3803)
        at java.awt.EventQueue.dispatchEvent(EventQueue.java:463)
        at java.awt.EventDispatchThread.pumpOneEventForHierarchy(EventDispatchThread.java:242)
        at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:163)
        at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:157)
        at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:149)
        at java.awt.EventDispatchThread.run(EventDispatchThread.java:110)

Kann mir jemand sagen, wo das Problem liegt?

Gruss und danke

 

[Der Müde Joe]

Zeile 17

Tippeingabe t...

muss

this.myTip = ....

heissen

 

[L-ectron-X]

Sind noch ein paar Dinge drin, probiers mal so:

public class GUI_Tippabgabe extends Frame implements ActionListener {

    private DBConnector dbc;
    private Tippabgabe myTipp;
    private Label lblTitel = new Label("Tippabgabe");
    private Button speichern = new Button("Speichern");

    /** Creates a new instance of GUI_Tippabgabe */
    public GUI_Tippabgabe(DBConnector dbconn)
    {
        int anzahl = 0;
        this.dbc = dbconn;
        setTitle("Tippabgabe");
        setSize(1000,300);

        myTipp = new Tippabgabe(dbconn);
        anzahl = myTipp.fillArrayLists();
        System.out.println(anzahl);

        Panel mainPanel = new Panel(new BorderLayout());

        Panel labelPanel = new Panel(new BorderLayout());
        labelPanel.add(lblTitel, BorderLayout.CENTER);

        Panel bottomPanel = new Panel(new BorderLayout());
        speichern.addActionListener(this);
        bottomPanel.add(speichern, BorderLayout.EAST);

        mainPanel.add(labelPanel, BorderLayout.NORTH);
        mainPanel.add(bottomPanel, BorderLayout.SOUTH);
        add(mainPanel);

        this.addWindowListener(new WindowAdapter() {
           public void windowClosing(WindowEvent e) {
              System.exit(0);
           }
        });

        setVisible(true);
    }

    public void actionPerformed(ActionEvent e)
    {
        if(e.getActionCommand().equals("Speichern")) {
           myTipp.test();
        }
    }
}

 

[Der Müde Joe]

das Problem ist

das globale Tippeingabe ist immer noch null

du machst eine lokale Variable myTipp neu
anstatt die Vorhandene Globale Variable zu
initialisieren.
das geschieht mit
this.myTipp = new .....

 

[L-ectron-X]

Ich glaub, da verwechselst du was, aber ich will mich nicht mit dir streiten.
OMG... :roll:

 

[Der Müde Joe]

hehe.....gesehen... thx

 

[Nathy]

Sorry, ich verstehe es nicht ganz. Wo liegt jetzt das Problem?

Gruss und danke

 

[Der Müde Joe]

Der Aufruf

Tippabgabe myTipp = new Tippabgabe(dbconn);

erzeugt eine NEUE LOKALE Variable.
diese ist an Ende des Konstruktors wieder weg (Garbage Collector)

Dabei bleibt die GLOBALE Variable myTipp unberührt.
also ist sie immer noch = null.

bei aPer() benutzt du dann global.myTipp
da diese aber null ist: java.lang.NullPointerException

du musst diese zuertst initialisieren
also

this.myTipp (die GLOBLAE) = new Tippabgabe(dbconn);

dann ist die globalee nicht mehr null und es funzt..
[/code]

 

[L-ectron-X]

Du erzeugst im Konstruktor ein Objekt vom Typ Tippabgabe und weist dieses der Referenz myTipp zu. Außerhalb des Konstruktors ist aber die Referenz noch null, weil es ein ganz andere Referenz ist. Variablen, die in Methoden deklariert wurden sind außerhalb der Methode nicht sichbar. Daher bleibt deine globale Referenz auch weiter null.

 

[Nathy]

Ich erzeuge ja in Zeile 5 eine globale Variable, welche ich noch nicht instanziere.
In Zeile 17 mache aber ich doch eine Instanz oder nicht?
Ich habe bereits versucht in Zeile 5 die Variable zu instanzieren. Das geht aber nicht, weil ich da noch kein dbconn habe.
Kannst du mir bitte genau den Code senden, damit ich es verstehe.

Gruss und danke

 

[Der Müde Joe]

public class MyClass{

private Tippabgabe myTippGLOBAL;

public MyClass(DBCon dbConn){

Tippabgabe myTippLOKAL = new Tippabgabe(dbConn);  //neue Lokale Variable myTipp

this.myTippGLOBAL = new Tippabgabe(dbConn); //initialisiert die Globale variable

}//hier ist die Lokale Variable wieder weg

}

public void useGlobal(){

myTippGLOBAL.justDoIt();

}

hoffe das bring dich weiter

 

[Eldar]

lass einfach den Typ in Zeile 17 (Tippabgabe) weg.
statt
Tippabgabe myTipp = new Tippabgabe(dbconn);
nur
myTipp = new Tippabgabe(dbconn);