System.Threading.Timer ใน C # ดูเหมือนว่าจะไม่ทำงาน มันทำงานเร็วมากทุกๆ 3 วินาที

ฉันเป็นวัตถุจับเวลา ฉันต้องการให้มันทำงานทุกนาที โดยเฉพาะอย่างยิ่งควรเรียกใช้OnCallBackวิธีการและไม่ได้ใช้งานในขณะที่OnCallBackวิธีการทำงาน เมื่อOnCallBackวิธีการเสร็จสิ้น (a OnCallBack) จะเริ่มการจับเวลาใหม่

นี่คือสิ่งที่ฉันมีในตอนนี้:

private static Timer timer;

private static void Main()
{
    timer = new Timer(_ => OnCallBack(), null, 0, 1000 * 10); //every 10 seconds
    Console.ReadLine();
}

private static void OnCallBack()
{
    timer.Change(Timeout.Infinite, Timeout.Infinite); //stops the timer
    Thread.Sleep(3000); //doing some long operation
    timer.Change(0, 1000 * 10);  //restarts the timer
}

อย่างไรก็ตามดูเหมือนว่าจะไม่ทำงาน มันทำงานเร็วมากทุกๆ 3 วินาที แม้ว่าจะเพิ่มระยะเวลา (1,000 * 10) ดูเหมือนว่ามันจะเมิน1000 * 10

ฉันทำอะไรผิด?

นี่ไม่ใช่การใช้งาน System.Threading.Timer ที่ถูกต้อง เมื่อคุณสร้างอินสแตนซ์ตัวจับเวลาคุณควรทำสิ่งต่อไปนี้เกือบตลอดเวลา:

_timer = new Timer( Callback, null, TIME_INTERVAL_IN_MILLISECONDS, Timeout.Infinite );

สิ่งนี้จะสั่งให้ตัวจับเวลาทำเครื่องหมายเพียงครั้งเดียวเมื่อพ้นช่วงเวลาดังกล่าว จากนั้นในฟังก์ชั่นการโทรกลับของคุณคุณจะเปลี่ยนตัวจับเวลาเมื่องานเสร็จสิ้นไม่ใช่ก่อนหน้านี้ ตัวอย่าง:

private void Callback( Object state )
{
    // Long running operation
   _timer.Change( TIME_INTERVAL_IN_MILLISECONDS, Timeout.Infinite );
}

ดังนั้นจึงไม่จำเป็นต้องมีกลไกการล็อคเนื่องจากไม่มีการทำงานพร้อมกัน ตัวจับเวลาจะเริ่มการโทรกลับครั้งต่อไปหลังจากผ่านช่วงเวลาถัดไป + เวลาของการดำเนินการที่ยาวนาน

หากคุณต้องการเรียกใช้ตัวจับเวลาที่ N มิลลิวินาทีแน่นอนฉันขอแนะนำให้คุณวัดเวลาของการทำงานที่ยาวนานโดยใช้นาฬิกาจับเวลาแล้วเรียกใช้วิธีการเปลี่ยนอย่างเหมาะสม:

private void Callback( Object state )
{
   Stopwatch watch = new Stopwatch();

   watch.Start();
   // Long running operation

   _timer.Change( Math.Max( 0, TIME_INTERVAL_IN_MILLISECONDS - watch.ElapsedMilliseconds ), Timeout.Infinite );
}

ฉันขอแนะนำให้ทุกคนที่ทำ. NET และใช้ CLR ที่ยังไม่ได้อ่านหนังสือของ Jeffrey Richter - CLR ผ่าน C #ให้อ่านโดยเร็วที่สุด มีการอธิบายตัวจับเวลาและเธรดพูลอย่างละเอียด

ไม่จำเป็นต้องหยุดตัวจับเวลาดูวิธีแก้ปัญหาที่ดีจากโพสต์นี้ :

"คุณสามารถปล่อยให้ตัวจับเวลาเริ่มต้นวิธีการโทรกลับต่อไป แต่ห่อรหัสที่ไม่กลับเข้าใหม่ของคุณไว้ในจอภาพลองป้อน / ออกไม่จำเป็นต้องหยุด / เริ่มตัวจับเวลาใหม่ในกรณีนั้นการโทรที่ทับซ้อนกันจะไม่ได้รับการล็อกและกลับทันที"

private void CreatorLoop(object state) 
 {
   if (Monitor.TryEnter(lockObject))
   {
     try
     {
       // Work here
     }
     finally
     {
       Monitor.Exit(lockObject);
     }
   }
 }

System.Threading.Timerบังคับใช้หรือไม่

หากไม่เป็นเช่นSystem.Timers.Timerนั้นมีประโยชน์Start()และStop()วิธีการ (และAutoResetคุณสมบัติที่คุณสามารถตั้งค่าเป็นเท็จเพื่อที่Stop()ไม่จำเป็นต้องใช้และคุณเพียงแค่เรียกใช้Start()หลังจากดำเนินการ)

ฉันจะทำ:

private static Timer timer;
 private static void Main()
 {
   timer = new Timer(_ => OnCallBack(), null, 1000 * 10,Timeout.Infinite); //in 10 seconds
   Console.ReadLine();
 }

  private static void OnCallBack()
  {
    timer.Dispose();
    Thread.Sleep(3000); //doing some long operation
    timer = new Timer(_ => OnCallBack(), null, 1000 * 10,Timeout.Infinite); //in 10 seconds
  }

และละเว้นพารามิเตอร์ period เนื่องจากคุณกำลังพยายามควบคุม periodicy ด้วยตัวเอง

รหัสเดิมของคุณทำงานเร็วที่สุดเนื่องจากคุณยังคงระบุ0สำหรับไฟล์dueTimeพารามิเตอร์ จากTimer.Change:

ถ้า DueTime เป็นศูนย์ (0) วิธีการโทรกลับจะถูกเรียกใช้ทันที

 var span = TimeSpan.FromMinutes(2);
 var t = Task.Factory.StartNew(async delegate / () =>
   {
        this.SomeAsync();
        await Task.Delay(span, source.Token);
  }, source.Token, TaskCreationOptions.LongRunning, TaskScheduler.Default);

source.Cancel(true/or not);

// or use ThreadPool(whit defaul options thread) like this
Task.Start(()=>{...}), source.Token)

ถ้าคุณชอบใช้ด้ายวนข้างใน ...

public async void RunForestRun(CancellationToken token)
{
  var t = await Task.Factory.StartNew(async delegate
   {
       while (true)
       {
           await Task.Delay(TimeSpan.FromSeconds(1), token)
                 .ContinueWith(task => { Console.WriteLine("End delay"); });
           this.PrintConsole(1);
        }
    }, token) // drop thread options to default values;
}

// And somewhere there
source.Cancel();
//or
token.ThrowIfCancellationRequested(); // try/ catch block requred.