-3

我有一个最佳实践问题。我有一个方法BuildTourTime，它需要一个 DateTime 和一个表示时间的字符串。

 internal DateTime BuildDateTime(DateTime date, string time)
    {
        return date.Add(TimeSpan.Parse(time));

    }

我将在多个位置使用它。目前它正在另一个类中使用

private IEnumerable<Tour> BuildTours(Response Data)
        {
            //Tour tour = new Tour();
            List<Tour> tours = new List<Tour>();
            foreach(Events ivEvents in Data.response.events)
            {
                bool isCancelled = true; //Replace this with logic
                tours.Add(new Tour(ivEvents.Id,Data.response.ID,Data.response.Title,BuildDateTime(ivEvents.Date,ivEvent.Start),isCancelled,ivEvents.Quantity-ivEvents.Remaining).);
            }
            return tours;
        }

我目前有 Tour 类，但没有无参数构造函数。据我所知，我有几个选择。将 BuildDateTime 声明为静态。或者为 Tour 添加一个无参数的构造函数。或者创建一个助手类并放入 BuildDateTime。这是使用静态方法的好地方吗？这就是我所倾向于的。

编辑：为什么投反对票？发布的链接（我在发布这个问题之前阅读过）没有回答我的问题。我的问题表明我已经调查了我可以使用的选项，并且我提出了一个确切的问题，并提供了相关代码。

4

2 回答 2

4

你可以做扩展方法：

internal static class DateTimeExtension
{
    internal static DateTime BuildDateTime(this DateTime date, string time)
    {
        return date.Add(TimeSpan.Parse(time));
    }
}

并使用它：

string time = ...
DateTime date = ...
date.BuildDateTime(time);
于 2014-10-24T17:18:14.430 回答
2

您的方法似乎并不特定于该类，但可以被所有其他类普遍使用。它似乎是 DateTime 结构的扩展方法的完美示例

public static class MyDateExtensions
{
    public static DateTime BuildDateTime(this DateTime date, string time)
    {
        return date.Add(TimeSpan.Parse(time));

    }
}

通过这种方式，您可以从任何地方调用它（前提是您添加了 using 命名空间）

// Addin a day plus one hour
DateTime newDate = DateTime.Now.BuildDateTime("01:01:00:00");

话虽如此，当 DateTime.Add 只是一行代码并且没有任何隐藏的复杂性时，使用这种方法似乎有点夸张

DateTime newDate = DateTime.Now.Add(TimeSpan.Parse("01:01:00:00"));

如果您不确定解析接收的输入，则应该首选扩展方法。例如，这可能是一种更安全的方法，但它在很大程度上取决于如果时间字符串无效，您希望发生什么。如果时间字符串无效，许多人更喜欢获取异常，而不是阻止异常并返回预定义的结果。

public static DateTime BuildDateTime(this DateTime date, string time)
{
    if(string.IsNullOrWhiteSpace(time))
        return date; 

    TimeSpan ts;
    if(TimeSpan.TryParse(time, out ts))
        return date.Add(ts);
    else 
        return date;
}
于 2014-10-24T17:20:06.727 回答