1

似乎我无法在我的域服务中的 include 上放置 where 子句 - 所以我们正在做一些循环来获取一些查找组所需的数据。

   var _lookupGroups = _lookupGroupRepository.All();
        var _lookupValues = _institutionLookupValueRepository.All().Where(x => x.InstitutionID == _userProfile.InstitutionID);

        int i = 0;

        foreach (var _group in _lookupGroups)
        {
            var _values = _lookupValues.Where(x => x.LookupGroupID == _group.LookupGroupID);

            foreach (var _value in _values)
            {
                _group.InstitutionLookupValues.Add(_value);

                i++;
            }

            Console.WriteLine(_group.GroupName + " " + i.ToString());
        }

        return _lookupGroups;

我输入的计数器是为了验证我所看到的——我们迭代的第一组是状态。我的计数器会说 50，但我的导航属性会说 100，显示我们拥有的每个机构的一组 50（我们有 2 组）。所以它正在获取所有查找值并将它们放入导航属性中，即使我的计数器显示为 50。我还删除了元数据文件中的所有包含。

澄清一下：我有附加到lookupGroups的lookupvalues。查找值分配给机构。因此，如果我去获取 STATES 查找组的值，我将获得 50 个查找值。上面的代码。当我运行此行时 _group.InstitutionLookupValues.Add(_value); 我的机构LookupValues = 100（无论机构如何，都采用所有lookupValues）。但我的计数器显示 50。在某个地方，我得到了所有查找值，不确定与特定机构相关联的值。

4

1 回答 1

3

我认为这是由于您正在访问修改后的闭包这一事实造成的，即使不是，您也不应该这样做。线

var _values = _lookupValues.Where(x => x.LookupGroupID == _group.LookupGroupID);

可能没有做你认为它正在做的事情（除非它是 C#5）。lambda 是一个委托，闭包将关闭变量，而不是值。这意味着您的代码_group.LookupGroupID将始终是枚举器最终迭代的最后一个值，因此您只会过滤 one 上的查找值_group.LookupGroupID。

尝试将块更改为：

foreach (var _group in _lookupGroups)
{
  var currentGroup = _group;    
  var _values = _lookupValues.Where(x => x.LookupGroupID == currentGroup.LookupGroupID);

  foreach (var _value in _values)
  {
     currentGroup.InstitutionLookupValues.Add(_value);

     i++;
  }

  Console.WriteLine(currentGroup.GroupName + " " + i.ToString());
}

我不确定这是否是问题的原因，但无论如何你都会得到其他奇怪的效果。通过在循环中创建自己的变量，可以避免关闭生成的枚举变量。

看看 Eric Lippert 的文章：关闭被认为有害的循环变量。令人惊讶的是，很少有人知道这个问题。

此外，折叠查询可能是个好主意：

var _lookupValues = _institutionLookupValueRepository.All().Where(x => x.InstitutionID == _userProfile.InstitutionID);

通过使用ToList()，或类似的。否则，您将为 中的每个循环重新评估此查询，_lookupGroups如果查询是针对数据库的，或者循环很大，这可能会对性能造成严重影响。

于 2012-04-11T22:11:11.127 回答