我应该始终锁定静态方法吗？

Question

在以下示例中，GetList 返回静态（共享）变量的实例。为了线程安全，需要加锁。

但是如果 DoSomething 在方法之外不使用任何静态变量呢？也需要加锁吗？

编辑：我想澄清一下，在这种特殊情况下，我希望 DoSomething 始终按顺序打印 0-100 （即没有 0123745...），无论调用线程的数量如何。或者，一般来说，不同的线程不会影响彼此的变量（打印到控制台只是一个示例）。以下示例中的语言是 VB.NET。

正如帕克斯迪亚布罗所说：

在这种情况下，似乎唯一触及的是局部变量 我每个函数调用都会有一个单独的副本。 换句话说，它不需要保护。

这正是我想要解决的问题。谢谢你！

Public Class TestClass

    Private Shared lock As New Object
    Private Shared list As List(Of Integer)

    Public Shared Function GetList() As List(Of Integer)

        SyncLock lock
            If list Is Nothing Then
                list = New List(Of Integer)
            End If
            Return list
        End SyncLock

    End Function

    Public Shared Sub DoSomething()

        Dim i As Integer

        For i = 0 To 100
            Console.WriteLine(i.ToString)
        Next

    End Sub

End Class

Answer 1

好吧，这主要取决于您未指定的语言，但通常，如果代码不触及另一个线程也可以触及的资源，则不必受到保护。

在这种情况下，似乎唯一触及的是局部变量i，它为每个函数调用都有一个单独的副本。换句话说，它不需要保护。

当然，可以说控制台也是一种资源，并且可能需要保护，例如，如果您不希望行相互干扰（同步写入）或希望将整个一百行作为一个单元输出（同步整个 for 循环）。

但这并不能真正保护控制台，只是保护使用它的代码块。如果不使用此方法，其他线程仍然能够写入控制台。

最重要的是，我认为第二种方法不需要同步锁。

---

如果您在 VB.Net 中使用 SyncLock（现在看来就是这种情况），那么下面的这一部分是不相关的。该语言保证无论您如何离开块，锁都会被释放。我会出于歇斯底里的目的而保留它。

我会有点担心第一个方法中的同步锁放置，特别是如果返回语句是将控制权转移回调用者（并且同步锁不会在范围更改时自动解锁） 。看起来你可以在不解锁的情况下返回，这将是一场灾难。我认为以下内容更合适：

Public Shared Function GetList() As List(Of Integer)
    SyncLock lock
        If list Is Nothing Then
            list = New List(Of Integer)
        End If
    End SyncLock
    Return list
End Function

Answer 2

一般来说，没有。

您需要清楚 GetList 应用锁定的原因。正如您在第一句话中所暗示的那样，它并不是因为它返回一个静态变量。您可以删除锁定代码，并且 GetList 仍然是线程安全的。但是，通过锁定，还有一个额外的保证 - 该列表只会创建一次，并且此代码的​​所有调用者都将收到对同一列表的引用。

Answer 3

为什么不完全避免锁定并这样做：

Public Class TestClass

    Private Shared lock As New Object
    Private Shared list As New List(Of Integer)

    Public Shared Function GetList() As List(Of Integer)
        Return list
    End Function

    Public Shared Sub DoSomething()
        Dim i As Integer
        For i = 0 To 100
            Console.WriteLine(i.ToString)
        Next
    End Sub

End Class