2008-10-02 8 views
2

J'essaie de combiner une liste de fonctions comme ça.Combiner plusieurs expressions LINQ à partir d'un tableau

J'ai ceci:

Func<int, bool>[] criteria = new Func<int, bool>[3]; 
criteria[0] = i => i % 2 == 0; 
criteria[1] = i => i % 3 == 0; 
criteria[2] = i => i % 5 == 0; 

Et je veux ceci:

Func<int, bool>[] predicates = new Func<int, bool>[3]; 
predicates[0] = i => i % 2 == 0; 
predicates[1] = i => i % 2 == 0 && i % 3 == 0; 
predicates[2] = i => i % 2 == 0 && i % 3 == 0 && i % 5 == 0; 

Jusqu'à présent, j'ai le code suivant:

Expression<Func<int, bool>>[] results = new Expression<Func<int, bool>>[criteria.Length]; 

for (int i = 0; i < criteria.Length; i++) 
{ 
    results[i] = f => true; 
    for (int j = 0; j <= i; j++) 
    { 
     Expression<Func<int, bool>> expr = b => criteria[j](b); 
     var invokedExpr = Expression.Invoke(
      expr, 
      results[i].Parameters.Cast<Expression>()); 
     results[i] = Expression.Lambda<Func<int, bool>>(
      Expression.And(results[i].Body, invokedExpr), 
      results[i].Parameters); 
    } 
} 
var predicates = results.Select(e => e.Compile()).ToArray(); 

Console.WriteLine(predicates[0](6)); // Returns true 
Console.WriteLine(predicates[1](6)); // Returns false 
Console.WriteLine(predicates[2](6)); // Throws an IndexOutOfRangeException 

Est-ce que quelqu'un sait ce que je Je fais mal?

Répondre

4

Ce fut une supposition, que je sais peu de choses sur ce genre de choses, mais cela semble fixer:

Func<int, bool>[] criteria = new Func<int, bool>[3]; 
criteria[0] = i => i % 2 == 0; 
criteria[1] = i => i % 3 == 0; 
criteria[2] = i => i % 5 == 0; 
Expression<Func<int, bool>>[] results = new Expression<Func<int, bool>>[criteria.Length]; 
for (int i = 0; i < criteria.Length; i++) 
{ 
    results[i] = f => true; 
    for (int j = 0; j <= i; j++) 
    { 
     int ii = i; 
     int jj = j; 
     Expression<Func<int, bool>> expr = b => criteria[jj](b); 
     var invokedExpr = Expression.Invoke(expr, results[ii].Parameters.Cast<Expression>()); 
     results[ii] = Expression.Lambda<Func<int, bool>>(Expression.And(results[ii].Body, invokedExpr), results[ii].Parameters); 
    } 
} 
var predicates = results.Select(e => e.Compile()).ToArray(); 

La clé est l'introduction de « ii » et « mau » (peut-être une seule question, Je n'ai pas essayé). Je pense que vous capturez une variable mutable à l'intérieur d'un lambda, et ainsi quand vous la référencez finalement, vous voyez la valeur mutée plus tard que la valeur originale.

+0

qui a été fixé il. Seul jj comptait. Merci –

5

Pas besoin de tirer dans les expressions ...

Func<int, bool>[] criteria = new Func<int, bool>[3]; 
    criteria[0] = i => i % 2 == 0; 
    criteria[1] = i => i % 3 == 0; 
    criteria[2] = i => i % 5 == 0; 

    Func<int, bool>[] predicates = new Func<int, bool>[3]; 

    predicates[0] = criteria[0]; 
    for (int i = 1; i < criteria.Length; i++) 
    { 
     //need j to be an unchanging int, one for each loop execution. 
     int j = i; 

     predicates[j] = x => predicates[j - 1](x) && criteria[j](x); 
    } 

    Console.WriteLine(predicates[0](6)); //True 
    Console.WriteLine(predicates[1](6)); //True 
    Console.WriteLine(predicates[2](6)); //False 
Questions connexes