Refactor zur Beseitigung der Wiederholung im Lambda -Ausdruck

Question

Diese beiden Methoden zeigen eine Wiederholung:

public static Expression<Func<Foo, FooEditDto>> EditDtoSelector()
{
    return f => new FooEditDto
    {
        PropertyA = f.PropertyA,
        PropertyB = f.PropertyB,
        PropertyC = f.PropertyC,
        PropertyD = f.PropertyD,
        PropertyE = f.PropertyE
    };
}

public static Expression<Func<Foo, FooListDto>> ListDtoSelector()
{
    return f => new FooDto
    {
        PropertyA = f.PropertyA,
        PropertyB = f.PropertyB,
        PropertyC = f.PropertyC
    };
}

Wie kann ich nacharbeiten, um diese Wiederholung zu beseitigen?

Update: Ups, ich habe es versäumt, einen wichtigen Punkt zu erwähnen. Fooeditdto ist eine Unterklasse von Foodto.

Answer 1

Wenn FooEditDto ist ein Sublass von FooDto Und Sie benötigen nicht die memberInitExpressionen, verwenden Sie einen Konstruktor:

class FooDto
 { public FooDto(Bar a, Bar b, Bar c) 
    { PropertyA = a;
      PropertyB = b;
      PropertyC = c;
    }
   public Bar PropertyA {get;set;}
   public Bar PropertyB {get;set;}
   public Bar PropertyC {get;set;}
 }

class FooEditDto : FooDto
 { public FooEditDto(Bar a, Bar b, Bar c) : base(a,b,c)
   public Bar PropertyD {get;set;}
   public Bar PropertyE {get;set;}
 }

public static Expression<Func<Foo, FooEditDto>> EditDtoSelector()
{
    return f => new FooEditDto(f.PropertyA,f.PropertyB,f.PropertyC)
    {
        PropertyD = f.PropertyD,
        PropertyE = f.PropertyE
    };
}

Answer 2

Nun, ich habe eine Wirklich schrecklich Wie du es schaffen könntest.

Sie können eine Methode schreiben, die Reflexion (mit mir!) Verwendete, um alle Eigenschaften für einen bestimmten Typ auszuarbeiten, und einen Delegierten (unter Verwendung von Reflexion.Emit) erstellt, um Eigenschaften von diesem Typ zu einem anderen zu kopieren. Verwenden Sie dann einen anonymen Typ, um sicherzustellen, dass Sie den Kopierdelegierten nur einmal erstellen müssen, damit er schnell ist. Ihre Methode würde dann so aussehen:

public static Expression<Func<Foo, FooEditDto>> EditDtoSelector()
{
    return f => MagicCopier<FooEditDto>.Copy(new { 
        f.PropertyA, f.PropertyB, f.PropertyC, f.PropertyD, f.PropertyC
    });
}

Die Nuancen hier:

• MagicCopier ist ein generischer Typ und Kopie ist eine generische Methode, damit Sie den Typ "Ziel" explizit angeben können, aber implizit den "Quell" -Typ angeben können.
• Es verwendet einen Projektionsinitialisierer, um die Namen der Eigenschaften aus den Ausdrücken zu schließen, mit der der anonyme Typ initialisiert wurde

Ich bin mir nicht sicher, ob es sich wirklich lohnt, aber es macht eine ziemlich lustige Idee ... ich muss es vielleicht trotzdem umsetzen :)

Bearbeiten: mit MitgliedInitexpression Wir könnten alles mit einem Ausdrucksbaum machen, was es viel einfacher macht als codiert. Ich werde es heute Abend versuchen ...

Bearbeiten: fertig, und es ist eigentlich ziemlich einfacher Code. Hier ist die Klasse:

/// <summary>
/// Generic class which copies to its target type from a source
/// type specified in the Copy method. The types are specified
/// separately to take advantage of type inference on generic
/// method arguments.
/// </summary>
public static class PropertyCopy<TTarget> where TTarget : class, new()
{
    /// <summary>
    /// Copies all readable properties from the source to a new instance
    /// of TTarget.
    /// </summary>
    public static TTarget CopyFrom<TSource>(TSource source) where TSource : class
    {
        return PropertyCopier<TSource>.Copy(source);
    }

    /// <summary>
    /// Static class to efficiently store the compiled delegate which can
    /// do the copying. We need a bit of work to ensure that exceptions are
    /// appropriately propagated, as the exception is generated at type initialization
    /// time, but we wish it to be thrown as an ArgumentException.
    /// </summary>
    private static class PropertyCopier<TSource> where TSource : class
    {
        private static readonly Func<TSource, TTarget> copier;
        private static readonly Exception initializationException;

        internal static TTarget Copy(TSource source)
        {
            if (initializationException != null)
            {
                throw initializationException;
            }
            if (source == null)
            {
                throw new ArgumentNullException("source");
            }
            return copier(source);
        }

        static PropertyCopier()
        {
            try
            {
                copier = BuildCopier();
                initializationException = null;
            }
            catch (Exception e)
            {
                copier = null;
                initializationException = e;
            }
        }

        private static Func<TSource, TTarget> BuildCopier()
        {
            ParameterExpression sourceParameter = Expression.Parameter(typeof(TSource), "source");
            var bindings = new List<MemberBinding>();
            foreach (PropertyInfo sourceProperty in typeof(TSource).GetProperties())
            {
                if (!sourceProperty.CanRead)
                {
                    continue;
                }
                PropertyInfo targetProperty = typeof(TTarget).GetProperty(sourceProperty.Name);
                if (targetProperty == null)
                {
                    throw new ArgumentException("Property " + sourceProperty.Name 
                        + " is not present and accessible in " + typeof(TTarget).FullName);
                }
                if (!targetProperty.CanWrite)
                {
                    throw new ArgumentException("Property " + sourceProperty.Name 
                        + " is not writable in " + typeof(TTarget).FullName);
                }
                if (!targetProperty.PropertyType.IsAssignableFrom(sourceProperty.PropertyType))
                {
                    throw new ArgumentException("Property " + sourceProperty.Name
                        + " has an incompatible type in " + typeof(TTarget).FullName);
                }
                bindings.Add(Expression.Bind(targetProperty, Expression.Property(sourceParameter, sourceProperty)));
            }
            Expression initializer = Expression.MemberInit(Expression.New(typeof(TTarget)), bindings);
            return Expression.Lambda<Func<TSource,TTarget>>(initializer, sourceParameter).Compile();
        }
    }

Und nennen es:

TargetType target = PropertyCopy<TargetType>.CopyFrom(new { First="Foo", Second="Bar" });

Answer 3

Die Wiederholung ist in den Namen, aber C# hat keine Ahnung, dass Propertya in einer Klasse mit Propertya in einer anderen verbunden ist. Sie müssen die Verbindung explizit herstellen. So wie du es gemacht hast, funktioniert es gut. Wenn Sie genügend davon hätten, könnten Sie in Betracht ziehen, Reflection zu verwenden, um eine Methode zu schreiben, die dies für alle Klassenpaare tun könnte.

Achten Sie auf die Auswirkungen auf die Leistung der von Ihnen gewählten Methode. Reflexion von sich aus ist langsamer. Sie könnten jedoch auch Reflection verwenden, um IL auszugeben, die so schnell so schnell laufen würden, wie Sie geschrieben haben. Sie können auch einen Ausdrucksbaum erzeugen und ihn in einen kompilierten Delegierten umwandeln. Diese Techniken sind etwas kompliziert, sodass Sie die Kompromisse abwägen müssen.

Answer 4

Sie können den Anrufer sein eigenes Objekt eines anonymen Typs mit nur den benötigten Eigenschaften zurückgeben lassen:

public static Expression<Func<Foo,T>> 
                             GetSelector<T>(Expression<Func<Foo,T>> f)
 { return f;
 }

/* ... */
var expr = GetSelector(f => new{f.PropertyA,f.PropertyB,f.PropertyC});