Search code examples
linqc#-4.0lambdaexpression-treesstatic-methods

ERROR Static method requires null instance, non-static method requires non-null instance


I am trying to create an expression tree. I need to read data from a data table and check its columns. The columns to be checked and also the number of columns to be checked are known at run time only. The column names are given to me as a string array and and each column has a List of strings to be checked. I tried out sample expression trees , like the one below.

Here I am encountering an error.

Static method requires null instance, non-static method requires non-null instance. Parameter name: instance

at the line

inner = Expression.Call(rowexp,mi, colexp);

Kindly help me out!!!

IQueryable<DataRow> queryableData = CapacityTable
    .AsEnumerable()
    .AsQueryable()
    .Where(row2 => values.Contains(row2.Field<string>("Head1").ToString()) 
                && values.Contains(row2.Field<string>("Head2").ToString()));

MethodInfo mi = typeof(DataRowExtensions).GetMethod(
     "Field", 
      new Type[] { typeof(DataRow),typeof(string) });

mi = mi.MakeGenericMethod(typeof(string));

ParameterExpression rowexp = Expression.Parameter(typeof(DataRow), "row");
ParameterExpression valuesexp = Expression.Parameter(typeof(List<string>), "values");
ParameterExpression fexp = Expression.Parameter(typeof(List<string>), "types");
Expression inner, outer, predicateBody = null;

foreach (var col in types)
{
    // DataRow row = CapacityTable.Rows[1];

    ParameterExpression colexp = Expression.Parameter(typeof(string), "col");
    //  Expression left = Expression.Call(pe, typeof(string).GetMethod("ToLower", System.Type.EmptyTypes));

    inner = Expression.Call(rowexp,mi, colexp);
    outer = Expression.Call(valuesexp, typeof(List<string>).GetMethod("Contains"), inner);
    predicateBody = Expression.And(predicateBody,outer);
}

MethodCallExpression whereCallExpression = Expression.Call(
    typeof(Queryable),
    "Where",
    new Type[] { queryableData.ElementType },
    queryableData.Expression,
    Expression.Lambda<Func<DataRow,bool>>(predicateBody, new ParameterExpression[] { rowexp }));

Solution

  • It means the method call you're trying to represent is a static one, but you're giving it a target expression. That's like trying to call:

    Thread t = new Thread(...);
    // Invalid!
    t.Sleep(1000);
    

    You're sort of trying to do that in expression tree form, which isn't allowed either.

    It looks like this is happening for the Field extension method on DataRowExtensions - so the "target" of the extension method needs to be expressed as the first argument to the call, because you actually want to call:

    DataRowExtensions.Field<T>(row, col);
    

    So you want:

    inner = Expression.Call(mi, rowexp, colexp);
    

    That will call this overload which is the way to call a static method with two arguments.