Определить, может ли десятичная цифра быть сохранена как int32

Question

Я делаю некоторые пользовательские сериализации, и для того, чтобы сэкономить некоторое пространство, я хочу сериализовать десятичные знаки как int, если возможно, значение мудрое. Производительность вызывает беспокойство, поскольку я имею дело с большим объемом данных. Текущий метод, который я использую:

if ((value > Int32.MinValue) && (value < Int32.MaxValue) && ((valueAsInt = Decimal.ToInt32(value)) == value)) 
{ 
    return true; 
} 

Можно ли улучшить это?

Answer 1

Есть ли у вас отрицательные значения? Я предполагаю, что да, так как у вас есть проверка MinValue, иначе вы можете пропустить ее. Вы даже можете использовать unsigned int, который позволит вам преобразовать больше ваших двойных значений в int.

Редактировать: Кроме того, если у вас есть более положительные числа, вы можете поменять первые два условия. Таким образом, первый из них, скорее всего, потерпит неудачу, уменьшив общее количество сравнений.

Answer 2

Ваши критерии признания недействительными являются:

1) Является ли это больше, чем MaxValue?

2) Он меньше, чем MinValue?

3) Имеет ли он дробный компонент?

Похоже, вы их покрыли. Моя реализация будет:

public bool IsConvertibleToInt(decimal value) 
{ 
    if(value > int.MaxValue) 
     return false; 

    if(value < int.MinValue) 
     return false; 

    if(Math.Floor(value) < value && Math.Ceiling(value) > value) 
     return false; 

    return true; 
} 

Answer 3

Вы бы не быть в состоянии просто сделать что-то вроде:

if(Decimal.ToInt32(value) == value) 
{ 
    return true; 
} 

Не специалист по .net, но я думаю, что должно быть все это было бы нужно. Кроме того, ваши два оператора сравнения должны быть «или равны», так как значения min/max также действительны.

Редактировать: Как указано в комментарии, это создаст исключение. Вы можете попытаться поймать исключение и вернуть false, но в этот момент, скорее всего, будет намного быстрее выполнить тестирование min/max самостоятельно.

Answer 4

Как насчет этого. Я думаю, что он должен принять меньше операций (по крайней мере, меньшее количество сравнений):

return (value == (Int32)value); 

запомнить Кроме того, если if оператор просто возвращает логическое значение, вы можете просто вернуть сравнение. Только это может сделать его быстрее (если компилятор уже не оптимизирует для этого). Если вы должны использовать, если заявление, вы можете так же сделать это:

if (value == (Int32)value) 
    { 
     //Do stuff... 
    return true; 
    } 
    else 
    { 
     //Do stuff... 
     return false; 
    } 

EDIT: Я понимаю, что это на самом деле не работает. Я думал, что бросок Int32 просто скопировал бы в первые 32 бита из десятичной дроби, оставив все оставшиеся биты (а не выбросив исключение), но, увы, это не сработало (не говоря уже о том, что это было бы неправильно для все отрицательные значения).

Answer 5

Нет необходимости в значении «ValueAsInt =». Я считаю (Decimal.ToInt32 (value) == value)) получает тот же результат с одним меньшим назначением. Вы используете valueAsInt как некоторый выходной параметр?

Answer 6

Это зависит от того, сколько десятичных разрядов у вас есть или действительно беспокоит. Если вы могли бы сказать, что мне только до 3 десятичных знаков, тогда наибольшее число, которое вы можете сохранить в int32, это int.MaxValue/1000. Если вы работаете только с положительными числами, вы можете получить большее число с помощью uint. В любом случае способ сделать это состоит в том, чтобы последовательно резервировать пространство для десятичной дроби и использовать * 1000 для их кодирования и/1000 для декодирования их в/из десятичного.