Whether the comparison is precarious or not should depend on how you handle the literal. If you handle 99.9 as BigDecimal("99.9") which precision can be controlled, you can trust the comparison.<br><br>Anyway, since it has different behavior for these two cases, it should be a bug, right? I just want to know it is Drools' bug or MVEL's... If it is MVEL's, I may need ask for MVEL guys' help...<br>
<br><div class="gmail_quote">On 26 March 2012 19:50, Wolfgang Laun <span dir="ltr"><<a href="mailto:wolfgang.laun@gmail.com">wolfgang.laun@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
4.0.7 is rather old. 5.1.1 and later do not exhibit this behaviour.<br>
<br>
However, it should be noted that comparing double values using == is a<br>
precarious approach. For instance, 99.9 (the mathematical number)<br>
cannot be represented as a double machine number. Hence, you do not<br>
truly test for<br>
equality with that number, just with its approximation by some machine<br>
number.<br>
<br>
-W<br>
<div class="im"><br>
On 26/03/2012, Zeke <<a href="mailto:xanadu860122@gmail.com">xanadu860122@gmail.com</a>> wrote:<br>
> Hi, guys:<br>
> I am using Drools 4.0.7. I find a strange behavior. When my LHS is<br>
> "Message( varDouble == 99.9 )" and "varDouble" equals "new Double(99.9)",<br>
> the condition is true. But when my LHS is "Message( types.doubleType ==<br>
> 99.9 )" and the "doubleType" field is also equal to "new Double(99.9)", the<br>
> condition is false. I debug the code, and find that:<br>
><br>
> When the LHS is "Message( varDouble == 99.9 )", the logic will handle the<br>
</div>> literal 99.9 as *Double.valueOf( “99.9” )*, and the comparison is between<br>
> *(new<br>
> Double(99.9)).doubleValue(*) and *Double.valueOf( “99.9” ).doubleValue()*,<br>
<div class="im">> When the LHS is "Message( types.doubleType == 99.9 )", the logic is<br>
</div>> different from above. The comparison is between *new BigDecimal((Double)<br>
> 99.9, MathContext.DECIMAL64)* (which double value is 99.90000000000001) and<br>
> *new BigDecimal(“99.9”, MathContext.DECIMAL64)* (which double value is<br>
<div class="im">> 99.9).<br>
><br>
> I think we need make things consistent. "varDouble" and<br>
> "types.doubleType" are of the same type and value. It doesn't make sense to<br>
> have different behavior just because of one field is nesting while the<br>
> other is not. I know Drools is using MVEL to evaluate the condition. But I<br>
> am not sure it is a MVEL bug or Drools uses MVEL wrong. It will be much<br>
> appreciated if you can give me any comments!<br>
><br>
<br>
</div>_______________________________________________<br>
rules-users mailing list<br>
<a href="mailto:rules-users@lists.jboss.org">rules-users@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/rules-users" target="_blank">https://lists.jboss.org/mailman/listinfo/rules-users</a><br>
</blockquote></div><br>