C# - 我是否替换了完美的代码？

Question

我有一个快速格式化日期的函数，这里是:

public static string archiveServerDateTime(string datetime)
    {
        DateTime tempDateTime = DateTime.ParseExact(datetime,"dd.MM.yyyy HH:mm:ss", null);
        return tempDateTime.ToString("yyyy/MM/dd:HH:mm:ss");
    }

只找到函数的输出 = 2009.10.22:16:21:03，令人惊讶的是这只在一台生产服务器上，测试服务器工作得很好......

所以现在我将函数重写为旧学校风格:

public static string archiveServerDateTime(string datetime)
    {
        DateTime tempDateTime = DateTime.ParseExact(datetime,"dd.MM.yyyy HH:mm:ss", null);
        string yearPart = Convert.ToString(tempDateTime.Year);
        string monthPart = Convert.ToString(tempDateTime.Month).PadLeft(2,'0');
        string dayPart = Convert.ToString(tempDateTime.Day ).PadLeft(2, '0');
        string hourPart = Convert.ToString(tempDateTime.Hour).PadLeft(2, '0');
        string minutePart = Convert.ToString(tempDateTime.Minute).PadLeft(2, '0'); 
        string secondPart = Convert.ToString(tempDateTime.Second).PadLeft(2,'0');

        return yearPart + @"/" + monthPart + @"/" + dayPart + ":" + hourPart + ":" + minutePart + ":" + secondPart; 
        //return tempDateTime.ToString("yyyy/MM/dd:HH:mm:ss");
    }

所以我想问问女士们先生们，我是不是一开始就替换了非常好的代码，或者这是某种 Microsoft 错误？我们真的可以相信这些看似不那么坚如磐石的新语言功能，还是我只是遗漏了什么？

Answer 1

您应该将 DateTimeFormatInfo.InvariantInfo 作为参数传递给 DateTime.ToString 方法。格式字符串中的 / 始终不表示 "/" 字符。它解析为当前区域性的日期分隔符:

return tempDateTime.ToString("yyyy/MM/dd:HH:mm:ss", DateTimeFormatInfo.InvariantInfo);

一般来说，始终在 ToString 和 Parse 方法(不仅仅是在 DateTime 上)明确指定区域性是一个很好的做法。 FxCop(VS 代码分析)的规则会在您不这样做时发出警告。它对非美国文化至关重要，而且其重要性常常被许多程序员低估。