Pimp my LINQ: un ejercicio de aprendizaje basado en otra publicación

Question

Decidí probar LINQ por primera vez para intentar y resolver this question.

Los resultados de mi primera incursión en el maravilloso mundo de LINQ eran los siguientes:

using System; 
using System.Collections.Generic; 
using System.Linq; 
using System.Text; 

namespace ConsoleApplication2 
{ 
    class Program 
    { 
     static void Main(string[] args) 
     { 
      List<string> list = new List<string>() 
      { 
       "fred-064528-NEEDED1", 
       "xxxx", 
       "frederic-84728957-NEEDED2", 
       "sam-028-NEEDED3", 
       "-----", 
       "another-test" 
      }; 

      var result = 
      from s in list 
      where (from c in s where c == '-' select c).Count() == 2 
      select s.Substring(s.LastIndexOf("-") + 1); 

      foreach (string s in result) 
       Console.WriteLine(s); 
      Console.WriteLine("Press Enter"); 
      Console.ReadLine(); 
     } 
    } 
} 

me gustaría saber cómo puedo mejorar la solución anterior a este pequeño ejemplo artificioso. No estoy demasiado interesado en si he usado el mejor método de validación, o cómo puedo localizar "Presione Enter" o algo por el estilo; Solo me interesa usar este ejemplo para aprender un poco más sobre LINQ.

Answer 1

var result = 
     from s in list 
     where s.Count(x => x == '=') == 2 
     select s.Substring(s.LastIndexOf("-") + 1); 

Answer 2

También se puede escribir usando expresiones lambda:

var result = 
      list.Where(s => (from c in s where c == '-' select c).Count() == 2).Select(
       s => s.Substring(s.LastIndexOf("-") + 1)); 

prefiero expresiones lambda más de sintaxis LINQ debido a la interfaz fluida. En mi humilde opinión es más humana legible.

Answer 3

Esto es bastante agradable, creo. Parcialmente LINQ.

var result = String.Join("-", inputData.Split('-').Skip(2)); 

Si no puede haber ningún '-' después de los dos primeros, entonces no sucederá (no LINQ):

var result = inputData.Split('-')[2]; //If the last part is NEE-DED then only NEE is returned. And will fail on wrong input 

Answer 4

Soy un gran fan de Lambdas también ...

static void Main(string[] args) 
    { 
     Func<string, char, int> countNumberOfCharsInString = 
      (str, c) => str.Count(character => character == c); 

     var list = new List<string>() 
     { "fred-064528-NEEDED1", 
      "xxxx", 
      "frederic-84728957-NEEDED2", 
      "sam-028-NEEDED3", 
      "-----", "another-test" 
     }; 

     list.Where(fullString => countNumberOfCharsInString(fullString,'-') == 2) 
      .ToList() 
      .ForEach(s => Console.WriteLine(s.Substring(s.LastIndexOf("-")+1))); 

     Console.WriteLine("Press Enter"); 
     Console.ReadLine(); 
    } 

Answer 5

no creo que esto es una mejora, ya que es menos fácil de leer, pero se puede hacer todo en una sola línea utilizando algunos de los métodos incorporados en la clase de lista:

list.FindAll(s => s.ToCharArray(). 
    Where(c => c == '-').Count() ==2). 
    ForEach(c => Console.WriteLine(c.Substring(c.LastIndexOf("-") + 1))); 

Personalmente encuentro esto bastante horrible, ¡así que es solo por interés!