如何正确编写这个函数？每次都让我失忆

Question

我的 While 循环中运行了一些代码：

while (Time <= EndPeriod)
{
      ... code ...

      Tine.AddTime(Time);
}

我遇到的问题是 Time.AddTime(Time) 执行的位置会造成内存泄漏。 AddTime 函数检查类属性，并根据开关中的 DateFormat 变量通过正确的方法（例如 AddDays、AddMonths 等）添加时间。如果我将开关放入循环中，一切都很好，但是如果我尝试引用此函数，则会出现堆栈溢出。

我的 AddTime 函数如下：

protected DateTime AddTime(DateTime Time)
{
    DateTime myTime = Time;
    switch (DatePart) {
        case "yy":
            myTime.AddYears(1);
            break;
        case "qq":
            myTime.AddDays(1);
            break;
        case "mm":
            myTime.AddMonths(1);
            break;
        case "dd":
            myTime.AddDays(1);
            break;
        case "hh":
            myTime.AddHours(1);
            break;
        case "ss":
            myTime.AddSeconds(1);
            break;
        default:
            myTime.AddMinutes(1);
            break;
    }
    return myTime;
}

我可能做错了什么？

谢谢

Answer 1

您应该更改：

myTime.AddSeconds(1);

以

// The same for all other methods which considered to modify 
// a DateTime value including your custom AddTime() method
myTime = myTime.AddSeconds(1);

这种方式，条件 Time 的值将发生更改，以便条件 (Time <= EndPeriod) 将永远被通过。

所有方法，例如 DateTime.AddYears() DateTime.AddDays() 不修改对象的值本身但返回新值。

微软软件定义网络：

此方法不会更改此 DateTime 对象的值。
相反，它返回一个新的 DateTime 对象，其值是以下结果
此操作。

Answer 2

堆栈很可能因大量 DateTime 分配而溢出。尝试将 DateTime 移到循环之外 &不断地覆盖它。

这里的想法是，b/c DateTime 分配到堆栈而不是堆，这可能会导致 Caling thead（while 循环）溢出。

Answer 3

（假设代码上有拼写错误并在此处更正它...

Time.AddTime(Time);

这会修改 Time 吗？看起来不像，至少不是直观上的。如果您不更改 的值Time 那么循环将无限地运行，永远不会达到其终止条件：

Time = Time.AddTime(Time);

此外，在方法内部使用诸如 myTime.AddYears() 和之类的东西。其他此类方法，您需要分配它们的返回值 。

作为一般的设计问题，您应该小心哪些方法会修改哪些对象，我承认 DateTime 上的 .AddXX() 方法是 在这方面不是很直观。