aktualisieren von Daten mit OLE DB in c#

Machte ich ein Projekt mit c# und Datenbank mithilfe von access accdb und verbunden zwischen den beiden. Ich habe 2 Schaltflächen, erste neue Kunde, der perfekt funktioniert, und die zweite eine Aktualisierung der Daten des Kunden (Vorname und Nachname), für einige Grund, die Schaltfläche "aktualisieren" funktioniert nicht, es gibt keine Fehlermeldung, wenn ich das Projekt ausführen, aber nachdem ich auf nichts passiert...

private void button2_Click(object sender, EventArgs e)
{
    connect.Open();
    string cid = textBox1.Text;
    string cfname = textBox2.Text;
    string clname = textBox3.Text;
    OleDbCommand command = new OleDbCommand();
    command.Connection = connect;
    command.CommandText = "UPDATE Tcostumers SET cfname= " + cfname + "clname= " + clname + " WHERE cid = " + cid;
    if (connect.State == ConnectionState.Open)
    {
        try
        {
            command.ExecuteNonQuery();
            MessageBox.Show("DATA UPDATED");
            connect.Close();
        }
        catch (Exception expe)
        {
            MessageBox.Show(expe.Source);
            connect.Close();
        }
    }
    else
    {
        MessageBox.Show("ERROR");
    }
}
  • Ist Ihre button1_Click code der anders als die Abfrage?
InformationsquelleAutor Amir | 2015-04-09
  1. 3

    Ich glaube, dein commandtext ist, wo das Problem liegt;

    command.CommandText = "UPDATE Tcostumers SET cfname= " + cfname + "clname= " + clname + " WHERE cid = " + cid;

    Benötigen Sie ein Komma zwischen den set-Anweisungen, und auch als Gino wies auf die speechmarks.

    Edit:

    Es ist besser, als Sie die Verwendung von Parametern für die Variablen, Ihre aktuelle Methode ist offen für SQL-injection, zB.

        private void button2_Click(object sender, EventArgs e)
    {
        OleDbCommand command = new OleDbCommand(@"UPDATE Tcostumers
                                                    SET cfname = @CFName,
                                                        clname = @CLName
                                                    WHERE cid = @CID", connect);

        command.Parameters.AddWithValue("@CFName", textBox2.Text);
        command.Parameters.AddWithValue("@CLName", textBox3.Text);
        command.Parameters.AddWithValue("@CID", textBox1.Text);

        try
        {
            connect.Open();
        }
        catch (Exception expe)
        {
            MessageBox.Show(expe.Source);
        }
        try
        {
            command.ExecuteNonQuery();

            MessageBox.Show("DATA UPDATED");
        }
        catch (Exception expe)
        {
            MessageBox.Show(expe.Source);
        }
        finally
        {
            connect.Close();
        }
    }

    Sein, wie ich neigen dazu, zu formatieren, meine code so tun, wie Sie mit ihm. Hoffe, es hilft.

    InformationsquelleAutor horHAY
  2. 2

    Es mag eine dumme Sache, aber...

    Sie aktualisieren strings, ints nicht, so versuchen Sie, hinzufügen ", um deine Saiten etwas wie: 

    command.CommandText = "UPDATE Tcostumers SET cfname= '" + cfname + "' clname='" + clname + "' WHERE cid = " + cid;
    InformationsquelleAutor Gino
  3. 0
    //my sample code for edit/update 
Table Name = StudentFIle
Fields = id,fname,lname

bool found = false;
OleDbConnection BOMHConnection = new OleDbConnection(connect);
string sql = "SELECT * FROM StudentFIle";
BOMHConnection.Open();
OleDbCommand mrNoCommand = new OleDbCommand(sql, BOMHConnection);
OleDbDataReader mrNoReader = mrNoCommand.ExecuteReader();
while (mrNoReader.Read())
{
    if (mrNoReader["id"].ToString().ToUpper().Trim() == idtextbox.Text.Trim())
    {
        mrNoReader.Close();
        string query = "UPDATE StudentFIle set fname='" +firstnametextbox.Text+ "',lname='"+lastnametextbox.Text+"' where id="+idtextbox.Text+" ";
        mrNoCommand.CommandText = query;
        mrNoCommand.ExecuteNonQuery();
        MessageBox.Show("Successfully Updated");

        found = true;
        break;
    }
    continue;
}
if (found == false)
{
    MessageBox.Show("Id Doesn't Exist !.. ");
    mrNoReader.Close();
    BOMHConnection.Close();
    idtextbox.Focus();
}
    • Man könnte es erklären, und geben einige Details...
    InformationsquelleAutor ricardo borinaga
Schreibe einen Kommentar