why this method retuurn null value

Imperor Awad

string sSql;
List vntArray;
sSql = "Select Emp_number,Name,leave_type_desc,Start_date,end_date from vemployee_leave_details";
vntArray = SqlHelper.FindRec(new SqlDataAdapter(sSql, ConnectionManager.BMP));
if (vntArray != null || vntArray.Count != 0)
{
// dbgDetail.DataSource = null;
dbgDetail.DataBind();
pnlNoData.Visible = true;
cmdPrint.Enabled = false;
}
else
{
dbgDetail.DataSource = vntArray.CopyToDataTable();
dbgDetail.DataBind();
dbgDetail.HeaderRow.TableSection = TableRowSection.TableHeader;
pnlNoData.Visible = false;
cmdPrint.Enabled = true;
}
}

2 Replies

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

Muhammad Imran Ansari4y ago
You need to apply && logical operator instead of || because If you didn't get any records in vntArray your object will not be null and your first condition becomes true and second condition (vntArray.Count != 0) will never run.
Try the following code:
string sSql;
List vntArray;
sSql = "Select Emp_number,Name,leave_type_desc,Start_date,end_date from vemployee_leave_details";
vntArray = SqlHelper.FindRec(new SqlDataAdapter(sSql, ConnectionManager.BMP));
if (vntArray != null && vntArray.Count > 0)
{
    dbgDetail.DataSource = vntArray.CopyToDataTable();
    dbgDetail.DataBind();
    dbgDetail.HeaderRow.TableSection = TableRowSection.TableHeader;
    pnlNoData.Visible = false;
    cmdPrint.Enabled = true;
}
else
{
    // dbgDetail.DataSource  = null;
    dbgDetail.DataBind();
    pnlNoData.Visible = true;
    cmdPrint.Enabled = false;
}
Rajanikant Hawaldar4y ago
move the if condition code to else part and else part code to if condition or change if condition as if(vntArray == null || vntArray.Count <= 0)