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

Question

我正在编写一个服务来获取特定类型的对象集合和output its primitive, string, and DateTime types 到CSV Format 中的字符串。我有以下两个陈述。我发现基于 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 project 改编了FindMemberExpression method of 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 （等）方法的签名。

Answer 2

您是否有任何计划让它更灵活，或者这就是它需要做的全部？

理想情况下，您应该拥有最简单的代码来完成您需要完成的工作，这样您就可以减少可能出错的事情的数量。

如果您这样做是为了证明概念，并且知道稍后您将需要 lambda 表达式，那么将它们保留在那里是有意义的，但是，如果这是最终产品，那么前者更易于阅读，如果其他人需要对代码进行更改，则不太可能引起混淆。