.net - 将频繁的文件 I/O 操作放在 SyncLock block 中是一个坏主意吗？

Question

假设我有一些代码可以做到这一点:

Public Function AppendToLogFile(ByVal s As String) As Boolean
    Dim success As Boolean = True
    Dim fs As IO.FileStream = Nothing
    Dim sw As IO.StreamWriter = Nothing

    Static LogFileLock As New Object()
    SyncLock LogFileLock
        Try
            fs = New IO.FileStream(LogFilePath)
            sw = New IO.StreamWriter(fs)
            sw.WriteLine(s)

        Catch ex As Exception
            success = False

        Finally
            If Not sw Is Nothing Then sw.Close()
            If Not fs Is Nothing Then fs.Close()
        End Try
    End SyncLock

    Return success
End Function

首先:我在 SyncLock 中有 Try/Catch/Finally 块是否有问题？

其次:假设这段代码在一个事件上运行，可能在很短的时间范围内运行多次——比如，一秒内运行十次。像这样拥有 SyncLock 是否可以，或者让它向队列添加一行，然后将队列中的所有行写入文件中的计时器，该计时器每秒关闭是否更有意义？

Answer 1

乍一看，这对我来说还不错，但有两个警告:

静态成员已经在幕后使用了一种线程安全锁定。因此，您可能只能捎带现有的锁，而不是显式锁。不过，我不确定那会是什么样子。

不返回状态代码。让异常传播到适当的级别。一旦你这样做了，你可以像这样重写你的代码:

.

Public Sub AppendToLogFile(ByVal s As String) As Boolean
    Static LogFileLock As New Object()
    SyncLock LogFileLock
        Using sw As New IO.StreamWriter(LogFilePath)
            sw.WriteLine(s)
        End Using
    End SyncLock
End Sub

这就是不到一半代码的所有功能。唯一的区别是您必须在调用代码中处理异常而不是检查返回状态。