Thoughts on Static Code Analysis Warning CA1806 for TryParse calls
- by Tim
I was wondering what people's thoughts were on the CA1806 (DoNotIgnoreMethodResults)  Static Code Analysis warning when using FxCop.
I have several cases where I use Int32.TryParse to pull in internal configuration information that was saved in a file.  I end up with a lot of code that looks like:
Int32.TryParse(someString, NumberStyles.Integer, CultureInfo.InvariantCulture, out intResult);
MSDN says the default result of intResult is zero if something fails, which is exactly what I want.
Unfortunately, this code will trigger CA1806 when performing static code analysis.  It seems like a lot of redundant/useless code to fix the errors with something like the following:
bool success = Int32.TryParse(someString, NumberStyles.Integer, CultureInfo.InvariantCulture, out intResult);
if (!success)
{
 intResult= 0;
}
Should I suppress this message or bite the bullet and add all this redundant error checking?  Or maybe someone has a better idea for handling a case like this?
Thanks!