servlet synchronisationsproblem

[TangoFan]

Hab mal wieder ein Problem :lol: Und zwar habe ich folgendes Servlet:

public class Test{
...
public void doGet(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
		
	synchronized(this)
	{
         ... speichere Daten in eine gemeinsame Datenstruktur
         }
	    	
		
}

Also der Code funktioniert einwandfrei. Auch die synchronisation klappt super und meine "gemeinsameDatenstruktur" wird nicht zerstört.



Lagere ich aber nun den Code im "doGet" aus in eine andere Methode dieser Klasse, dann gibts wieder Probleme:

public class Test{
...
public void doGet(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
			
	this.handleRequest(request, response);    		
}  

private void handleRequest (HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {

         synchronized(this)
	{
         ... speichere Daten in eine gemeinsame Datenstruktur
         }
}

Unterliegt die Methode "doGet" von Haus aus einer speziellen Behandlung oder warum gibts synchronisationsprobleme nur wenn ich auslagere?

 

[SlaterB]

was sind denn die 'synchronisationsprobleme'?
was ist die gemeinsame Datenstruktur (Exemplarvariablen oder globaler)?

dir ist bekannt, dass evtl. mehrere Objekte der Servlet-Klasse unterwegs sind
und sich nicht gegenseitig synchronisieren wenn du nur auf this synchronisierst?

-------

inwiefern das was mit der Unteroperation zu tun hat ist mir persönlich aber völlig schleierhaft,
sehe da keinen relevanten Unterschied und auch keine Möglichkeit eines externen Eingriffs

 

[Guest]

dir ist bekannt, dass evtl. mehrere Objekte der Servlet-Klasse unterwegs sind
und sich nicht gegenseitig synchronisieren wenn du nur auf this synchronisierst?

Ich vermute mal das wird es sein. Worauf soll ich dann synchronisieren?

was sind denn die 'synchronisationsprobleme'?

Wenn im sync-Block versucht wird aus einer ArrayList einen bestimmmten Eintrag zu entfernen dann ist der plötzlich nicht mehr da, weils wohl schon eine andere Servlet-Instanz gelöscht hat. Der kommt also ziemlich durcheinander.

was ist die gemeinsame Datenstruktur (Exemplarvariablen oder globaler)?

Die Datenstruktur ist eine zusätzliche Singleton-Klasse, in welcher mittels getter/setter die Daten eingefügt/entfernt werden. Wie gesagt, sobald der Code komplett im "doGet" liegt, funktioniert auch alles super.

 

[SlaterB]

statt this auf die Klasse synchronisieren (Test.class) oder auf das Singleton,

wenn du dir aber sicher bist, dass es nicht nur zufällig mal ging und mal nicht,
dann würde ich an deiner Stelle einige Zeit in die Analyse stecken,
denn es ist ja immer noch unverständlich warum es mal zum Problem kommt und mal nicht

z.B. in dem Singleton mitloggen, welcher Request zu welcher Zeit welche Änderung macht,
insbesondere welches Servlet dabei aktiv war
(Objekt der Servlet-Klasse am hashcode() von anderen unterscheiden
oder im Konstuktor jedem Objekt der Servlet-Klasse eine eindeutige Nummer zuweisen)

evtl. dann noch vor, in und nach jedem Synchronzised-Block ne Logmeldung,
evtl. das ganze mal ganz ohne Synchronisierung laufen lassen und und und,
aber muss ja nicht sein

 

[TangoFan]

Also wenn ich die komplette Methode auf synchronized umstelle dann funktioniert wieder alles:

private synchronized void handleRequest (HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException { 

           ... speichere Daten in eine gemeinsame Datenstruktur 
       
}

Tu ich aber den synchronized-Block umändern auf z.B.:

         synchronized(Test.class) 
         { 
         ... speichere Daten in eine gemeinsame Datenstruktur 
         }

dann klappts nicht. Ist halt schon seltsam dass wenn der Code im "doGet()" steht dort funktioniert und hier in dieser Methode nicht.