[Java] NullPointer bij Singleton

[2playgames]

ik heb een singleton klasse gemaakt (het voorbeeld heb ik van Wikipedia):

Java:

public class MainApp {

    public MainWindow mainWindow;
    public ControlStreamReader socketReader;

    /**
     * Start de applicatie
     * @param args De commandline argumenten. Voor deze applicatie niet van toepassing
     */
    public static void main(String[] args) {
        MainApp.get();
    }

    /**
     * Verkrijg de hoofdklasse van de applicatie
     * Hij wordt gemaakt als hij nog niet bestaat
     */
    public static MainApp get() {
        return SingletonHolder.instance;
    }
    private static class SingletonHolder {
        private static MainApp instance = new MainApp();
    }
    
    // private constructor volgens het Singleton principe
    private MainApp() {
        mainWindow = new MainWindow();
    }

}

de eerste aanroep van get() in main() gaat goed, en het programma start normaal. als ik dan later in een andere klass MainApp.get().methode() aanroep, krijg ik een de volgende error voor mn kiezen:

Exception in thread "main" java.lang.ExceptionInInitializerError
at ica.elo.MainApp.get(MainApp.java:31)
at ica.elo.MainApp.main(MainApp.java:23)
Caused by: java.lang.NullPointerException
at ica.elo.MainWindow.showConnectDialog(MainWindow.java:129)
at ica.elo.MainWindow.access$0(MainWindow.java:124)
at ica.elo.MainWindow$1.widgetSelected(MainWindow.java:65)
at org.eclipse.swt.widgets.TypedListener.handleEvent(TypedListener.java:227)
at org.eclipse.swt.widgets.EventTable.sendEvent(EventTable.java:66)
at org.eclipse.swt.widgets.Widget.sendEvent(Widget.java:938)
at org.eclipse.swt.widgets.Display.runDeferredEvents(Display.java:3673)
at org.eclipse.swt.widgets.Display.readAndDispatch(Display.java:3284)
at ica.elo.MainWindow.<init>(MainWindow.java:39)
at ica.elo.MainApp.<init>(MainApp.java:39)
at ica.elo.MainApp.<init>(MainApp.java:38)
at ica.elo.MainApp$SingletonHolder.<clinit>(MainApp.java:34)
... 2 more

regel 129 van MainWindow.java bevat dus "MainApp.get().methode();"

wat kan ik aan dit supersimpele principe nou fout hebben gedaan dat het niet werkt?

bij voorbaat dank

edit: ik heb eerst ook nog dit geprobeerd, maar kreeg toen ook een nullpointerfout

Java:

public class MainApp {

private static MainApp instance = null;

public static MainApp get() {
    if(instance == null) instance = new MainApp();
    return instance;
}

// ... constructor en de rest

}

[ Voor 6% gewijzigd door 2playgames op 09-05-2007 18:51 ]

[Michali]

Compile en probeer dit eens:

Java:

public class Test
{
    public static void main(String[] args)
    {
        getInstance().printHello();
    }

    private static Test instance;

    public static Test getInstance()
    {
        if ( instance == null )
        {
            instance = new Test();
        }
        return instance;
    }

    public void printHello()
    {
        System.out.println("Hello");
    }
}

Hier werkt het iig.

[NetForce1]

Misschien komt het omdat de constructor van MainWindow er voor zorgt dat een event gefired wordt, vervolgens probjeer je in je event handler aan de singleton instance te refereren. Die is echter nog niet geset omdat de constructor nog bezig is. Je tweede probeersel brengt me echter aan het twijfelen over deze theorie, ik had hier nl een stack overflow verwacht dan.

[2playgames]

@Michali: ja dat werkt
@NetForce1: de tweede get() wordt pas aangeroepen nadat de applicatie is gestart en ik een dialoogvenster heb laten zien en ingevuld, dus de constructor is dan al klaar

edit: ik heb het gevonden. het lijkt een beetje op de suggestie van NetForce1. wat er gebeurt is dat in MainApp() een nieuwe MainWindow gemaakt wordt. in deze constructor wordt een SWT Shell gemaakt en daarna komt de eventloop (die niet afloopt tot het programma gesloten wordt). ik ga het oplossen door deze code dus buiten de constructors te halen.

[ Voor 48% gewijzigd door 2playgames op 09-05-2007 19:37 ]

[NetForce1]

Dan is op de een of andere wazige fiets je klasse verdwenen uit de classloader, anders kan ie nooit die static init nog een keer uit gaan voeren

[ Voor 4% gewijzigd door NetForce1 op 09-05-2007 19:38 ]

[2playgames]

zie edit in vorige post

[BalusC]

Mag ik vragen waarom je een uitvoerbare Java Applicatie als een singleton wil ontwerpen? Wellicht zijn er veel betere oplossingen

[voodooless]

code:

Caused by: java.lang.NullPointerException
at ica.elo.MainWindow.showConnectDialog(MainWindow.java:129)

Daar zit dus de fout, regel 129 van MainWindow.

[dawuss]

voodooless schreef op woensdag 09 mei 2007 @ 21:28:

code:

Caused by: java.lang.NullPointerException
at ica.elo.MainWindow.showConnectDialog(MainWindow.java:129)

Daar zit dus de fout, regel 129 van MainWindow.

Niet zo gek natuurlijk, daar probeert hij een methode aan te roepen op de singleton-class:

Java:

MainApp.get().methode()

Dit kan voorkomen omdat je (in mijn ogen) een gammele implementatie van het Singleton pattern hebt geschreven: wat de get() methode teruggeeft is tenslotte afhankelijk van het al dan niet eerder geïnstantieerd zijn van de klasse MainApp. Wat in je commentaar staat en wat je klasse daadwerkelijk doet is niet hetzelfde.

Je roept deze nota bene zelf verkeerd aan in de static main(): daar weet je absoluut zeker dat die MainApp.get() null teruggeeft!

Probeer het zo eens:

Java:

public class MainApp { 

    private MainWindow mainWindow; // private (good coding practice)
    private ControlStreamReader socketReader; // private (good coding practice)

    private static MainApp mainApp; // this is the Singleton instance!

    /** 
     * Start de applicatie 
     * @param args De commandline argumenten. Voor deze applicatie niet van toepassing 
     */ 
    public static void main(String[] args) { 
        System.out.println(MainApp.get()); 
    } 

    /** 
     * Verkrijg de hoofdklasse van de applicatie 
     * Hij wordt gemaakt als hij nog niet bestaat 
     * @ensure instanceOf(MainApp.get()) == MainApp && MainApp.get() != null
     */ 
    public static MainApp get() { 
        if(mainApp == null) {
            mainApp = new MainApp();
        }

        return mainApp;
    } 
    
    // private constructor volgens het Singleton principe 
    private MainApp() { 
        mainWindow = new MainWindow(); 
    } 

     // getters (evt ook setters)
    public MainWindow getMainWindow() {
        return this.mainWindow;
    }

    public ControlStreamReader getSocketReader() {
        return this.socketReader;
    }

}

[ Voor 53% gewijzigd door dawuss op 09-05-2007 22:07 . Reden: oplossing erbij ]

[voodooless]

Ah, ik zie het ja. Lekker recursief

Hmm, toch niet Volgens mij gaat er gewoon wat mis op die regel en heeft dat verder niets met de singleton te maken.

En als je er niet uit komt, gooi het dan ff in de debugger

[ Voor 78% gewijzigd door voodooless op 09-05-2007 22:08 ]

[dawuss]

voodooless schreef op woensdag 09 mei 2007 @ 21:59:
Ah, ik zie het ja. Lekker recursief

Hmm, toch niet Volgens mij gaat er gewoon wat mis op die regel en heeft dat verder niets met de singleton te maken.

Nee, het probleem is gewoon dat MainWindow.get() null kan opleveren, ik vermoed omdat de SingletonHolder private is (maar ik vind dat ook maar een slecht leesbare constructie).

[voodooless]

Dat kan alleen als de constructie van MainApp() faalt (dus java.lang.ExceptionInInitializerError). Er gaat dus wel degelijk wat mis in de de constructor van MainApp() (nullpointer @ regel 129).

[BalusC]

In plaats van om zijn singletonprobleem te oplossen kunnen we ook wachten op het antwoord waarom hij per-se een uitvoerbare java applicatie in een singleton wil gieten. Dat je specifieke utilityklassen als een singleton ontwerpt, oké, maar een uitvoerbare applicatie? Hier steekt meer 'bad design' achter

[dawuss]

voodooless schreef op woensdag 09 mei 2007 @ 22:11:
Dat kan alleen als de constructie van MainApp() faalt (dus java.lang.ExceptionInInitializerError). Er gaat dus wel degelijk wat mis in de de constructor van MainApp() (nullpointer @ regel 129).

Ik denk dat die hele constructor nooit wordt aangeroepen in dit stukje code:

Java:

    public static MainApp get() { 
        return SingletonHolder.instance; 
    }
 
    private static class SingletonHolder { 
->        private static MainApp instance = new MainApp(); 
    }

Daar staat impliciet dat die statische variabele geïnitialiseerd moet worden, maar volgens mij werkt dat zo niet.

BalusC schreef op woensdag 09 mei 2007 @ 22:29:
In plaats van om zijn singletonprobleem te oplossen kunnen we ook wachten op het antwoord waarom hij per-se een uitvoerbare java applicatie in een singleton wil gieten. Dat je specifieke utilityklassen als een singleton ontwerpt, oké, maar een uitvoerbare applicatie? Hier steekt meer 'bad design' achter

Daar ben ik inderdaad ook wel benieuwd naar

[ Voor 24% gewijzigd door dawuss op 09-05-2007 22:35 ]

[voodooless]

BalusC schreef op woensdag 09 mei 2007 @ 22:29:
In plaats van om zijn singletonprobleem te oplossen kunnen we ook wachten op het antwoord waarom hij per-se een uitvoerbare java applicatie in een singleton wil gieten.

Zoals ik al zei, er is vermoedelijk helemaal geen singleton probleem

Neemt natuurlijk niet weg dat het een vreemde ontwerpbeslissing is...

[voodooless]

dawuss schreef op woensdag 09 mei 2007 @ 22:34:
Daar staat impliciet dat die statische variabele geïnitialiseerd moet worden, maar volgens mij werkt dat zo niet.

Dat kan prima werken. Maar het gaat dus in dit geval mis, vandaar die vreemde java.lang.ExceptionInInitializerError, die je nu krijgt. De fout zit dus in die regel 129.

[ Voor 4% gewijzigd door voodooless op 09-05-2007 22:44 ]

[2playgames]

erm...mensen...is het jullie opgevallen dat het allang opgelost is??
2playgames in "[Java] NullPointer bij Singleton"
het ging dus fout omdat mn constructor pas klaar was aan het einde van de applicatie

In plaats van om zijn singletonprobleem te oplossen kunnen we ook wachten op het antwoord waarom hij per-se een uitvoerbare java applicatie in een singleton wil gieten. Dat je specifieke utilityklassen als een singleton ontwerpt, oké, maar een uitvoerbare applicatie? Hier steekt meer 'bad design' achter

omdat de hoofdklasse dan makkelijk overval vandaan te bereiken is, zonder dat deze in allerlei constructoren meegegeven moet worden. ja, misschien is het lui en slecht ontwerp, maar richtlijnen zijn er om het makkelijk te maken, niet om per se te volgen
wikipedia:

[Singleton Classes are] useful when exactly one object is needed to coordinate actions across the system.

naar mijn mening geldt dit voor mijn klasse

[ Voor 14% gewijzigd door 2playgames op 09-05-2007 23:13 ]

[voodooless]

Als je het goed ontwerpt is het helemaal niet nodig om de hoofdklasse overal in mee te geven