Избегайте повторения при выполнении A Выберите из IEnumerable

Question

У меня есть IEumerable<XElement> под названием foo.

У меня есть этот код

foo.Select(x => new 
     { 
      bars = x.Descendants(namespace + "bar") != null 
       ? x.Descendants(namespace + "bar").Select(z => z.Value).ToArray() 
       : new string[0] 
     }) 
     .ToArray(); 

Как я могу написать это немного аккуратней? Я действительно не хочу, чтобы повторить эту часть

x.Descendants(namespace + "bar") 

Answer 1

Использование дополнительных Select:

foo.Select(x => x.Descendants(namespace + "bar")) 
    .Select(x => new 
      { 
       bars = x != null ? x.Select(z => z.Value).ToArray() : new string[0] 
      }) 
      .ToArray(); 

Или:

foo.Select(x => 
       { 
        var elements = x.Descendants(namespace + "bar"); 
        return new 
        { 
         bars = elements != null ? elements.Select(z => z.Value).ToArray() : new string[0] 
        } 
       }) 
      .ToArray(); 

BTW, я не думаю, что Descendants вернется null. Вероятно, вы должны проверить, есть ли какие-либо элементы, используя Any способ.

Answer 2

Вы можете использовать другое выражение лямбда, чтобы избежать повторения, как показано ниже, где вы также можете реализовать нулевую проверку.

Function<XElement,IEnumerable<XElement>> f = 
    x => x.Descendants(namespace + "bar") ?? Enumerable.Empty<XElement>(): 

foo.Select(x => new 
      { 
       f(x).Select(z => z.Value).ToArray(); 
      }) 
      .ToArray(); 

Answer 3

foo.Select(x => new 
    { 
     bars = x.Descendants(namespace + "bar") != null 
      ? x.Descendants(namespace + "bar").Select(z => z.Value).ToArray() 
      : new string[0] 
    }) 
    .ToArray(); 

Как я могу написать это немного аккуратней? Я действительно не хочу повторять эту часть

foo.Select(x => new { bars = x.Descendants(namespace + "bar") 
           .Select(z => z.Value).ToArray()}).ToArray(); 

Вам не нужно проверять NULL в качестве потомков возвращает отложила перечислимы, например, он всегда будет иметь экземпляр IEnumerable, это может быть пустой (нет результатов), но . Выберите (z => z.Value) .ToArray() автоматически произведет новую строку [0] {}, если результатов не будет.