C# Linq代码工作不正常

我编写了以下代码，以便在网格视图中组合选中复选框（true）的所有行的列值“lbljudiction”

if (grdView.Rows.Count > 0)
{
    foreach (GridViewRow row in grdView.Rows)
    {
        CheckBox chkbox = row.FindControl("chkbox") as CheckBox;
        Label lblJurisdiction = row.FindControl("lblJurisdiction") as Label;
        bool saveThis = chkbox.Checked;
        if (saveThis == true)
        {
            List<String> Items = new List<String>();
            Items.Add(lblJurisdiction.Text);
            Items.Sort();
            List<string> Unique = Items.Distinct().ToList();
            string ReplacedJurisdiction = string.Join(",", Unique.ToArray());
            hdnJurisdiction.Value = ReplacedJurisdiction;
        }
    }
}

if（grdView.Rows.Count>0）
{
foreach（grdView.Rows中的GridViewRow行）
{
复选框chkbox=row.FindControl（“chkbox”）作为复选框；
Label LBLWITHORITION=行.FindControl（“LBLWITHORITION”）作为标签；
bool saveThis=chkbox.Checked；
if（saveThis==true）
{
列表项=新列表（）；
添加项（lbljudiction.Text）；
Items.Sort（）；
List Unique=Items.Distinct（）.ToList（）；
string ReplacedJurisdiction=string.Join（“，”，Unique.ToArray（））；
hdnJurisdiction.Value=ReplacedJurisdiction；
}
}
}

例如，网格视图包含

• [对][阿拉巴马州][有些价值]
• [正确][阿拉斯加][有些价值]
• [假][纽约][一些价值]
• [假][加州][部分价值]

现在隐藏的区域应该包括阿拉巴马州和阿拉斯加

---

它只合并了一个，即阿拉斯加

您应该连接字符串，现在您要在隐藏字段中覆盖它们

if(hdnJurisdiction.Value.Length > 0 && ReplacedJurisdiction != "")
   hdnJurisdiction.Value += ", ";

hdnJurisdiction.Value += ReplacedJurisdiction;

---

您应该连接字符串，现在在隐藏字段中覆盖它们

if(hdnJurisdiction.Value.Length > 0 && ReplacedJurisdiction != "")
   hdnJurisdiction.Value += ", ";

hdnJurisdiction.Value += ReplacedJurisdiction;

---

你应该改变你的变量范围！您应该在

foreach

循环之外声明列表，并在完成计算后设置值。我还更改了您的代码，以匹配C#中常见的命名和编码约定

if（grdView.Rows.Count>0）
{
var states=新列表（）；
foreach（grdView.Rows中的GridViewRow行）
{
var chkbox=row.FindControl（“chkbox”）作为复选框；
如果（chkbox.Checked）
{
var lbljudiction=row.FindControl（“lbljudiction”）作为标签；
添加（lbljudiction.Text）；
}
}
hdnJurisdiction.Value=string.Join（“，”，states.Distinct（）.OrderBy（x=>x））；
}

您应该更改变量范围！您应该在

foreach

循环之外声明列表，并在完成计算后设置值。我还更改了您的代码，以匹配C#中常见的命名和编码约定

if（grdView.Rows.Count>0）
{
var states=新列表（）；
foreach（grdView.Rows中的GridViewRow行）
{
var chkbox=row.FindControl（“chkbox”）作为复选框；
如果（chkbox.Checked）
{
var lbljudiction=row.FindControl（“lbljudiction”）作为标签；
添加（lbljudiction.Text）；
}
}
hdnJurisdiction.Value=string.Join（“，”，states.Distinct（）.OrderBy（x=>x））；
}

原始代码根本不使用LINQ。选择所有选中行的标签的LINQ查询可以是：

var labels= grdView.Rows.OfType<GridViewRow>() 
               .Where(row=>(row.FindControl("chkbox") as CheckBox)?.Checked)
               .Select(row=>(row.FindControl("lblJurisdiction") as Label)?.Text)
               .Distinct()
               .OrderBy(x=>x);

---

原始代码根本不使用LINQ。选择所有选中行的标签的LINQ查询可以是：

var labels= grdView.Rows.OfType<GridViewRow>() 
               .Where(row=>(row.FindControl("chkbox") as CheckBox)?.Checked)
               .Select(row=>(row.FindControl("lblJurisdiction") as Label)?.Text)
               .Distinct()
               .OrderBy(x=>x);

---

我们不是通灵者。我刚才问这个问题是为了解决这个问题，这是stackover flow Deverepers的工作。不是。你在一个免费的网站上，要求人们放弃他们的时间，免费帮助你。这里没有工作。一点谦虚会让你走很长的路。在你的问题中提供完整的信息也是如此。这不是我们的工作！我们在帮助你@NIDAW LINQ代码在哪里？这只是一个循环，我们不是通灵者。我问这个问题是为了解决这个问题，这个问题是叠加流导出器的工作。不是。你在一个免费的网站上，要求人们放弃他们的时间，免费帮助你。这里没有工作。一点谦虚会让你走很长的路。在你的问题中提供完整的信息也是如此。这不是我们的工作！我们在帮助你@NIDAW LINQ代码在哪里？这只是一个

foreach

循环。更好的选择是实际使用LINQ，选择所有标签文本，然后使用Join。这将避免创建单独的列表更好的选择是实际使用LINQ，选择所有标签文本，然后使用Join。这将避免创建单独的列表

var text=String.Join(labels);