[NejcZ]

Hi, with your previus help, I managed to get most of the things sorted out. now there is a problem with my DataGridView. When I update row in DataGridView, it stays there until i close the program, so I suppose, changes are only made to loaded DataTable. I searched alot for the solution on the internet, but nothing worked.

Here is the update function (only updating one row at the moment):

void update(int row)
        {
            DataGridViewCell cell = dataGridView2.CurrentCell as DataGridViewCell;
            if (cell.IsInEditMode)
            {
                string query;
                sqlcon.Open();
                query = "update tabelaZaposlenih set ime='" + dataGridView2.Rows[row].Cells[1].EditedFormattedValue + "'";
                sqlcmd = new SqlCommand(query, sqlcon);
                // i added this line afterwards, but no joy...
                sqlcmd.Parameters.AddWithValue("ime", dataGridView2.Rows[row].Cells[1].EditedFormattedValue);
                sqlcmd.ExecuteNonQuery();
                sqlcon.Close();
                MessageBox.Show("Podatki uspešno posodobljeni", "Podatki uporabnika");
            }
        }

What am I doing wrong this time? Thanks.

[Momerath]

Are you using a file based database?

[trichardson]

1. I would remove line #11 since you are not using a stored procedure to update the database.
2. Make sure

void update (int)

is actually being called when the time comes (perhaps through an event?).
3. "When I update row in DataGridView, it stays there until i close the program, so I suppose, changes are only made to loaded DataTable." What do you mean by "stays there?"

[Curtis Rutland]

You're using parameters incorrectly, for one thing. You're doing string concatenation as well as adding parameters. What you should be doing is something more like this:

var imeValue = dataGridView2.Rows[row].Cells[1].EditedFormattedValue;
query = "update tabelaZaposlenih set ime=@ime";
sqlcmd = new SqlCommand(query, sqlcon);
sqlcmd.Parameters.AddWithValue("@ime", imeValue);

That's a more correct and safe query. Whether or not that will make your code work, I don't know, because you could have connection problems, or other issues. But it's the correct way to use parameters in a query.

trichardson, on 03 January 2013 - 10:42 AM, said:

1. I would remove line #11 since you are not using a stored procedure to update the database.

One has nothing to do with the other. Parameters aren't just for stored procedures, they're the right way to do any query that needs input.