Преобразование вложенных циклов for в один оператор LINQ - PullRequest
Новая
       33

Преобразование вложенных циклов for в один оператор LINQ

11 голосов
/ 06 июля 2010

Может кто-нибудь помочь мне превратить эту вложенную структуру в один оператор LINQ? 

        EventLog[] logs = EventLog.GetEventLogs();
        for (int i = 0; i < logs.Length; i++)
        {
            if (logs[i].LogDisplayName.Equals("AAA"))
            {
                for (int j = 0; j < logs[i].Entries.Count; j++)
                {
                    if (logs[i].Entries[j].Source.Equals("BBB"))
                    {
                        remoteAccessLogs.Add(logs[i].Entries[j]);
                    }
                }
            }
        }

Ответы [ 5 ]

14 голосов
/ 06 июля 2010

Вложенные циклы обычно заканчиваются несколькими предложениями from (которые преобразуются компилятором в вызовы SelectMany): 

var remoteAccessLogs = from log in EventLogs.GetEventLogs()
                       where log.LogDisplayName == "AAA"
                       from entry in log.Entries
                       where entry.Source == "BBB"
                       select entry;

(при условии, что remoteAccessLogs пуст перед этим вызовоми что вы счастливы, перебирая его напрямую - вы можете позвонить ToList(), если хотите List<T>.)

Вот форма записи в виде точек: 

var remoteAccessLogs = EventLogs.GetEventLogs()
                                .Where(log => log.LogDisplayName == "AAA")
                                .SelectMany(log => log.Entries)
                                .Where(entry => entry.Source == "BBB");

Или длясписок: 

var remoteAccessLogs = EventLogs.GetEventLogs()
                                .Where(log => log.LogDisplayName == "AAA")
                                .SelectMany(log => log.Entries)
                                .Where(entry => entry.Source == "BBB")
                                .ToList();

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

0 голосов
/ 06 июля 2010

Посмотрите: 

List<Entries> result = new List<Entries>();

GetEventLogs().Where(x => x.LogDisplayName.Equals("AAA")).ToList().ForEach(delegate(Log en)
{
    en.Entries.Where(y => y.Source.Equals("BBB", StringComparison.InvariantCultureIgnoreCase)).ToList().ForEach(delegate(Entries ent)
    {
        result.Add(ent);
    });
});
0 голосов
/ 06 июля 2010

Попробуйте следующее: 

from log in logs 
where log.LogDisplayName.Equals("AAA")
select 
   (from entry in log.Entries
    where entry.Source.Equals("BBB")
    select entry);
0 голосов
/ 06 июля 2010

У меня есть это решение, и я предполагаю, что remoteAccessLogs имеет тип List 

remoteAccessLogs.AddRange(

        from log in EventLog.GetEventLogs()
        from entry in log.Entries.Cast<EventLogEntry>()
        select entry
    );

Edit

Я забыл о выражениях where 

List<EventLogEntry> remoteAccessLogs = new List<EventLogEntry>();


    remoteAccessLogs.AddRange(

        from log in EventLog.GetEventLogs()
        where log.LogDisplayName.Equals("AAA")
        from entry in log.Entries.Cast<EventLogEntry>()
        where entry.Source.Equals("BBB")
        select entry
    );
0 голосов
/ 06 июля 2010

Попробуйте это: 

 EventLog[] logs = EventLog.GetEventLogs(); 
 remoteAccessLogs.AddRange(
   logs.Where(l => l.LogDisplayName.Equals("AAA"))
     .Select(l => l.Entries)
     .Where(le => le.Source.Equals("BBB"));

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

...