Compiler-Fehler Variable nicht mit null initialisiert, trotzdem: NullPointerException

[DaCrazyJavaExpert]

Hey,
in meinem Code habe ich diese Variable: this.priceShopCursor = shopData.getPrice_Shop_Cursor(); Auf sie wird auch die Exception verwiesen.

Spoiler: Hier die koplette Exception:

java.lang.NullPointerException
at de.m_ver.cookieclicker.gui.CookieShop.initialVariables(CookieShop.java:62)
at de.m_ver.cookieclicker.gui.CookieShop.<init>(CookieShop.java:42)
at de.m_ver.cookieclicker.gui.CookieClickerGUI.<init>(CookieClickerGUI.java:82)
at de.m_ver.cookieclicker.main.Main$1.run(Main.java:15)
at java.awt.event.InvocationEvent.dispatch(Unknown Source)
at java.awt.EventQueue.dispatchEventImpl(Unknown Source)
at java.awt.EventQueue.access$500(Unknown Source)
at java.awt.EventQueue$3.run(Unknown Source)
at java.awt.EventQueue$3.run(Unknown Source)
at java.security.AccessController.doPrivileged(Native Method)
at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(Unknown Source)
at java.awt.EventQueue.dispatchEvent(Unknown Source)
at java.awt.EventDispatchThread.pumpOneEventForFilters(Unknown Source)
at java.awt.EventDispatchThread.pumpEventsForFilter(Unknown Source)
at java.awt.EventDispatchThread.pumpEventsForHierarchy(Unknown Source)
at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
at java.awt.EventDispatchThread.run(Unknown Source)

Die Exception java.lang.NullPointerException sagt doch, dass eine Variable mit null initialisiert ist, was bei mir nicht der Fall ist, meine Variable ist mit shopData.getPrice_Shop_Cursor(); initialisiert. Dieser Wert wird aus meiner Datenbank-Klasse geholt price_Shop_Cursor = resultSet.getInt("PRICE_SHOP_CURSOR"); Diese Variable enhält einen Wert der Datenbank, welcher definitif auch nicht null ist. Dies hab ich mit einem sysout auch üerprüfen können: Preis Cursor: 400(Beim allerersten Start des programmes wird der Wert in der Datenbank auf 400 gesetzt) Weiß zufällig jemand, woran das Problem liegen kann? Vielen Dank für potentielle Hilfe!

 

[Robat]

Sicher, dass die Variable shopData auch nicht null ist?
Schau dir dein Code mal an und Überleg ob die Reihenfolge der Aufrufe stimmt.

 

[DaCrazyJavaExpert]

Sicher, dass die Variable shopData auch nicht null ist?
Schau dir dein Code mal an und Überleg ob die Reihenfolge der Aufrufe stimmt.

Ah stimmt, ich hab die Inztanz der Klasse nicht initialisiert. Ich dachte das wäre bei Klassen nicht umbedingt nötig. Allerdings weiß ich dann nicht, wie ich eine Inztanz der Klasse erstellen kann ohne sie damit aufzurufen.

 

[Robat]

wie ich eine Inztanz der Klasse erstellen kann ohne sie damit aufzurufen.

Was meinst du damit?

 

[DaCrazyJavaExpert]

Wenn man eine Klasse hat welche einen Kontruktor mit Befehlen hat, diese Klasse dann in einer anderen Klasse inztanziert, TestKlasse irgendeineKlasse = new TestKlasse()
wird die Klasse/der Konstruktor aufgerufen.
Und genau dass will ich nicht. Ich rufe diese Klasse schon in einer anderen Klasse auf, und doppelt aufrufen führ nur zu Fehlern. Ich brauche bloß eine Instatztvariable um aif die Werte ihrer Klasse zugreifen zu können.

 

[Robat]

Dann musst du die erste Instanz der Klasse mittels Dependency Injection in die Klasse übergeben, wo du auf die Werte zugreifen musst.
Hier mal ein Minimalbeispiel:

public class Foo{
    private int x;
    public Foo(){ x = 5; }
    public void doSmth() { /* fancy stuff */ }
    public int getX() { return x; }
}

public class FooTester{
     private Foo foo;
     public FooTester(Foo foo){
           this.foo = foo;
           System.out.println(foo.getX());
     }
}

public class Main{
     public static void main(String[] args){
           Foo foo = new Foo();
           foo.doSmth();
           FooTester tester = new FooTester(foo);
     }
}

 

[DaCrazyJavaExpert]

Dass heißt also, ich nehme einfach die Inztanz die noch nicht initialisiert ist und übergebe sie den Kontruktor als Parameter?
Ich habe meine Klasse einfach so instanziert gehabt. TestKlasse test; Und dann immer test benutzt um auf die Klasse der Inztanz zuzugreifen

 

[Robat]

Du musst die Instanz der Klasse schon initialisieren.

 

[DaCrazyJavaExpert]

Und ich brauche aber den Parameter? Was soll ich denn da einsetzen, wenn ich den Kontruktor der Klasse in einer anderen Klasse brauche?
#Edit: Mit new Foo() ?
Ich habe jetzt einen Beitrag dazu in Google gefunden. Hab es alllerdings nur halb verstanden. Es gibt wohl irgendwie 3 Möglichkeiten dazu(?), also irgendwas mit Interfaces und noch 2 andere, 1 wharsxheinlich dann die, die dur mir hier gezeigt hast.

 

[Sasuke]

Hey,

ich glaube du hast da ein paar zentrale Konzepte noch nicht verstanden. Woran das genau liegt kann man noch nicht ganz genau sagen. Ich versuche einmal einen kleinen Grundbedarf an Begriffen und Funktionalitäten zu decken:

Deklarierung

Object someObject;

Initialisierung

someObject = new Object();

Der Zugriff auf eine Instanz der Klasse Object

String hashCode = someObject.hashCode();

Erst nachdem du ein Object initialisiert hast kannst du es in dem Sinne auch verwenden. Versuchst du, auf eine nicht initialisierte Variable zuzugreifen bekommst du eine NullPointerException. Was würdest du tun, wenn jemand sagt: "Hüpf!"? Bist du gemeint? Vielleicht meinte man auch jemand anderen?

Und ich brauche aber den Parameter? Was soll ich denn da einsetzen, wenn ich den Kontruktor der Klasse in einer anderen Klasse brauche?
#Edit: Mit new Foo() ?
Ich habe jetzt einen Beitrag dazu in Google gefunden. Hab es alllerdings nur halb verstanden. Es gibt wohl irgendwie 3 Möglichkeiten dazu(?), also irgendwas mit Interfaces und noch 2 andere, 1 wharsxheinlich dann die, die dur mir hier gezeigt hast.

Ich kann leider nur raten, was du mir damit sagen oder was du mich fragen willst. Du hast oben bereits ein kleines Beispiel bekommen, wie es aussehen kann. Danke dafür @Robat. Egal wie du es angehst, ob Dependency Injection, ganz egal, ein Code wie

TestKlasse test;

ist keine Initialisierung und die Variable 'test' verweist nach dieser Anweisung noch auf keine Instanz der Klasse 'TestKlasse'. Es wäre toll, wenn du erklären würdest, auf was du dich mit

Und ich brauche aber den Parameter? Was soll ich denn da einsetzen, wenn ich den Kontruktor der Klasse in einer anderen Klasse brauche?

beziehst. Welchen Konstruktor mit welchen Parametern in welcher Klasse?

Mit freundlichen Grüßen
Sasuke

 

[T_T]

Benutze doch einfach das Singleton Pattern für deine Klasse, die du nur einmal initialisieren darfst/willst/sollst...

ShopData shopData = ShopData.getInstance();

 

[Joose]

Das Singleton Pattern sollte man vermeiden und lieber auf DependencyInjection setzen

 

[T_T]

Das Singleton Pattern sollte man vermeiden und lieber auf DependencyInjection setzen

Im Grunde macht DI auch nichts anderes als ein Singleton zu definieren, sofern man den standard scope belässt und nicht auf "prototype" umschaltet.
----
Aber mich persönlich würde es interessieren, warum man das Pattern vermeiden sollte, habe ich jetzt zum ersten Mal gehört. Falls es hier in diesem Thread unpassend ist, schreibe mir mal bitte eine PN.

 

[mrBrown]

Im Grunde macht DI auch nichts anderes als ein Singleton zu definieren, sofern man den standard scope belässt und nicht auf "prototype" umschaltet.
----
Aber mich persönlich würde es interessieren, warum man das Pattern vermeiden sollte, habe ich jetzt zum ersten Mal gehört. Falls es hier in diesem Thread unpassend ist, schreibe mir mal bitte eine PN.

DI kennt keinen Scope Das ist nur eine Umsetzung dieses Patterns.


Verpönt ist das Singleton-Pattern, nicht, dass es einzelne Instanzen einer Klasse gibt.

Die Klasse selbst besorgt sich beim Singleton-Pattern ihr Abhängigkeiten selbst über meist statische Methoden, und nach außen hin ist nicht sichtbar, welche Abhängigkeiten überhaut benötigt wird.
Dadurch hat man als "Client" kaum Möglichkeit, die Abhängigkeiten zu tauschen und schränkt sich zB bei der Testbarkeit massiv ein.
Bei DI hat man einerseits ganz explizite Abhängigkeiten, und andererseits kann man die Implementierungen beliebig austauschen, zB mit einer gemockten Instanz

 

[Flown]

Verpönt ist das Singleton-Pattern, nicht, dass es einzelne Instanzen einer Klasse gibt.

Kann ich dir zustimmen, weil es oft falsch verwendet wird!

Dadurch hat man als "Client" kaum Möglichkeit, die Abhängigkeiten zu tauschen und schränkt sich zB bei der Testbarkeit massiv ein.

Mittelerweile nicht mehr. Hat auf die Testbarkeit keinen Einfluss mehr (PowerMock).

 

[DaCrazyJavaExpert]

Database shopData = Database.getInstance(); funktioniert ohnehin irgendwie nicht... der Fehler gibt Pries, dass die Methode getInstance(); nicht in der Klasse Database vorhanden ist. Somit muss ich eigentlich mit DI arbeiten. Ich habs jetzt so probiert:

private Database shopData;

    public CookieShop(Database shopData) {
        this.shopData = shopData;
}

//Und in der Klasse in der CookieShop aufgrerufen wird:
this.cookieShop = new CookieShop(new Database());
//Das hier im Konstruktor

private Database shopData;

    public CookieShop(Database shopData) {
        this.shopData = shopData;
}

//Und in der Klasse in der CookieShop aufgrerufen wird:
this.cookieShop = new CookieShop(new Database());
//Das hier im Konstruktor

 

[T_T]

DI kennt keinen Scope Das ist nur eine Umsetzung dieses Patterns.

Hups, zu viel mit Spring in letzter Zeit gearbeitet

Database shopData = Database.getInstance(); funktioniert ohnehin irgendwie nicht... der Fehler gibt Pries, dass die Methode getInstance(); nicht in der Klasse Database vorhanden ist

Ist auch logisch, wenn deine Klasse die Methode nicht besitzt. Das musst du schon selbst machen.

 

[DaCrazyJavaExpert]

Hups, zu viel mit Spring in letzter Zeit gearbeitet



Ist auch logisch, wenn deine Klasse die Methode nicht besitzt. Das musst du schon selbst machen.

Ich habe keine Ahnung was in so eine get Instance methode reinmuss... Wenn ichs wüsste wäre ich bestimmt schon drauf gekommen. (Oder vielleicht auch nicht)

 

[T_T]

Hier ist ein Beispiel von einem Singleton: https://javabeginners.de/Design_Patterns/Singleton_-Pattern.php
Oder hier: https://de.wikipedia.org/wiki/Singleton_(Entwurfsmuster)

Deine Klasse muss ein wenig verändert werden, sie braucht eine Methode mit getInstance(), die eine Instanz der Klasse zurück gibt, sofern es bereits eine gibt - ansonsten eine neue Instanz erstellen...

 

[thecain]

Und schon haben wir ein Singleton, welches nach Bugs schreit...

Wieso diskutieren wir über die nachteile und machen es dann trotzdem einfach?

 

[DaCrazyJavaExpert]

Ok, danke. Mit getInctance() die Inztanz zu holen macht echt keinen Sinn. Wie mache ich dass nun mit DI? @Robat 's Vorlschlag enötigt einen Parameter im Konstruktor der Klasse, in dem man eine Inztanz einer anderen Klasse haben möchte. Das will ich eigentlich nicht, da ich eigentlich auch nicht so genau weiß, wass ich später als Parameter benutzen soll. Bei Google verstehe ich das Prinzig von DI auch nicht komplett. Kann mir vielleicht jemand noch eine guten Link schicken, der zu meinem Thema passt, oder mich ein wenig aufklären?
Hier nochmal ein paar relevante Informationen:

//Von dierser Klasse will ich eine Inztanz in der Folgenden Klasse haben. Allerdings soll durch das Erstellen der Inztanz nicht diese Klasse aufgerufen werden.
public class Database {
public Database;
}

public class CookieShop { //Die Klasse, in der ich eine Inztanz von Database haben will.

    private Database shopData; // Das hier geht nicht, weil die Variable nicht mit null initialisiert sein darf

public CookieShop() {
//hier irgendwelche anderen Methoden, welche durch den Kontruktor in der Hauptklasse aufgerufen werden.
}
}

Vielen Dank!

 

[Robat]

Das will ich eigentlich nicht, da ich eigentlich auch nicht so genau weiß, wass ich später als Parameter benutzen soll

Dann solltest du dich erstmal hinsetzen und dir mit Blatt und Papier die Abhängigkeiten der einzelnen Klassen aufmalen / dir überlegen. Hier jetzt zu raten um am Ende doch die falsche Lösung zu nennen ist nicht im Sinne des Erfinders. Überleg dir in welchen Klassen du welche Objekte brauchst und wie die dein Projekt gescheit strukturierst - dann fällt sowas auch wesentlich leichter.

 

[DaCrazyJavaExpert]

Dann solltest du dich erstmal hinsetzen und dir mit Blatt und Papier die Abhängigkeiten der einzelnen Klassen aufmalen / dir überlegen. Hier jetzt zu raten um am Ende doch die falsche Lösung zu nennen ist nicht im Sinne des Erfinders. Überleg dir in welchen Klassen du welche Objekte brauchst und wie die dein Projekt gescheit strukturierst - dann fällt sowas auch wesentlich leichter.

Ich weiß wie ich mein Projekt strukturieren muss und welche Klassen/Objekte ich brauche. Der letzte Schritt zu fertigstellen meines Programms ist eigentlich nur noch die Inztanz der Database-Kalsse zu bekommen ohne dass die Klasse damit aufgerufen wird.

 

[Robat]

Dann musst du eine bereits existierende Instanz an die Klasse, die die Instanz benötigt, übergeben. Gibt es eine Klasse wo du eine Instanz von Database und CookieShop anlegst?

 

[DaCrazyJavaExpert]

Dann musst du eine bereits existierende Instanz an die Klasse, die die Instanz benötigt, übergeben. Gibt es eine Klasse wo du eine Instanz von Database und CookieShop anlegst?

Ja in der GUI-Main-Klasse. Aber um die getter davon zu bekommen brauche ich auch erstmal die Inztanz dieser GUI-Klasse.(Also es führ kein Weg dran vorbei) Oder gibts da eine andere Möglichkeit? Wäre jetzt so meine Idee.

 

[Robat]

Warum dann nicht einfach sowas machen:

public class GuiKlasse{

       public GuiKlasse(){
             Database db = new Database();
             CookieShop shop = new CookieShop(db);

             // .. irgendwelcher GUI Code...
       }
}

Da musst du natürlich den Konstruktor der CookiShop Klasse um einen Parameter vom Typ Database erweitern.

 

[DaCrazyJavaExpert]

Oh ja es hat geklappt, ich hab es mal mit new Database probiert gehabt. Das hatte nicht geklappt. Danke.
Und wenn ich jetzt eine Inztanz der Main in der CookieShop-Klasse brauche, was muss ich da machen? Da kann ich ja schlecht das gleiche machen, da sonst die Main in der Shop-Klasse aufgerufen wird. Oder kann ich in der Main einfach eine Inztanz der Main erstellen, ohne dass die Main dann doppelt aufgerufen wird.

 

[Robat]

Dann übergibst du dem Konstruktor der CookieShope-Klasse einfach this.

public class GuiKlasse{

       public GuiKlasse(){
             Database db = new Database();
             CookieShop shop = new CookieShop(this);

             // .. irgendwelcher GUI Code...
       }
}

Wobei der Konstruktor der CookieShop-Klasse dann natürlich ein Parameter vom Typ GuiKlasse braucht

 

[DaCrazyJavaExpert]

Dann übergibst du dem Konstruktor der CookieShope-Klasse einfach this.

public class GuiKlasse{

       public GuiKlasse(){
             Database db = new Database();
             CookieShop shop = new CookieShop(this);

             // .. irgendwelcher GUI Code...
       }
}

Wobei der Konstruktor der CookieShop-Klasse dann natürlich ein Parameter vom Typ GuiKlasse braucht

Oh stimmt ja, hab überhaupt nicht daran gedacht, dass this auf die Klasse verweist. Danke für die Hilfe!