c# - 我应该在以下情况下使用锁吗

Question

我设计了一个 Stack 包装器类。我的困惑是，我是否应该在将对象弹出或推送到堆栈变量“ParameterStack”时使用 lock。请让我知道，这个类是否是线程安全的。

public static class StackManager
{
    private static Stack ParameterStack = new Stack();

    public static T Pop<T>()
    {
        T RawObject;
        T Result = default(T);

        lock (ParameterStack)
        {
            RawObject = (T)ParameterStack.Pop();
        }

        if (RawObject != null && RawObject is T)
            Result = (T)RawObject;

        return (T)Result;
    }

    public static void Push<T>(T Data)
    {
        lock (ParameterStack)
        {
            ParameterStack.Push(Data);
        }
    }
}

我出于学习目的创建了这个 StackManager 类。

Answer 1

看起来不错。有一个(相当理论上的)论点认为锁定 ParameterStack 本身并不完全安全，因为您不拥有代码。假设 Stack 中的某处执行了 lock(this) ，您可能会死锁。

public static class StackManager
{
    private static Stack parameterStack = new Stack();
    private static object stackLock = new object();

    // now use lock(stackLock) instead of lock(ParameterStack)

}