复选框列表选择问题

Question

有5个目标和5名员工。每个目标都可以分配给这 5 名员工中的任意数量。因此，我为每个目标设置了 5 个 CheckBoxList，每个 CheckBoxList 都将这 5 名员工的姓名作为项目。

我想从数据库中检索哪些员工已分配了哪些目标。我有以下代码：

            List<CheckBoxList> checkboxlists = new List<CheckBoxList>();
            checkboxlists.Add(CheckBoxList1);
            checkboxlists.Add(CheckBoxList2);
            checkboxlists.Add(CheckBoxList3);
            checkboxlists.Add(CheckBoxList4);
            checkboxlists.Add(CheckBoxList5);


            for (int z = 1; z <= checkboxlists.Count; z++)
            {
                SqlCommand check = new SqlCommand("SELECT ISGoal1, ISGoal2,ISGoal3, ISGoal4,ISGoal5 FROM PRM2011_EMPLOYEE_GOAL WHERE EmployeeID = '" + employeeid[z - 1] + "'", con);
                SqlDataReader y = check.ExecuteReader();

                y.Read();

                for (int j = 1; j <= 5; j++)
                {
                    if (null != y && y.HasRows)
                    {
                        string yes_or_no = y["ISGoal" + j].ToString().Trim();
                        if (yes_or_no == "Yes")
                        {
                            checkboxlists[j-1].Items[z-1].Selected = true;

                        }
                        //else checkboxlists[j - 1].Items[z - 1].Selected = false;
                    }
                }

                y.Close();
           }

我的问题是，即使我为一名员工选择一个目标，与该特定员工对应的所有复选框都会被选中。为什么会发生这种情况？

相应地，如果我注释掉发布的代码中的 else 部分，并且未选择任何目标，则与该员工对应的所有复选框都将被取消选中。请帮忙。

Answer 1

一些想法：

1. 让 for 循环从 0 到小于 checkboxlists.Count 以及从 0 到小于 5。这样您就可以避免在各处处理所有减法。
2. 在 checkboxlists[j-1].Items[z-1].Selected = true 行上，不应该是 checkboxlists[z-1].Items[j-1]。 Selected = true 因为我假设您正在使用 z 来迭代您的 CheckBoxLists。

现在已经很晚了，所以我的大脑可能有点模糊，但似乎#2 可能是你的问题。尝试一下这些想法，如果您仍然遇到问题，我会与您联系。

Answer 2

您的代码中有很多问题，我将告诉您主要问题，然后列出代码的其他缺陷。

造成这种情况的原因是：

for 循环外部存在方法y.Read();。
因为Read();的功能是读取数据库中的下一行。所以基本上你的代码读取第一个值，让我们假设该值为“Yes”，这样它将导致 ListBox 检查员工，而不是再次调用 y.Read(); 以便它移动到下一行，它不会！ ..因此该值保持为“是”，因此列表中的所有复选框都将被选中。

解决方案：

很简单，只需将 y.Read(); 从循环外部移入循环即可。

像这样：

        List<CheckBoxList> checkboxlists = new List<CheckBoxList>();
        checkboxlists.Add(CheckBoxList1);
        checkboxlists.Add(CheckBoxList2);
        checkboxlists.Add(CheckBoxList3);
        checkboxlists.Add(CheckBoxList4);
        checkboxlists.Add(CheckBoxList5);


        for (int z = 1; z <= checkboxlists.Count; z++)
        {
            SqlCommand check = new SqlCommand("SELECT ISGoal1, ISGoal2,ISGoal3, ISGoal4,ISGoal5 FROM PRM2011_EMPLOYEE_GOAL WHERE EmployeeID = '" + employeeid[z - 1] + "'", con);
            SqlDataReader y = check.ExecuteReader();


            for (int j = 1; j <= 5; j++)
            {
               y.Read();

                if (null != y && y.HasRows)
                {
                    string yes_or_no = y["ISGoal" + j].ToString().Trim();
                    if (yes_or_no == "Yes")
                    {
                        checkboxlists[j-1].Items[z-1].Selected = true;

                    }
                    //else checkboxlists[j - 1].Items[z - 1].Selected = false;
                }
            }

            y.Close();
       }

---

关于代码的额外注释

首先

您需要编辑SqlCommand以使用SqlParameters

SqlCommand check = new SqlCommand("SELECT ISGoal1, ISGoal2,ISGoal3, ISGoal4,ISGoal5 FROM PRM2011_EMPLOYEE_GOAL WHERE EmployeeID = @EmpID", con);

check.Parameters.AddWithValue("@ImpID", employeeid[z - 1]);

---

第二

如果如果你正在尝试构建一个真正的应用程序，那么这是一个非常糟糕的做法。即使您没有真正构建这个应用程序，我也不认为这是练习的方式。

for (int j = 1; j <= 5; j++)

您的循环应如下所示：

for (int j = 1; j <= checkboxlists.Count; j++)

---

Third

使用字符串来表示是/否值也是一种不好的做法。您应该将其用于所有 ISGoal 列数据库，数据类型BIT。因此，您需要将 C# 代码中的局部变量的 DateType 从 string 更改为 bool。

---

第四

checkboxlists[j-1].Items[z-1].Selected = true;

你应该像 Ryan 所说的那样进行切换，因为 z 表示 CheckBoxLists ， j 表示 a 的项目给定 CheckBoxList

所以它可能是这样的：

checkboxlists[z-1].Items[j-1].Selected = true;

注意：我一开始没有注意，我认为 [z-1] 是某种 LINQ 表达式 :D!! ..这是我的错，但我的意思是当我第一次开始编程时我曾经这样做过，但我仍然无法识别它..这不是我认为的最佳实践！ ..我的建议尊重使用的从零开始的编号。

---

最后

您不必每次都检查 y.Read(); 是否有效。是否为空。所以我认为这段代码更有意义，而且对循环的条件进行硬编码是一种非常糟糕的做法，因此我们将使用 while 循环并添加 int 类型的局部变量，然后我们将在我们的内部递增它循环代码，以便您可以使用它来访问 CheckBoxList 项目

                int j = 1;
                while (y.Read())
                {
                        string yes_or_no = y["ISGoal" + j].ToString().Trim();
                        if (yes_or_no == "Yes")
                        {
                            checkboxlists[j-1].Items[z-1].Selected = true;
                            //use our counter "j" here

                        }
                        //else checkboxlists[j - 1].Items[z - 1].Selected = false;
                        //use our counter "j" here
                        j++;
                    }

..祝你好运；）