EchoNullify commented on code in PR #3121:
URL: https://github.com/apache/tinkerpop/pull/3121#discussion_r2112728816
##########
gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/util/NumberHelper.java:
##########
@@ -312,43 +356,84 @@ private NumberHelper(final BiFunction<Number, Number,
Number> add,
this.cmp = cmp;
}
- public static Class<? extends Number> getHighestCommonNumberClass(final
Number... numbers) {
- return getHighestCommonNumberClass(false, numbers);
- }
-
- public static Class<? extends Number> getHighestCommonNumberClass(final
boolean forceFloatingPoint, final Number... numbers) {
+ static NumberInfo getHighestCommonNumberInfo(final boolean
forceFloatingPoint, final Number... numbers) {
int bits = 8;
boolean fp = forceFloatingPoint;
for (final Number number : numbers) {
if (!isNumber(number)) continue;
final Class<? extends Number> clazz = number.getClass();
if (clazz.equals(Byte.class)) continue;
if (clazz.equals(Short.class)) {
- bits = bits < 16 ? 16 : bits;
+ bits = Math.max(bits, 16);
} else if (clazz.equals(Integer.class)) {
- bits = bits < 32 ? 32 : bits;
+ bits = Math.max(bits, 32);
} else if (clazz.equals(Long.class)) {
- bits = bits < 64 ? 64 : bits;
+ bits = Math.max(bits, 64);
} else if (clazz.equals(BigInteger.class)) {
- bits = bits < 128 ? 128 : bits;
+ bits = 128;
} else if (clazz.equals(Float.class)) {
- bits = bits < 32 ? 32 : bits;
+ bits = Math.max(bits, 32);
fp = true;
} else if (clazz.equals(Double.class)) {
- bits = bits < 64 ? 64 : bits;
+ bits = Math.max(bits, 64);
fp = true;
} else /*if (clazz.equals(BigDecimal.class))*/ {
- bits = bits < 128 ? 128 : bits;
+ bits = 128;
fp = true;
break; // maxed out, no need to check remaining numbers
}
}
- return determineNumberClass(bits, fp);
+ return new NumberInfo(bits, fp);
+ }
+
+ public static Class<? extends Number> getHighestCommonNumberClass(final
Number... numbers) {
+ return getHighestCommonNumberClass(false, numbers);
+ }
+
+ public static Class<? extends Number> getHighestCommonNumberClass(final
boolean forceFloatingPoint, final Number... numbers) {
+ NumberInfo numberInfo = getHighestCommonNumberInfo(forceFloatingPoint,
numbers);
+ return determineNumberClass(numberInfo.bits, numberInfo.fp);
+ }
+
+ private static Number mathOperationWithPromote(final MathOperation
mathOperation, final boolean forceFloatingPoint, final Number a, final Number
b) {
+ if (null == a || null == b) return a;
+ NumberInfo numberInfo = getHighestCommonNumberInfo(forceFloatingPoint,
a, b);
+ Number result = 0;
+ while (true) {
+ try {
+ result = mathOperation.apply(numberInfo, a, b);
+ if (result.getClass() == BigInteger.class || result.getClass()
== BigDecimal.class)
+ {
+ return result;
+ }
+ if (Double.isInfinite(result.doubleValue()))
+ {
+ throw new ArithmeticException("Floating point overflow
detected");
Review Comment:
I found it was needed for cases where you start with a big intereger /
decimal
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
