Multi Thread application

Question

0.00/5 (No votes)

See more:

hey guys I've a program like this (with a background worker) in backgroundWorker1_DoWork. For example I move a button to left and right:

C#

public partial class Form1 : Form
    {
        bool f = false;

        //--------------------------------------------------------------
        public Form1()
        {
            InitializeComponent();
        }

        //--------------------------------------------------------------
        private void Form1_MouseMove(object sender, MouseEventArgs e)
        {
            if (e.Y > 40)
            {
                this.BeginInvoke((ThreadStart)delegate()
                {
                    MenuShow();
                });
            }
            else if (e.Y < 100)
            {
                this.BeginInvoke((ThreadStart)delegate()
                {
                    MenuHide();
                });
            }
            //am I used this correctly?:~
            backgroundWorker1.RunWorkerAsync();
        }

        //--------------------------------------------------------------
        private void MenuHide()
        {
            while (menuStrip1.Top != 0)
            {
                menuStrip1.Top++;
                Thread.Sleep(10);
                menuStrip1.Update();
            }
        }

        //--------------------------------------------------------------
        private void MenuShow()
        {
            while (menuStrip1.Top != -25)
            {
                menuStrip1.Top--;
                Thread.Sleep(10);
            }
        }

        public delegate void InvokeDelegate();

        //--------------------------------------------------------------
        private void Invoke_Click(object sender, EventArgs e)
        {
            textBox1.BeginInvoke(new InvokeDelegate(InvokeMethod));
        }

        //--------------------------------------------------------------
        public void InvokeMethod()
        {
          textBox1.Text = "Executed the given delegate";
        }

        //--------------------------------------------------------------
        private void Form1_MouseClick(object sender, MouseEventArgs e)
        {
            f = true;
        }

        //--------------------------------------------------------------
        private void backgroundWorker1_DoWork(object sender, DoWorkEventArgs e)
        {
           while(!f)  //is this good way to stop the action?:~
           {
               {
                   this.BeginInvoke((ThreadStart)delegate()
                   {
                       //Invoke is a button name
                       while (InvokeButton.Left != 30)
                       {
                           InvokeButton.Left--;
                           Thread.Sleep(10);
                        }
                        while (InvokeButton.Left != 120)
                        {
                            InvokeButton.Left++;
                            Thread.Sleep(10);
                        }
                    });
                }
            }
        }
}

Only the button movement works. The show/hide menu doesn't work. How can I make these works do separately?

Posted 12-Mar-10 17:53pm

Nima.naqipoor

Updated 14-Mar-10 2:54am

#realJSOP

v4

Add a Solution

1 solution

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

#realJSOP · Accepted Answer · 2010-03-14T03:01:00

First, you don't need the variable f - you'll see why in a moment.

Second, your BackgroundWorker object should support cancellation so that it cleans up correctly if the user does something to dismiss the form while the worker is busy.

The while statement should look something like this:

C#

private void backgroundWorker1_DoWork(object sender, DoWorkEventArgs e)
{
    BackgroundWorker worker = sender as BackgroundWorker;
    while(!worker.CancellationPending)  
    {
        // In here, invoke a MoveButton method that actually moves the 
        // button in the desired direction
    }
}

Third, ALL UI updates should be performed through delegates (using Invoke). You may need more than one depending on what you want to accomplish.

Lastly, the name "Invoke" is a poor choice for your button control, especially since there is an Invoke() method. Change it to buttonInvoke, or something like that.