Contact US

Question

C# Thread Basic Please correct this (Beginner)

atzsea (Programmer)

(OP)

29 Mar 13 01:37

This does not get reflected in the Form.
I want somebody to simply correct this.

CODE --> C#

using System;
using System.Windows.Forms;

namespace WindowsFormsApplication1
{
    public partial class Form1 : Form
    {
        private Timer Timer1;
        private System.Threading.Thread thr;
        public Form1()
        {
            InitializeComponent();

            Thread1();
        }

        private void Thread1()
        {
            thr = new System.Threading.Thread(new System.Threading.ThreadStart(threadmethod1));
            thr.Start();
        }

        private void threadmethod1()
        {
            Timer1 = new Timer();
            Timer1.Interval = 200;

            Timer1.Tick += new EventHandler(Timer1_Tick);

            Timer1.Enabled = true;
            Timer1.Start();
            
        }

        private void Timer1_Tick(object sender, EventArgs e)
        {
            DateTime dt = new DateTime();
            this.label1.Text = dt.ToLongTimeString();
        }


    }
}

Moregelen (Programmer) · Answer 1 · 2013-05-08 11:16:30

For one thing, you don't need to run your timer in a separate thread; the timer already creates its own thread.. and you also want to use System.Timers.Timer instead. Generally speaking you won't have to get into directly threading things.. the only time I find myself ever directly creating a Thread is for BIG file operations.

You also need to make the update thread safe.

CODE

using System;
using System.Windows.Forms;

namespace WindowsFormsApplication1
{
    public partial class Form1 : Form\cf3
    {
        private System.Timers.Timer Timer1;

        public Form1()
        {
            InitializeComponent();
        }

        private void Form1_Load(object sender, EventArgs e)
        {
            this.label1.Text = DateTime.Now.ToString("MM:HH:ss");
            StartTimer();
        }

        private void StartTimer()
        {
            Timer1 = new System.Timers.Timer();
            Timer1.Elapsed += new System.Timers.ElapsedEventHandler(Timer1_Tick);
            Timer1.Interval = 1000;
            Timer1.Enabled = true;
            Timer1.Start();

        }

        private void Timer1_Tick(object sender, EventArgs e)
        {
            if (InvokeRequired)
            {
                this.Invoke(new Action<object, EventArgs>(Timer1_Tick), new object[] { sender, e });
            }
            this.label1.Text = DateTime.Now.ToString("MM:HH:ss");
        }

    }
}

Come Join Us!

Posting Guidelines

C# Thread Basic Please correct this (Beginner)

C# Thread Basic Please correct this (Beginner)

C# Thread Basic Please correct this (Beginner)

CODE --> C#

RE: C# Thread Basic Please correct this (Beginner)

CODE

Red Flag This Post

Red Flag Submitted

Reply To This Thread

Posting in the Tek-Tips forums is a member-only feature.

Join Tek-Tips^® Today!

Contact US

Come Join Us!

Posting Guidelines

C# Thread Basic Please correct this (Beginner)

C# Thread Basic Please correct this (Beginner)

C# Thread Basic Please correct this (Beginner)

CODE --> C#

RE: C# Thread Basic Please correct this (Beginner)

CODE

Red Flag This Post

Red Flag Submitted

Reply To This Thread

Posting in the Tek-Tips forums is a member-only feature.

Join Tek-Tips® Today!

Join Tek-Tips^® Today!