c# - 在 SqlDataReader while-loop 中覆盖变量

问题描述

我已经阅读了一些似乎与此类似的线程，但找不到解决我的问题的方法，我没有太多使用堆栈溢出，所以请多多包涵

我有一个使用 SqlDataReader 的 while 循环，它从数据库中提取信息并将其放入开发请求列表中，如下所示

public ListOfDevelopmentRequestsModel GetDevRequests(List<SelectListItem> evaluators)
        {
            SqlCommand cmd = new SqlCommand(StoredProcedures.DevRequests.GetDevRequests, Conn);
            cmd.CommandType = CommandType.StoredProcedure;
            ListOfDevelopmentRequestsModel ListOfDevRequests = new ListOfDevelopmentRequestsModel();
            Conn.Open();
            SqlDataReader reader = cmd.ExecuteReader();

            while (reader.Read())
            {
                DateTime requestDate = Convert.ToDateTime(reader["DateCreated"].ToString());
                string requestorFirstName = reader["Staff First Name"].ToString();
                string requestorLastName = reader["Staff Last Name"].ToString();
                string requestorEmailAddress = reader["Staff Email"].ToString();
                string solutionName = reader["SolutionName"].ToString();
                string solutionDescription = reader["SoultionDescription"].ToString();
                string solutionElementName = reader["SolutionElementName"].ToString();
                string solutionElementDescription = reader["SolutionElementDescription"].ToString();
                string itemToChange = reader["ItemChange"].ToString();
                string changeDetails = reader["ChangeDetail"].ToString();

                List<SelectListItem> evaluatorList = new List<SelectListItem>(DisplayCurrentEvaluator(evaluators, evaluator));

                DevelopmentRequestModel DevRequest = new DevelopmentRequestModel
                {
                    RequestDate = requestDate,
                    RequestorName = $"{requestorFirstName} {requestorLastName}",
                    RequestorEmailAddress = requestorEmailAddress,
                    SolutionName = solutionName,
                    SolutionDescription = solutionDescription,
                    SolutionElementName = solutionElementName,
                    SolutionElementDescription = solutionElementDescription,
                    ItemToChange = itemToChange,
                    ChangeDetails = changeDetails,
                    AccordionHeading = $"{(changeID.PadLeft(4, '0'))} - {requestorFirstName} {requestorLastName} - {itemToChange}"
                };

                ListOfDevRequests.DevelopmentRequests.Add(DevRequest);
            }
            Conn.Close();

            return ListOfDevRequests;
        }

我还有一个获取请求评估者的列表

        public static List<SelectListItem> GetEvaluators()
        {
            List<SelectListItem> evaluators = new List<SelectListItem>();
            SqlCommand cmd = new SqlCommand(StoredProcedures.DevRequests.GetEvaluators, Conn);
            cmd.CommandType = CommandType.StoredProcedure;
            Conn.Open();
            SqlDataReader reader = cmd.ExecuteReader();
            while (reader.Read())
            {
                evaluators.Add(
                    new SelectListItem
                    {
                        Text = reader["Staff Name"].ToString(),
                        Value = reader["Staff Code"].ToString(),
                    });
            }
            Conn.Close();
            return evaluators;
        }

最后我有一个列表，它将传递上述评估者列表和从数据库中提取的评估者：string evaluator = reader["Evaluator"].ToString();并将根据评估者名称是否与文本值匹配设置选择列表的默认值，并将其设置为选中选择列表项。

        public List<SelectListItem> DisplayCurrentEvaluator(List<SelectListItem> evaluators, string evaluator)
        {
            foreach (var item in evaluators)
            {
                if (item.Text == evaluator)
                {
                    item.Selected = true;
                }
                else
                {
                    item.Selected = false;
                }
            }

            return evaluators;

        }

问题是循环中的第一项具有评估器“Bill”并且选择了“Bill”，并且工作正常，但是循环中的第二项是“John”，并且当它将“John”设置为选中时，它用“John”替换“Bill”作为第一项中的选定值

代码最终变得一团糟，因为我尝试了多种不同的方法来修复，但我很难过，希望能得到帮助。

抱歉，如果帖子格式不适合阅读，我可以尝试重新格式化并在需要时提供更多信息。

干杯

编辑代码：

GetDevRequests()

 public ListOfDevelopmentRequestsModel GetDevRequests(List<SelectListItem> evaluators)
        {
            SqlCommand cmd = new SqlCommand(StoredProcedures.DevRequests.GetDevRequests, Conn);
            cmd.CommandType = CommandType.StoredProcedure;
            ListOfDevelopmentRequestsModel ListOfDevRequests = new ListOfDevelopmentRequestsModel();
            Conn.Open();
            SqlDataReader reader = cmd.ExecuteReader();
            List<SelectListItem> evaluatorList = new List<SelectListItem>();

            while (reader.Read())
            {
                string changeID = reader["ChangeID"].ToString();
                string evaluator = reader["Evaluator"].ToString();
                string status = reader["Status"].ToString();
                string priority = reader["Priority"].ToString();
                string eliteID = reader["RequestorID"].ToString();
                DateTime requestDate = Convert.ToDateTime(reader["DateCreated"].ToString());
                string requestorFirstName = reader["Staff First Name"].ToString();
                string requestorLastName = reader["Staff Last Name"].ToString();
                string requestorEmailAddress = reader["Staff Email"].ToString();
                string solutionName = reader["SolutionName"].ToString();
                string solutionDescription = reader["SoultionDescription"].ToString();
                string solutionElementName = reader["SolutionElementName"].ToString();
                string solutionElementDescription = reader["SolutionElementDescription"].ToString();
                string itemToChange = reader["ItemChange"].ToString();
                string changeDetails = reader["ChangeDetail"].ToString();

                evaluatorList = DisplayCurrentEvaluator(evaluators, evaluator);

                DevelopmentRequestModel DevRequest = new DevelopmentRequestModel
                {
                    ChangeID = (changeID.PadLeft(4, '0')),
                    Evaluator = evaluator,
                    Evaluators = evaluatorList,
                    Status = status,
                    Priority = priority,
                    EliteID = eliteID,
                    RequestDate = requestDate,
                    RequestorName = $"{requestorFirstName} {requestorLastName}",
                    RequestorEmailAddress = requestorEmailAddress,
                    SolutionName = solutionName,
                    SolutionDescription = solutionDescription,
                    SolutionElementName = solutionElementName,
                    SolutionElementDescription = solutionElementDescription,
                    ItemToChange = itemToChange,
                    ChangeDetails = changeDetails,
                    AccordionHeading = $"{(changeID.PadLeft(4, '0'))} - {requestorFirstName} {requestorLastName} - {itemToChange}"
                };

                ListOfDevRequests.DevelopmentRequests.Add(DevRequest);
            }
            Conn.Close();

            return ListOfDevRequests;
        }

DisplayCurrentEvaluator()

        public List<SelectListItem> DisplayCurrentEvaluator(List<SelectListItem> selectListItems, string selectListDefaultItem)
        {
            foreach (var item in selectListItems)
            {
                item.Selected = item.Text == selectListDefaultItem;
            }

            return selectListItems;

        }

标签： c#while-loopsqldatareader

解决方案

问题出在这一行：

List<SelectListItem> evaluatorList = new List<SelectListItem>(DisplayCurrentEvaluator(evaluators, evaluator));

首先，这也可以写成

List<SelectListItem> evaluatorList = DisplayCurrentEvaluator(evaluators, evaluator);

您DisplayCurrentEvaluator已经返回了正确的列表，因此无需将其复制到新列表中。

但这是一个小问题，因为据我所知，您没有使用该 evaluatorList 。在该 while 循环的每次迭代中，您都在创建一个新的（这可能不是您想要的），然后您就忘记了它。我也看不到在哪里evaluator设置，但这可能是你没有显示的代码。

因此，您需要在循环之外生成一次此列表并保留它（可能在类级别的字段或属性中）。

还有一个额外的提示，DisplayCurrentEvaluator 方法也可以写成

public List<SelectListItem> DisplayCurrentEvaluator(List<SelectListItem> evaluators, string evaluator)
{
    foreach (var item in evaluators)
    {
        item.Selected = item.Text == evaluator;
    }

    return evaluators;
}

在显示设置evaluator并使用结果的代码后编辑evaluatorList

您的 DisplayCurrentEvaluator 更新原始评估者列表并将其返回。这有效地导致每个 evaluatorList 指向同一个列表，最后一个更新获胜。因此，请确保返回一个新列表。

public List<SelectListItem> DisplayCurrentEvaluator(List<SelectListItem> evaluators, string evaluator)
{
    var result = new List<SelectListItem>(evaluators.Count);

    foreach (var item in evaluators)
    {
        result.Add(new SelectListItem { Text = item.Text, Value = item.Value, Selected= item.Text == evaluator};      
    }

    return result;
}

此外，在循环内（仅）声明 evaluatorList。

 var evaluatorList = DisplayCurrentEvaluator(evaluators, evaluator);

c# - 在 SqlDataReader while-loop 中覆盖变量

问题描述

解决方案

推荐阅读