使用联锁答案

【问题标题】：Using Interlocked使用联锁
【发布时间】：2010-02-09 16:59:09
【问题描述】：

这段代码是线程安全的吗？或者这样说：

无论如何调用 GetIt() 并且 GetIt() 将返回相同的数字到 2 个不同的线程

Private Shared hitCount As Long = 1

Public Shared Function GetIt() As Long
     Threading.Interlocked.Increment(hitCount)
     DoSomethingQuick(hitCount)
     Return hitCount
End Function

好像有可能，那我是应该用Interlocked.Read()还是把整个东西锁在一个块里？

【问题讨论】：

我认为一旦您到达Integer.MaxValue 就会遇到溢出异常？ hitCount 是 Long 但您返回的是 Integer。

标签： .net performance multithreading interlocked

【解决方案1】：

是的，有可能：

线程 1 运行 Threading.Interlocked.Increment(hitCount)
线程 2 运行 Threading.Interlocked.Increment(hitCount)
线程 1 运行 Return hitCount
线程 2 运行 Return hitCount

在第 3 步和第 4 步中，hitCount 将是相同的值。

但修复很容易 Interlocked.Increment 返回增量值，因此只需将代码更改为：

Private Shared hitCount As Long = 1L

Public Shared Function GetIt() As Long
     Return Threading.Interlocked.Increment(hitCount)
End Function

编辑或者现在根据你的编辑，你有一个相当多的时间漏洞。无论如何，这就是你想要的：

Public Shared Function GetIt() As Long
     Dim localHitCount As Long = Threading.Interlocked.Increment(hitCount)
     Console.Writeline("Something, something....")
     Return localHitCount 
End Function

编辑然后执行此操作（这正是 Michael 下面的建议）

Private Shared hitCount As Long = 1L

Public Shared Function GetIt() As Long
     Dim localHitCount As Long = Threading.Interlocked.Increment(hitCount)
     DoSomethingQuick(localHitCount )
     Return localHitCount 
End Function

【讨论】：

:) 我不知道它正在返回最终值，不错。
刚刚更新了代码，使其与我的原始代码相同（对不起，我写的示例代码很糟糕）。 AFAIK 在这个更新的代码中，我必须用锁包裹整个东西。
您可以只缓存 Interlocked.Increment 的返回值，并在调用 DoSomethingQuick 和返回时使用它。
谢谢迈克尔！该死！为什么我不这么认为？ :) 这很明显不是吗：D
你不需要，但整个事情就是一个锁，除非你依赖 hitCount 在运行期间根本不改变，而不是仅仅确保你有一个唯一的值。