0

我有一个库存界面的特定部分，要求员工从组合框中选择他或她的姓名，然后将产品扫描到分配给员工姓名的表中。

我的好奇心是：当点击EDIT, ADDORDELETE按钮时，它知道从带有该员工姓名的Switch - Case语句中执行此功能的表。问题是，对于每个员工来说，这段代码都很长，尤其是对于每个都有Switch - Case语句的 9 名员工来说。

关于如何简化或缩短代码的任何建议？我确实提前了解了我无法使用的参数化 SQL 。只是想先完成这个。

private void btnAdd_Click(object sender, EventArgs e)
    {
        ActiveControl = txtSerialN;
        if (!string.IsNullOrEmpty(txtSerialN.Text) && !string.IsNullOrEmpty(cboEmpName.Text))

            switch (cboEmpName.SelectedItem.ToString().Trim())
            {
                case "John Doe":
                    try
                    {
                        connection.Open();
                        OleDbCommand command = new OleDbCommand();
                        command.Connection = connection;

                        command.CommandText = "INSERT INTO JohnDoe(SerialNumber,PartNumber,DateEntered,Customer) values ('" + txtSerialN.Text + "','" + txtPart.Text + "','" + txtDate.Text + "','" + txtCustomer.Text + "')";
                        command.ExecuteNonQuery();
                        MessageBox.Show("Inventory Added".PadLeft(23));

                        connection.Close();
                        txtSerialN.Clear();
                        txtPart.Clear();
                        txtDate.Clear();
                        txtCustomer.Clear();

                        command.CommandText = "SELECT * FROM JohnDoe ORDER BY PartNumber";
                        OleDbDataAdapter db = new OleDbDataAdapter(command);
                        DataTable dt = new DataTable();
                        db.Fill(dt);
                        dataGridEmpParts.DataSource = dt;
                    }

                    catch (OleDbException)
                    {
                        string strmsg = "THIS SERIAL NUMBER ALREADY EXISTS ! , Please try again";
                        MessageBox.Show(strmsg, "YOU CAN'T ENTER THE SAME ONE AGAIN", MessageBoxButtons.OK,
                        MessageBoxIcon.Exclamation,
                        MessageBoxDefaultButton.Button1);
                        connection.Close();
                    }
                    break;
            }
    }
4

2 回答 2

1

我宁愿建立一个查找表，其中包含 EmployeeName、AssignedTable 等列，并根据参数值动态构造命令文本。

于 2016-06-07T01:19:56.857 回答
0

我怀疑这个问题可以通过更改数据库更有效地解决。甚至可能就像为员工姓名添加一个字段一样简单。

于 2016-06-07T02:51:14.077 回答