問題leppieに

Question

感謝：それは低下し、エラーを与える「= mailMergeItems」になると 現在、私は

Expression<Func<vwMailMerge,bool>> whereClause= null; 
List<vwMailMerge> mailMergeItems = null; 

int personType = mailMergeSettings.PersonType.ToInteger(); 
if (personType > 0) 
{ 
    whereClause = this.MailMergeWhereClause(whereClause, f => f.MemberTypeId == personType); 
} 
if (mailMergeSettings.PersonIds != null) 
{ 
    var personIds = mailMergeSettings.PersonIds.ToGuidArray(); 
    if (personIds != null && personIds.Length > 0) 
    { 
     var personList = personIds.ToList(); 
     whereClause = this.MailMergeWhereClause(whereClause, f => personList.Contains(f.UserId)); 
    } 
} 

mailMergeItems = this.ObjectContext.vwMailMerges.Where(whereClause).ToList(); 
private Expression<Func<vwMailMerge, bool>> MailMergeWhereClause(params Expression<Func<vwMailMerge, bool>>[] wheres) 
{ 
    if (wheres.Length == 0) 
    { 
     return x => true; 
    } 
    Expression result; 
    if (wheres[0] == null) 
    { 
     result = wheres[1].Body; 
     return Expression.Lambda<Func<vwMailMerge, bool>>(result, wheres[1].Parameters); 
    } 
    else 
    { 
     result = wheres[0].Body; 
     for (int i = 1; i < wheres.Length; i++) 
     { 
      result = Expression.And(result, wheres[i].Body); 
     } 
     return Expression.Lambda<Func<vwMailMerge, bool>>(result, wheres[0].Parameters); 
     }  
    } 
} 

を得ましたエンティティクエリ式に。

私は唯一の人々のために、またはのみmembertypeIdをチェックするときに、それが正しく動作していることに気づいた..しかし、第二は、それの「F =>」私が思うにエラーを与える組み合わせました。

Answer 1

あなたはより良いコードツールであなたの質問を再フォーマットできます。あなたの周りに浮いこれらすべてのFUNC表現を避けるために、このように問題にアプローチできように見えるしかし

：

this.ObjectContext.vwMailMerges.Where(mm=>IsValidMailMerge(mm,personType)).ToList() 


private bool IsValidMailMerge(YourType mailmerge, YourType2 personType) 
{ 
if(...) // type specific criteria here 
return true; 
else 
return false; 
} 

Answer 2

あなたはFuncを使用傾ける、あなたはExpression<Func>を使用する必要があります。

+はExpression.Andを介して行うことができます。

更新（テストしていません）：

Expression<Func<vwMailMerge, bool>> whereClause = null; 
... 
Expression<Func<vwMailMerge, bool>> MailMergeWhereClause(
    params Expression<Func<vwMailMerge, bool>>[] wheres) 
{ 
    if (wheres.Length == 0) return x => true; 
    Expression result = wheres[0].Body; 
    for (int i = 1; i < wheres.Length; i++) 
    { 
    //probaby needs a parameter fixup, exercise for reader 
    result = Expression.And(result, wheres[i].Body); 
    } 
    return Expression.Lambda<Func<vwMailMerge,bool>>(result, wheres[0].Parameters); 
} 

アップデート2：

上記のアプローチは、私が期待通りに失敗しました。 ExpressionVistorクラスを使用して.NET 4で解決するのは簡単でしょう。 .NET 3.5の場合（または上記があまりにも難しい場合）は、次のように動作するはずです。

somequery.Where(x => x.foo).Where(x => x.bar).Where(x => x.baz) 

IOW、必要に応じて、あなただけのそれらを追加することができますが、それはロジック/フローにいくつかの変更が必要になります：

アプローチは、あなたがで終わるので、直接IQueryableでwhere句を追加していますあなたが貼り付けたコードの