Recently, I've seen the following statement quite a lot:
object o;
// assign o
return "" + o;
(Basically the same as return String.Concat(o);
)
Why would anyone want to do that? Why not just call .ToString()
on the object (after checking object is not null, of course)?
Sure, the code is shorter and does away with the null check, but I find it confusing to read. Are there any other benefits to this? I've also seen the same statement with regard to integer values.
I expect to see above code in loosely typed languages such as JavaScript or PHP – but not in C#.
IMO, Just a bad approach to avoid Null Reference Exception.
Probably the original developer's intent was to use as less code as possible in returning string
without checking null
or Type
No, you would not want to do that in C#. You are better off with calling the ToString method on the object. There are some languages where that is the preferred syntax, such as with JavaScript. So you might be seeing some cross-language leak.
If you love us? You can donate to us via Paypal or buy me a coffee so we can maintain and grow! Thank you!
Donate Us With