CODEWITHSUNDEEP
c#
user1400995
user1400995

Reputation:

How can I deal with this double.NaN

I have the a Convert.ToDecimal() which occasionally throws an exception with the message

Value was either too large or too small for a Decimal

, because the call to DataContainer.GetValue(ColKeyId, index) returns double.NaN.

if (Convert.ToDecimal(DataContainer.GetValue(ColKeyId, index)) != Convert.ToDecimal(newValueToSet))
{
    DataContainer.SetValue(ColKeyId, index, newValueToSet);
}

I cannot change the API implementation call of the GetValue(). What would be the best approach to deal with the conversion to decimal of a NaN double?

Upvotes: 3

Views: 2794

Answers (3)

w.b
w.b

Reputation: 11228

You can check if the value is double.NaN before setting:

if (!Double.IsNaN(DataContainer.GetValue(ColKeyId, index)))
{
    if (Convert.ToDecimal(DataContainer.GetValue(ColKeyId, index)) != Convert.ToDecimal(newValueToSet))
    {
        DataContainer.SetValue(ColKeyId, index, newValueToSet);
    }
}

Upvotes: 0

Jon Skeet
Jon Skeet

Reputation: 1499730

Okay, so it sounds like you just need to detect whether or not it's a NaN value:

double value = DataContainer.GetValue(ColKeyId, index);
if (double.IsNaN(value) || double.IsInfinity(value) ||
    (decimal) value != (decimal) newValueToSet)
{
    DataContainer.SetValue(ColKeyId, index, newValueToSet);
}

To be honest, it's not clear why you're converting from double to decimal at all, but this code at least shows how you can detect NaN/infinite values. Note that I've changed the calls to Convert.ToDecimal to simple casts to make the code simpler - but you could use Convert.ToDecimal if you want, of course..

Upvotes: 4

AntDC
AntDC

Reputation: 1907

Decimal.TryParse

So then you will know if it converted or not and if not assigne a defasult value or branch off down some other avenue.

HTH

Upvotes: 0

Related Questions