Question

我每隔几秒钟收到一个包含“1”的字符串。我试图获取收到的字符串和下一个字符串之间的经过时间以返回经过的时间。我做错了什么。我得到的结果是0，而字符串恰好每秒更新一次，所以我应该读1.我很确定逻辑中有错误，但我看不出它在哪里。这应该运行几个小时，每次我获得字符串“giriRicevuti”的更新时都会更新。

class Rpm
{
    public void CalcolaRPM(string giriRicevuti, out long RPM)
    {
        Stopwatch stopWatch = new Stopwatch();
        stopWatch.Start();

        if (giriRicevuti == "1")
        {
            stopWatch.Stop();
        }
            long duration = stopWatch.ElapsedMilliseconds;

            RPM =(duration/1000);
    }
}

Answer 1

如果你想在中间调用它，那么你需要在CalcolaRPM()方法之外设置一个秒表。

最简单的方法是将其添加为类中的私有字段。

另一个问题是，当giriRicevuti不是“1”时，您需要返回最后一个已知的RPM - 我们也可以通过将最后已知的RPM保持在私有字段中来解决这个问题。

另一个问题是，第一次计算RPM时，它不准确，因为没有先前的时间来比较它。我们将通过返回-1来解决此问题，直到我们有正确的报告时间。

接下来，您将计算经过的RPM作为整数计算。现在想象一下，如果事情略微偏离，那么经过的时间总是999毫秒。只有一毫秒，但你的RPM = 999/1000的计算将导致零。

您有多种选择，但最有可能是：

返回双人。
将值四舍五入到最接近的RPM。

我去了四舍五入。 RPM计算不正确，所以我在同一时间纠正：

lastRPM = (int) Math.Round(60000.0/((int) stopWatch.ElapsedMilliseconds));

总而言之，这是一个可编译的测试程序（控制台应用程序）：

using System;
using System.Diagnostics;
using System.Collections.Generic;
using System.Threading;

namespace Demo
{
    class Rpm
    {
        private Stopwatch stopWatch = new Stopwatch();
        private int lastRPM = -1;

        // RPM will be -1 until we have received two "1"s       

        public int CalcolaRPM(string giriRicevuti)
        {
            if (giriRicevuti == "1")
            {
                if (stopWatch.IsRunning)
                    lastRPM = (int) Math.Round(60000.0/((int) stopWatch.ElapsedMilliseconds));

                stopWatch.Restart();
            }

            return lastRPM;
        }
    }

    class Program
    {
        void run()
        {
            test(900);
            test(1000);
            test(1100);
            test(500);
            test(200);
        }

        void test(int interval)
        {
            Rpm rpm = new Rpm();

            for (int i = 0; i < 10; ++i)
            {
                Thread.Sleep(interval);
                rpm.CalcolaRPM("0");
                rpm.CalcolaRPM("1").Print();
                rpm.CalcolaRPM("2");
            }
        }

        static void Main()
        {
            new Program().run();
        }
    }

    static class DemoUtil
    {
        public static void Print(this object self)
        {
            Console.WriteLine(self);
        }

        public static void Print(this string self)
        {
            Console.WriteLine(self);
        }

        public static void Print<T>(this IEnumerable<T> self)
        {
            foreach (var item in self) Console.WriteLine(item);
        }
    }
}

Answer 2

感谢您的意见和建议，我最终得到了这个解决方案。我还使用return和一些浮动变量来简化方法以获得更高的准确性。这个适用于我的应用程序。

class Turns
{
    static DateTime prevTimeInstance = DateTime.Now;
    static float RPM = 0;

    public float Counts(int getTurn)
    {
        TimeSpan currentTimeSpan = TimeSpan.Zero;
        if (getTurn.Equals(1))
        {
            currentTimeSpan = DateTime.Now.Subtract(prevTimeInstance);
            prevTimeInstance = DateTime.Now;
            if (currentTimeSpan.TotalSeconds != 0)
                RPM = 60.0f / (float)currentTimeSpan.TotalSeconds;
        }
        return RPM;
    }
}

我要感谢Mattew给予我的大帮助。

经过的时间没有给我预期的结果

2 个答案: