C#でハマった

Wed, Feb 20, 2019

最近久しぶりにC#でハマったので備忘録です。
lock中で要素を列挙したかったのですが、ロックの外での列挙になってしまいSystem.InvalidOperationException: コレクションが変更されました。列挙操作は実行されない可能性があります。が発生してしまっていました。

簡易的なコードは下記

正しいコード

 1
 2
 3
 4
 5
 6
 7
 8
 9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47


using System;
using System.Collections.Generic;
using System.Linq;

namespace Sample
{
    class Program
    {
        static object _lock = new object();

        static IEnumerable<int> Enum1()
        {
            Console.WriteLine("ENTER: Enum1");
            try
            {
                lock (_lock)
                {
                    foreach (var i in Enumerable.Range(1, 3))
                        yield return i;
                }
            }
            finally
            {
                Console.WriteLine("EXIT: Enum1");
            }
        }

        static void Loop1()
        {
            Console.WriteLine("ENTER: Loop1");
            try
            {
                foreach (var i in Enum1())
                    Console.WriteLine(i);
            }
            finally
            {
                Console.WriteLine("EXIT: Loop1");
            }
        }

        static void Main(string[] args)
        {
            Loop1();
        }
    }
}

結果

ENTER: Loop1
ENTER: Enum1
1
2
3
EXIT: Enum1
EXIT: Loop1

間違ったコード

 1
 2
 3
 4
 5
 6
 7
 8
 9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60


using System;
using System.Collections.Generic;
using System.Linq;

namespace Sample
{
    class Program
    {
        static object _lock = new object();

        static IEnumerable<int> InnerEnum2()
        {
            Console.WriteLine("ENTER: InnerEnum2");
            try
            {
                foreach (var i in Enumerable.Range(1, 3))
                    yield return i;
            }
            finally
            {
                Console.WriteLine("EXIT: InnerEnum2");
            }
        }

        static IEnumerable<int> Enum2()
        {
            Console.WriteLine("ENTER: Enum2");
            try
            {
                lock (_lock)
                {
                    return InnerEnum2();
                }
            }
            finally
            {
                Console.WriteLine("EXIT: Enum2");
            }
        }

        static void Loop2()
        {
            Console.WriteLine("ENTER: Loop2");
            try
            {
                foreach (var i in Enum2())
                    Console.WriteLine(i);
            }
            finally
            {
                Console.WriteLine("EXIT: Loop2");
            }
        }

        static void Main(string[] args)
        {
            Loop2();
        }
    }
}

結果

ENTER: Loop2
ENTER: Enum2
EXIT: Enum2
ENTER: InnerEnum2
1
2
3
EXIT: InnerEnum2
EXIT: Loop2

Enum2を下記のように修正すればOK

static IEnumerable<int> Enum2()
{
    Console.WriteLine("ENTER: Enum2");
    try
    {
        lock (_lock)
        {
            foreach (var i in InnerEnum2())
                yield return i;
        }
    }
    finally
    {
        Console.WriteLine("EXIT: Enum2");
    }
}