Question

我一直在研究这段代码，出于某种原因，所有的if都让我疯狂了，还有一堆重复的代码。有没有更好的方法来做到这一点？

public Program(string[] args)
    {
        try
        {
            WriteToLogFile("Starting ImportTask");
            if (args.Length == 0)
            {
                Import(DateTime.Now,DateTime.Now);
                MarkRecordsAsDeleted();
            }
            else if (args.Length == 1)
            {
                DateTime dateToImport;
                bool isValidDate = DateTime.TryParse(args[0], out dateToImport);
                if (isValidDate)
                {
                    Import(dateToImport,dateToImport);
                    MarkRecordsAsDeleted();
                }
                else
                    WriteToLogFile(String.Format("The Import date specified was invalid. - {0}", args[0]));
            }
            else if (args.Length == 2)
            {
                DateTime importStartDate;
                bool isValidStartDate = DateTime.TryParse(args[0], out importStartDate);
                DateTime importEndDate;
                bool isValidEndDate = DateTime.TryParse(args[0], out importEndDate);
                if (isValidStartDate && isValidEndDate)
                {
                    if (importStartDate > importEndDate)
                    {
                        WriteToLogFile(String.Format("Invalid date range provided. Start date = {0} End date {1}",importStartDate,importEndDate));
                        return;
                    }
                    Import(importStartDate, importEndDate);
                    MarkRecordsAsDeleted();
                }
                else
                    WriteToLogFile(String.Format("The Import date specified was invalid. - {0}", args[0]));
            }
            else
            {
                WriteToLogFile("Invalid Command Line Parameters Specified");
            }

        }
        catch (Exception ex)
        {
            WriteToLogFile("Error in Import Process = " + ex.StackTrace);
        }
    }

Answer 1

你在这里得到的是一个双参数函数，期望可以解析为日期的字符串。如果缺少第二个参数，则使用第一个参数;如果缺少第一个，则使用DateTime.Now。

所以，写一个两个参数的函数，它需要两个日期。叫它。有点像：

switch(args.length) {
    case 0: myFunc(DateTime.Now, DateTime.Now); break;
    case 1: myFunc(toDate(args[0]), toDate(args[0])); break;
    case 2: myFunc(toDate(args[0]), toDate(args[1])); break;
}

顺便说一下，你的功能似乎并没有提到args [1];那可能是一个错误。

Answer 2

看看strategy pattern。它需要一些重要的重新分解和相当多的新类，但它应该解决你想要解决的问题。

暂时忽略这些花哨的名字，这个想法其实很简单。

您有一个抽象或接口类，它定义了一个要调用的方法。然后，您有几个派生类，您可以将if的内容移动到。

作为一个简单示例，我们可以执行以下操作：

 interface Ifoo
 {
     void myAction();
 }

 class MyCustomActionBar : Ifoo
  {
      public void myAction()
      {
          //.... details ... - Contents of single if statement.
      }
  }

 class MyCustomActionTar : Ifoo
  {
      public void myAction()
      {
          //.... details ... - Contents of next but single if statement.
      }
  }

一旦你设置了所有课程，你就可以改变你的if，如下所示。

public Program(string[] args)
{
   Ifoo _myFoo;
    try
    {
        WriteToLogFile("Starting ImportTask");
        if (args.Length == 0)
        {
            _myFoo = new MyCustomActionBar();
        }
        else if (args.Length == 1)
        {
            _myFoo = new MyCustomActionTar ();
        }
        //... etc....
        }
        else
        {
              _myFoo = new MyErrorAction(); //Definition not illustrated above
        }
        _myFoo.myAction();   //Getting your actual return values and stuff would be design details that I'm leaving out.

    }
    catch (Exception ex)
    {
         //...
    }
}

请原谅示例中的错误或拼写错误，我很快就做到了。这只是为了说明简单地介绍维基百科文章的一般想法，以提供一个快速的快照的想法..细节将更多涉及。

Answer 3

这是第一遍：

public Program(string[] args)
{
    try
    {
        WriteToLogFile("Starting ImportTask");
        DateTime startTime;
        DateTime endTime;
        GetDateRange(args, out startTime, out endTime);

        Import(startTime, endTime);
        MarkRecordsAsDeleted();
    }
    catch (Exception ex)
    {
        WriteToLogFile("Error in Import Process = " + ex);
        throw;
    }
}

    private static void GetDateRange(string[] args, out DateTime startTime, out DateTime endTime)
    {
        switch (args.Length)
        {
            case 0:
                startTime = DateTime.Now;
                endTime = DateTime.Now;
                break;
            case 1:
                {
                    DateTime dateToImport;
                    var isValidDate = DateTime.TryParse(args[0], out dateToImport);
                    if (!isValidDate)
                    {
                        throw new Exception(
                            String.Format(
                                "The Import date specified was invalid. - {0}",
                                args[0]));
                    }

                    startTime = dateToImport;
                    endTime = dateToImport;
                }
                break;
            case 2:
                {
                    DateTime importStartDate;
                    bool isValidStartDate = DateTime.TryParse(args[0], out importStartDate);
                    DateTime importEndDate;
                    bool isValidEndDate = DateTime.TryParse(args[1], out importEndDate);
                    if (!isValidStartDate || !isValidEndDate)
                    {
                        throw new Exception(
                            String.Format(
                                "The Import dates specified was invalid. - {0}, {1}",
                                args[0], args[1]));
                    }

                    if (importStartDate > importEndDate)
                    {
                        throw new Exception(
                            String.Format(
                                "Invalid date range provided. Start date = {0} End date {1}",
                                importStartDate, importEndDate));
                    }

                    startTime = importStartDate;
                    endTime = importEndDate;
                }
                break;
            default:
                throw new Exception("Invalid Command Line Parameters Specified");
        }
    }

Answer 4

这是我的两分钱。请注意，DateTime解析失败默认返回DateTime.Now，这很可能不是你想要的。如果是这样，只需按您的要求处理。从您的源代码中，您可以记录它并完成而不会抛出错误。根据您处理问题的方式，您可能希望在（startDate＆gt; endDate）时执行相同的操作。

以下是未经测试的代码...

public Program(string[] args)
{
    WriteToLogFile("Starting ImportTask");

    DateTime startDate = DateTime.Now;
    DateTime endDate = DateTime.Now;

    if (args.Length > 0)                    
        if (!DateTime.TryParse(args[0], out startDate)
            WriteToLogFile(string.format("Warning: StartDate argument, {0}, is invalid; value has reverted to current Date/Time.", args[0]) );

    if (args.Length > 1)
        if (!DateTime.TryParse(args[1], out endDate)
            WriteToLogFile(string.format("Warning: EndDate argument, {0}, is invalid; value has reverted to current Date/Time.", args[]) );

    if (startDate > endDate)
    {       
        WriteToLogFile(String.Format("Invalid date range provided. Start date = {0} End date {1}",importStartDate,importEndDate));
        return;
    }

    Import(startDate, endDate);
    MarkRecordsAsDeleted();
}

用Ifs重构帮助

4 个答案: