2

我正在尝试按照此处的建议实现多线程:Spawn Multiple Threads for work then wait until all finished

代码如下所示:

var resetEvent = new ManualResetEvent(false);
            var clientsCount = IDLocal.UserSessions.Count;

            // Load sessions:
            IDLocal.UserSessions = new IDSessions();

            // Start thread for each client 
            foreach (IDSession session in IDLocal.UserSessions)
            {
                var clnt = session;
                new Thread(
                    delegate()
                    {
                        Debug.WriteLine(Thread.CurrentThread.ManagedThreadId);
                        clnt.FailedToRespond = !this.SendDirect(data, this.GetHostAddress(clnt.HostName), clnt.PortNumber);

                        // If we're the last thread, signal 
                        if (Interlocked.Decrement(ref clientsCount) == 0) resetEvent.Set();
                    })
                    .Start();
            }

在这里,我收到 ReSharper 警告:if (Interlocked.Decrement(ref clientsCount) == 0)

它表明我正在访问修改后的闭包(clientsCount)

这是一个有效的建议吗?

4

1 回答 1

3

该警告旨在涵盖代码,例如

int i = 1;
Func<int> f = () => i + 1;
i = 2;
int j = f(); // What does this set j to?

i调用时将使用的更新值在哪里f。警告建议将其更改为

int i = 1;
int icopy = i;
Func<int> f = () => icopy + 1;
i = 2;
int j = f(); // Now what does this set j to?

如果您想f查看i创建委托时的值。在foreach循环的上下文中,很容易出现不直观的行为。foreach正因为如此,C# 5中循环的含义发生了变化。

由于这不是您想要的,因为您确实想查看对捕获变量的更改,所以不要更改代码中的任何内容。

于 2012-09-20T22:02:19.463 回答