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

System.NullReferenceException was unhandled

Hi all,
i'm getting this error and im trying everything to resolve it.  below is the code that is the problem.
when i run it,
this.HeadersNeededSearch(myHeaderData);
pulls back the following error:
"System.NullReferenceException was unhandled"
      - "Object reference not set to an instance of an object."

Basically, what it does, passes a number of variables into a <list>.
any ideas where this is wrong?what am i missing?
public delegate void HeadersNeededSearchHandler(HeaderSearchData e);
public event HeadersNeededSearchHandler HeadersNeededSearch;
        
private void HeaderSearch(ManagementInterfaces.EDIStatusSelection HeaderStatus, DateTime OrderDate, string SupplierID, string PONo)
        {
            this.Cursor = System.Windows.Forms.Cursors.WaitCursor;
 
            HeaderSearchData myHeaderData = new HeaderSearchData();
            HeadersNeededSearchHandler test = HeadersNeededSearch;
 
            myHeaderData.HeaderStatus = HeaderStatus;
            myHeaderData.TargetDate = OrderDate;
            myHeaderData.SupplierID = SupplierID;
            myHeaderData.PONo = PONo;
 
            this.HeadersNeededSearch(myHeaderData);
 
            this.NoOrdersSearchLabel.Visible = !(myHeaderData.HeaderCount > 0);
            this.OrderSearchView.DataSource = myHeaderData.Headers;
 
            this.Cursor = System.Windows.Forms.Cursors.Default;
        }

Open in new window

0
zell71
Asked:
zell71
  • 3
  • 3
  • 2
  • +2
1 Solution
 
p_davisCommented:
have you gone into that method to see where the error occurs-- i would suggest breakpoint at the top of the
 this.HeadersNeededSearch(myHeaderData) method.
0
 
NIMTUG_SimonCommented:
The error is in side the HeadersNeededSearch find this method and add a break line on the first line of code inside the method and then step over(F10), otherwise break no the HeaderNeedSearch(myHeaderData) and step into (F11) rather than step over this line.
0
 
dev_kittuCommented:
this.headerneededsearch(myheaderdata)
here the object is referring a null object , , , i'm sure i will get an error.
0
Industry Leaders: 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!

 
p_davisCommented:
asker has that object instantiate before making that call -- as well as setting property values, so it shouldn't be null.
0
 
intrwrksCommented:
Not sure what Line # 9 (HeadersNeededSearchHandler test = HeadersNeededSearch;) is accomplishing. I would remove that line9 and then check for null on the event. If you are not subscribed to the event in whatever form you are using this control, then the event call will throw a NullReferenceException. I have attached recommended new code below.

Jason
public delegate void HeadersNeededSearchHandler(HeaderSearchData e);
public event HeadersNeededSearchHandler HeadersNeededSearch;
        
private void HeaderSearch(ManagementInterfaces.EDIStatusSelection HeaderStatus, DateTime OrderDate, string SupplierID, string PONo)
        {
            this.Cursor = System.Windows.Forms.Cursors.WaitCursor;
 
            HeaderSearchData myHeaderData = new HeaderSearchData();
             
            myHeaderData.HeaderStatus = HeaderStatus;
            myHeaderData.TargetDate = OrderDate;
            myHeaderData.SupplierID = SupplierID;
            myHeaderData.PONo = PONo;
 
            if (this.HeadersNeededSearch != null)
            {
                 this.HeadersNeededSearch(myHeaderData);
            }
 
            this.NoOrdersSearchLabel.Visible = !(myHeaderData.HeaderCount > 0);
            this.OrderSearchView.DataSource = myHeaderData.Headers;
 
            this.Cursor = System.Windows.Forms.Cursors.Default;
        }
 
Open in New Window 

Open in new window

0
 
zell71Author Commented:
ok intrwrks, firstly HeadersNeededSearchHandler test = HeadersNeededSearch; was used as testing and not removed for purpose for you guys to see..so that can be ignored.
using your method, it works and passes over this.HeadersNeededSearch(myHeaderData);
however doesnt return expected data, any other suggestions out there please feel free to post.  im just testing over code
0
 
intrwrksCommented:
What do you mean it doesn't return the expected data? Do you mean the event is not firing?

If the HeadersNeededSearch is null, then that means the form or class using the control has not subscribed to the event properly.

Are you subscribing to the event somewhere in your code? Example in code window.

void someFunction()
{
     YourCustomControl.HeadersNeededSearch += new HeadersNeededSearchHandler(YourCustomControl_HeadersNeededSearch);
}
 
protected void YourCustomControl_HeadersNeededSearch(object sender, HeaderSearchData e)
{
      // process data here
}

Open in new window

0
 
zell71Author Commented:
abit of background..it was a project left last year, so i've been asked to pick it up and finish it.
I think your right..its not hitting this..which is surely the event.
so
YourCustomControl.HeadersNeededSearch += new HeadersNeededSearchHandler(YourCustomControl_HeadersNeededSearch); is the subscription code?

class clsServerConnection
    {
        public clsServerConnection(EDIManagementConsole FormToConnect)
        {
        FormToConnect.HeadersNeededSearch += new EDIManagementConsole.HeadersNeededSearchHandler(HandleHeadersNeededSearch);
        }
...

Open in new window

0
 
intrwrksCommented:
That code should subscribe to the event as long as the value passed into the Constructor is a reference to the proper control. You should then have a function in that class named HandleHeadersNeededSearch that would actually handle the event and receive the appropriate data.

Jason
protected void HandleHeadersNeededSearch(object sender, EDIManagementConsole.HeaderSearchData e)
{
      // handle data.
      // e should contain the values from the event.
}

Open in new window

0
 
zell71Author Commented:
Jason, cheers mate, a little bit of backwards engineering sorted it.
Basically, gonna have to scrap the events and start it in my way.
i have it working now so if i want to call from events i can do it once i got more time.
A rating because you replied fast and VERY helpful.  your a star *
0

Featured Post

What does it mean to be "Always On"?

Is your cloud always on? With an Always On cloud you won't have to worry about downtime for maintenance or software application code updates, ensuring that your bottom line isn't affected.

  • 3
  • 3
  • 2
  • +2
Tackle projects and never again get stuck behind a technical roadblock.
Join Now