单圈时间格式代码令人困惑

Question

我从学校得到了这段代码，但我似乎无法准确理解它在做什么。我知道它会显示单圈时间，就像运动中的那样，以分钟、秒和毫秒转换，但我不明白 2 字符串变量是什么以及它们的作用。

代码如下：

public String getTimeFormat(long ms){
    String s1 = ""+ms;
    Date date = null;
    try {
        date = new SimpleDateFormat("SSS").parse(s1);
    } catch (ParseException e){
        e.printStackTrace();
    }
    String s2 = new SimpleDateFormat("mm:ss:SS").format(date);
    return s2;
}

Answer 1

首先，在我看来，这是非常糟糕的代码。

它将原始 ms 值转换为字符串（因此 35968 将变为“35968”），然后将其解析为 Date，就好像从仅指定数字的格式毫秒...然后解释为“自格式化程序时区中的 Unix 纪元以来的毫秒数”。

然后，它会将 Date 值转换为分钟/秒/毫秒格式，方法是使用另一个 SimpleDateFormat 对其进行格式化。毫秒仅格式化为 2 位小数，尽管在这里使用 : 而不是 . 作为分隔符很奇怪。 （它使它看起来像一个小时/分钟/秒值，而不是分钟/秒/毫秒。）

两种格式都使用相同的时区，这至少某种可以避免出现问题。然而：

• 如果系统默认时区在 Unix 纪元前后发生了转换，仍然可能存在一些异常情况
• 基本上它试图通过将其视为日期和时间来格式化持续时间。这是个坏主意
• 异常“处理”是可笑的——如果在解析时抛出异常，则将打印第一个异常，并在下一行抛出NullPointerException，因为date仍然为null
• 如果持续时间超过一个小时，信息会静默丢失；目前尚不清楚此处所需的行为是什么
• 即使您确实想要以这种方式对其进行格式化，使用new Date(ms) 然后将其格式化为SimpleDateFormat 并将时区设置为UTC 会更简单。这里不需要任何字符串解析

如果不了解更多要求，我无法轻易提供更好的代码，但如果您尝试替换此代码，您应该考虑：

• 考虑到你有毫秒，你肯定只想要几十毫秒的显示精度吗？
• 如果ms 是否定的，你想做什么？
• 如果ms代表一个多小时，你想做什么？
• 您确定要将: 作为秒和毫秒之间的分隔符吗？

然后您可能想要使用String.format 来格式化该值。不幸的是，Java 8 中的 Joda Time 和 java.time 都没有执行自定义 Duration 格式的好方法，而这正是您真正想要的。