嵌套的 foreach 循环只返回不同的

Question

我有一个数据库，其中每个电子邮件地址都应该属于一个唯一的客户，但我有很多重复项。我使用 sql 查询列出客户 ID、电子邮件地址对，每次出现多个客户 ID 映射到单个电子邮件地址。结果看起来像这样（更改地址以保护无辜者）

Customer ID   Email
101233        bob@myaddress.com
108993        bob@myaddress.com
113224        bob@myaddress.com
89223         mary@otherdomain.com
188223        mary@otherdomain.com

在 c# 中，我将其填充到一个名为 dt 的 DataTable 中，其中包含 722 行。我用它来制作第二个名为 distinctTbl 的 DataTable，它有 344 行，只包含不同的电子邮件地址：

DataTable distinctTbl = dt.AsDataView().ToTable(true, "Email");

我正在尝试使用嵌套循环为每个电子邮件地址创建一个整数列表（客户 ID）：

foreach (DataRow dr in distinctTbl.Rows)
{
    // for each email address:
    List<int> idNums = new List<int>();

    foreach (DataRow myRow in dt.Rows) 
    {
        // for every customerID / email pair in the original table
        if (myRow["Email"] == dr["Email"])
        {
            idNums.Add((int)myRow["CustomerID"]);
        }
    }

    // Do something with the List<int> before exiting outside loop
}

当我运行这段代码时，每个整数列表都包含一个值。该值是正确的，但每个电子邮件地址至少应该有两个。我已经进行了足够的调试以发现它始终正确识别第一个，但跳过任何后续匹配。我确定我遗漏了一些明显的东西，但是有人看到发生了什么吗？

Answer 1

放弃foreach 循环。

您可以使用 Linq 更轻松地获取您要查找的信息。

Dictionary<string, List<int>> emailIDs =
    dt.Rows.OfType<DataRow>()
           .GroupBy(row => (string)row["Email"])
           .ToDictionary(grp => grp.Key,
                         grp => grp.Select(row => (int)row["CustomerID"]).ToList());

Answer 2

一种快速简便的解决方案是使用Dictionary<string,List<int>> 而不是列表：

    Dictionary<string, List<int>> idNums = new Dictionary<string, List<int>>();
    foreach (DataRow myRow in dt.Rows)
    {
        string email = myRow["Email"].ToString()
        if (idNums.ContainsKey(email))
        {
            idNums[email].Add((int)myRow["CustomerID"]);
        }
        else
        {
            idNums.Add(email, new List<int> { (int)myRow["CustomerID"] });
        }
    }

现在 idNums 将包含与每封电子邮件关联的 id 列表。