C#, "Object reference not set to an instance of an object." error

Question

I've got this code...

namespace YellowBox
{
    public partial class Form1 : Form
    {

        private string sid = "";

        FileTransferManager fm = new FileTransferManager();
        Jid _jid = new Jid();


        public Form1()
        {
            InitializeComponent();

            fm.OnError += fm_OnError;
            fm.OnEnd += fm_OnEnd;
            fm.OnStart += fm_OnStart;
            fm.OnProgress += fm_OnProgress;
        }

        private void btn_pickFile_Click(object sender, System.EventArgs e)
        {
            var of = new OpenFileDialog();
            if (of.ShowDialog() == DialogResult.OK)
            {
                tb_file.Text = of.FileName;

                var fi = new FileInfo(of.FileName);
                //lblSize.Text = Util.HumanReadableFileSize(fi.Length);

                btn_sendFile.Enabled = true;
            }
        }

        private void btn_sendFile_Click(object sender, System.EventArgs e)
        {

        _jid.Server = "xxx";
        _jid.User = "xxx";  /// EDIT, added the _jid values.
        _jid.Resource = "xxx";

            sid = fm.Send(_jid, tb_file.Text, ""); /// HERE IT SAYS "Object reference not set to an instance of an object." ???
            btn_sendFile.Enabled = false;
            btn_pickFile.Enabled = false;
        }

...

And when I hit the btn_sendFile it gives me a "Object reference not set to an instance of an object." error. But I had instanced the fm object in FileTransferManager fm = new FileTransferManager(); , didn't I?

SOLVED: Appears it was missing fm.XmppClient = xmppClient;

Set a breakpoint on that line, and check which variable is null. — Blorgbeard
– Blorgbeard, Commented Mar 11, 2011 at 15:37
FileTransferManager comes from a separate dll. But I guess it loads fine, as if it wouldn't the error would come from instancing it... — Roger
– Roger, Commented Mar 11, 2011 at 15:56

Manish Basantani · Accepted Answer · 2011-03-11 15:52:30Z

2

Reading your comments, i don't think any of the parameters you are passing to "Send" is null.

I Would say, there is a usability issue in the "FileTransferManager" class. It is possiby expecting something more from the user (something like, init,configure).

You will need to set breakpoint inside the FileTransferManager and then debug. No other choice.

answered Mar 11, 2011 at 15:52

Manish Basantani

17.6k23 gold badges77 silver badges107 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Roger Over a year ago

THANKS EVERYBODY! Here's what appeared to be missing fm.XmppClient = xmppClient;

Lav · Accepted Answer · 2011-03-11 15:38:26Z

2

But what about your tb_file object.

I dont see instantiation or definition of that object anywhere in your code.

answered Mar 11, 2011 at 15:38

Lav

1,89615 silver badges17 bronze badges

3 Comments

Roger Over a year ago

never mind the tb_file, its a texbox, that gets its value assigned from tb_file.Text = of.FileName;

Lav Over a year ago

@Roger I would recommend like others, adding try catch block around the line of code thats giving you trouble. That will help you pinpoint it. May be after that you can provide us with more informaiton about the exception?

Roger Over a year ago

Did so, the trouble comes only from "sid = fm.Send(_jid, tb_file.Text, "myFile");"

krtek · Accepted Answer · 2011-03-11 15:39:08Z

1

Is it possible that the error is on tb_file.Text ?

There's no definition of this variable anywhere in the code. If this is the problem, you should also correct the assignation in the btn_pickFile_Click method.

answered Mar 11, 2011 at 15:39

krtek

26.7k5 gold badges60 silver badges84 bronze badges

1 Comment

Roger Over a year ago

never mind the tb_file, its a texbox, that gets its value assigned from tb_file.Text = of.FileName;

SecretDeveloper · Accepted Answer · 2011-03-11 15:39:34Z

1

Can you debug the application and set a breakpoint on the line of code that is throwing the exception. Hover you mouse over each object on that line and it will show you which one is null. That will let you know where the problem is, after that it is a matter of figuring out why it is null. I cant tell anything more from the code you have posted.

answered Mar 11, 2011 at 15:39

SecretDeveloper

3,1502 gold badges31 silver badges36 bronze badges

1 Comment

Roger Over a year ago

Did so, the trouble comes only from "sid = fm.Send(_jid, tb_file.Text, "myFile");"

MattC · Accepted Answer · 2011-03-11 15:45:45Z

1

Probably wise to stick a try..catch block around the sid = fm.Send(_jid, tb_file.Text, ""); call snd then in the catch you will be able to see the stack trace which should tell you where the exception is orginating.

It could be being generated from inside you FileTransferManager class.

answered Mar 11, 2011 at 15:45

MattC

4,0241 gold badge37 silver badges50 bronze badges

1 Comment

MattC Over a year ago

Based on your stack trace it's definately comnig from within your Matrix.Xmpp.Client.FileTransferManager object.

Kumar · Accepted Answer · 2011-03-11 15:55:34Z

1

Break point at the line and hover to see the value fm. Or tb_file.Text might be the one.

Other Check - Before calling sid = fm.Send(_jid, tb_file.Text, "");, can you print all the parameters and verify the values.

edited Mar 11, 2011 at 15:55

answered Mar 11, 2011 at 15:38

Kumar

1,0155 silver badges8 bronze badges

1 Comment

Kumar Over a year ago

Then something must be going on inside the method fm.Send. Can you able to step in and see.

Collectives™ on Stack Overflow

C#, "Object reference not set to an instance of an object." error

6 Answers 6

1 Comment

3 Comments

1 Comment

1 Comment

1 Comment

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

6 Answers 6

1 Comment

3 Comments

1 Comment

1 Comment

1 Comment

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related