从C＃中的Form2调用Form1中的Function

Question

我在Form1中具有以下功能，并且在单击Form2中的按钮时想调用它。

   public object GetStudents()
    {
        using (var DB = new myView1())
        {
            var studentList = (from s in DB.View123
                            select new { s.StudentName, s.StudentNumber, s.ClassName, s.StreamName, s.ParentName }).ToList();
            dataGridViewStudents.DataSource = students;
            return students.ToList();
        }
    }

上面的函数在Form1中，如何从Form2中的button_Click事件中调用它。 这在VB.NET中很简单

Answer 1

您需要给您的Form2实例一个对Form1实例的引用，这时它就像form1.GetStudents()一样简单。

如果Form1是您的主要表单，则只需执行

var students = ((Form1)ApplicationContext.MainForm).GetStudents();

但是，此代码的某些部分散发出难闻的代码气味：

• Form内的数据库代码（这实际上不是表单的责任）
• 返回object而不是IList
• 不清楚什么ApplicationContext.MainForm是没有上下文的（推论：不要这样做）

恕我直言，您应该借此机会重构您现有的代码，并使其与可接受的惯例保持一致。

Answer 2

在C＃中也很简单。 您只需要一个Form1实例，然后可以在该实例上调用此方法。

但是，有更好的方法。 该代码实际上并不属于表单。 表单用于UI交互，而不用于业务逻辑和数据访问。 最好将这段代码移到两个表格都可以访问的公共位置。

创建一个数据访问类。 像这样：

public class StudentRepository
{
    public static object GetStudents()
    {
        using (var DB = new myView1())
        {
            var studentList = (from s in DB.View123
                        select new { s.StudentName, s.StudentNumber, s.ClassName, s.StreamName, s.ParentName }).ToList();
            dataGridViewStudents.DataSource = students;
            return students.ToList();
        }
    }
}

还要注意，该方法现在是static ，这意味着它不需要调用实例。 （这类似于VB中的Shared关键字。）因此，您的表单可以仅调用：

var students = StudentRepository.GetStudents();

您要解决的另一件事是返回类型，尽管它不在此问题的范围内。 object不是很具体。 您应该有一个Student类，并且该方法应返回IList<Student> 。 像这样：

public class Student
{
    public string StudentName { get; set; }
    public int StudentNumber { get; set; }
    public string ClassName { get; set; }
    public string StreamName { get; set; }
    public string ParentName { get; set; }
}

和：

public class StudentRepository
{
    public static IList<Student> GetStudents()
    {
        using (var DB = new myView1())
        {
            var studentList = (from s in DB.View123
                        select new Student { StudentName = s.StudentName, StudentNumber = s.StudentNumber, ClassName = s.ClassName, StreamName = s.StreamName, ParentName = s.ParentName }).ToList();
            dataGridViewStudents.DataSource = students;
            return students.ToList();
        }
    }
}

我对字段的类型做了一些假设，如果它们是错误的，您应该能够纠正它们。 如果结果不变（也就是说，如果它只是学生的枚举，而不是要添加到列表中或从列表中删除的列表，则可能甚至希望它返回IEnumerable<Student>而不是IList<Student> ）。

您还可以在这里做更多的事情，例如为其他数据元素（例如Class （可能需要一个不同的名称，只是为了干净））， Parent ， Stream等等创建一个对象。这可能需要更好的名字。一般你的变量命名需要一点的工作。喜欢的东西Form1或myView1不传达的意图非常好。）

Answer 3

我没有使用C＃，但是它应该类似于VB.Net，因为它仍在同一框架上。 您是否尝试过将函数放在单独的资源文件中并将其添加到项目中？ 您似乎无法调用该函数的原因似乎是由于它们格式不同而引起的范围问题。 资源文件应注意这一点，并且由于它减少了混乱，因此往往使代码更易于维护。