Stringbuilder verwenden

[Bhaldur]

Tag zusammen,

ich habe ein Problem mit dem Stringbuilder.

public ResultSet getStatistics(String Startebene,
				String Zielebene, String groesse, String gewicht, String geschlecht) throws SQLException {

			StringBuilder str = null;
			
			str.append("SELECT sum(t.costs) AS Gesamtkosten,"
					+ "  avg(t.costs) AS Durchschnittskosten,"
					+ "  sum(t.behandlungsdauer) AS Gesamtdauer,"
					+ "  avg(behandlungsdauer) AS Durchschnittsdauer "
					+ "FROM therapydata_all t ");
			
			if((groesse + gewicht + geschlecht)!=null){
				str.append(", patientdata_all p ");
			}
...

Bei

str.append("SELECT sum(t.c

bekomme ich eine Nullpointerexception die ich nicht wirklich verstehe. (Warum ist es an dieser Stelle überhaupt wichtig, nicht null zu sein?).
Einen Charwert kann ich dem Stringbuilder str nicht übergeben, es wird "null" verlangt. Wenn ich aber nicht mit einem null als Startwert beginnen kann, kann ich praktisch gar nichts mit dem Stringbuilder machen.

Kann mir jmd. verraten, wie ich dieses Problem umgehen kann?

 

[SlaterB]

mal abgesehen von dem trivialen Problem,
was spricht dagegen
StringBuilder beispiel java - Google-Suche
auszuführen?

 

[max40]

StringBuilder str = new StringBuilder();
			
str.append("SELECT sum(t.costs) AS Gesamtkosten,");
str.append("  avg(t.costs) AS Durchschnittskosten,");
str.append("  sum(t.behandlungsdauer) AS Gesamtdauer,");
str.append("  avg(behandlungsdauer) AS Durchschnittsdauer ");
str.append("FROM therapydata_all t ");

if((groesse + gewicht + geschlecht)!=null){    // sieht etwas komisch aus
	str.append(", patientdata_all p ");
}

 

[maki]

StringBuilder für SQL Statements ist imho nicht der richtige Weg, PreparedStatements dagegen schon.

 

[Noctarius]

Ich mag die Version:
Querystrings (mit Platzhaltern) in ein Properties-File, von dort lesen, ab in PreparedStatement, Platzhalter ersetzen, ausführen

 

[maki]

Ich mag die Version:
Querystrings (mit Platzhaltern) in ein Properties-File, von dort lesen, ab in PreparedStatement, Platzhalter ersetzen, ausführen

Sieh dir doch mal die SQLMaps von iBatis an
Da hast du dann gar nix mehr mit JDBC am Hut(try/catch Block im Try/catch Block etc.), und SQL Statement nur noch in XML Dateien und super Spring Unterstützung

 

[Noctarius]

Ich mach mit Queries eh nur im Ausnahmefall etwas, JPA über Eclipselink (alternativ früher Torque)

 

[maki]

Ach so

 

[Noctarius]

Am Anfang fand ich JPA etwas seltsam aber man gewöhnt sich dran. Ich hoffe nur, dass bald eine saubere Criteria API kommt. Die von Torque war besser als auch nicht so richtig TypeSafe.

 

[maki]

Am Anfang fand ich JPA etwas seltsam aber man gewöhnt sich dran. Ich hoffe nur, dass bald eine saubere Criteria API kommt. Die von Torque war besser als auch nicht so richtig TypeSafe.

Ich bin am überlegen ob JDO nicht doch die bessere Wahl ist, verglichen mit JPA.
JDO kann einiges mehr, gibt es ja schon seit 1999, aber erst seit ein paar Jahren auch als kostenlose Implementierungen.
Hat aber einen Riesennachteil: JDO braucht kein RDBMS

Ist natürlich schlecht für Oracle & IBM..

Nachtrag: Bin letztens hierüber gestolpert
Java Data Objects (JDO) - Home

 

[SchonWiederFred]

StringBuilder str = null;
str.append(...

str.append bedeutet: Führe die Methode append auf dem Objekt aus, auf das die Referenz zeigt, die in str gespeichert ist. Da in str leider keine gültige Referenz auf ein bestehendes Objekt (sondern null) gespeichert ist, kommt es zu einer NullPointerException. Du musst einen StringBuilder erzeugen und die Referenz darauf in str speichern.

 

[Noctarius]

Ich bin am überlegen ob JDO nicht doch die bessere Wahl ist, verglichen mit JPA.
JDO kann einiges mehr, gibt es ja schon seit 1999, aber erst seit ein paar Jahren auch als kostenlose Implementierungen.
Hat aber einen Riesennachteil: JDO braucht kein RDBMS

Ist natürlich schlecht für Oracle & IBM..

Nachtrag: Bin letztens hierüber gestolpert
Java Data Objects (JDO) - Home

JPA kann auch anderes als RDBMS, solange es einen passenden Vendor Adapter gibt. Was kann JDO denn mehr? Immerhin soll der Standard ja bald komplett "abgelöst" werden durch JPA2

 

[maki]

Was kann JDO denn mehr?

Java Data Objects (JDO) - JDO .v. JPA

Sehr interessant sind imho die sog. "fetch groups", da kann ein Objekt übergeben (fetch group), das aussagt, welche Attribute mitgeladen werden sollen. Ist so etwas angedacht für JPA 2.0?

JPA kann auch anderes als RDBMS, solange es einen passenden Vendor Adapter gibt.

Selbst mit Vendoradapter muss es eine RDBMS sein.

Immerhin soll der Standard ja bald komplett "abgelöst" werden durch JPA2

Wenn es nach dem Willen der "Expert Group" geht die JPA definniert schon

Beschäftige mich im Moment nur theoretisch mit JDO, aber je mehr ich lese, desto verwunderter bin ich über JPA (besonders 1.0), da hat man doch wirklich das Rad neu erfunden, aber eckig.
Wie gesagt, JDO existiert seit 1999.
Wird wohl Zeit für ein bisschen praktische Erfahrung durch Experimente..

 

[Noctarius]

Hmm ich denke ich bleibe bei Eclipselink Immerhin hab ich zur Not noch Moxy. Ansonsten brauch ich ehh immer nur RMDBS und JPA ist schön in Spring integriert

 

[Bhaldur]

Danke für eure Antworten!

Bin nun vom Stringbuilder abgesprungen und versuche es über PreparedStatements... allerdings tauchen auch hier Probleme auf die mich in Verlegenheit bringen.

Bekomme eine java.sql.exception "before start of the result set".

Hier die neue Methode:

Zur Erklärung: Das Statement wird je nach Parameterübergabe ausgewählt, da es hier zu unterschiedlichen Anforderungen kommen kann. Manchmal ist die Größe nicht von belangen, manchmal benötigt man noch eine andere View etc. Daher der versuch es über die if-Fälle semi-dynamisch zu gestalten.

public double[] getStatistcs2(String Startebene, String Zielebene,
			String groesse, String gewicht, String geschlecht) {
		if (Startebene + Zielebene + groesse + gewicht + geschlecht == null)
			return null;
		String[] s = { Startebene, Zielebene, groesse, gewicht, geschlecht };

		for (int i = 0; i < s.length; i++) {
			if (s[i] == null) {
				if (i == 1)
					s[i] = "therapydata1";
				if (i == 2)
					s[i] = "therapydata1";
				if (i == 3 || i == 4)
					s[i] = "0";
				if (i == 5)
					s[i] = "'weiblich'";
			}

		}

		String query = "SELECT sum(t.costs) AS Gesamtkosten,"
				+ "  avg(t.costs) AS Durchschnittskosten,"
				+ "  sum(t.behandlungsdauer) AS Gesamtdauer,"
				+ "  avg(behandlungsdauer) AS Durchschnittsdauer "
				+ " FROM therapydata_all t, ";

		if (Startebene != null) {

			if (Startebene.equals("therapydata3")
					|| Startebene.equals("therapydata2")
					|| Startebene.equals("therapydata1")) {

				query += Startebene + " d";
			}
		}
		query += ", patientdata_all p" + " WHERE t.pid = d.pid"
				+ "  AND t.pid = p.id"
				+ "  AND t.behandlungsdauer = d.behandlungsdauer"
				+ "  AND t.versorgungsebene = d.versorgungsebene"
				+ "  AND t.versorgungsebene = ?" + "  AND p.gender=?;"
				+ "  AND p.height>?" + "  AND p.weight>?";

		double[] k = new double[4];
		for (int i = 0; i <= 3; i++) {
			try {
				k[i] = getResultset(query, s).getDouble((i + 1));
			} catch (SQLException e) {
				// TODO Auto-generated catch block
				e.printStackTrace();
			}
		}
		return k;
	}

Evt. liegt der Fehler auch nicht hier, sondern in der main?

public static void main(String proConGes[]) {
		Preloader preloader = new Preloader("./seplogo.png", 32, 1);
		preloader.setVisible(true);
		preloader.setFinishedCallback(new Runnable() {
			public void run() {
				//System.out.println("---> test");
				ControlUnit cu = new ControlUnit();
				String str = "SELECT * FROM therapydata_all WHERE pid=?";
				String[] a = { "25" };
				try {
				ResultSet t = cu.getResultset(str, a);
				System.out.println(t.getDouble(1)); // <- java.sql.SQLException
				} catch (SQLException e) {
				// // TODO Auto-generated catch block
				e.printStackTrace();
				}
				
				cu.start();
			}
		});
		preloader.startFading();
	}

Denke der Fehler ist von gröberer Natur, etwas dass ich an den result sets / prepared statements noch nicht verstehe... könnt ihr mir noch mal weiterhelfen?

 

[SlaterB]

bevor man Daten von einem ResultSet abfragen kann, muss man es mindestens auf den ersten Eintrag setzen:
resultSet.next();

schau dir doch erstmal Tutorials an, wie die auf ResultSets zugreifen