C# WinForms传递MySQL表名变量引发SQL语法错误求助

我一眼就看出问题所在了——你的Form1构造函数执行时，DatabaseTableName属性还没被赋值！

梳理下代码的执行顺序你就明白了：

• 在MainWindow的菜单点击事件里，你先执行new Form1()，这会立刻触发Form1的构造函数
• 构造函数里直接调用了PopulateDataGridView()，但此时你还没执行后面的form.DatabaseTableName = "products";，DatabaseTableName还是默认的null
• 拼接出来的SQL语句变成了SELECT * FROM ，这完全不符合MySQL语法，自然就抛出了那个错误

下面给你几个可行的解决方案：

方案1：使用带参数的构造函数 #

给Form1新增一个接收表名的构造函数，创建实例时直接传入表名，确保PopulateDataGridView执行时变量已有有效值：

public partial class Form1 : Form { 
    private DataTable dt; 
    private BindingSource bs; 
    public string DatabaseTableName { get; set; } 

    // 保留无参构造函数（WinForms设计器需要）
    public Form1() { 
        InitializeComponent(); 
        bs = new BindingSource(); 
    }

    // 新增带表名参数的构造函数
    public Form1(string tableName) : this() {
        DatabaseTableName = tableName;
        this.PopulateDataGridView(); 
    }

    private void PopulateDataGridView() { 
        string query = String.Format("SELECT * FROM {0}", DatabaseTableName); 
        DataTable data = GetData(query); 
        bs.DataSource = data; 
        dataGridView1.DataSource = bs; 
        bindingNavigator1.BindingSource = bs; 
    }

    // 其余GetData等代码保持不变...
}

然后修改MainWindow里的调用逻辑：

private void listaKupacaToolStripMenuItem_Click(object sender, EventArgs e) { 
    // 直接传入表名创建Form1实例
    Form1 form = new Form1("products"); 
    form.ShowDialog(); 
}

方案2：将数据加载逻辑移到窗体Load事件 #

把PopulateDataGridView()的调用从构造函数移到Form1的Load事件中，这样你给DatabaseTableName赋值后，窗体加载时才会执行数据加载：

修改Form1的构造函数：

public Form1() { 
    InitializeComponent(); 
    bs = new BindingSource(); 
    // 移除构造函数里的PopulateDataGridView调用
    // 绑定Load事件
    this.Load += Form1_Load;
}

private void Form1_Load(object sender, EventArgs e) {
    this.PopulateDataGridView();
}

这种方案下，MainWindow的代码可以完全保持不变，因为赋值操作会在窗体加载之前完成。

额外提醒：SQL注入风险 #

虽然你现在是通过菜单点击固定赋值表名，相对安全，但要注意：直接用字符串拼接表名存在SQL注入风险。如果未来这个表名需要来自用户输入，一定要做严格的白名单验证（比如检查是否是预先允许的表名），避免恶意输入破坏数据库。

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