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,令人惊讶的是这只是在1台生产服务器上,测试服务器工作得很好......

所以现在我把功能重写为旧式风格:

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");
    }

所以我问女士们,先生们,我是否首先替换了完美的代码,或者这是某种微软的错误？我们真的可以相信这些看起来不那么坚固的新语言功能,还是我只是缺少某些东西？

Answer 1

您应该DateTimeFormatInfo.InvariantInfo作为参数传递给DateTime.ToString方法./在格式字符串中并不一直意味着"/"字符.它解析为当前文化的日期分隔符:

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

一般来说,总是明确指定文化ToString和Parse方法(不仅仅是DateTimes)是一种很好的做法.FxCop(VS代码分析)具有在不执行此操作时吐出警告的规则.这对非美国文化至关重要,许多程序员经常低估其重要性.