C#中多单选按钮使用及Access数据库存储问题排查

嘿，我来帮你搞定这两个问题！先从单选按钮的使用说起，再解决你遇到的数据库插入异常~

一、如何在C#中正确使用多个单选按钮？ #

首先，单选按钮（RadioButton）要分组才能实现互斥选择——同一组内的单选按钮只能选一个，不同组的互不影响。你可以用GroupBox或者Panel来分组：

• 把同一类的单选按钮拖到同一个GroupBox/Panel里（比如性别、年级、状态各一个GroupBox）
• 这样用户选择时，同一组内自动互斥，不用额外处理

另外，你现在的写法是每个单选按钮写一个CheckedChanged事件，其实可以简化：

• 给同一组的单选按钮设置相同的事件处理方法，然后通过sender判断是哪个按钮被选中，再赋值对应的字符串。比如：

// 给所有性别单选按钮绑定同一个CheckedChanged事件
private void GenderRadio_CheckedChanged(object sender, EventArgs e)
{
    RadioButton rb = sender as RadioButton;
    if (rb != null && rb.Checked)
    {
        Gender = rb.Text; // 比如按钮Text是"Male"/"Female"，直接用这个值
    }
}

// 年级单选按钮同理
private void YearRadio_CheckedChanged(object sender, EventArgs e)
{
    RadioButton rb = sender as RadioButton;
    if (rb != null && rb.Checked)
    {
        year = rb.Text; // 按钮Text设为"1st"/"2nd"等
    }
}

这样能减少重复代码，维护起来更方便~

二、解决Access数据库插入异常的问题 #

看你的代码，发现两个关键问题：

1. INSERT语句遗漏了year字段：你说添加year单选按钮组后执行INSERT出错，但看你的button4_Click代码里，INSERT的字段列表里根本没有year，值列表也没加year变量！这就是直接赋值文本能保存，但用变量就出错的核心原因——你没把year字段加入插入语句里。
2. 严重的SQL注入风险：你用字符串拼接的方式生成SQL语句，不仅容易因为特殊字符（比如名字里有单引号）导致语法错误，还会被SQL注入攻击，必须改成参数化查询。

下面是修复后的完整代码，解决了这两个问题：

public partial class student : UserControl
{
    // 把连接字符串提出来，方便维护
    private readonly string connectionString = @"Provider = Microsoft.ACE.OLEDB.12.0; Data Source =C:\Users\ronaldcaymo\Desktop\Student Information System.accdb";
    private string Gender;
    private string year;
    private string Status;

    private void button4_Click(object sender, EventArgs e)
    {
        // 使用using语句自动释放连接，避免忘记关闭
        using (OleDbConnection con = new OleDbConnection(connectionString))
        {
            try
            {
                con.Open();
                // 参数化查询，避免SQL注入和语法错误
                // 注意：year是Access保留字，用方括号括起来避免冲突
                string insertSql = @"INSERT INTO table1(Lname,Fname,Mname,Course,Major,Minor,semester,Gender,Status,[year]) 
                                     VALUES(@Lname,@Fname,@Mname,@Course,@Major,@Minor,@Semester,@Gender,@Status,@Year)";
                using (OleDbCommand cmd = new OleDbCommand(insertSql, con))
                {
                    // 添加参数，OleDb的参数顺序要和SQL里的字段顺序一致
                    cmd.Parameters.AddWithValue("@Lname", tblname.Text);
                    cmd.Parameters.AddWithValue("@Fname", tbfname.Text);
                    cmd.Parameters.AddWithValue("@Mname", tbmname.Text);
                    cmd.Parameters.AddWithValue("@Course", cbcourse.Text);
                    cmd.Parameters.AddWithValue("@Major", cbmajor.Text);
                    cmd.Parameters.AddWithValue("@Minor", cbminor.Text);
                    cmd.Parameters.AddWithValue("@Semester", cbsemester.Text);
                    cmd.Parameters.AddWithValue("@Gender", Gender);
                    cmd.Parameters.AddWithValue("@Status", Status);
                    cmd.Parameters.AddWithValue("@Year", year);

                    cmd.ExecuteNonQuery();
                    MessageBox.Show("Student Added");
                }
            }
            catch (Exception ex)
            {
                // 捕获异常，方便排查问题
                MessageBox.Show($"插入出错：{ex.Message}");
            }
        }
    }

    // 简化的单选按钮事件处理（以性别为例，年级、状态同理）
    private void GenderRadio_CheckedChanged(object sender, EventArgs e)
    {
        RadioButton rb = sender as RadioButton;
        if (rb != null && rb.Checked)
        {
            Gender = rb.Text;
        }
    }

    private void YearRadio_CheckedChanged(object sender, EventArgs e)
    {
        RadioButton rb = sender as RadioButton;
        if (rb != null && rb.Checked)
        {
            year = rb.Text;
        }
    }

    private void StatusRadio_CheckedChanged(object sender, EventArgs e)
    {
        RadioButton rb = sender as RadioButton;
        if (rb != null && rb.Checked)
        {
            Status = rb.Text;
        }
    }
}

关键修复点： #

• 在INSERT语句里添加了[year]字段（用方括号避免保留字冲突）和对应的@Year参数
• 用using语句管理数据库连接和命令，确保资源自动释放，避免连接泄漏
• 改用参数化查询，彻底解决SQL注入问题，同时避免因特殊字符导致的语法错误
• 添加了异常捕获，方便你在出错时看到具体的错误信息，更容易排查问题

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