Problem mit Scope in JSP - Bean hält Daten ewig

[longfang]

und einen Tag. Hi.

Ich hab ein Problem mit folgender Konstellation:

Ich habe eine Klasse, die Daten aus einer Tabelle holt. Dies macht sie bei der Initialisierung. Jeder Datensatz wird dann einem Bean zugeordnet und diese dann in einer Array-List abgelegt.
In weiterer Folge prüft die Steuerklasse, ob dieses Array bereits gefüllt ist und greift auf diese zu anstatt wieder auf die Datenbank zuzugreifen.
Für die Anzeige steht dann ein JSP bereit, das von der Controller-Klasse über Delegate mit der Bean-Arrayliste gefüttert wird. Das Ganze passiert über den Sessionscope (HttpSession.setAttribute)

Geht ansich ja soweit.

Einziges Problem dabei ist, dass die Session scheinbar auch noch nach dem Schließen des Browsers bestehen bleibt. Klar tut sie das, ich war jedoch der Meinung, dass beim erneuten Öffnen des Browsers diesem eine neue Session zugewiesen wird. Ist doch so, oder täusch ich mich da?
Jedenfalls greift das Ding jetzt ständig auf die einmal erstellte Arrayliste zu.

Wo hab ich hier den Denkfehler, was hab ich übersehen?

p.s. ist so eine Vorgehensweise wie ich es beschrieben hab eigentlich sinnvoll? Soll ich das Zwischenspeichern ganz lassen und immer auf die DB zugreifen? Wie macht ihr das?

Danke im Voraus

Mike

 

[Bleiglanz]

>>Dies macht sie bei der Initialisierung

Initialisierung wovon? Hoffentlich bei der Session mit einem Listener? oder wo?

Wenn deine ArrayList z.B. ein Member eines Servlets ist, dann ist das schon OK?

>>p.s. ist so eine Vorgehensweise wie ich es beschrieben hab
>>eigentlich sinnvoll?

hängt davon ab

>>Soll ich das Zwischenspeichern ganz lassen
>>und immer auf die DB zugreifen? Wie macht ihr das?

wenn die Performance OK ist auf jeden Fall immer neu lesen (ist einfacher und es gibt keine Probleme mit veralteten daten)!

Eine Cache würde ich nur einsetzen, wenn es wirklich notwendig ist, d.h. wenn es messbar was bringt und die App anderweitig zu langsam ist

 

[KSG9|sebastian]

speicherst du die gefüllte arraylist als Instanz-Variable ???

 

[longfang]

Initialisierung ist hier wohl der falsche Ausdruck gewesen.

Die Funktion, die die Liste generiert, entscheidet aufgrund des Arrays (leer oder net) ob jetzt zuerst die Datenbank abgefragt werden oder gleich die vorhandene Liste übergeben werden soll.

By the way: Was meinst du mit 'bei der Session mit einem Listener'?

Jedenfalls ist die Liste ein Member des Servlets, die die Steuerung des Datenzugriffes auf die Tabelle übernimmt.
Vielleicht sollte ich hier ausholen:


Usertabelle in DB

usermanager - klasse steuert insert, update, select usw.
hier wird auch die Arrayliste mit den einzelnen Datensatzobjekten gehalten

userbean - container für die einzelnen Datensätze. Hat nur Getters und Setters

controller - servlet: is ein httpservlet und wird von einem Formular angesprochen. Eingaben werden gecheckt und entschieden, was damit erfolgen soll (ruft usermanager funktionen auf)
hier wird auch die oben beschriebene Listenfunktion aufgerufen.

HttpSession userSession = request.getSession(true);
userSession.setAttribute("liste",liste);

die Welcome.jsp krallt sich dann das so gefüllte usermanagerbean:

<jsp:useBean id="users" class="xxx.com.servlets.login.user.usermanager" scope="session" />

<%
    ArrayList userliste = users.getUserList();
    Iterator iter = userliste.iterator();
    while(iter.hasNext())
    {
...

p.s. Das macht imho so net wirklich Sinn, war nur ein Verzweiflungsversuch den Scope des Beans irgendwie auf Sessionbasis zu bringen.
ursprünglich hab ich mir die Daten via session.getAttribute("users"); geholt, die ich im Controllerservlet vorher zugewiesen hatte.

Ja, da steh ich jetzt, Anfänger... und versuche, all die aufgeschnappten Fetzen in ein vernünftiges Proof-of-Concept umzuwandeln.

Zwischenspeichern werde ich wohl net brauchen, aber mittlerweile gehts mir ums Verständnis, wie Java mit Sessionobjekten umgeht und wie man die Lebensdauer der Objekte für mich nachvollziehbar steuern kann

@sebastian:
wenn du damit das meinst:

private ArrayList userArray = new ArrayList();

...dann ja.

(Gott, ich komm mir so klein vor )