Sollte man dies kapseln?

[CarloC]

Hallo zusammen,

meine Applikation beinhaltet einige JTables. Die Daten dieser Tabellen speicher ich immer in einem Vector ab. Für das füllen des Vectors habe ich mir eine Methode geschrieben (datensaetzeEinfuegen).

Diese Methode steht momentan noch in jeder Klasse der einzelnen Tabellen. Sauberer wäre es aber sicherlich wenn ich sie kapseln würde, da ich sie ja in mehrern Klassen benutze.

Der Aufbau der Methode ist an sich immer gleich. Sie unterscheidet sich allerdings beim SELECT und bei der Art von Objekten die für den Vector gebildet werden (beim ersten Beispiel Organisationseinheiten und beim zweiten Beispiel Gliederungseinheiten).

		public void datensaetzeEinfuegen()
		{
			Sql.connect();

	        try
			{
	        	String select = "SELECT * FROM dbo.organisationsplan WHERE [...]";
	        	rSet = Sql.select(select);

	        	while (rSet.next())
	        	{   		
	        		
	        		OrganisationsEinheit orgEinheitObjekt = new OrganisationsEinheit(
	        			new ImageIcon(),
						rSet.getString("orgeinheit"),
						rSet.getString("bezeichnung"),
						rSet.getInt("id")
	        		);

	        		daten.add(orgEinheitObjekt);
	        	}
			}
	        catch (Exception e)
			{
	        	[...]
	     	}
		}

zweites Beispiel:

	public void datensaetzeEinfuegen()
	{
		Sql.connect();

        try
		{
        	String select = "SELECT * FROM dbo.gliederungsplan WHERE [...]";
        	System.out.println(select);
        	rSet = Sql.select(select);

        	while (rSet.next())
        	{   		
        		
        		GliederungsEinheit eaua = new GliederungsEinheit(
        			new ImageIcon(),
					rSet.getString("eaua"),
					rSet.getString("bezeichnung"),
					rSet.getInt("id")
        		);

        		daten.add(eaua);
        	}
		}
        catch (Exception e)
		{
        	[...]
     	}
	}

Jetzt stellt sich für mich die Frage ob man die Methoden wirklich kapseln sollte oder ob ich es so lassen soll. Wenn man es kapseln soll, wie unterscheide ich dann die Art von Objekten die gebildet werden müssen?

Wie würdet ihr das machen?

Vielen Dank für die Hilfe!

 

[LoN_Nemesis]

Bin mir nicht sicher ob ich das richtig verstehe, also kannst du der Methode nicht einfach 2 Parameter geben? Der erste bestimmt welcher String ausgewählt wird und der zweite die Klasse des einzufügenden Objektes.

Sorry wenn ich was falsch verstanden habe.

 

[CarloC]

Meinst du, dass ich als String den Tabellennamen übergebe? Dann könnte der SELECT so lauten:

SELECT * FROM dbo." + tabellenName + "WHERE [...]

Bei der Übergabe des Objektes gibt es das Problem, dass sich die Spalten der Tabelle unterscheiden.
Bei einem gibt es dann "orgeinheit, bezeichnung, id" und bei dem anderen "gliederungseinheit, bezeichnung, id".
Es kann allerdings auch sein, dass sich alle drei Spalten unterscheiden oder es noch mehr Spalten gibt.

Ich könnte höchstens mit einer IF-Abfrage prüfen um was für ein Objekt es sich handelt.

Wenn Organisationseinheit dann -->

        		OrganisationsEinheit orgEinheitObjekt = new OrganisationsEinheit(
        			new ImageIcon(),
					rSet.getString("orgeinheit"),
					rSet.getString("bezeichnung"),
					rSet.getInt("id")
        		);

Wenn Gliederungseinheit dann --> usw.
Da könnte man dann natürlich auch einfach nur einen String übergeben, der besagt um was es sich handelt.

Ist es denn sinnvoll etwas zu kapseln wenn ich es dann mit IF-Abfragen unterscheide?

 

[SlaterB]

nur bedingt sinnvoll,

was du einigermaßen sinnvoll machen könntest, wäre die Datenanfrage auszulagern,

also eine Operation, die den kompletten SQL-String bekommt und ganz generisch ohne Unterscheidung das Ergebnis in ein Object[][] tut,
dazu wahrscheinlich noch ein ein Array mit den Spaltennamen falls die wichtig sind,
ansonten müsste man mit Spalte 1,2,3 arbeiten und hoffen dass sich die Reihenfolge nie ändert..

dann noch Einzeloperationen ohne Connection, SQL, ResulSet und try-catch,
dafür mit Object[][] und ähnlicher Übertragung auf die zu erstellenden Objekte

----------


wäre ganz hübsch, aber recht aufwendig,
dazu der Overhead des zu erstellenden Object[][] + die Frage der Spaltennamen,
musst du wissen,

auf jeden Fall eine Operation pro zu erstellenes Objekt

 

[CarloC]

Danke!