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; 
                     }
                 }
        }
    }
}

我有一个对象列表（EPSElement）。在EPSElement中有一个字段ColumnValue，它从数据表中获取数据（dt = ds.Tables [1];）。为了达到这个目的，我在那里使用foreach循环。有更好的方法来做到这一点。每个循环我都不满意三个。可以用LINQ替换它。

Answer 1

我永远不会尝试用LINQ替换它。首先，你在foreach的主体内部进行变异，这是一个错误的想法，试图用LINQ做。 LINQ是关于查询（LIN Q 中的'Q'是 q uery），所以不应该有副作用。但是，即使假设您要将突变替换为新的对象序列，将其转换为LINQ也会非常难以阅读;嵌套逻辑太多了。别管它了。

Answer 2

这相当好。每个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)

然后，实现可以是我的查询或原始代码中的逻辑，基于您的喜好。

用linq替换foreach

2 个答案: