我可以/应该 Linqify 这个 foreach/if/foreach/if/invoke 代码吗？

Question

我认为代码可以是 Linq：

    foreach (var key in _dic.Keys) // for each observed key
        if (_changedKeys.Contains(key)) // if it changed
            foreach (var item in _dic[key]) // then for each observer
                if (_webSitePage.Session[key] != null) // , as long as the value is something
                    item(_webSitePage.Session[key]); // call the registered delegate

在周围代码的上下文中：

public class WebSiteContext
{
    private WebSitePage _webSitePage;

    internal void SetSessionValue<T>(string key, T value)
    {
        object current = _webSitePage.Session[key];
        if (current != null && current.Equals(value)) return;
        _webSitePage.Session[key] = value;
        _changedKeys.Add(key);
    }

    Dictionary<string, List<Action<object>>> _dic = new Dictionary<string, List<Action<object>>>();

    List<string> _changedKeys = new List<string>();

    internal void WhenSessionValueChanges(string key, Action<object> action)
    {
        if (!_dic.ContainsKey(key)) _dic[key] = new List<Action<object>>(); // create on demand
        _dic[key].Add(action);
    }

    internal void PageLoadComplete()
    {
        foreach (var key in _dic.Keys) // for each observed key
            if (_changedKeys.Contains(key)) // if it changed
                foreach (var item in _dic[key]) // then for each observer
                    if (_webSitePage.Session[key] != null) // , as long as the value is something
                        item(_webSitePage.Session[key]); // call the registered delegate

    }
}

public class WebSitePage : System.Web.UI.Page
{
    public WebSitePage()
    {
        this.WebSiteContext = new WebSiteContext(this);
    }

    public WebSiteContext WebSiteContext { get; set; }

    protected override void OnLoadComplete(EventArgs e)
    {
        base.OnLoadComplete(e);
        this.WebSiteContext.PageLoadComplete();
    }
}

Answer 1

它可以更改为 LINQ，但由于您调用 item(...) 是为了产生副作用，我认为在这里使用 LINQ 是不合适的，并且 foreach 循环更好。

不过你可以这样写：

 foreach (var key in _dic.Keys.Where(key => _changedKeys.Contains(key))
 {
     foreach (var item in _dic[key])
     {
         var value = _webSitePage.Session[key];
         if (value != null)
         {
             item(value);
         }
     }
 }

Answer 2

var keys = from key in _dic.Keys
           where _changedKeys.Contains(key) && _webSitePage.Session[key] != null
           from item in _dic[key]
           select item;

foreach (var key in keys)
    item(_webSitePage.Session[key]);

编辑：

以上是严重错误的。抱歉。我打算使用查询理解来回答这个问题。这是一个尝试：

var tuples = from key in _dic.Keys
       where _changedKeys.Contains(key) && _webSitePage.Session[key] != null
       from item in _dic[key]
       select new { key, item };

foreach (var t in tuples)
    t.item(_webSitePage.Session[t.key]);

Answer 3

这会缩短一点。请注意，我移动了 _webSitePage 检查，因为它不依赖于 item 的值：

foreach (var kvp in _dic.Where(kvp => !_changedKeys.Contains(kvp.Key) && _webSitePage.Session[kvp.Key] != null))
    foreach (var item in kvp.Values) // then for each observer
        item(_webSitePage.Session[key]); // call the registered delegate

Answer 4

var keys = _dic.Keys.Where(key => _changedKeys.Contains(key) && 
                                  (_webSitePage.Session[key] != null));
foreach (var key in keys)
  foreach (var item in _dic[key])
    item(_webSitePage.Session[key]);

（第二个测试不需要在内部 foreach 循环中，除非 item 修改 _webSitePage.Session）