VB.net：我的线程安全列表解决方案真的安全吗？

Question

我已将以下扩展添加到我的项目中，以便创建线程安全列表：

扩展 如果我想在我的列表上进行一项简单的操作

   <Extension()> _
    Public Sub Action(Of T)(ByVal list As List(Of T), ByVal action As Action(Of List(Of T)))
        SyncLock (list)
            action(list)
        End SyncLock
    End Sub

如果我想传递多个参数，我可以简单地用更多项目扩展它...

<Extension()> _
Public Sub Action(Of T)(ByVal list As List(Of T), ByVal action As Action(Of List(Of T), T), ByVal item As T)
    SyncLock (list)
        Action(list, item)
    End SyncLock
End Sub

操作 我创建了以下操作示例：

Private Sub Read(Of T)(ByVal list As List(Of T))
    Console.WriteLine("Read")
    For Each item As T In list
        Console.WriteLine(item.ToString)
        Thread.Sleep(10)
    Next
End Sub

还有一个带有参数的操作示例：

Private Sub Write(Of T)(ByVal list As List(Of T), ByVal item As T)
    Thread.Sleep(100)
    list.Add(item)
    Console.WriteLine("Write")
End Sub

启动 然后在我的各种线程中，我将使用以下方式调用我的操作：

list.Action(AddressOf Read)

或者

list.Action(AddressOf Write2, 10)

这些扩展方法线程安全吗？或者您有其他建议吗？它类似于执行操作的 List.Foreach 方法。

Answer 1

是的，它们看起来是线程安全的。但只有正确使用时，这可能会变得难以管理。您必须非常小心地观察死锁情况。

有“最佳实践”指出您不应该锁定列表本身（因为其他代码也可能如此），但您违反了该建议。在实践中这并不是一个迫在眉睫的问题，而是潜伏在阴影中等待不太可能发生的巧合的事情。