.NET :이 Math.Ceiling 표현식을 단순화 할 수 있습니까?

Question

이 Math.Ceiling 표현식을 단순화 할 수 있습니까?

decimal total 
decimal? quantity, multiplier 
int? days 

total = (decimal)Math.Ceiling((double)quantity.Value * (double)days.Value * (double)multiplier); 

EDIT이 코드는 Silverlight 코드이므로 모든 캐스트가 두 번 입력되었음을 잊어 버렸습니다.

Answer 1

왜 모든 것을 2 배로 변환 하시겠습니까?

이

total = decimal.Ceiling(quantity.Value * days.Value * multiplier.Value); 

(당신은 대신 Math.Ceiling(decimal)을 사용할 수 있습니다,하지만 난 당신이 decimal.Ceiling를 사용하는 경우가 decimal를 사용하고 있음을 명확하게 느낀다.)

, 두 배로 모든 변환이 연산을 수행 한 다음 변환 : 내가 사용하는 것이 10 진수로 돌아 가면 정보가 손실 될 가능성이 큽니다. 아주 소수의 IMO와 2 진 IMO 사이의 변환은 거의 없습니다.

Answer 2

정보를 많이 제공하지는 않았지만 다음은 하나의 아이디어입니다.

Null 가능 유형 (?으로 끝나는 유형)을 사용하지만 실제로 null인지 확인하지는 않습니다. (생략 한 코드가있을 수 있습니다.) null이 아님을 알고 있으면 Nullable 유형을 사용하지 마십시오. 그러면 표현식에있는 모든 용어가 제거됩니다. 당신이 double 각 승수를 캐스팅하는 이유를 모르겠어요

total = (decimal)Math.Ceiling((double)quantity * (double)days * (double)multiplier); 

:

이

는로 표현을 변경합니다. double으로 전송하기 전에 모두 함께 사용하는 것이 좋습니다. 이것은 당신의 표정을 바꿀 것

(. 정밀도의 손실을주의 깊게 확인) :

이

total = (decimal)Math.Ceiling((double)(quantity * days * multiplier)); 

전반적으로, 그 날에 간단하게 보이는, 그냥 좋은해야한다.
(테스트 전용 확실히 말할 것이다!)

Answer 3

방법에 대해 :

decimal total; 
decimal? quantity = null, multiplier = null; 
int? days = null; 

total = Math.Ceiling(quantity ?? 0 * days ?? 0 * multiplier ?? 0);