列出<t>意外清除For循环</t>

时间:2015-01-15 01:32:13

标签: c# list class foreach

这个错误困扰了我!本质上,代码接收一个名为“Parking”的对象列表。我想将Parking个对象与Title类似。为此,我考虑在列表中创建一个列表 - TerminalCollection是一个Parking对象列表和一个名称供参考。

为了帮助我,我创建了一个新的 - 临时 - terminal对象,它是Parking个对象的列表。每当我在同一类别中找到Parking对象时,我会将其添加到临时terminal,直到找到一个不适合的对象。发生这种情况时,我将terminal列表与名称组合在一起以创建TerminalCollection对象。然后我通过向终端添加下一批类似的Parking对象来重新开始这个过程。

我发现的问题是,当我想在创建包含此列表副本的terminal对象后清除TerminalCollection列表时,该列表将从TerminalCollection清除对象也是!!我不知道为什么,因为我以为我正在创建一个新列表并为其分配terminal的副本?为什么新列表与原始terminal共享相同的内存?有没有我误解的基本概念?非常感谢您提供任何帮助和建议来解决这个问题! :)

private static List<TerminalCollection> Terminals(List<Parking> input)
    {
        List<TerminalCollection> Terminals = new List<TerminalCollection>();
        List<Parking> terminal = new List<Parking>();
        int current = 0;
        int currentpos = 0;
        string currentname = "";
        bool clearbuffer = false;
        foreach (Parking attempt in input)
        {
            if (clearbuffer)
            {
                terminal.Clear();
                clearbuffer = false;
            }
            if (current != 0)
            {
                int checknew = int.Parse(Regex.Match(attempt.Title, @"\d+").Value);
                if (checknew - current == 0 && Regex.Match(attempt.Title, @"\d+").Index == currentpos)
                {
                    //Matches same terminal, so slot added in temporary terminal list  
                    terminal.Add(attempt);
                    currentname = Regex.Match(attempt.Title, @"^.*?\d+").Value;
                } 
                else
                {
                    //Clears and starts again (resets current and currentpos)
                    current = checknew;
                    currentpos = Regex.Match(attempt.Title, @"\d+").Index;
                    TerminalCollection final = new TerminalCollection();
                    final.Slots = terminal;
                    final.Name = currentname;
                    //One terminal added to global collection of terminals
                    Terminals.Add(final);
                    //Temporary terminal list cleared
                    clearbuffer = true;
                }
            }
            else { 
            //Gets first integer  
            current = int.Parse(Regex.Match(attempt.Title, @"\d+").Value);
            currentpos = Regex.Match(attempt.Title, @"\d+").Index;
            terminal.Clear();
            //Adds first element to start a new terminal
            terminal.Add(attempt);
            }
        }
        return null;
    }

1 个答案:

答案 0 :(得分:7)

Clear 会生成新列表,也不会进行分配。您只需在循环之前创建一次列表,因此每个对象都将指向它。

您将此清单视为&#34;其他&#34;列出清算(实际上它们都是相同的清单)。

您需要更改为:

if (clearbuffer)
{
   terminal = new List<Parking>();
   clearbuffer = false;
}

现在每次迭代都会有自己的拥有列表对象,但您不会看到对其他实例的更改。

总而言之,考虑重新设计该循环,其真的令人困惑。