5 replies to this topic

[Omar (OZ)]

Video 1-


Video 2-


Code -

        public void Pulse(int onTime, int offTime, bool onFirst, uint repeat)
        {
            bool doit = false;
            _worker = new Thread(
                delegate() 
                { 
                    doit = doWork(onTime, offTime, onFirst, repeat); 
                }
                );
            _worker.Start();


        }

        private bool doWork(int onTime, int offTime, bool onFirst, uint repeat)
        {
            if (onFirst)
            {
                for (uint c = 0; c < repeat; c++)
                {
                    _pwmPort.SetDutyCycle((uint)Power);
                    Thread.Sleep(onTime);
                    _pwmPort.SetDutyCycle((uint)0);
                    Thread.Sleep(offTime);
                } 
            }
            else
            {
                for (uint c = 0; c < repeat; c++)
                {
                    Thread.Sleep(offTime);
                    _pwmPort.SetDutyCycle((uint)Power);
                    Thread.Sleep(onTime);
                    _pwmPort.SetDutyCycle((uint)0);
                }
            }
            
            return true;
        }

[Chris Walker]

Multi-threading, the use of anonymous methods, and thread synchronization--wow, pretty sophisticated. Quick note: In your doWork() call from yourPublicMethod, you're passing one parameter to doWork--which has three. For your sample code, you might want to add two other parameters to make it complete. Chris

[Omar (OZ)]

Multi-threading, the use of anonymous methods, and thread synchronization--wow, pretty sophisticated.

Quick note:
In your doWork() call from yourPublicMethod, you're passing one parameter to doWork--which has three. For your sample code, you might want to add two other parameters to make it complete.

Chris

Thanks for catching that, didn't change it.

[Dave]

Actually since you're not using the value returned you could do something like this, and do it all in one line of code....

       public void Pulse(int onTime, int offTime, bool onFirst, uint repeat)
        {
            new Thread(() => doWork(onTime, offTime, onFirst, repeat).Start();
        }

--Dave

[Omar (OZ)]

Actually since you're not using the value returned you could do something like this, and do it all in one line of code....

       public void Pulse(int onTime, int offTime, bool onFirst, uint repeat)
        {
            new Thread(() => doWork(onTime, offTime, onFirst, repeat).Start();
        }

--Dave

I see, but I think its still better to do _worker = new Thread(.....), because without that _worker =, then you can't Join the thread

[Michel Trahan]

Just leaving a breadcrum to remember Thanks Omar !