Преобразование простого ForEach в оператор Linq

Question

Я разработчик SQL по профессии, поэтому Linq (и C# в целом) немного чуждо мне, тем не менее у меня есть простой цикл foreach, который, как я думаю, может быть лучше обслуживаться Linq-запрос. если ничего больше, это будет полезно узнать, даже если проблема не является сложной задачей. Вот мой код:

bool fireAgain = true; 
foreach (var connMan in Dts.Connections) 
{ 
    Dts.Events.FireInformation 
     (0 
      , "" 
      , String.Format("Connection Manager {0} has connection string {1}" 
      , connMan.Name 
      , connMan.ConnectionString) 
      , "" 
      , 0 
      , ref fireAgain 
     ); 
} 

Я могу понять:

from connMan in Dts.Connections select connMan 

достаточно легко, но как же я затем передать мой Connman в мой вызов метода FireInformation?

Answer 1

В целом LINQ будет использоваться для запроса информации. В этом случае вы также запрашиваете информацию.

Что вы можете сделать, однако, следующее:

Dts.Connections.ForEach((conMan) => 
{ 
    Dts.Events.FireInformation(0, "", String.Format("Connection Manager {0} has connection string {1}", connMan.Name, connMan.ConnectionString), "", 0, ref fireAgain); 
} 

То есть, если Dts.Connections является список, в противном случае вы не можете сделать это. (не могу сказать из вашего вопроса, является ли это IEnumerable или List)

Обходным решением было бы создать метод расширения и использовать его в любом случае. Например:

public void ForEach<T>(this IEnumerable<T> source, Action<T> action) 
{ 
    foreach(T item in source) 
    { 
     action(item); 
    } 
} 

После этого вы можете использовать метод ForEach на вашем источнике, как показано в примере.

Answer 2

Dts.Connections 
    .ForEach(connMan => Dts.Events.FireInformation(0, "", String.Format("Connection Manager {0} has connection string {1}", connMan.Name, connMan.ConnectionString), "", 0, ref fireAgain)); 

Если Соединения не являются IEnumerable, измените первую строку;

Dts.Connections.ToList() 

Answer 3

Я бы не предложил попробовать преобразовать это в Linq, потому что Linq предназначен для функционального программирования, а не для императива. Если вы хотите сравнить его с SQL, то это будет без DML.

Dts.Events.FireInformation, похоже, не возвращает значение, которое вы можете выбрать. Вероятно, он запускает событие или что-то еще, и он также использует параметр ref. Этот фрагмент кода совсем не похож на запрос, и я думаю, что его нельзя вставлять в него. Петля foreach в этом случае отлично подходит, поскольку вы на самом деле выполняете внутри нее работу и не запрашиваете данные (для чего нужен linq).

Если вам необходимо отфильтровать коллекцию на основе какого-либо условия, если вам нужно присоединиться к отдельным коллекциям или что-то в этом роде, то Linq может быть лучшим инструментом для работы.