C#中互斥锁的使用

Question

我在 c# 和一般情况下的线程方面有点新意， 在我的程序中，我使用mutex 只允许 1 个线程进入关键部分，并且由于未知原因进行一些 cw 打印，我可以看到超过 1 个线程进入我的关键部分，这是我的代码：

Mutex m = new Mutex();
m.WaitOne();
<C.S> // critical section here
m.ReleaseMutex();

我非常想知道我是否在这里做错了，提前感谢您的帮助。

编辑：

我的代码包含类，所以基本上看起来更像这样：

public class test
{
    private mutex m;
    public test()
    {
         m = new mutex();
    }
    public func()
    {
         m.WaitOne();
         <C.S> // critical section here
         m.ReleaseMutex();
     }


    } 

Answer 1

这里的问题是你所有的调用者都在使用一个不同的互斥体；您需要将锁定对象共享，通常是通过将其设为字段。例如，切换到更简单的lock 隐喻：

private readonly object syncLock = new object();
public void ThreadSafeMethod() {
    lock(syncLock) {
        /* critical code */
    }
}

或使用互斥锁：

private readonly Mutex m = new Mutex();
public void ThreadSafeMethod() {
    m.WaitOne();
    try {
        /* critical code */
    } finally {
        m.ReleaseMutex();
    }
}

Answer 2

看起来你给每个线程一个自己的互斥锁。那是行不通的。

在大多数情况下，互斥锁是矫枉过正的。你只需要：

private static object syncLock = new object();  // just 1 instance

....

lock(syncLock)
{
    // critical section
}

Answer 3

这种模式根本没有锁定。每个线程创建一个新的 Mutex 对象并立即拥有它的锁。其他线程自己创建和使用新的 Mutex。

考虑使用常规的 lock()！

lock(_lockobject) {
   // do inside what needs to be done - executed on a single thread only
} 

其中 _lockobject 是类中的一个简单私有变量：

private object _lockobject; 

编辑：感谢评论者！存在一些情况，其中 lock(this) 可能很危险。所以我删除了那个。

Answer 4

Mutex 用于识别运行的应用实例。

 using (Mutex mutex = new Mutex(true, "app name", out createdNew))
            {
                if (createdNew)//check app is already run
                {
                    KillOthers();
                    StartApp();
                }
                else
                {
                    MessageBox.Show("Another instance already running!");
                }
            }

Answer 5

我可以对已接受的答案进行更正吗？

private readonly Mutex m = new Mutex();
public void ThreadSafeMethod() {
    while(!m.WaitOne()){}
    try {
        /* critical code */
    } finally {
        m.ReleaseMutex();
    }
}