C# Egg Timer Please Help!!!!

clairey4u

Hi Folks i am pretty new to c# and i cant get this to work. Basically its a timer application and when i start the thread i then cant use the stop or start button can anyone help. Here is my onpaint function where the threading happens


I have set true initially to false


protected override void OnPaint(PaintEventArgs pe)
{
base.OnPaint(pe);
Graphics g = pe.Graphics;



if (second >= 10)
{
g.DrawString(min + ":" + sec, f, Bbrush, 630, 30);

}


else if (second < 10)
{

g.DrawString(min + ":0" + sec, f, Bbrush, 630, 30);

}
if (start == true)
{


decrement_sec();
Thread.Sleep(1000);
sec = second.ToString();
min = minute.ToString();
this.Refresh();

}


Then this is what i have in my stop and start events


Stop Event


private void button2_Click(object sender, EventArgs e)
{


start = false;
Console.WriteLine("Clicked");
Application.DoEvents();
Button b = sender as Button;
this.Refresh();
}


Start Event

private void button1_Click(object sender, EventArgs e)
{
start = true;
this.Refresh();
}



and my decrement sec method is just this that im calling in the onPaint Function

public void decrement_sec()
{
if (second > 0) second--;
else
{
second = 59;
if (minute > 0) minute--;
}
}



Many thanks for the help

Webmonkey

I didn't really look too closely to what you doing there but please note if you want to access any GUI code from outside of the main thread you will have to queue such events with the dispatcher.

Have a look at Dispatcher.Invoke...

And by the way, the Software Dev forum would be a better place.

clairey4u

hmmm not sure how i use this in my program because my thread is a UI thread i dont know if that makes a difference.

aidan_walsh

Moved to the Development forum.

dazberry

clairey4u wrote: »

protected override void OnPaint(PaintEventArgs pe)
        {
            base.OnPaint(pe);
            Graphics g = pe.Graphics;

            if (second >= 10)
            {
                g.DrawString(min + ":" + sec, f, Bbrush, 630, 30);
            }
            else if (second < 10)
            {
              g.DrawString(min + ":0" + sec, f, Bbrush, 630, 30);               
            }
          if (start == true)
           {             
             decrement_sec();
             [B]Thread.Sleep(1000);[/B]
             sec = second.ToString();
             min = minute.ToString();
             [B]this.Refresh();[/B] 
           }

The OnPaint handler isn't a separate thread, it's part of the main application thread, so Sleep(1000) will freeze your application for 1 second, and constantly forcing a refresh will keep it painting/freezing.

Consider using a timer instead.

D.