我的时间计算给了我一个负数

如何解决我的时间计算给了我一个负数

以下内容效果很好：

using System;
using System.Threading;

namespace cursor_position
{
    class Program
    {
        static void Main(string[] args)
        {
            DateTime startTime = DateTime.Now;;
            TimeSpan timeRemaining;
            int intCount = 119;
            
            for (int i = 0; i <= intCount; i++)
            {
                Thread.Sleep(2000);
                
                Console.Clear();

                timeRemaining = TimeSpan.FromTicks(DateTime.Now.Subtract(startTime).Ticks * (intCount - (i+1)) / (i+1));

                Console.SetCursorPosition(0,0);
                Console.Write("ETA: ");
                Console.Write(String.Format("{0} Minutes,{1} Seconds",timeRemaining.Minutes,timeRemaining.Seconds));

                Console.WriteLine();
            }
        }
    }
}

但是，最后，我得到的ETA计算值为负数。代码运行的时间越长（如果我增加sleep或intCount），则ETA减去数字就越大。

有人知道为什么吗？

解决方法

当您尝试使用这样的数学计算时差时，总是有问题的。最好完全不使用计数器来增强代码的抗性。

尝试以下代码：

DateTime startTime = DateTime.Now;
DateTime targetTime = startTime.AddMinutes(1.0);

while (true)
{
    DateTime now = DateTime.Now;
    if (now > targetTime)
    {
        break;
    }
    else
    {
        TimeSpan timeRemaining = targetTime.Subtract(now);

        Console.Write("ETA: ");
        Console.Write(String.Format("{0} Minutes,{1} Seconds",timeRemaining.Minutes,timeRemaining.Seconds));

        Console.WriteLine();

        Thread.Sleep(1000);
    }
}

---

我认为在您当前的问题中使用Thread.Sleep(1000)（在您的原始问题中）和Thread.Sleep(2000)使我不清楚您要计算不确定过程（例如复制）的剩余时间一份文件。在我看来，您只是在进行“计时器倒计时”。

让我们写一个通用函数来估计不确定过程的剩余时间。

public TimeSpan ComputeRemaining((int count,DateTime time) start,(int count,DateTime time) current,int end) =>
    current.count - start.count == 0
        ? TimeSpan.MaxValue
        : TimeSpan.FromSeconds((end - current.count) * current.time.Subtract(start.time).TotalSeconds / (current.count - start.count));

鉴于起始计数和DateTime，给定的current计数和DateTime以及所需的end计数，此函数将产生剩余时间。

它可以随着计数的增加或减少而工作。

让它真正发挥作用。

var now = DateTime.Now;
var start = (count: 0,time: now.Subtract(TimeSpan.FromSeconds(5.0)));
var current = (count: 50,time: now);
var end = 100;
Console.WriteLine(ComputeRemaining(start,current,end).TotalSeconds);

因此，该示例是5.0秒钟前开始的一个过程，现在已经完成了50％-需要多长时间？还剩5.0秒。此代码正确写出5。

让我们在示例应用中尝试一下：

static void Main(string[] args)
{
    var random = new Random();
    
    DateTime startTime = DateTime.Now;
    int intCount = 30;

    for (int i = 0; i <= intCount; i++)
    {
        Thread.Sleep(random.Next(100,2000));

        TimeSpan timeRemaining = ComputeRemaining((0,startTime),(i,DateTime.Now),intCount);

        Console.Write("ETA: ");
        Console.Write(String.Format("{0} Minutes,timeRemaining.Seconds));

        Console.WriteLine();
    }
}

当我运行它时，我得到：

ETA: 48 Minutes,5 Seconds
ETA: 0 Minutes,54 Seconds
ETA: 0 Minutes,39 Seconds
ETA: 0 Minutes,42 Seconds
ETA: 0 Minutes,41 Seconds
ETA: 0 Minutes,33 Seconds
ETA: 0 Minutes,32 Seconds
ETA: 0 Minutes,31 Seconds
ETA: 0 Minutes,29 Seconds
ETA: 0 Minutes,28 Seconds
ETA: 0 Minutes,27 Seconds
ETA: 0 Minutes,25 Seconds
ETA: 0 Minutes,22 Seconds
ETA: 0 Minutes,20 Seconds
ETA: 0 Minutes,18 Seconds
ETA: 0 Minutes,17 Seconds
ETA: 0 Minutes,16 Seconds
ETA: 0 Minutes,14 Seconds
ETA: 0 Minutes,12 Seconds
ETA: 0 Minutes,11 Seconds
ETA: 0 Minutes,10 Seconds
ETA: 0 Minutes,8 Seconds
ETA: 0 Minutes,7 Seconds
ETA: 0 Minutes,6 Seconds
ETA: 0 Minutes,4 Seconds
ETA: 0 Minutes,3 Seconds
ETA: 0 Minutes,2 Seconds
ETA: 0 Minutes,1 Seconds
ETA: 0 Minutes,0 Seconds

请注意，在我的示例的早期阶段，估计值实际上是根据随机输入的时间计算得出的。

我相信这就是您想要的。

,

似乎您有括号问题。

假设我们已经设置了intCount = 1，那么根据您的代码，for循环将被执行两次。一次用于i=0，一次用于i=1。

在i = 0时，我们有：1234 * (intCount - (i + 1)) / (i + 1) ==> 1234*(1 - (0 + 1)) / (0 + 1)

以上代码等于1234 * A / B，其中A为(intCount - (i + 1))，B为(i + 1)。因此，首先将计算1234 * A，然后将结果除以B！

所以我们有1234 * (1-(0+1))等于0，然后是0 / 1等于0。

当i = 1为`1234 *（intCount-（i + 1））/（i + 1）==> 1234 *（1-（1 + 1））/（1 + 1）= -617

我们有1234 * (1-(1+1))等于-1234，然后-1234 / 1等于-1234。

要解决此问题，您需要将for循环更改为：

for (int i = 0; i < intCount; i++)

在这种情况下，循环将执行一次（i=0），并且您将不再获得负值。并也进行此修改以解决括号问题：

timeRemaining = TimeSpan.FromTicks(DateTime.Now.Subtract(startTime).Ticks * ((intCount - (i + 1)) / (i + 1)));