带有负值的即时加号

Question

我有这个代码

Instant now = Instant.now();
if (amountDays >= 0) {
    now = now.plus(amountDays, ChronoUnit.DAYS);
} else {
    now = now.minus(Math.abs(amountDays), ChronoUnit.DAYS);
}

我想像这样简化它

Instant now = Instant.now();
now = now.plus(amountDays, ChronoUnit.DAYS);

但是，我不确定 plus 是否可以在负值下正常工作，或者是否会弄乱结果。

我可以像这样使用plus，可能是负值吗？

Answer 1

看看minus是如何实现的

@Override
public Instant minus(long amountToSubtract, TemporalUnit unit) {
    return (amountToSubtract == Long.MIN_VALUE ? plus(Long.MAX_VALUE, unit).plus(1, unit) : plus(-amountToSubtract, unit));
}

当然，为了可读性，在适当的情况下同时使用plus 和minus 是有意义的，但检查amountDays 是否大于或小于0 似乎是某种内部Instant.plus 逻辑和绝对无助于提高代码的可读性

Answer 2

plus 带负值

plus 方法支持从其documentation 中添加负时间以回到过去：

amountToAdd - 添加到结果中的单位数量，可能为负数

一切都好，你可以这样使用它，它会按预期工作。

---

实施

琐事，minus 的current implementation 甚至以-amountToSubtract 作为值委托给plus：

return (amountToSubtract == Long.MIN_VALUE
    ? plus(Long.MAX_VALUE, unit).plus(1, unit)
    : plus(-amountToSubtract, unit));

---

注意事项

一般来说，如果您只是想回到过去，最好使用minus 以提高可读性。

在您的特定情况下，我会坚持使用plus，但不会不必要地膨胀代码和逻辑。相反，更喜欢添加评论

// amountDays may be negative

或确保您的 javadoc 清楚这一点。

小幅改进，您可以将代码从两条语句简化为一条：

Instant now = Instant.now().plus(amountDays, ChronoUnit.DAYS);