列表项目删除问题

Question

不要害怕大量的代码。问题很普遍。我只是提供代码以更好地理解问题。

我试图找到一种处理具有多对多关系的表的标准方法。我快完成了。这里Teacher和Course有M:M关系。我的课程设计如下：

Teacher - class：

public class Teacher
{
    public int ID{get;set;}
    public string TeacherName{get;set;}
    private List<Course> _items = null;
    public List<Course> Items
    {
        get 
        {   if (_items == null) {_items = Course.GetCoursesByTeacherID(_ID);}
            return _items;
        }
        set {_items = value;}
    }
    public int Save() 
    {   //...
        CourseTeacher.DeleteCoursesByTeacherID(tc, id);
        CourseTeacher.SaveCoursesWithTeacherID(tc, id, this.Items);
        //...
    }
    public bool Update()
    {   //...
        CourseTeacher.DeleteCoursesByTeacherID(tc, this.ID);
        CourseTeacher.SaveCoursesWithTeacherID(tc, this.ID, this.Items);
        //...
    }
    public static Teacher Get(int id)
    {   //...
        item.Items = CourseTeacher.GetCoursesByTeacherID(tc, item.ID);//...
    }
    public static List<Teacher> Get()
    {   //...
        items[i].Items = CourseTeacher.GetCoursesByTeacherID(tc, items[i].ID);//...
    }
    public static List<Teacher> GetTeachersByCourseID(int id)
    {   //...
        items = CourseTeacher.GetTeachersByCourseID(tc, id);//...
    }
    public bool Delete()
    {   //...
        CourseTeacher.DeleteCoursesByTeacherID(tc, this.ID);//...
    }
}

Course 与 Teacher - class 绝对相似。映射类如下：

public class CourseTeacher
{
    public int CourseID{get;set;}
    public int TeacherID{get;set;}  
    public static void SaveCoursesWithTeacherID(TransactionContext tc, int teacherID, List<Course> items){}
    public static void SaveTeachersWithCourseID(TransactionContext tc, int courseID, List<Teacher> items){}
    private void Save(TransactionContext tc){}
    public static void DeleteCoursesByTeacherID(TransactionContext tc, int teacherID){}
    public static void DeleteTeachersByCourseID(TransactionContext tc, int courseID){}
    public static List<Teacher> GetTeachersByCourseID(TransactionContext tc, int courseID){}
    public static List<Course> GetCoursesByTeacherID(TransactionContext tc, int teacherID){}
}

现在我的问题是，这段代码不起作用？

Teacher professorXyz = Teacher.Get(2);                        
Course cpp = Course.Get(3);
Course java = Course.Get(2);
professorXyz.Items.Remove(cpp);
professorXyz.Items.Remove(java);
professorXyz.Update();

这不起作用，因为它可能找不到匹配项或获取访问器返回只读列表。

我应该如何重构我的教师/课程类来实现这一目标？

也不例外。持久化代码没有问题。项目不会被删除。

为什么 professorXyz.Items.Contains(cpp); 返回 false？

需要检查什么？

Answer 1

这不是一个直接的答案，但是......

你的设计是非常（非常）相关的。这使得持久化到数据库变得更容易，但你没有合适的面向对象模型。也许您应该考虑在 DataSet 中使用 DataTable 并获得 Relation 类的好处。

---

试一下：

Teacher professorXyz = Teacher.Get(2);                        
Course cpp = Course.Get(3);

我怀疑 cpp 课程被加载了两次，并且内存中有该课程的 2 个实例。你的设计造成了非常糟糕的后果。默认情况下，这两个实例不相等，这就是 Remove 不起作用的原因。您可以重载 Equals、== 和 GethashCode 但这是对于可变类型不推荐。
您真正需要的是一种设计，其中对于给定的教师或课程，内存中永远不会存在超过 1 个实例。

评论：OO 中的 MxM 关系如下所示：

class Teacher
{
   public readonly List<Course> Courses = ...;
}

class Course
{
   public readonly List<Teacher> Teachers = ...;
}

这将需要更多的工作来写入数据库，但它解决了许多其他问题。

Answer 2

你想做什么？您的示例看起来像是您想要构建一个用 C# 实现的关系数据库表。

如果你想要一个面向对象的表示，那么就去掉整个 CourseTeacher 类。这与 OO 完全无关。

Answer 3

似乎您已经解决了这个问题，但请考虑以下代码，其中我覆盖了 bool Equals; C# 无法知道如何将新的 cpp 实例与 List 中的另一个实例进行比较，因此我们需要通过创建更专门的 来告诉它等于方法：

class Teacher
{
    private List<Course> items = new List<Course>();

    public int ID { get; set; }
    public List<Course> Items { get { return items; } }
}

class Course
{
    public int ID { get; set; }

    public override int GetHashCode()       { return base.GetHashCode(); }
    public override bool Equals(object obj) { return Equals(obj as Course); }
    public bool Equals(Course another)
    {
        return another != null && this.ID.Equals(another.ID);
    }
} 

static void Main(string[] args)
{
    Teacher teacher = new Teacher { ID = 2 };
    teacher.Items.AddRange(
        new Course[] { 
            new Course{ ID = 2 },       // java
            new Course{ ID = 3 } });    // cpp

    Course cpp = new Course { ID = 3 }; // previous problem: another instance
    teacher.Items.Contains(cpp);        // now returns true
    teacher.Items.Remove(cpp);          // now returns true
}

Answer 4

亨克是正确的；你的设计是非常非常相关的。不过，对于这种情况，您最好关注对象中的行为，并使用对象关系映射 (ORM) 工具在对象和数据库之间进行转换。

ADO.NET 的 DataTable 和 DataSet 并不真正提供对象关系映射功能；而是提供对象关系映射功能。因为它们与底层数据库模式紧密耦合，所以当您真正想从教师和课程的角度思考时，它们会迫使您从列、表和关系的角度进行思考。

对于这种情况，我强烈建议您查看 Castle ActiveRecord 。它使用与您的示例相同的方法 - static Teacher.Get() 来检索实例，myTeacher.Save() 来保存您的更改 - 但是您的示例缺少很多必要的复杂性，并且使用 ORM 框架将允许您可以忽略这种复杂性并专注于您自己项目的需求。

这是多对多关联的示例 Castle ActiveRecord 文档可能会对您有所帮助。

Answer 5

添加和删​​除在教师班级中完成怎么样？

public class Teacher
{
        //.... Original implementations
    public bool AddCourse(Course course) {
        if(_items.Contains(course)) return false;

        _items.Add(course);
        return true;
    }

        // similarly for remove course

}