35

Why does the second one of these produce an exception while the first one doesn't?

string s = null;
MessageBox.Show(s);
MessageBox.Show(s.ToString());

Updated - the exception I can understand, the puzzling bit (to me) is why the first part doesn't show an exception. This isn't anything to do with the Messagebox, as illustrated below.

Eg :

string s = null, msg;
msg = "Message is " + s; //no error
msg = "Message is " + s.ToString(); //error

The first part appears to be implicitly converting a null to a blank string.

3
  • The safe way to do- see How to do ToString for a possibly null object? Aug 5, 2016 at 7:15
  • 6
    To avoid null damage on a string or object, obj?.ToString() ?? ""
    – bvj
    Jan 5, 2017 at 6:34
  • 1
    For second line, the error happens because you invoke a method (.ToString()) on a null string reference. This is why you get an exception. Jul 16, 2020 at 10:26

9 Answers 9

31

because you cannot call instance method ToString() on a null reference.

And MessageBox.Show() is probably implemented to ignore null and print out empty message box.

1
  • 2
    First bit is fine, but I think the MessageBox bit of my question is distracting, hence my update.
    – MartW
    Jan 25, 2011 at 19:18
14

It is because MessageBox.Show() is implemented with pinvoke, it calls the native Windows MessageBox() function. Which doesn't mind getting a NULL for the lpText argument. The C# language has much stricter rules for pure .NET instance methods (like ToString), it always emits code to verify that the object isn't null. There's some background info on that in this blog post.

0
14

As this question ranks quite high on Google for a search for "c# toString null", I would like to add that the Convert.ToString(null) method would return an empty a null string, which is ignored by the messagebox.

However, just to reaffirm the other answers, you can use string.Concat("string", null) in this example.

Edit - modified answer in line with HeyJude's comment below. As pointed out, a method like Convert.ToString(null).Length will throw an exception.

2
  • 3
    Convert.ToString(null) returns a null value, not an empty string (this can be verified by calling Convert.ToString(null) == null, which returns true). However, passing a null variable indeed equals to null, so that object v = null; Convert.ToString(v) == string.Empty returns true (see this answer).
    – OfirD
    Jul 15, 2020 at 9:15
  • 1
    @HeyJude - well spotted, thank you! I'll update accordingly.
    – RooiWillie
    Jul 16, 2020 at 10:08
7

Behind the scenes concat is being called in your follow up question / update E.g

string snull = null;

string msg = "hello" + snull;

// is equivalent to the line below and concat handles the null string for you.
string msg = String.Concat("hello", snull);

// second example fails because of the toString on the null object
string msg = String.Concat("hello", snull.ToString());

//String.Format, String.Convert, String.Concat all handle null objects nicely.
4
  • 2
    Can you show us this behind the scenes because I surely can't find it in mscorlib... Oct 2, 2012 at 17:46
  • So you mean String utility methods just convert it to 'null' string if value returned is null ? Apr 8, 2015 at 11:29
  • @RobertKoritnik Use ildasm.exe on compiled sample code, and you will see what James wrote.
    – Kobor42
    Oct 6, 2015 at 13:15
  • I'm sorry but, this is gold when you have nullable integers. You can write string s = "" + myInt; Thanks, 6 years later ! Sep 19, 2018 at 14:15
3

You are trying to execute the ToString() method on a null. You need a valid object in order to execute a method.

1

The .show function must have null checking and handle it.

2
  • As others have said, the exception is raised because a method is called on a null reference.
    – Vinzz
    Jan 25, 2011 at 12:44
  • 1
    Indeed. That is the obvious part.
    – rcravens
    Jan 25, 2011 at 13:06
0

Because, the second call is expecting an object of "s" to satisfy a ToString() method request. so, before .Show() is called, the s.ToString() would failed with an attempt to call a method.

Interestingly, While .Show() is implemented correctly, many such methods expect non null instances to be passed in. Usually, that is when you use a NullObject pattern so that the caller should not have to deal with this kind of behavior.

0

Probably the Show method handles a null value and just shows nothing. The second use of s - s.ToString() fails because you there is no ToString method to run.

0

ToString() can't operate on null reference of s vairable

The shortest way

obj?.ToString() 

Another correct ways

obj?.ToString() ?? string.Empty  
obj?.ToString() ?? "default string value"  

https://www.informit.com/articles/article.aspx?p=2421572

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service, privacy policy and cookie policy

Not the answer you're looking for? Browse other questions tagged or ask your own question.