Question

I’m using System.Windows.Forms.Timer to display a transfer file update progress(timeleft, speed ..etc)

and I also use backgroundworker to send the file

• backgroundWorker1_DoWork calls timer1.Start();
• backgroundWorker1_RunWorkerCompleted calls timer1.Stop();

It works fine only in the first call for timer1.Strat, but when it called again after timer1.Stop(). It doesn’t work.

timer1.Enabled = True; 
timer1.Interval = 1000;
private void timer1_Tick(object sender, EventArgs e)
    {
        long speed = sumAll - prevSum;
        prevSum = sumAll;
        labelSpeed.Text = CnvrtUnit(speed) + "/S";
        if (speed > 0)
        {
            totalSeconds++;
            labelTime.Text = FormatRemainingText(TimeSpan.FromSeconds((sizeAll - sumAll) / speed));
            labelTotalTime.Text = FormatRemainingText(TimeSpan.FromSeconds(totalSeconds));
        }
    }

What’s wrong with it and how do I fix it?

Answer 1

I’ve figured it out, I use System.Timers.Timer instead of System.Windows.Forms.Timer

System.Timers.Timer timer1 = new System.Timers.Timer(1000);

In class constructor I added:

    public FileTransfer()
    {
        InitializeComponent();
        timer1.Elapsed += timer1_Tick;
    }


    private void timer1_Tick(object sender, EventArgs e)
    {
        long speed = sumAll - prevSum;
        Console.WriteLine(speed);
        prevSum = sumAll;
        Speed(CnvrtUnit(speed) + "/S");
        if (speed > 0)
        {
            totalSeconds++;
            Timeleft(FormatRemainingText(TimeSpan.FromSeconds((sizeAll - sumAll) / speed)));
            TotalTime(FormatRemainingText(TimeSpan.FromSeconds(totalSeconds)));
        }
    }

    private void Timeleft(string value)
    {
        if (InvokeRequired)
        {
            this.Invoke(new Action<string>(Timeleft), new object[] { value });
            return;
        }
        labelTime.Text = value;
    }
    private void TotalTime(string value)
    {
        if (InvokeRequired)
        {
            this.Invoke(new Action<string>(TotalTime), new object[] { value });
            return;
        }
        labelTotalTime.Text = value;
    }
    private void Speed(string value)
    {
        if (InvokeRequired)
        {
            this.Invoke(new Action<string>(Speed), new object[] { value });
            return;
        }
        labelSpeed.Text = value;
    }

Now it works every time I call timer1.Start(), no need to “AutoReset”.