Frage zu ThreadLocal

[Generic1]

Hallo,

ich hatt folgendes Problem in der Klasse unten, also eine Klasse welche Singleton ist, es müsste also nur ein Objekt dieser Klasse geben, Meine Applikation läuft in einem JBoss- context.

Wenn ich jetzt das Programm ausgeführt habe, dann würde mit der Methode setCurrentUser ein user gesetzt. Im weiteren Programmablauf würde die Methode getCurrentUser aufgerufen und diese Methode gab dann "null" zurück -> und das kann ich mir jetzt nicht erklären, es gibt ja nur ein Objekt von der Klasse MyClass und daher auch nur einen ThreadLocal "user".
Warum dann null zurückgegeben wird obwohl vorher schon ein User mit setCurrentUser gesetzt wurde ist mir absolut nicht klar.
Mein workaround war jetzt, dass ich den ThreadLocal "static" mache, dann gehts natürlich und es deutet für mich auch darauf hin, dass es mehrere Instanzen der Klasse MyClass gibt, wie das aber sein kann bei einer Singleton- Klasse ist mir nicht klar.

Hat vielleicth jemand eine Idee, warum das sein kann,
Vielen Dank,
lg

pulic class MyClass extends BaseClass implements MyInterface {
    public final static MyClass INSTANCE = new MyClass();
  
    protected ThreadLocal<IUser> user = new ThreadLocal<IUser>()

    private MyClass() {}

    public void setCurrentUser(IUser user) {
      if(user != null )
        this.user.set(user)
       }

    public IUser getCurrentUser() {
       return this.user.get();
       }

   }

 

[FArt]

Wie hast du sichergestellt, dass es nur eine Instanz von dieser Klasse geben kann?
Singleton (static) bedeutet nicht eine Instanz in einer VM, sondern für jeden Classloader, der die Klasse laden kann. Gerade in einem Applicationserver, der Classloader selber verwaltet (und auch noch konfigurierbar) sollte man auf solche Konstrukte verzichten, denn sie funktionieren nicht wie ein Java Programmier das erwartet.
Warum funktioniert es nicht wie du glaubst? Sei nicht so hilflos und hilf dir selber mal. Baue Logging in deinen Code ein ... Instanziierung der Klasse und setzen und holen von Userns. Hilfreich ist dabei, wenn du die Instanz identifizieren kannst (falls es mehrere gibt) und den aktuell gültigen Kontextclassloader und den aktuellen Thread.

 

[Generic1]

Also, ich hab mich jetzt mal nicht so angestellt 🙂 und hab in den Konstruktor ein System.out.println("...") reingegeben -> dieser Konstruktor von MyClass wird nur einmal aufgerufen, Trotdem bekomme ich bei einem Aufruf von getCurrentUser "null" zurück,
Also ganz versteh ich das jetzt nicht, vielleicht hat jemand noch eine Idee, was es da haben kann.
lg

 

[FArt]

Dann ist vielleicht der setztende Thread nicht der holende Thread... siehe mein Anmerkung zu Logging oben.
Außerdem: dein Konstrukt kann in einer Enterprise-Umgebung grundsätzlich in die Hose gehen, abhängig von dem Kontext in dem es verwendet wird.

Was soll denn eigentlich wann (in welchem Kontext) erreicht werden?

 

[Marco13]

Wie hast du sichergestellt, dass es nur eine Instanz von dieser Klasse geben kann?
Singleton (static) bedeutet nicht eine Instanz in einer VM, sondern für jeden Classloader, der die Klasse laden kann.

Siehe auch Geek And Poke: The Geek&Poke Pattern Tip Of The Week 😀

 

[FArt]

Siehe auch Geek And Poke: The Geek&Poke Pattern Tip Of The Week 😀

Bedeutet was in deinem Fall?

 

[Marco13]

Nichts. Ähm. Mittagspause...? :bahnhof: Passte nur zu deinem Hinweis 😉

 

[tuttle64]

Ein typisches Pattern für ein Singleton ist es, die MemberVariable des Typs Referenz als static zu deklarieren und innerhalb des Konstruktors zu prüfen, ob diese Variable null ist, denn nur dann wird eine Instanz erzeugt. Das könnte in etwa so aussehen (ohne extends BaseClass implements MyInterface):

public class MyClass {
	// static bedeutet user ist ein Klassenobjekt
	private static ThreadLocal<IUser> user = null;

	public MyClass() {
		if (user == null) {
			user = new ThreadLocal<IUser>();
			System.out.println("Neuer User erzeugt.");
		} else {
			System.out.println("Neuer User konnte nicht erzeugt werden.");
		}
	}

	public IUser getCurrentUser() {
		if (user != null) {
			return this.user.get();
		} else
			return null;
	}

	public static void main(String[] args) {
		MyClass c1 = new MyClass();
		MyClass c2 = new MyClass();
	}
}

 

[SlaterB]

@tuttle64
das scheint mir nicht gerade ein gängiges Beispiel, die statische Variable kannst du direkt initialisieren,
ansonsten passiert das beim ersten MyClass-Objekt, danach nie wieder,

ein User wird dabei noch nicht angelegt, kann auch gar nicht ohne eigene ThreadLocal-Klasse mit initialValue(),
eine Thread-Unterscheidung findet nicht wirklich statt

 

[FArt]

Nichts. Ähm. Mittagspause...? :bahnhof: Passte nur zu deinem Hinweis 😉

Hoppla, ich habe nicht gesehen dass du gar nicht der TS warst... und habe den Zusammenhang deswegen nicht erkannt, denn Generic1 hatte ja schon gesagt es wäre definitiv nur eine Instanz.

Ein typisches Pattern für ein Singleton ist es, die MemberVariable des Typs Referenz als static zu deklarieren und innerhalb des Konstruktors zu prüfen, ob diese Variable null ist, denn nur dann wird eine Instanz erzeugt.

Ok, aber das ist ja auch nur eine Spielart des "normalen" Singleton und weist in einer Enterpriseumgebung (oder anderen Umgebungen, die Classloader managen) die selben Schwächen auf.

 

[Generic1]

OK, ich werd dann die Instanz von ThreadLocal "static" machen.