foreach를 linq으로 바꿉니다

Question

dt = ds.Tables[1]; 

foreach (EPSFromElement element in elementList) 
{ 
    foreach (DataRow row in dt.Rows) 
    { 
     foreach (DataColumn column in dt.Columns) 
      { 
       var ColumnName = column.ColumnName; 
       var ColumnData = row[column].ToString(); 
       var currentElement = Regex.Replace(element.Field_Label, @"\W", ""); 

       if (element.Module_Field_ID != null) 
       { 

        if (currentElement == ColumnName) 
         element.ColumnValue = ColumnData;         
       } 
       else 
       { 
        if (element.Field_Type_Name != "Checkbox") 
        { 
         if ("Q_" + element.Column_Name_ID == ColumnName) 
          element.ColumnValue = ColumnData; 
        } 
        else 
        { 
         if ("Q_" + element.Column_Name_ID + "_" + element.Value_Column_Name_ID == ColumnName) 
            element.ColumnValue = ColumnData; 
        } 
       } 
     } 
    } 
} 

나는 List of object (EPSElement)를 가지고 있습니다. EPSElement에는 데이터 테이블 (dt = ds.Tables [1];)에서 데이터를 가져 오는 ColumnValue 필드가 있습니다. 이것을 달성하기 위해 foreach 루프를 사용하고 있습니다.이 작업을 수행하는 더 좋은 방법이 있습니까? 각 루프마다 세 가지로 너무 만족하지 않습니다. LINQ로 바꿀 수 있습니까?

Answer 1

이것은 상당히 잘 번역됩니다. foreach -es 각각은 from으로 변환되며, 몇 가지 변수는 각각 let이고, if은 모두 where입니다.

var query = from element in elementList 
      from row in dt.Rows.Cast<DataRow>() 
      from column in dt.Columns.Cast<DataColumn>() 
      let ColumnName = column.ColumnName 
      let ColumnData = row[column].ToString() 
      let currentElement = Regex.Replace(element.Field_Label, @"\W", "") 
      where (element.Module_Field_ID != null && currentElement == ColumnName) 
      || (element.Field_Type_Name != "Checkbox" 
        && "Q_" + element.Column_Name_ID == ColumnName 
       || element.Field_Type_Name == "Checkbox" 
        && "Q_" + element.Column_Name_ID + "_" + element.Value_Column_Name_ID == ColumnName) 
      select new { Element = element, ColumnData = ColumnData }; 

foreach (var item in query) 
    item.Element.ColumnValue = item.ColumnData; 

참고 Where의 논리가 조금 복잡한이기 때문에, 그것은 당신이 같은 것을 가지고있는 그것의 자신의 방법으로 그것을 밖으로 리팩토링하는 것이 가장 수 있음 : 다음

where IsValid(element, ColumnName, currentElement) 

구현 할 수 있습니다 내 쿼리 또는 원본 코드의 논리에 따라 원하는 것을 기반으로하십시오.

Answer 2

나는 이 아니며은 LINQ로 바꾸려고합니다. 먼저, foreach의 본문 내부에서 돌연변이를 일으키고 있습니다. 은입니다. LINQ를 사용하려고 시도했습니다. LINQ는 쿼리에 관한 것입니다 (LIN Q의 Q는 q입니다) 부작용이 없어야합니다. 그러나 당신이 돌연변이를 새로운 순서의 투영법으로 대체하고 LINQ로 변환하면 이 매우이됩니다. 너무 많은 중첩 된 논리. 그냥 놔둬.