Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

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 }));
like image 875
Jani5e Avatar asked Oct 22 '11 04:10

Jani5e


1 Answers

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.

like image 85
Jon Skeet Avatar answered Sep 20 '22 08:09

Jon Skeet