Invalidate help needed!

[mxmillercell]

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Text;
using System.Windows.Forms;
 
namespace timertest
    {
    public partial class Form1 : Form
        {
 
        public Form1()
            {
            InitializeComponent();
            }
 
        int a = 0;
 
        System.Windows.Forms.Timer timer = new Timer();
 
        System.Windows.Forms.Timer timer2 = new Timer(); 
 
        public void flash(object sender, EventArgs e)
            {
            if (this.BackColor == System.Drawing.Color.Red)
                {
                this.BackColor = System.Drawing.Color.White;
                }
            else 
                {
                this.BackColor = System.Drawing.Color.Red;
                }
            Invalidate();
            }
 
        public void noflash(object sender, EventArgs e)
            {
                {
                timer.Stop;
                timer2.top;
                }
            }
 
        private void button1_Click(object sender, EventArgs e)
            { 
            timer.Interval = 300;
            timer2.Interval = 3000;
            timer.Start();
            timer2.Start();
            timer.Tick += new EventHandler(flash);
            timer2.Tick += new EventHandler(noflash);            
            }
        }
    }

//This is code I wrote to test a routine to cause a windows form to flash when a specific event occurs. In this code the event is button1_click. The problem I have is when I click the button the form flashes until timer2 stops at which point I presume the "noflash" method has been called by the timer2.Tick event. But when I click the button again the form does nothing. When I click the button the third time, the form flashes again, acting as if the flash action has been toggled.
Why is this happening, are the timers not actually being stopped, is timer2 really stopping or not. I am very confused, please help a guuy out.

[Plater]

You should not be assigning MORE event handlers everytime a button is clicked. They should be assigned once. Or assigned, then removed, and assigned again, although there's no reason for that.

[mxmillercell]

I'm not sure how that answers my question. Is assigning an event the reason the form is only flashing every other time I click? How do I get the form to flash every time I click the button?

You should not be assigning MORE event handlers everytime a button is clicked. They should be assigned once. Or assigned, then removed, and assigned again, although there's no reason for that.

[mxmillercell]

Whoa, I know what you meant, it was the problem. I moved it into the Form class declaration Thank you very much. I was pulling my hair out over this one.

I'm not sure how that answers my question. Is assigning an event the reason the form is only flashing every other time I click? How do I get the form to flash every time I click the button?