使用lambda表达式来避免使用"魔术字符串"来指定属性

Question

我正在编写一个服务来获取特定类型的对象集合,并将其原始,字符串和日期时间类型输出为CSV格式的字符串.我有以下两个陈述都有效.我发现基于lambda的版本更清洁.

魔术弦版

string csv = new ToCsvService<DateTime>(objs)
    .Exclude("Minute")
    .ChangeName("Millisecond", "Milli")
    .Format("Date", "d")
    .ToCsv();

与Lambda版本

string csv = new ToCsvService<DateTime>(objs)
    .Exclude(p => p.Minute)
    .ChangeName(p => p.Millisecond, "Milli")
    .Format(p => p.Date, "d")
    .ToCsv();

根据Jon Skeet的建议,所有lambda方法都共享一个类似的方法签名

public IToCsvService<T> Exclude<TResult>(
        Expression<Func<T, TResult>> expression)

然后我传递expression.Body给FindMemberExpression.我已经从nhlambdaextensions项目FindMemberExpression的ExpressionProcessor.cs方法中修改了代码.我非常相似的版本如下:FindMemberExpression

private string FindMemberExpression(Expression expression)
{
    if (expression is MemberExpression)
    {
        MemberExpression memberExpression = (MemberExpression)expression;

        if (memberExpression.Expression.NodeType == ExpressionType.MemberAccess
            || memberExpression.Expression.NodeType == ExpressionType.Call)
        {
            if (memberExpression.Member.DeclaringType.IsGenericType
                && memberExpression.Member.DeclaringType
                .GetGenericTypeDefinition().Equals(typeof(Nullable<>)))
            {
                if ("Value".Equals(memberExpression.Member.Name))
                {
                    return FindMemberExpression(memberExpression.Expression);
                }

                return String.Format("{0}.{1}",
                    FindMemberExpression(memberExpression.Expression),
                    memberExpression.Member.Name);
            }
        }
        else
        {
            return memberExpression.Member.Name;
        }
    }

    throw new Exception("Could not determine member from "
        + expression.ToString());
}

我正在测试足够的病例FindMemberExpression吗？鉴于我的用例,我正在做什么矫枉过正？

Answer 1

编辑:使这个更简单的核心是将方法的签名更改为结果类型中的通用:

public IToCsvService<TSource> Exclude<TResult>(
    Expression<Func<TSource, TResult>> expression)

这样您就不会得到转换表达式,因为不需要转换.例如,由于类型推断,p => p.Minute将最终Expression<Func<DateTime, int>>自动结束.

对我来说这看起来有点矫枉过正,因为目前您需要的只是一个属性 - 至少,这就是您的样本所展示的一切.

为什么不从识别属性开始,如果需要,可以在以后扩展它？

编辑:这是一个简短但完整的示例,不显示任何转换:

using System;
using System.Linq.Expressions;

class Test
{
    static void Main()
    {
        Expression<Func<DateTime, int>> dt = p => p.Minute;
        Console.WriteLine(dt);
    }
}

如果您更改表达式类型Expression<Func<DateTime, long>>然而,它确实显示出Convert(...)位.我怀疑你需要改变你的Exclude(等)方法的签名.