Re: While MouseDown - C# Discussion Boards

17-Oct-09 16:21

Probably manufacturer specific. My Windows 7, nor Server 208 boxes doesn't have it.

only two letters away from being an asset

While MouseDown

Zap-Man17-Oct-09 3:27

17-Oct-09 3:27

I am trying to do some events while the mouse button is down.
Here is what I have

private static bool IsKeyDown = false;

private void button1_MouseDown(object sender, MouseEventArgs e) {
IsKeyDown = true;
while (IsKeyDown) {
Winamp.Winamp.VolumeDown();

}

}

private void button1_MouseUp(object sender, MouseEventArgs e) {

IsKeyDown = false;

}

The problem is when it hits the mouse down event It continues to run the while loop and does not hit the mouse up event.

I don't want to use a timer to do this.

I did see some code that I think will work better in my app but I cant seem to get it to work can some one please help.

http://stackoverflow.com/questions/1152525/c-winform-looping-event-on-press-button[^]

private bool mouseDown = false;

private void buttonScrollUp_MouseDown(object sender, MouseEventArgs e)
{
mouseDown = true;
new Thread(() => {
while (mouseDown)
{
Invoke(new MethodInvoker(() => [DO_THE_SCROLLING_HERE));
Thread.Sleep([SET_AUTOREPEAT_TIMEOUT_HERE);
}
})
.Start();
}

private void buttonScrollUp_MouseUp(object sender, MouseEventArgs e)
{
mouseDown = false;
}

Lyon Sun17-Oct-09 4:00

Lyon Sun

17-Oct-09 4:00

Zap-Man wrote:
private static bool IsKeyDown = false;

Confused. Can this variable be changed ever? Correct me if I am wrong... Smile | :)

Zap-Man17-Oct-09 4:09

17-Oct-09 4:09

yes it changes on the mouse down to true. but the mouse up event never fires because the loop has not finished. but it will change in the mouse up event. if the value never changed then I would have to set it to const or read only. A static variable will change. private means this variable is not accessible from another class. when the form is initialized the variable is set to false. When the mouseDown event is fired then the variable is set to true. This is working. the problem lies in the loop. Because it is in the loop the MouseUp event never executes.

Richard MacCutchan17-Oct-09 4:44

Richard MacCutchan

17-Oct-09 4:44

Zap-Man wrote:
IsKeyDown = true;
while (IsKeyDown) {
Winamp.Winamp.VolumeDown();
}

This loop will never terminate so your event handler button1_MouseDown will never notify the framework that it is complete. This blocks your application from doing anything else, so you need to separate the functions somehow.

BTW it helps if you put <pre></pre> tags around your code snippet so it is easier to read.

Luc Pattyn17-Oct-09 5:08

Luc Pattyn

17-Oct-09 5:08

Hi,

this is all wrong.

1. event handlers get executed on the main thread (aka GUI thread), hence MouseUp cannot do anything as long as MouseDown is active.

2. Your while loop will never exit.

3. Your while loop has no delay, so at what rate will VolumeDown be called?

4. Using Thread.Sleep would not be a good approach as that blocks the GUI; using a timer (a Windows.Forms.Timer) is the perfect solution: start the timer on MouseDown, stop it on MouseUp (or better yet have the Tick handler check Control.MouseButtons).

Zap-Man wrote:
I don't want to use a timer

why? because you don't understand them? because you don't like a clean and natural solution?

Smile | :)

Luc Pattyn

I only read code that is properly indented, and rendered in a non-proportional font; hint: use PRE tags in forum messages

Local announcement (Antwerp region): Lange Wapper? Neen!

Zap-Man17-Oct-09 5:43

17-Oct-09 5:43

Look I am not looking for Critics. If you cant help then don't bother posting. I understand timers perfectly. My app has a timer which runs a massive about of code. I understand what the program is doing. I am looking for a way around it.

Not Active17-Oct-09 6:17

17-Oct-09 6:17

This response is uncalled for, Luc's comments are valid.

only two letters away from being an asset

Not Active17-Oct-09 5:42

17-Oct-09 5:42

private AutoResetEvent autoEvent = null;
private void OnMouseDown(object sender, MouseEventArgs e)
{
autoEvent = new AutoResetEvent(false);
ThreadPool.QueueUserWorkItem(VolumeDown);
}

private void OnMouseUp(object sender, MouseEventArgs e)
{
autoEvent.Set();
}

private void VolumeDown(object state)
{
bool lowerVolume = true;
do
{
BeginInvoke(new MethodInvoker(delegate() { Winamp.Winamp.VolumeDown(); }), null);
if(autoEvent.WaitOne(100))
lowerVolume = false;
}
while(lowerVolume);
}

only two letters away from being an asset

Zap-Man17-Oct-09 5:46

17-Oct-09 5:46

Thank you Mark This is excatly what Im looking for. Congrats to you!!! Thumbs Up | :thumbsup:

Not Active17-Oct-09 6:15

17-Oct-09 6:15

You're welcome

only two letters away from being an asset