Benutzt Mutex

Question

Ich habe zwei Instanzen des gleichen Windows-Dienstes ausgeführt. Sie überprüfen die Gesundheit der anderen und berichten, wenn ein Problem gefunden wird. Ich habe einen kritischen Job, der ausgeführt werden muss, damit ich ihn mit einem Failover-Ansatz ausführen kann, er läuft im Master und wenn der Master nicht antwortet, läuft er im Slave. Dieser Job muss über eine bestimmte serielle Schnittstelle kommunizieren. Ich versuche, Mutex zu verwenden, um den Race-Zustand zu überprüfen. Ich habe keinen Zugriff auf die Produktion, also möchte ich vor der Bereitstellung sicherstellen, dass mein Ansatz in Ordnung ist. Bitte schlage also vor, ob meine Verwendung von Mutex für den gegebenen Fall in Ordnung ist.

if (iAmRunningInSlave) 
{ 
    HealthClient hc = new HealthClient(); 
    if (!hc.CheckHealthOfMaster()) 
     return this.runJobWrapper(withMutex, iAmRunningInSlave); 
    else 
     return true; //master is ok, we dont need to run the job in slave 
} 
return this.runJobWrapper(withMutex, iAmRunningInSlave); 

Und dann in runJobWrapper

bool runJobWrapper(bool withMutex, bool iAmRunningInSlave) 
{ 
    if (!withMutex) 
     return this.runJob(iAmRunningInSlave); //the job might be interested to know 
    Mutex mutex = null; 
    string mutexName = this.jobCategory + "-" + this.jobTitle; //this will be unique for given job 
    try 
    { 
     mutex = Mutex.OpenExisting(mutexName); 
     return false; //mutex is with peer, return false which will re-trigger slave 
    } 
    catch 
    { 
     try 
     { //mean time mutex might have created, so wrapping in try/catch 
     mutex = new Mutex(true /*initiallyOwned*/, mutexName); 
     return this.runJob(iAmRunningInSlave); //the job might be interested to know where I am running 
     } 
     finally 
     { 
     if (null!=mutex) mutex.ReleaseMutex(); 
     } 
     return false; 
    } 
} 

Answer 1

hatte ich ein ähnliches Problem vor kurzem.

Das Design der Mutex Klasse ist ein bisschen seltsam/anders als die normalen Klassen in .NET.

Mit OpenMutex auf eine vorhandene Mutex zu überprüfen ist nicht wirklich nett, da Sie eine Ausnahme fangen müssen.

Ein besserer Ansatz ist es, die

Mutex(bool initiallyOwned, string name, out bool createdNew) 

Konstruktor zu verwenden, und überprüfen Sie die von createdNew zurückgegebene Wert.

Answer 2

Sie suchen nicht, um den Rückgabewert von runJobWrapper überall zu überprüfen - ist das absichtlich? Es ist nicht offensichtlich, was der Rückgabewert tatsächlich bedeutet. Auch sollten Sie wirklich nicht jede Ausnahme fangen, die OpenExisiting möglicherweise werfen könnte - nicht genügend Speicher? Paketüberfluss? usw. etc. Fangen Sie einfach die, die Sie richtig handhaben wollen.

Auch Ihr Code sieht etwas fragil aus - ich wäre nicht überrascht, wenn Sie Race Conditions haben.

Answer 3

Ich bemerkte, dass mutex.ReleaseMutex() den Mutex nicht wurde die Freigabe immediately..I hatte GC.Collect()