Insert befehl geht nicht

[tachchen]

hallo,

versuche einen Datensatz in eine DB anzulegen, doch es kommt immer die Fehlermeldung:

[Microsoft][ODBC Microsoft Access Driver] Syntaxfehler in der INSERT INTO-Anweisung.
at sun.jdbc.odbc.JdbcOdbc.createSQLException(Unknown Source)
at sun.jdbc.odbc.JdbcOdbc.standardError(Unknown Source)
at sun.jdbc.odbc.JdbcOdbc.SQLExecDirect(Unknown Source)
at sun.jdbc.odbc.JdbcOdbcStatement.execute(Unknown Source)
at bean.DBBean.neuenDatensatzAnlegen(DBBean.java:95)
at bean.TestDB_Student.main(TestDB_Student.java:13)

der relevante Code ist:

conn.createStatement().execute(
						"Insert Into student"+ "("
						+name+","+Studiengang+","+Studienbeginn+","+mail+","+Geschlecht+")"+
						" Values"+
						"('"
						+ name +"','"
						+ Studiengang +"','"
						+ Studienbeginn +"','"
						+ mail +"','"
						+ Geschlecht +"');");

wo soll den hier der Fehler sein?? thx

 

[SebiB90]

eine insert into anweisung ist so aufgebaut:
INSERT INTO tabelle(spaltenname) VALUES(inhalt)
du hast aber bei spaltenname auch den inhalt angegeben

 

[SamHotte]

anders gesagt: in deiner Codezeile 4 müssen die Spaltennamen der Datenbanktabelle stehen.

 

[tachchen]

ja aber ich hab die Methode so definiert, das ich den Inhalt als Parameter übergebe. Die Methode sieht so aus:

public void neuenDatensatzAnlegen(String name,String Studiengang,String Studienbeginn,
			String mail,String Geschlecht){
		this.name=name;
		this.Studiengang=Studiengang;
		this.Studienbeginn=Studienbeginn;
		this.mail=mail;
		this.Geschlecht=Geschlecht;
		
		
		
		try {
	
				
				conn.createStatement().execute(
						"Insert Into student"+ "("
						+name+","+Studiengang+","+Studienbeginn+","+mail+","+Geschlecht+")"+
						" Values"+
						"('"
						+ name +"','"
						+ Studiengang +"','"
						+ Studienbeginn +"','"
						+ mail +"','"
						+ Geschlecht +"');");
							
		} catch (SQLException e) {
			System.out.println(" Fehler beim anlegen");

oder geht das auch nicht? wenn ja wieso?

danke

 

[tachchen]

anders gesagt: in deiner Codezeile 4 müssen die Spaltennamen der Datenbanktabelle stehen.

ja das sind die Spaltennamen, aber nicht alle, denn die erste Spalte hab ich als userID angegeben. Ich muss ja nicht alle Spaltennamen angeben oder?

 

[SamHotte]

Es kann sein, dass in den Werten ein Zeichen vorkommt, das dir das SQL-Statement zerschießt - besser benutzt du ein PreparedStatement für sowas.

Hmm, die genaue Fehlermeldung könnte jetzt helfen. Lass dir vielleicht im catch-Block mit 'e.getMessage()' mal den genauen Grund sagen.

 

[tachchen]

Es kann sein, dass in den Werten ein Zeichen vorkommt, das dir das SQL-Statement zerschießt - besser benutzt du ein PreparedStatement für sowas.

Hmm, die genaue Fehlermeldung könnte jetzt helfen. Lass dir vielleicht im catch-Block mit 'e.getMessage()' mal den genauen Grund sagen.

die Fehlermeldung lautet nun:

Fehler beim anlegen des Datensatzes
java.sql.SQLException: General error
at sun.jdbc.odbc.JdbcOdbc.createSQLException(Unknown Source)
at sun.jdbc.odbc.JdbcOdbc.standardError(Unknown Source)
at sun.jdbc.odbc.JdbcOdbc.SQLExecDirect(Unknown Source)
at sun.jdbc.odbc.JdbcOdbcStatement.execute(Unknown Source)
at bean.DBBean.neuenDatensatzAnlegen(DBBean.java:96)
at bean.TestDB_Student.main(TestDB_Student.java:13)

ich hab nun die paltennamen in feste Strings geändert, da ja die Feldnamen fest sein müssen..

also die Methide sieht nun so aus:

public void neuenDatensatzAnlegen(String name,String Studiengang,String Studienbeginn,
			String mail,String Geschlecht){

		
		try {
				
				conn.createStatement().execute(
						"Insert Into student(name,Studiengang,Studienbeginn,mail,Geschlecht)"+
						" Values"+
						"('"
						+ name +"','"
						+ Studiengang +"','"
						+ Studienbeginn +"','"
						+ mail +"','"
						+ Geschlecht +"');");
				
	
		} catch (SQLException e) {
			System.out.println(" Fehler beim anlegen des Datensatzes");
			
			e.getMessage();
			e.getNextException();
			e.printStackTrace();
			e.getErrorCode();
			e.getStackTrace();
			
		}
		
		
		
		
	}

 

[SamHotte]

- ist der Primärschlüssel schon vergeben?
- wie heißen denn die Spalten?
- Statement wird nicht geschlossen.

 

[tachchen]

- ist der Primärschlüssel schon vergeben?
- wie heißen denn die Spalten?
- Statement wird nicht geschlossen.

Primärschlüssel ist userID

erste Spalte ist userID, die restlichen sind wie in der Methode.

Statement schließe ich wie? conn.close() ?

 

[Leroy42]

stmt.close()
conn.close() kommt später.

Bin jetzt seit einer Woche dabei selbst mit Datenbanken zu hantieren, und da mir die
SQL-Syntax noch nicht in Fleisch und Blut übergegangen ist und ich selbst noch
über so manche Kleinigkeiten stolpere, mache ich folgendes:

1. Deinen SQL-String zusammenbasteln und per System.out erstmal ausgeben und anschauen.

2. Weiterhin habe ich mir eine kleine JSP-Seite geschrieben, über die ich SQL.Abfragen und
-Befehle direkt verschicken kann.

Punkt 1 solltest du auf jeden Fall beherzigen und uns evtl. den SQL-String posten.

 

[SamHotte]

Grundidee beim Insert:

    String sql = "INSERT INTO meinetabelle(spalte1,spalte2) VALUES (?, ?)";
    PreparedStatement pstmt = null;
    try {
      pstmt = meineConnection.prepareStatement(sql);

      pstmt.setString(1, "Wert für spalte1");
      pstmt.setString(2, "Wert für spalte2");
      int rows = pstmt.executeUpdate();
      if (rows == 1) {
        // eine Zeile eingefügt, genau wie geplant
        System.out.println("Hat geklappt.");
      }
      else {
        System.err.println("Hat nicht geklappt.");
      }
    }
    catch (SQLException e) {
      System.err.println("Fehler: " + e.getMessage());
    }
    finally {
      try {
        pstmt.close();
      }
      catch (SQLException e) {
        System.err.println("Fehler: " + e.getMessage());
      }
    }

 

[Leroy42]

Es kann sein, dass in den Werten ein Zeichen vorkommt, das dir das SQL-Statement zerschießt - besser benutzt du ein PreparedStatement für sowas

Da bin ich auch schon drüber gestolpert.

Wegen Java benutze ich für SQL-Statements als Zeichenbegrenzung single quotes.
Da muß ich diese selbst verdoppeln, kann aber double quotes direkt benutzen.
Fragen:

1. Brauche ich mich mit preparedStatements darum nicht zu kümmern? Kann ich also
einfach schreiben:

pstmt.setString("Mal ein Quote \' und ein double quote \"");

2. Gibt es außer den Anführungszeichen noch andere spezielle Zeichen auf die
ich achten muß? Zum Beispiel können bei Formulareingaben ja auch newlines vorkommen.

 

[SamHotte]

Gute Frage, so genau kann ich das auch nicht sagen, da ich immer die Daten aus irgendwelchen Textfeldern meinen Stringsvariablen zuweise und von dort in die Datenbank schreibe - ich bau die SQL-Statements nicht im Code zusammen.

Newlines sind bei mir kein Problem, weder beim einlesen noch beim 'rausscheiben, hab Memofelder in meiner Access-DB und befülle die regelmäßig mit sowas. Echte Anführungszeichen (") stehen definitiv auch in der Datenbank und die preparedStatements funzen. Im Zweifel einfach mal ausprobieren.

Das einzige, was nicht klappt hier, ist das schreiben von NULL. Da weiß ich aber nicht, ob das an Access liegt oder an Java.

 

[tachchen]

So ich bin dem Prblem nun näher gerückt,

wenn ich folgende Methode aufrufe, und dann erst die INSERT methode aufrufe geht es komischerweise??
:? vielleicht mus ich vorher die Result Klasse aufrufen ? oder an was liegt da??

hier die andere Methode:

public void alleDatenInListe(){
		
	try {
		
		liste=new LinkedList();
		
		st = conn.createStatement();
		rs =st.executeQuery("SELECT * FROM student;");
		
		while (rs.next()) {
			
			
			name = rs.getString("name");
			Studiengang = rs.getString("Studiengang");
			Studienbeginn = rs.getString("Studienbeginn");
			mail = rs.getString("mail");
			Geschlecht = rs.getString("geschlecht");
                            
			liste.add(name+" "+Studiengang+" "+Studienbeginn+" "+mail+" "+Geschlecht);

		}
		
		} catch (Exception e) {
		System.out.println("Fehler beim Listeneintrag aus der DB	" + e.getMessage() );
	}	
		
	}

oder muss man vorher erst eine Abfrage durchführen? versteh nicht woran es liegt

 

[SamHotte]

Vllt. ist irgendwo noch ein Statement, das du nicht schließt? Ich würde immer einen finally-Block anhängen (in deinem Code grad an Zeile 26), in dem "st.close()" steht. Zumindest war das neulich in 'nem anderen Zusammenhang die Fehlerquelle - da wurden nur 9 Datensätze eingefügt, obwohl die Schleife 10-mal lief; lag damals am fehlenden .close().

 

[personenkult]

Normalerweise macht ein vergessenes close() keine Probleme. Versuch mal anstatt dem execute ein executeUpdate zunehmen. Bei Update, Insert usw. muss man darauf achten die richtige Methode aufzurufen.

executeUpdate = Insert, Update etc.
executeQuery = Select etc.