Events are running twice

[=?Utf-8?B?amRrYzRk?=]

I have been designing an application in Visual Studio 2005, and just about
everything is working, except all of my events run twice. They run through
correctly, but then they randomly run through a second time. For example, if
I click cancel on the main form, it properly triggers the event, but it does
so twice. The main program is program.cs, and it calls the others. Here is
the source code, its too long to list here, so here are some links:

http://www.joshkelahan.com/slu/AutoL...Program.cs.pdf
http://www.joshkelahan.com/slu/AutoL...rm_main.cs.pdf
http://www.joshkelahan.com/slu/AutoL...rm_cncl.cs.pdf
http://www.joshkelahan.com/slu/AutoL...t_about.cs.pdf

And here are the Designer files created by Visual Studio
http://www.joshkelahan.com/slu/AutoL...esigner.cs.pdf
http://www.joshkelahan.com/slu/AutoL...esigner.cs.pdf
http://www.joshkelahan.com/slu/AutoL...esigner.cs.pdf

Any help you can provide on how I can get these events to trigger only once
would be greatly appreciated.

Thanks!

[Chris Mullins [MVP - C#]]

You're hooking the events up twice. :)

Do a quick search for "+=" on the event, put a breakpoint there, on see how
many times it gets called. I would be willing to bet, it's exactly two
times.

If you add an event handler, you need to remembe to remove it. Otherwise,
you can add it multiple times, and then it's called multiple times.

--
Chris Mullins

"jdkc4d" <jd****@discussions.microsoft.comwrote in message
news:F0**********************************@microsof t.com...

>I have been designing an application in Visual Studio 2005, and just about
everything is working, except all of my events run twice. They run
through
correctly, but then they randomly run through a second time. For example,
if
I click cancel on the main form, it properly triggers the event, but it
does
so twice. The main program is program.cs, and it calls the others. Here
is
the source code, its too long to list here, so here are some links:

http://www.joshkelahan.com/slu/AutoL...Program.cs.pdf
http://www.joshkelahan.com/slu/AutoL...rm_main.cs.pdf
http://www.joshkelahan.com/slu/AutoL...rm_cncl.cs.pdf
http://www.joshkelahan.com/slu/AutoL...t_about.cs.pdf

And here are the Designer files created by Visual Studio
http://www.joshkelahan.com/slu/AutoL...esigner.cs.pdf
http://www.joshkelahan.com/slu/AutoL...esigner.cs.pdf
http://www.joshkelahan.com/slu/AutoL...esigner.cs.pdf

Any help you can provide on how I can get these events to trigger only
once
would be greatly appreciated.

Thanks!

[Chris Mullins [MVP - C#]]

"Miro" <mi******@beero.netwrote:

Would you mind posting an example of code that was causing this... i
wouldn't mind just seeing what the cause was.

Was it something like "handles += onclick or something ?

If your code looked like:
Button1.OnClick += MyButtonHandler(...);
Button1.OnClick += MyButtonHandler(...);

When the user clicks button1 a single time, your method ("MyButtonHandler")
will actually be called twice.

If you're adding event handlers based on state, then you need to remember to
remove them as well:
Button1.Onclick -= MyButtonHandler(...);

--
Chris Mullins

[Miro]

Chris Mullins [MVP - C#] wrote:

"Miro" <mi******@beero.netwrote:

>Would you mind posting an example of code that was causing this... i
wouldn't mind just seeing what the cause was.

Was it something like "handles += onclick or something ?

If your code looked like:
Button1.OnClick += MyButtonHandler(...);
Button1.OnClick += MyButtonHandler(...);

When the user clicks button1 a single time, your method ("MyButtonHandler")
will actually be called twice.

If you're adding event handlers based on state, then you need to remember to
remove them as well:
Button1.Onclick -= MyButtonHandler(...);

--
Chris Mullins

Thank you

Miro