C# - 使用 foreach 比较两个对象列表以进行更改

Question

对于我的课堂作业，我需要将旧的学生列表与新列表进行比较，并将新学生、删除的学生和更改的学生添加到单独的列表中。教师指定使用嵌套的 foreach 循环而不是 LINQ，但我的问题是一旦旧学生列表匹配新学生中的条目并移动到旧列表中的下一个学生，就会打破循环。

我的代码现在通过嵌套的 foreach 运行，将条目与旧列表中的第一个条目进行比较，结果出现没有 ID 匹配的结果，因此它将它们放入已删除的列表中并结束循环而不移动到旧列表中的下一个学生。

public static void CompareStudents(List<Student> oldList, List<Student> newList)
    {
        foreach (Student o in oldList)
        {
            foreach (Student n in newList)
            {
                if (FindStudent(o.ID, n.ID))
                {
                    if (CheckChanges(o, n))
                    {
                        changed.Add(n);
                        break;
                    }
                }
                else
                {
                    removed.Add(o);
                    break;
                }
            }
        }
    }

    private static bool FindStudent(string oldID, string newID)
    {
        if (newID.Equals(oldID))
        {
            return true;
        }
        else
        {
            return false;
        }
    }

    public static bool CheckChanges(Student oldStu, Student newStu)
    {
        if (oldStu.FirstName.Equals(newStu.FirstName) && 
            oldStu.LastName.Equals(newStu.LastName) &&
            oldStu.StudentYear.Equals(newStu.StudentYear) &&
            oldStu.StudentRank.Equals(newStu.StudentRank))
        {
            return false;
        }
        else
        {
            return true;
        }
    }

Answer 1

如果您的 Student 类覆盖了 Equals 方法，那么您可以执行以下操作：

public static void CompareStudents(List<Student> oldList, List<Student> newList)
{
    List<Student> added = new List<Student>();
    List<Student> removed = new List<Student>();
    List<Student> changed = new List<Student>();

    foreach(Student n in newList){
        // the added list is a subset of the newList so we begin by cloning the newList
        added.Add(n);
    }

    foreach (Student o in oldList)
    {
        bool existsInNewList = false;
        // we remove every o from the added list
        added.Remove(o);

        foreach (Student n in newList)
        {
            if (o.ID.Equals(n.ID))
            {
                // o and n have the same Id
                existsInNewList = true;
                if (!o.Equals(n))
                {
                    // o and n have the same Id but are different
                    changed.Add(n);
                    added.Remove(n);
                }
                // eventually add a break; here so you don't loop after you've found a n that matches o
            }
        }
        if(!existsInNewList){
            // none of the newStudents have the same Id as o
            removed.Add(o);
        }
    }
}

最后，您应该将所有三个列表 added、removed 和 changed 填入正确的 Students。

Answer 2

使用旗帜怎么样？

        foreach (Student o in oldList)
        {
            bool flag = false;
            foreach (Student n in newList)
            {
                if (FindStudent(o.ID, n.ID))
                {
                    if (CheckChanges(o, n))
                    {
                        changed.Add(n);
                        flag = true;
                        break;
                    }
                }
                else
                {
                    removed.Add(o);
                    flag = true;
                    break;
                }
            }
            if(flag) continue;
        }

Answer 3

我认为你应该使用如下代码：

List<Student> OldStudents = new List<Student>();
List<Student> NewStudents = new List<Student>();
List<Student> StudentsEdit = new List<Student>();

 foreach (var oStud in OldStudents)
     {
        foreach (var nStud in NewStudents)
        {
          if (oStud != nStud)
            StudentsEdit.Add(oStud);
        }
     }

Instructor 建议我解释 LINQ，我的错， 希望这会有所帮助。

问候，

N 鲍阿