endrjuh

Wydaje mi się, że iterowanie po wszystkich węzłach i wyciąganie danych tylko z tych węzłów, które nas interesują nie jest najlepszym pomysłem. Lepiej wyciągnąć tylko węzły znaczące, np. najpierw stworzyć kolekcję węzłów row i następnie z nich wyciągać te węzły col które mają dla zadanego atrybutu konkretne wartości - np. wszystkie col które w wartości atrybutu name zawierają ciąg 'name'. Można to zrobić takim np. zapytaniem (wyświetli wyciągnięte dane oddzielone spacją):

List<string> rowsDetails =
                xml.Descendants()
                   .Where(x => x.Name == "row")
                   .Select(row =>
                        string.Join(" ", row.Descendants()
                                            .Where(x => x.Name == "col" && ((string)x.Attribute("name")).Contains("name"))
                                            .Select(y => y.Value)
                        )
                    ).ToList();

rowsDetails.ForEach(Console.WriteLine);

Oczywiście jeżeli zamierzamy robić coś więcej z danymi z XMLa to zamiast List<string> należy użyć listy obiektów prezentujących dane z każdego wiersza - np List<RowDetail> i wówczas można to zrobić tak:

public class RowDetails
{
  public string Name1 { get; set; }
  public string Name2 { get; set; }
  public string Name3 { get; set; }
  ...
}
...
List<RowDetails> rowsDetails = xml.Descendants().Where(x => x.Name == "row").Select(row =>
{
  var desc = row.Descendants().Where(x => x.Name == "col");
  var xElements = desc as List<XElement> ?? desc.ToList();

  var n1 = xElements.FirstOrDefault(x => (string)x.Attribute("name") == "name1")?.Value;
  var n2 = xElements.FirstOrDefault(x => (string)x.Attribute("name") == "name2")?.Value;
  var n3 = xElements.FirstOrDefault(x => (string)x.Attribute("name") == "name3")?.Value;
  ...
    
  return new RowDetails()
  {
  	Name1 = n1,
  	Name2 = n2,
  	Name3 = n3
    ...
  };
}).ToList();

Nazwanie używania 'var' lenistwem programistycznym to zdecydowanie nadużycie. Dla przykładu, w moim subiektywnym odczuciu dużo wygodniej czyta się

var systemTimeZonesList = TimeZoneInfo.GetSystemTimeZones().ToList();

niż

List<TimeZoneInfo> systemTimeZonesList = TimeZoneInfo.GetSystemTimeZones().ToList();

Szczególnie uwzględniając fakt, że po najechaniu kursorem na słowo kluczowe 'var' wyświetlany jest w eleganckim tooltipie typ zmiennej. Chyba, że ktoś nie korzysta z VS

Myślę, że używając 'var'a kluczowe jest nadawanie odpowiednich nazw zmiennych. Gdyby w powyższym przykładzie zamiast systemTimeZonesList programista nazwał zmienną stzl, czytający faktycznie mógłby mieć problem z rozszyfrowaniem kodu. 

Sam do niedawna wzbraniałem się przed korzystaniem z 'var', ale teraz nie wyobrażam sobie życia bez niego, szczególnie w połączeniu z LINQ i typami anonimowymi.

Oczywiście, można prowadzić wielogodzinne dyskusje czy korzystanie z 'var' jest dobre czy złe, czy to dobra praktyka czy lenistwo programistyczne - ale po co? Każdy używa jak mu wygodniej albo tak jak ustali zespół