.NET 2.x Системная информация — как улучшить код?

Question

[ArteSDeveloper]

string savePath = @"C:\Sys.txt";
                   using (FileStream file = new FileStream(savePath, FileMode.Append))
                           {
                   using (StreamWriter twf = new StreamWriter(file, Encoding.UTF8))
                           {
                   foreach (ManagementBaseObject avResult in avsearcher.Get())
                           {
                               pcAntivirus = "& " + (avResult.GetPropertyValue("displayName")).ToString();
                               pcAntivirus = (pcAntivirus.StartsWith("& ")) ? pcAntivirus.Substring(1) : pcAntivirus;
                               twf.WriteLine(" антивирус: " + pcAntivirus);
                           }
                           foreach (ManagementObject gr in gs.Get())
                           {
                               foreach (PropertyData pty in gr.Properties)
                               {
                                   if (pty.Name == "Description")
                                   {
                                       pcGPU += pty.Value.ToString();
                                       twf.WriteLine("  Видеоадаптер: " + pcGPU);
                                   }
                               }
                           }
foreach (ManagementObject gob in sar.Get())
                           {
                               twf.WriteLine("  Модель компьютера: " + gob["Manufacturer"] + gob["Model"]);
                           }

Проблема в том что слишком много foreach ! как можно сократить это всё?

Comments

[Рифат]

А чем они Вас не устраивают? Вложенный foreach можно заменить на что то типа
foreach(PropertyData pty in gs.Get().SelectMany(x => x.Properties))
{
}

Или можно Select'ом сконструировать строку которую будете писать в файл. Код уменьшиться, но эффективности не прибавит.

[Алексей Немиро]

foreach - не проблема. Проблема тут в первую очередь в форматировании, а точнее в его полном отсутствии :-)

Понятно, что Visual Studio сама все отформатирует, но в вопросе с просьбой сделать код лучше, отсутствие форматирования - это как минимум странно.

pcAntivirus.StartsWith("& ")

Каким образом может быть иначе, если строчкой выше в переменную pcAntivirus добавляется подстрока "& "? И зачем далее удалять добавленный амперсанд? Зачем его вообще добавлять? ;-)

pcGPU += pty.Value.ToString();

Аналогично, нет смысла, можно сразу передавать в twf.WriteLine().

Ко всему прочему, учитывая, что значение добавляется к переменной (+=), то это логическая ошибка. Если видеоадаптеров будет больше одного, то при следующей итерации в файл будут записываться информация обо всех предыдущих адаптерах.

Answer 1

[Александр Евсеев]

Как вариант использовать LINQ и его метод SelectMany (возможно несколько раз). (см. professorweb.ru/my/LINQ/base/level2/2_2.php)
И методы Concat.
Т.е. будет примерно так

IEnumerable<string> strings = avsearcher.Get().Select(avResult => avResult.ToDisplayName()) // это метод расширение, который делает нужную вам строку
    .Concat(
        gs.Get().SelectMany(gr => gr.Properties).Where(prt => prt.Name == "Description").Select(pry => prt.Value.ToString()) // здесь тоже можно использовать метод расширение
    ).Concat(
       sar.Get().Select(gob => gob.ToDisplayName()) // тоже метод расширение
    ); // и т.д.
// ну а дальше можно уже даже через foreach или любым другим способом записать в twf