There is no row at position 0.

You have a few issues in your code here:

1. First off, do not append the values directly to your SQL query as it can leads to SQL Injection attack, even if the value is coming from the GridView and even if it's not coming from the user input still I would not recommend it. It's a big No No and security risk. Check this article for an example: https://www.c-sharpcorner.com/article/protect-your-data-prevent-sql-injection/

2. Break down your code into lines so you can easily debug it.

3. Always check for the result rows before accessing the values from it. For example, you can do something like this:

var data = dm.GetData(string.Format("select IDUOM from dbo.UOM where NameUOM ='{0}'", GVitem.SelectedRow.Cells[4].Text.Trim()));
if (data.Rows.Count > 0)
{
string SesnId1 = data.Rows[0][0].ToString();
}

1 Reply

Know the answer? Post it — somebody with the same question will find it here.

Vincent Maverick Durano7y ago
Accepted answer
You have a few issues in your code here:

1. First off, do not append the values directly to your SQL query as it can leads to SQL Injection attack, even if the value is coming from the GridView and even if it's not coming from the user input still I would not recommend it. It's a big No No and security risk. Check this article for an example: https://www.c-sharpcorner.com/article/protect-your-data-prevent-sql-injection/

2. Break down your code into lines so you can easily debug it.

3. Always check for the result rows before accessing the values from it. For example, you can do something like this:
var data = dm.GetData(string.Format("select IDUOM from dbo.UOM where NameUOM ='{0}'", GVitem.SelectedRow.Cells[4].Text.Trim()));
        if (data.Rows.Count > 0)
        {
            string SesnId1 = data.Rows[0][0].ToString();
        }