• Status: Solved
  • Priority: Medium
  • Security: Public
  • Views: 681
  • Last Modified:

Null Value in Control While trying to populate drop down list

I am trying to dynamically populate the controls on a web page.  Regardless of whether there are 1 dropdpwnlist or several, I would like to call the same method to populate.  The problem I am having is trying to get the dropdown to work dynamically by carrying the name.  This way I do not have to write code to populate every field.   The dropdown casting does not give me an error.  It is when I try to assign the data source that it gives me a runtime error that the "Object reference not set to an instance of an object."  What I am doing wrong or am I doing it completely wrong?  

Thanks,
Doug
populateDropDownList(controlID, sql);
 
protected void populateDropDownList(String controlID, String sql)
    {
        DBAccessUsingDB2 db2Access = new DBAccessUsingDB2();
        CustomUtilities cu = new CustomUtilities();
        DataSet ds = new DataSet();
        DropDownList ddlId = (DropDownList)cu.FindControl(controlID);
        ds = db2Access.FindRecords(ds, sql);
        if (ds.Tables[0].Rows.Count > 0)
        {
            DDLID.DataSource = ds.Tables[0];
            DDLID.DataTextField = ds.Tables[0].Columns[1].ColumnName.ToString();
            agencyIDDDL.DataValueField = ds.Tables[0].Columns[0].ColumnName.ToString();
            agencyIDDDL.DataBind();
        }

Open in new window

0
anewlevel4u
Asked:
anewlevel4u
  • 4
1 Solution
 
Priest04Commented:
Looking at these two lines

DropDownList ddlId = (DropDownList)cu.FindControl(controlID);
DDLID.DataSource = ds.Tables[0];

these two variables have different casing, are therefore are not pointing to the same object. Should it be
protected void populateDropDownList(String controlID, String sql)
    {
        DBAccessUsingDB2 db2Access = new DBAccessUsingDB2();
        CustomUtilities cu = new CustomUtilities();
        DataSet ds = new DataSet();
        DropDownList ddlId = (DropDownList)cu.FindControl(controlID);
        ds = db2Access.FindRecords(ds, sql);
        if (ds.Tables[0].Rows.Count > 0)
        {
            ddlId.DataSource = ds.Tables[0];
            ddlId.DataTextField = ds.Tables[0].Columns[1].ColumnName.ToString();
            agencyIDDDL.DataValueField = ds.Tables[0].Columns[0].ColumnName.ToString();
            agencyIDDDL.DataBind();
        }

Open in new window

0
 
anewlevel4uAuthor Commented:
I figured the solution.  The casing was not the problem.  I simply needed to send the control type as a parameter in the method call.  See below the new code.
populateDropDownList(agencyNameDDL, sql);
 
//simply send the control type through the parameter in the method
protected void populateDropDownList(DropDownList ddlId, String sql)
{
     DBAccessUsingDB2 db2Access = new DBAccessUsingDB2();
     DataSet ds = new DataSet();
     ds = db2Access.FindRecords(ds, sql);
     if (ds.Tables[0].Rows.Count > 0)
     {
         //The value send to the method can be called directly.
         //This could be updated for whatever type you need.
         ddlId.DataSource = ds.Tables[0];
         ddlId.DataTextField = ds.Tables[0].Columns["NAMEFEIN"].ColumnName.ToString();
         ddlId.DataValueField = ds.Tables[0].Columns[0].ColumnName.ToString();
         ddlId.DataBind();
     }        
}

Open in new window

0
 
Priest04Commented:
It shouldnt make any difference, both methods should work. As you said it above, it throws an error on this line.

DDLID.DataSource = ds.Tables[0];

"Object reference not set to an instance of an object."

this means that DDLID is not instantiated, or is equal to NULL. I dont see where have you declared this variable, and in your latter example you use ddlId, which is of different casing. So I dont understand how can you say its not the casing problem? :)
0
 
Priest04Commented:
I find that anewlevel4u's statement that the resolution was to send the control reference is not correct. It doesn't make any difference if control ID is passed to a FindControl method, or a control reference is passed, both will have the same result - reference to a ListBox control.

This is the line that caused the exception:

DDLID.DataSource = ds.Tables[0];

"Object reference not set to an instance of an object."

This without doubt indicates that DDLID is not pointing to an object on the heap.  There can be several reasons for this:

1) DropDownList ddlId = (DropDownList)cu.FindControl(controlID);

As it can be seen, ddlID and DDLID are not the same. Could be that DDLID was declared somewhere outside the procedure, so there was a mistake in casing.

2) a wrong Control ID is passed to a method, resulting in ddlId to have a NULL reference.

In either case, this has nothing to do with passing the control reference as a parameter. And ss it can be seen in anewlevel4u's solution, casing WAS corrected.

I disagree with people denying the correct answer, just to get refunded points. anewlevel4u did not even replied to my remarks, could be because he (she) didn't have an argument to pass.
0

Featured Post

Independent Software Vendors: We Want Your Opinion

We value your feedback.

Take our survey and automatically be enter to win anyone of the following:
Yeti Cooler, Amazon eGift Card, and Movie eGift Card!

  • 4
Tackle projects and never again get stuck behind a technical roadblock.
Join Now