Почему anyValue% 1 «глупая математика» в сонаре, когда anyValue является двойным?

Question

SonarQube вызывает серьезное нарушение Silly math should not be performed в моем коде. В описании написано:

Определенные математические операции просто глупые и не должны быть выполнены , потому что их результаты предсказуемы.

В частности, anyValue% 1 глупо, потому что он всегда будет возвращать 0.

В моем случае, хотя, anyValue является двойной. И это работает as intended для меня. Вот реальный код:

double v = Double.parseDouble(Utils.formatDouble(Double.valueOf(value.getValue()), accuracy.intValue())); 
boolean negative = v < 0; 
v = Math.abs(v); 
long deg = (long) Math.floor(v); 

v = (v % 1) * 60; 

ли анализатор при условии, моя переменная является int (который является их ошибка)? Или я пропустил что-то еще?

Answer 1

Это действительно ошибка, так что спасибо большое за информацией об этом.

Проблема здесь в коде: https://github.com/SonarSource/sonar-java/blob/3.9/java-checks/src/main/java/org/sonar/java/checks/ConstantMathCheck.java#L117

где нет абсолютно никакой проверки на тип левого операнда оператора%.

Я просто подал следующую ошибку, чтобы справиться с этим: https://jira.sonarsource.com/browse/SONARJAVA-1457

Answer 2

Вы можете сделать ваше выражение более явным, изменив его использовать явный двойной константы как:

(v % 1.0d) * 60