使用可为空的 DateTime 转换表达式会产生错误

Question

在一个类中，我有一个静态方法通过提供一个表达式来获取 PropertyInfo。

public static PropertyInfo GetPropertyInfo<T>(Expression<Func<T, object>> expressie)
        {
            MemberExpression me;
            switch (expressie.Body.NodeType)
            {
                case ExpressionType.Convert:
                case ExpressionType.ConvertChecked:
                    var ue = expressie.Body as UnaryExpression;
                    me = ((ue != null) ? ue.Operand : null) as MemberExpression;
                    break;
                default:
                    me = expressie.Body as MemberExpression;
                    break;
            }
            if (me == null)
            {
                throw new InvalidOperationException("Expression does not refer to a property: " + expressie.ToString());
            }

            return (PropertyInfo)me.Member;
        }

为了使我的代码更安全，我更改了在调用此函数的部分方法中使用的表达式。

// Old
Expression<Func<T, Object>> expression;

// New (TProp instead of Object)
Expression<Func<T, TProp>> expression;

因此，我不能再使用旧的“GetPropertyInfo”方法，因为表达式与请求的参数不匹配。所以我创建了一个新的。

public static PropertyInfo GetPropertyInfo<T, TProp>(Expression<Func<T, TProp>> expressie)
        {
            MemberExpression me;
            switch (expressie.Body.NodeType)
            {
                case ExpressionType.Convert:
                case ExpressionType.ConvertChecked:
                    var ue = expressie.Body as UnaryExpression;
                    me = ((ue != null) ? ue.Operand : null) as MemberExpression;
                    break;
                default:
                    me = expressie.Body as MemberExpression;
                    break;
            }
            if (me == null)
            {
                throw new InvalidOperationException("Expression does not refer to a property: " + expressie.ToString());
            }

            return (PropertyInfo)me.Member;
        }

这个新表达式的主体与现有的完全相同。因为我想坚持 DRY 原则，所以我寻找了一种方法，在这两种情况下都使用一个方法体。这可以通过首先将 'Expression>' 转换为 'Expression>' 并将转换后的表达式传递给 'old' GetPropertyInfo(...) 方法来实现。我无法完全删除旧的“GetPropertyInfo”方法，因为其他代码仍然依赖它。

我使用的转换方法如下：

private static Expression<Func<T, object>> convertToObjectExpression<T, TProp>(Expression<Func<T, TProp>> expression)
        {
            return Expression.Lambda<Func<T,object>>(
                    Expression.Convert(expression.Body, typeof(object)),
                    expression.Parameters);
        }

这很好用，除非我使用带有可为空的 DateTime (DateTime?) 的表达式作为 TProp 类型。在这种情况下，以下行会导致 'me' 为空。

me = ((ue != null) ? ue.Operand : null) as MemberExpression;

有没有人知道是什么导致了这个问题？

我有一个包含问题的fiddle。

Answer 1

我建议你反其道而行之。不要从新的类型安全方法传递到旧方法（带有所有附加问题），而是从旧方法传递到新方法：

// relay old interface to new one for compatibility
public static PropertyInfo GetPropertyInfo<T>(Expression<Func<T, object>> expressie)
{
    return GetPropertyInfo<T, object>(expressie);
}

public static PropertyInfo GetPropertyInfo<T, TProp>(Expression<Func<T, TProp>> expressie)
{ /* your code, as you provided it */ }

如果您对此仍有疑问，请提供您的调用代码

编辑

因与问题无关而被删除

Answer 2

有没有人知道是什么导致了这个问题？

您调用新函数的方式（使用DateTime? 从普通的DateTime 属性转换）和实现它的方式（通过convertToObjectExpression）导致有两个Convert 表达式，因此出现了问题。

为了修复它，请考虑用户 grek40 的建议，和/或按如下方式更改实现

public static PropertyInfo GetPropertyInfo<T>(Expression<Func<T, object>> expressie)
{
    var body = expressie.Body;
    while (body.NodeType == ExpressionType.Convert || body.NodeType == ExpressionType.ConvertChecked)
        body = ((UnaryExpression)body).Operand;
    var me = body as MemberExpression;
    var property = me != null ? me.Member as PropertyInfo : null;
    if (property == null)
    {
        throw new InvalidOperationException("Expression does not refer to a property: " + expressie.ToString());
    }
    return property;
}