c # Método looping y control
-
06-09-2019 - |
Pregunta
Tengo un método que me gustaría correr una y otra again.I le gustaría ser capaz de iniciar y detener este proceso.
He estado usando este patrón para un trabajo zócalo y me pregunto qué puedo hacer mejoras?
public delegate void VoidMethod();
public class MethodLooper
{
private VoidMethod methodToLoop;
private volatile bool doMethod;
private readonly object locker = new object();
private readonly Thread loopingThread;
public void Start()
{
if (!doMethod)
{
doMethod = true;
loopingThread.Start();
}
}
public void Stop()
{
if (doMethod)
{
doMethod = false;
loopingThread.Join();
}
}
public void ChangeMethod(VoidMethod voidMethod)
{
if (voidMethod == null)
throw new NullReferenceException("voidMethod can't be a null");
Stop();
lock (locker)
{
methodToLoop = voidMethod;
}
}
public MethodLooper(VoidMethod voidMethod)
{
if (voidMethod == null)
throw new NullReferenceException("voidMethod can't be a null");
methodToLoop = voidMethod;
loopingThread = new Thread(new ThreadStart(_MethodLoop));
}
private void _MethodLoop()
{
VoidMethod methodToLoopCopy;
while (doMethod)
{
lock (methodToLoop)
{
methodToLoopCopy = methodToLoop;
}
methodToLoopCopy();
}
}
}
Solución
Una versión más segura sería la de hacerlo de esta manera:
private readonly object m_locker = new object(); // readonly so it can never be null
private readonly Thread m_workerThread; // readonly so must set in constructor,
// and never can be null afterwards
private Action m_methodToRun;
private volatile bool m_keepGoing = true; // needs to be volatile or else you need to lock around accesses to it.
public Constructor()
{
m_workerThread = new Thread(ThreadWorker);
}
public void SetMethod(Action action)
{
lock(m_locker)
m_methodToRun = action;
}
private void ThreadWorker()
{
while(m_keepGoing)
{
// use a lock to take a local copy in case another thread sets m_methodToRun to null
// while we are processing things
Action methodLocal;
lock(m_locker)
methodLocal = m_methodToRun;
methodLocal(); // call it
// Note: Remember that the underlying method being pointed to must ALSO be
// thread safe. Nothing you do here can make up for that if it is not.
}
}
private void Stop()
{
m_keepGoing = false;
m_workerThread.Join(); // BLOCK and wait for it to finish
}
private void Start()
{
m_keepGoing = true;
m_workerThread.Start();
}
esta otra pregunta de los puntos más finos en la volátil vs bloqueo
Otros consejos
Bueno, usted debe expresar la doMethodToLoop tan volátil. Como se indica en el MSDN:
"La palabra clave volátil indica que un campo se puede modificar en el programa por algo como el sistema operativo, el hardware, o un hilo que se ejecutan simultáneamente."
Estoy en un apuro, sino que debe salir este tutorial del interruptor en el código.
Licenciado bajo: CC-BY-SA con atribución
No afiliado a StackOverflow