如何解决SonarQube提示“将减法运算操作数转为long”的问题？

原始代码与检测问题 #

我们有个计算密码修改提醒的老旧后端服务，代码如下：

import java.util.concurrent.TimeUnit;

int daysBeforePasswordExpires = 60;
Integer daysBeforeReminder = 30;
long daysLeftToRemind = TimeUnit.DAYS.toMillis(daysBeforeReminder - daysLeftToRemind);

SonarQube检测到如下问题：

Cast one of the operands of this subtraction operation to a "long".

问题指向long daysLeftToRemind = TimeUnit.DAYS.toMillis(daysBeforeReminder - daysLeftToRemind);这一行。

注意：原始代码存在严重逻辑错误——daysLeftToRemind是待赋值的变量，右侧计算直接使用未初始化的它，这在Java中会导致编译失败，大概率是笔误，应该是用daysBeforePasswordExpires替代右侧的daysLeftToRemind（比如计算daysBeforePasswordExpires - daysBeforeReminder，即距离提醒触发的剩余天数）。

SonarQube提示的本质原因 #

daysBeforeReminder是Integer类型，和另一个int/Integer类型变量做减法时，会以int类型执行运算。当数值超出int的取值范围（-2147483648 到 2147483647）时，会触发整数溢出，导致计算结果错误。而TimeUnit.DAYS.toMillis接收的参数是long类型，所以需要提前将减法运算的操作数转为long，避免溢出风险。

正确修复步骤 #

1. 先修正逻辑笔误 #

先把变量使用错误改过来，比如正确逻辑是计算过期前多少天触发提醒：

// 示例：计算距离提醒触发的剩余天数（假设逻辑是过期天数减提前提醒天数）
long daysLeftToRemind = TimeUnit.DAYS.toMillis(daysBeforePasswordExpires - daysBeforeReminder);

2. 解决SonarQube的类型转换问题 #

有两种安全的修复方式，均不会改变计算逻辑：

• 方式一：转换其中一个操作数为long
  让减法运算以long类型执行，避免溢出：

// 转换第一个操作数
long daysLeftToRemind = TimeUnit.DAYS.toMillis((long) daysBeforePasswordExpires - daysBeforeReminder);
// 或转换第二个操作数
long daysLeftToRemind = TimeUnit.DAYS.toMillis(daysBeforePasswordExpires - (long) daysBeforeReminder);

• 方式二：提前将变量转为long，再计算
  代码可读性更高，尤其适合多变量运算的场景：

long expireDays = daysBeforePasswordExpires;
long remindThreshold = daysBeforeReminder;
long daysLeftToRemind = TimeUnit.DAYS.toMillis(expireDays - remindThreshold);

关于你修改的代码的说明 #

你修改的long daysLeftToRemind = TimeUnit.DAYS.toMillis(daysBeforeReminder - (long) daysLeftToRemind );，如果忽略变量未初始化的笔误，单纯从类型转换角度看，这种写法是安全的——转换其中一个操作数为long后，减法会自动提升为long运算，计算结果和原逻辑（如果原逻辑正确）完全一致，不会影响提醒天数的计算。但核心必须先修正变量未初始化的笔误，否则代码根本无法正常运行。

内容的提问来源于stack exchange，提问作者newProgrammer005