OleDbCommand mit Variabeln? - iB-InformatikBoard.ch

Stoked · Registriert seit: 09.08.2005

Saaali zemme..

Ich bin gerade an den Arbeiten für ein Schulpojekt in C#..

.. Wir sind dabei ein PRogramm zu schreiben, das am Ende eine Klassen(Also im Sinne von Schulklassen)-Verwaltung darstellen soll. Das Ziel des Projektes ist es, die Daten in eine Datenbank zu schreiben und da wieder rauszulesen.

Mit dem herauslesen habe ich wirklich keine Probleme funktioniert alles wunderbar. Aber mit dem Einfügen schon.

Code:

        private void btnKadd_Click(object sender, EventArgs e)
        {
            k = new Klasse();
            k.ClearKlasse();
            k.setKlasseName(Convert.ToString(txtKName.Text));
            k.setKlassenArt(Convert.ToString(txtKart.Text));

            OleDbCommand cmd = new OleDbCommand();
            cmd.Connection = AllePersonen.getInstanz().getOpenConnection();
            cmd.CommandText="INSERT INTO Klasse (m_ID,m_KName,m_KArt) VALUES (...);

            ListViewRefresh();

        }

Das hier ist der Code den ich bisher habe. Zur Erklärung: Die Daten die reingeschrieben werden sollen werden von den beiden Textfeldern "txtKName" und "txtKArt" eingelesen. die ID wird per Konstruktor vergeben. Die beiden eingelesen strings werden dann zum Abfangen von Fehlerhaften oder nicht gemachten eingaben in der Klasse "Klasse" in die Membervariablen abgefüllt und von dort sollten sie mit der get-Methode wieder eingelesen werden.

Mein Problem ist jetzt, dass ich mit dem "INSERT INTO" kommande nicht feste werte, sondern jedesmal die gerade eingelesenen Daten in die Datenbank einfügen möchte. Das wollte ich eigentli mit Variablen machen. Funktionierte aber nicht wirklicht.

Kann mir jemand eine Lösung bieten? oder zumindest einen Tipp geben?

achja: Die Methode ListviewRefresh() aktualisiert, wie der name schon sagt, die ListView.

Phil · Registriert seit: 01.06.2004

Hi Stoked

Sowie du das gemacht hast, wird's nicht funktionieren. Du hast zwar das SQL Command richtig zugewiesen, du musst es nur noch ausführen. Benutze dazu die Methode ExecuteNonQuery.

gruss
Phil

Stoked · Registriert seit: 09.08.2005

hi Phil

joa.. so hatte ich das auch anfangs... dann merkte ich dass es nicht funktionirt und hab alles wieder abgerissen und nochmal vorne angefangen... deshalb ist es jetzt nicht mehr da

Ich bin jetzt zu dem Schluss gekommen, dass dies, so wie ich es hier mache nicht zum gewünschten Ergebnis führt und Arbeite jetzt mit dem OleDbDataAdapter und DataSet.. Mal schauen was dabei raus kommt

Aber trotzdem danke für die Hilfe.

Stoked · Registriert seit: 09.08.2005

Nachdem ich weitere 3h daran gearbeitet habe gebe ich auf.

Ich komme wieder zurück zu diese Lösung hier.

Code:

        private void btnKadd_Click(object sender, EventArgs e)
        {
            k = new Klasse();
            k.ClearKlasse();
            k.setKlasseName(Convert.ToString(txtKName.Text));
            k.setKlassenArt(Convert.ToString(txtKart.Text));

            string t1 = "blubb";
            string t2 = "blöbb";

            OleDbCommand cmd = new OleDbCommand();
            cmd.Connection = AllePersonen.getInstanz().getOpenConnection();
            cmd.CommandText = "INSERT INTO Klassen (m_ID, m_KName, m_KArt) VALUES(" + k.getm_KID(); "','" + k.getm_KName(); "','" + k.getm_KArt(); "')";
            try
            {
                OleDbDataAdapter da = new OleDbDataAdapter(cmd);
                cmd.ExecuteNonQuery();
            }
            catch (Exception ex)
            {
                MessageBox.Show("Fehler:" + ex);
            }
            ListViewRefresh();
        }

So.. sieht es jetzt komplett aus.

Also die jeweils Fett und Rot Markierten Teile verursachen den Compilerfehler CS0201 (Nur assignment-, call-, increment-, und "new object"-Ausdrücke können als Anweisung verwendet werden.).
Nur.. weiss ich nicht so recht was er mir damit sagen will. Und vorallem weiss ich nicht warum es bei der ersten Methode funkioniert und nachher nicht mehr.

Ich dreh noch durch hier...

ahja: :lol: der Blubber und der Blöbber string sind nur da weil ich die mal zum Testen nutzte...

Wenn ich das ganze nur mit Text also Ohne die Methoden mache funktioniert es einwandfrei...

Phil · Registriert seit: 01.06.2004

Code:

cmd.CommandText = "INSERT INTO Klassen (m_ID, m_KName, m_KArt) VALUES(" + k.getm_KID(); "','" + k.getm_KName() + "','" + k.getm_KArt() + "')";

so sollte es funktionieren. Du hattest zu viele Semikolons hintereinander, wie du ja sicherlich weisst wird durch ein Semikolon eine Anweisung eindeutig abgeschlossen.

Stoked · Registriert seit: 09.08.2005

danke schonmal

werds zuhause ausprobieren.

Aber zu dem Thema mit den Semikolons: Das weiss ich sehr wohl ja... Aber der Compiler meckerte jedesmal wenn ich es Ohne Semikolons machte.. dass da jeweils ein Smikolon fehlt...

Phil · Registriert seit: 01.06.2004

Zitat:

Zitat von Stoked

danke schonmal

werds zuhause ausprobieren.

Aber zu dem Thema mit den Semikolons: Das weiss ich sehr wohl ja... Aber der Compiler meckerte jedesmal wenn ich es Ohne Semikolons machte.. dass da jeweils ein Smikolon fehlt...

Ja das ist klar, aber da CommandText ein String ist, muss du den String mittels + zusammensetzen (siehe Posting oben!).

softi · Registriert seit: 16.10.2006

Zitat:

Zitat von Phil

Code:

cmd.CommandText = "INSERT INTO Klassen (m_ID, m_KName, m_KArt) VALUES(" + k.getm_KID(); "','" + k.getm_KName() + "','" + k.getm_KArt() + "')";

so sollte es funktionieren. Du hattest zu viele Semikolons hintereinander, wie du ja sicherlich weisst wird durch ein Semikolon eine Anweisung eindeutig abgeschlossen.

Dies hat aber zwei Proleme:
1. Wenn einer der String selbst ein ' enthält, wird eine exception ausgelöst. Oder in einem noch schlimmeren Fall, manipuliert jemand per SQL-Injection die Daten. Jenach Anwendungsbereich ist dies recht gefährlich.

2.Wenn die Strings leer sind, müsste eigentlich nicht '' eingefügt werden sondern NULL (ohne ' ). Jetzt kommt es aber noch auf die Datenbank an die dahintersteht. Evt. ist es der DB egal, evt. gibt es ein Fehler oder es ist eben doch nicht so wahr was ich nun eben geschrieben habe.

Ich würde es darum etwa so machen:

OleDbCommand cmd = new OleDbCommand();
cmd.Connection = AllePersonen.getInstanz().getOpenConnection();
cmd.CommandText = "INSERT INTO Klassen (m_ID, m_KName, m_KArt) VALUES(?,?,?)";
cmd.Parameters.Add("m_KID", OleDbType.Integer) = k.getm_KID;
cmd.Parameters.Add("m_KName", OleDbType.VarChar) = k.getm_KName;
cmd.Parameters.Add("m_KArt", OleDbType.VarChar) = k.getm_KArt;
cmd.ExecuteNonQuery();

gruss

Themen-Optionen
Druckbare Version zeigen Jemanden per E-Mail auf dieses Thema hinweisen