C#子例程使用疑问：static与public修饰符及跨窗体变量

疑问1：按钮点击调用类中子例程，用static void还是public？ #

取决于子例程的功能是否依赖类的实例：

• 如果子例程是不依赖类实例的通用操作（比如登录验证、工具方法），用public static修饰（可以是static void或带返回值的static方法），调用时直接通过类名访问（如User.Login(...)）。
• 如果子例程是属于类实例的专属操作（比如修改当前用户的积分），用public void修饰，调用前需要先创建类的实例（如var user = new User(); user.UpdatePoints(10);）。

疑问2：跨多个窗体使用的类变量，用public还是static？ #

优先选择public实例变量，配合实例共享的方式（比如创建全局单例实例、在窗体间传递实例），尽量避免用static：

• static变量是全局共享的，一旦赋值就会一直占据内存，容易导致状态混乱（比如多用户场景下数据串用），还可能引发内存泄漏。
• public实例变量更可控，你可以明确管理实例的生命周期，跨窗体共享时只需传递这个实例的引用即可。

关于登录逻辑的重构建议 #

把按钮点击事件中的登录逻辑重构到User类时，更适合做成static方法，因为登录验证不需要依赖已有的User实例，是一个独立的身份校验操作。同时建议把UI逻辑（如MessageBox）和业务逻辑分离，让User类只负责数据和业务处理。

重构后的代码示例 #

User类修改：

public class User
{
    public string Username;
    public string Password;
    public string FirstName;
    public string LastName;
    public string Gender;
    public int Age;
    public int TotalPoints;

    // 静态登录方法，返回验证结果及用户实例
    public static (bool IsValid, User? LoggedInUser) Login(string inputUsername, string inputPassword)
    {
        using (OleDbConnection connection = new OleDbConnection(@"Provider=Microsoft.ACE.OLEDB.12.0;Data Source=C:\Users\RAV21001310\OneDrive\Database1.accdb;"))
        {
            connection.Open();

            using (OleDbCommand command = new OleDbCommand("SELECT * FROM tblUser WHERE Username=@Username AND Password=@Password", connection))
            {
                command.Parameters.AddWithValue("@Username", inputUsername);
                command.Parameters.AddWithValue("@Password", inputPassword);
                
                using (OleDbDataReader reader = command.ExecuteReader())
                {
                    if (reader.Read())
                    {
                        // 读取数据库数据，创建User实例
                        User user = new User
                        {
                            Username = reader["Username"].ToString(),
                            Password = reader["Password"].ToString(),
                            FirstName = reader["FirstName"].ToString(),
                            LastName = reader["LastName"].ToString(),
                            Gender = reader["Gender"].ToString(),
                            Age = Convert.ToInt32(reader["Age"]),
                            TotalPoints = Convert.ToInt32(reader["TotalPoints"])
                        };
                        return (true, user);
                    }
                    return (false, null);
                }
            }
        }
    }
}

按钮点击事件修改：

private void button1_Click(object sender, EventArgs e)
{
    var (isValid, loggedInUser) = User.Login(username.Text, password.Text);

    if (isValid)
    {
        MessageBox.Show("Username and password is correct");
        // 示例：将登录用户实例保存为全局变量，供其他窗体使用
        // Global.CurrentUser = loggedInUser;
        // 打开主窗体并关闭当前登录窗体
        // new MainForm().Show();
        // this.Close();
    }
    else
    {
        MessageBox.Show("Username or password incorrect");
    }
}

补充说明 #

• 不要直接把按钮的private void方法放到User类里，因为按钮事件是UI层的逻辑，属于窗体类，而User类应该专注于用户数据和业务规则。
• 原代码中判断count > 1的场景，正常设计下用户名应该是唯一的，数据库可以给Username字段加唯一约束，避免出现重复数据。

内容的提问来源于stack exchange，提问作者Muhammad Ravat