我写了一个方法来将给定数字从天转换为毫秒:
private long expireTimeInMilliseconds; ... public void setExpireTimeInDays(int expireTimeInDays) { expireTimeInMilliseconds = expireTimeInDays * 24 * 60 * 60 * 1000; }
我很难弄明白我做错了什么.现在我的问题是: 这个错误是如此明显吗?
纠正方法:
private long expireTimeInMilliseconds; ... public void setExpireTimeInDays(int expireTimeInDays) { expireTimeInMilliseconds = ((long) expireTimeInDays) * 24 * 60 * 60 * 1000; }
如果我在计算之前没有将整数转换为long,那么我得到一个完全错误的结果.
这很明显吗?我想这取决于你使用Java多长时间以及你需要多少次处理毫秒.当然,它应该可以持续大约24天......
我认为最大的提示应该是System.currentTimeMillis()
返回一个long
.这是一个很好的迹象,可以在几毫秒内变大.您设置的变量类型也应该是一个很好的提示.
当然,您还必须知道如果使用整数进行算术运算,结果将是int
溢出时的回绕.这是否足够明显可以辩论,但这将是一个非常毫无意义的讨论.在C#中,如果你打开溢出检查,你很快就会发现这个错误 - 但是没有多少开发人员这样做(事实上,我不是,但我可能应该这样做).
是的,如果你以前做过,那就很明显了.每当你看到一串数字成倍增加时,你应该自动开始考虑整数溢出错误.在这种情况下,如果expireTimeInDays
超过24,则设置为溢出.从技术上讲,任何时候使用整数时都应该考虑溢出错误,但是像这样增加一组它们应该是一个非常大的红旗.