Question

我需要以编程方式识别索引器何时出现在表达式中，但结果表达式树不是我所期望的。

class IndexedPropertiesTest
{
    static void Main( string[] args ) { new IndexedPropertiesTest(); }

    public string this[int index]
    {
        get { return list[index]; }
        set { list[index] = value; }
    }
    List<string> list = new List<string>();

    public IndexedPropertiesTest()
    {
        Test( () => this[0] );
    }

    void Test( Expression<Func<string>> expression )
    {
        var nodeType = expression.Body.NodeType;
        var methodName = ((MethodCallExpression)expression.Body).Method.Name;
    }
}

在上面的代码中，nodeType是“呼叫”，methodName是“get_Item”。为什么？ expression.Body不应该等同于Expression.Property( Expression.Constant( this ), "Item", Expression.Constant( 0 ) )吗？这就是我的预期。

我需要能够以非常一般的方式检测索引器 - 只考虑任何表达式。对预期表达式树的这种破坏会影响我执行此操作的能力。依赖于方法名称为“get_Item”太脆弱了。另外，IndexerNameAttribute可能还用于重命名索引器属性。

那么有没有让编译器生成预期的表达式树？请不要建议手动构建表达式，因为这不是一个选项。或者有没有办法以编程方式确保我拥有的是索引器？

Answer 1

我认为你有答案：这就是C＃编译器的工作方式。

我将您的代码翻译成VB.NET。无足够的，VB.NET不会调用方法get_Item，而是通过你给它的名称来调用它。在下面的示例中，最终会得到get_MyDefaultProperty。

Sub Main
    Dim x as IndexedPropertiesTest = New IndexedPropertiesTest()

End Sub

' Define other methods and classes here
Class IndexedPropertiesTest

    Private list as New List(Of String) From { "a" }
    Default Property MyDefaultProperty(index as Integer) as String
        Get
            Return list(index)
        End Get
        Set(value as String)
            list(index) = value
        End Set
    End Property

    Public Sub New
        Test( Function() Me(0))
    End Sub

    Public Sub Test(expression as Expression(Of Func(Of String)))

        Dim nodeType as ExpressionType = expression.Body.NodeType
        Dim methodName as String = CType(expression.Body, MethodCallExpression).Method.Name

        'expression.Dump() 'Using LINQPad

    End Sub

End Class

但是，所有内容都不会丢失：您可以撰写访问者，尝试将get_Item电话填回IndexExpression。我在这里开始了：

public class PropertyFixerVisitor : ExpressionVisitor
{
    protected override Expression VisitMethodCall(MethodCallExpression node)
    {
        if (node.Method.Name.StartsWith("get_"))
        {
            var possibleProperty = node.Method.Name.Substring(4);
            var properties = node.Method.DeclaringType.GetProperties()
                .Where(p => p.Name == possibleProperty);

            //HACK: need to filter out for overriden properties, multiple parameter choices, etc.
            var property = properties.FirstOrDefault();
            if (property != null)
                return Expression.Property(node.Object, possibleProperty, node.Arguments.ToArray());
            return base.VisitMethodCall(node);

        }
        else
            return base.VisitMethodCall(node);
    }
}

然后您可以安全地修改您的Test方法：

void Test(Expression<Func<string>> expression)
{
    var visitor = new PropertyFixerVisitor();
    var modExpr = (Expression<Func<string>>)visitor.Visit(expression);

    var indexExpression = (modExpr.Body as IndexExpression); //Not Null
}

Answer 2

我最近也遇到了同样的问题，并得到了以下解决方案（根据您的示例）：

void Test(Expression<Func<string>> expression)
{
    if (expression.Body.NodeType == ExpressionType.Call)
    {
        var callExpression = (MethodCallExpression)expression.Body;
        var getMethod = callExpression.Method;
        var indexer = getMethod.DeclaringType.GetProperties()
            .FirstOrDefault(p => p.GetGetMethod() == getMethod);
        if (indexer == null)
        {
            // Not indexer access
        }
        else
        {
            // indexer is a PropertyInfo accessed by expression
        }
    }
}

因此，基本上，我依赖于以下内容，而不是依赖于索引器以特定方式命名：

可以将MethodInfo的两个对象与相等（或不相等）进行比较（operator ==和operator !=都针对MethodInfo实现了。）
索引器具有get方法。通常，属性可能没有get方法，但首先不能使用这种属性来构造具有lambda语法的表达式。
对非索引器属性的访问将产生MemberExpression而不是MethodCallExpression（即使不会，代表简单属性的PropertyInfo始终可以与具有GetIndexParameters的代表索引器的区别方法，因为所有索引器都有至少一个参数。

如果一个类中存在多个indexer exst，这种方法也适用，因为它们每个都有自己的MethodInfo，并且只有一个等于表达式中使用的那个。

注意：以上方法既不适用于私有索引器，也不适用于具有私有get方法的索引器。为了概括该方法，应该使用GetProperties和GetGetMethod的正确重载：

// ...
var indexer = getMethod.DeclaringType.GetProperties(
        BindingFlags.Instance | BindingFlags.Public | BindingFlags.NonPublic)
    .FirstOrDefault(p => p.GetGetMethod(nonPublic: true) == getMethod);
// ...

希望对某人有帮助。

在LINQ表达式

2 个答案: