It show an error : there is no row at the position

mohaammed talha

5y816.NET

when the row is empty then its not move to another row and it show an error:there is no row at the position

public void loadAct()
{
string qryAct = "";
DateNow = DateTime.Now.ToString("yyyy-MM-dd");
DataTable dt2 = new DataTable();
if (txtDate.Text != "")
{
qryAct = "select Line, MAX(TotalPairs)Pairs from ProdHourlyRep where Date= '" + txtDate.Text + "' and Team='C' group by Line";
}
else
{
qryAct = "select Line, MAX(TotalPairs)Pairs from ProdHourlyRep where Date='" + DateNow + "' and Team='C' group by Line";
}
SqlDataAdapter objAdapter2 = new SqlDataAdapter(qryAct, strConn);
objAdapter2.Fill(dt2);
if (dt2.Rows.Count > 0)
{
lblActl1.Text = dt2.Rows[0][1].ToString();
lblActl2.Text = dt2.Rows[1][1].ToString();
lblActl3.Text = dt2.Rows[2][1].ToString();
//lblActl4.Text = dt2.Rows[3][1].ToString();
}
}

4 Replies

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

Rijwan Ansari5y ago
Accepted answer
Better check the null value before converting to string.
However, instead of
lblActl1.Text = dt2.Rows[0][1].ToString();
lblActl2.Text = dt2.Rows[1][1].ToString();
lblActl3.Text = dt2.Rows[2][1].ToString();
//lblActl4.Text = dt2.Rows[3][1].ToString();
Use this way
lblActl1.Text = Convert.ToString( dt2.Rows[0][1]);
lblActl2.Text = Convert.ToString(dt2.Rows[1][1]);
lblActl3.Text = Convert.ToString( dt2.Rows[2][1]);
lblActl4.Text = Convert.ToString(dt2.Rows[3][1]);
Convert.ToString handles null value and return ".
Convert.ToString(dt2.Rows[3][1]) will return '' for null row position.
But dt2.Rows[3][1].ToString(); will throw error.
Salman Beg5y ago
Hey. Better check for the null and empty condition and then you never face such type of errors. Thanks.
Sachin Singh5y ago
remove empty rows then,
public void loadAct()
{
string qryAct = "";
DateNow = DateTime.Now.ToString("yyyy-MM-dd");
DataTable dt2 = new DataTable();
if (txtDate.Text != "")
{
qryAct = "select Line, MAX(TotalPairs)Pairs from ProdHourlyRep where Date= '" + txtDate.Text + "' and Team='C' group by Line";
}
else
{
qryAct = "select Line, MAX(TotalPairs)Pairs from ProdHourlyRep where Date='" + DateNow + "' and Team='C' group by Line";
}
SqlDataAdapter objAdapter2 = new SqlDataAdapter(qryAct, strConn);
objAdapter2.Fill(dt2);

DataTable dt = dt2.Rows
                 .Cast()
                 .Where(row => !row.ItemArray.All(f => f is DBNull ||
                                  string.IsNullOrEmpty(f as string ?? f.ToString())))
                 .CopyToDataTable();


if (dt.Rows.Count > 0)
{
lblActl1.Text = dt2.Rows[0][1].ToString();
lblActl2.Text = dt2.Rows[1][1].ToString();
lblActl3.Text = dt2.Rows[2][1].ToString();
//lblActl4.Text = dt2.Rows[3][1].ToString();
}
}
Amit Gupta5y ago
Check the count of rows( in your case 3) and use Convert.ToString(dt2.Rows[0][1]);