Please don't have direct queries like this. It's just bad practice. Instead, take a look at using Command Parameters - they are your friend, and they remove a lot of vulnerabilities from your application (take a look at this[^] article). Also take a look at this[^] article on passing values between forms. Deja View - the feeling that you've seen this post before. My blog | My articles

Heres the code below. dataGridView.CurrentCell.Selected = false; dataGridView.Rows[e.RowIndex].Cells[e.ColumnIndex].Selected = true; dataGridView.CurrentCell = dataGridView.SelectedCells[0];

As someone else said, this can't work, you need the old name in order to find the right record to update, unless you have a unique Id ( definately a better idea ) to identify the one record you want to change. Your code is also insecure, you need to read up on SQL injection attacks. Christian Graus - Microsoft MVP - C++ "I am working on a project that will convert a FORTRAN code to corresponding C++ code.I am not aware of FORTRAN syntax" ( spotted in the C++/CLI forum )

You might want to simplify it by not using the OleDbDataAdapter, but use an OleDbCommand directly. string strConnection = "Provider=Microsoft.Jet.OLEDB.4.0;" + "Data Source=datacapt.mdb"; OleDbConnection conn = new OleDbConnection(strConnection); conn.Open(); string strCommand = "INSERT INTO compdata(ID,CompanyName,ContactName) Values ('" + txID.Text + "','" + txtName.Text + "','" + txtcontact.Text + "')"; OleDbCommand comm = new OleDbCommand(strCommand, conn); int t1 = command.ExecuteNonQuery(); conn.Close(); if (t1 > 0) { MessageBox.Show("added successfully!"); }

Welcome to CodeProject. There are a large number of C# Database Articles here follow the link[^]

You can easily find a lot of C# tutorial online. Here you can have two links im providing, go through any of them or use both for reference. http://www.softsteel.co.uk/tutorials/cSharp/contents.html[^] http://www.csharp-station.com/Tutorial.aspx[^]