Frage bezüglich Codedesigns

[WeirdAl]

Hi,
ich habe innerhalb meines Projekts ein paar statische Klassen die Methoden anbieten, die verschiedenste SQL Selects, Updates, Deletes auf einer mySQL DB ausführen. Die einzelnen Methoden haben verschiedene Paramater und Rückgabewerte, aber der Kern der Methoden ist gleich:

Methode die eine SQLException wirft
...
PreparedStatement  pst = null;
ResultSet  rs = null;
Connection cn = null;
	    	    
	    try {
	    	 // load driver and connect
	    	 Class.forName( driver );
	         cn = DriverManager.getConnection( dbUrl, dbLogin, dbPW );
	         

	         pst = cn.prepareStatement("...");
	         pst.setInt(1, ...);

                 ...               

	         rs = pst.executeQuery();
         
	         rs.next(); 
	         ....
	         } catch (ClassNotFoundException e) {
				System.err.println(e);
			} 
				
			finally {
				pst.close();
		        rs.close();
		        cn.close(); 
			}

Der Teil ist praktisch in allen Methoden gleich. Einzig die speziellen PreparedStatements und deren "Verarbeitung" unterscheiden sich innerhalb dieses Codeblocks. Kann ich die "gleichen" Teile irgendwie auslagern? Ich habe keine Idee wie ich dies machen könnte, aber mich stört es, dass ich innerhalb einer Klasse 10x diesen Try/Catch/Final habe.

Vielleicht weiß jemand von euch Rat, Danke

Cu
Alex

 

[Azubi]

Schreib dir doch ne SQL Klasse.
Sollte man ausserdem nicht eh beim starten des Programmes EINE Connection erstellen und nicht immer wieder eine öffnen, closen, öffnen, closen?

Dann kannste nämlich mit deiner Klasse eine Connection erstellen, die über die Laufzeit des Programms erhalten bleibt.
Die Klasse kann ja dann noch Methoden haben wie:
executeStatement(); (für update, delete, ...)
getQueryStatement(); (für Select z.b. Was dann eben ein ResultSet zurückgibt)
Den 2 Methoden kannst du dann z.B. entweder nen String mit dem Statement was ausgeführt werden soll übergeben, oder aber direkt ein Statement. Bleibt ja dir überlassen wie dus handhaben willst.

Hilft dir das?

 

[WeirdAl]

Das mit der SQL Klasse muss ich mir mal ansehen (also generelles Vorgehen usw., ich hab mir da bislang noch gar nix angesehen). Mit der Connection weiss ich nicht was besser ist. Da das Projekt eine JSF Webapplikation ist, müsste ich dann pro Nutzer eine Connection zur Verfügung stellen, da ansonsten 100 Nutzer über eine Connection "geschliffen" werden.

Jedenfalls danke für die Denkanstöße . Wie Du durch meinen obigen Text sicher mitbekommen hast, muss ich erstmal googlen und mir klar werden, wie ich nun weitermachen werde

Cu
Alex

 

[Yzebär]

Du könntest eine statische Methode schreiben, der ein PrepareStatement übergeben wird und die ein ResultSet zurückgibt (wenn es eines gibt, ansonsten ohne Rückgabewert).

Ich verwende generell immer eine eigene Klasse, die alle Zugriffe auf die DB kapselt und die Dinge die sich stets ändern (Statements) als Parameter bekommen und die Ergebnisse die dann verarbeitet werden sollen (Resultset) als Returnwert haben.

Das mit der Connection würde ich auch ändern, ist doch egal, ob alle User über eine einzige Connection von der DB bedient werden. Wenn es sehr, sehr viele Zugriffe gibt, könnte man überlegen einen Connectionpool zu verwenden. Wäre auf jeden Fall effizienter, als jedesmal eine Connection zu öffnen und danach wieder zu schließen.

 

[Guest]

Nimm in Webanwendungen auf jeden Fall immer ConnectionPools.

Außerdem darfst du dich nicht wundern, daß du in einer Architektur, die auf statischen Methoden beruht, wiederholende Codestellen findest. Dazu gibt es die OOP; schreibe eine Basisklasse, die generische select, insert/update und delete-Statements ausführen kann und leite Basisklassen ab, die diese generischen Methoden benutzen.

Dringend abraten würde ich auch von der Rückgabe eines ResultSets, am besten noch in die View. Dazu benutzt man Transfer-Objekte, die ResultSets sollten dann schon längst geschlossen sein.