Question

尝试读取csv文件，并获取流中的第一个单词，将其放入字典中，同时将以下单词添加到该字典中的列表中。

然而，我发现（在调试期间），在我的循环中我决定清除我的列表时，它之前添加到字典中的所有值都被清除了。我想我错误地假设它复制了一个列表，它实际上只是引用相同的列表？我应该在每次迭代时创建一个新列表吗？代码如下：

public class TestScript : MonoBehaviour {

// Use this for initialization
void Start() {

    Dictionary<string, List<string>> theDatabase = new Dictionary<string, List<string>>();
    string word;
    string delimStr = ",.:";
    char[] delimiter = delimStr.ToCharArray();
    List<string> theList = new List<string>();


    using (StreamReader reader = new StreamReader("testComma.csv")) {
        while (true) {
            //Begin reading lines
            string line = reader.ReadLine();
            if (line == null) {
                break;
            }
            //Begin splitting lines, adding to array.
            string[] split2 = line.Split(delimiter, StringSplitOptions.RemoveEmptyEntries);

            //Loop to hold the first word in the stream
            for(int i = 0; i <= 0; i++) {
                word = split2[i];

                //loop to hold the following words in to list.
                for (int y = 1; y < split2.Length; y++) {
                    theList.Add(split2[y]);
                }

                //Add word/list combo in to the database
                theDatabase.Add(word, theList);

                //clear the list.
                theList.Clear();
            }
        }
    }

    foreach (KeyValuePair<string, List<string>> pair in theDatabase) {
        string keys;
        List<string> values;

        keys = pair.Key;
        values = pair.Value;
        print(keys + " = " + values);

    }
  }
}

底部的foreach循环就是这样，我可以看到结果。此外，由于我是初学者，因此欢迎任何关于如何写作的批评。

Answer 1

是的，您正在将相同的对象添加到字典中。

你可以改变：

theDatabase.Add(word, theList);

致：

theDatabase.Add(word, theList.ToList());

方法ToList()制作了List<T>

的浅层副本

Answer 2

C＃以引用方式传递。
因此，theList和Dictionary中的列表是同一个对象。

最简单的解决方案是停止清除List并每次创建一个新的

for(int i = 0; i <= 0; i++) {
    List<string> theList = new List<string>(); // it is in a loop now

    word = split2[i];

    //loop to hold the following words in to list.
    for (int y = 1; y < split2.Length; y++) {
        theList.Add(split2[y]);
    }

    //Add word/list combo in to the database
    theDatabase.Add(word, theList);

    //clear the list.
    //theList.Clear(); - not required anymore
}

更具可读性和清晰度的解决方案：创建列表，插入项目，将列表粘贴到字典中，继续迭代。
由于没有List清除，它的性能也更高 - List<T>.Clear()是一个线性操作，需要O(n)个操作。

Answer 3

是的，正如大家所说，列表是参考类型。您需要制作副本以避免.Clear()清除所有列表。

您可以随时编写代码：

void Start()
{
    string delimStr = ",.:";
    Dictionary<string, List<string>> theDatabase = 
        File
            .ReadAllLines("testComma.csv")
            .Select(line => line.Split(delimStr.ToCharArray(), StringSplitOptions.RemoveEmptyEntries))
            .ToDictionary(x => x[0], x => x.Skip(1).ToList());

    /* foreach here */
}

}

这对列表引用没有问题。

清除List <string>将从字典的字典列表中删除所有值，C＃

3 个答案: