Refactor लैम्ब्डा अभिव्यक्ति में दोहराव समाप्त करने के लिए

Question

इन दो तरीकों प्रदर्शनी पुनरावृत्ति:

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 
    }; 
} 

मैं इस पुनरावृत्ति को खत्म करने refactor कर सकते हैं कैसे?

अद्यतन: ओह, मैंने एक महत्वपूर्ण बिंदु का उल्लेख करने की उपेक्षा की। FooEditDto FooDto का एक उप-वर्ग है।

Answer 1

हैं FooEditDtoFooDto का एक उपन्यास है और आपको सदस्यInitExpressions की आवश्यकता नहीं है, एक कन्स्ट्रक्टर का उपयोग करें:

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

पुनरावृत्ति नामों में है, लेकिन सी # को यह नहीं पता है कि एक वर्ग में संपत्ति ए दूसरे में PropertyA से जुड़ा हुआ है। आपको कनेक्शन को स्पष्ट रूप से बनाना होगा। जिस तरह से आपने किया वह ठीक काम करता है। यदि आपके पास पर्याप्त था, तो आप एक विधि लिखने के लिए प्रतिबिंब का उपयोग करने पर विचार कर सकते हैं जो कक्षाओं की किसी भी जोड़ी के लिए ऐसा कर सकता है।

जो भी तरीका आप चुनते हैं उसके प्रदर्शन के प्रभाव पर ध्यान दें। खुद से प्रतिबिंब धीमा है। हालांकि, आप आईएल को उत्सर्जित करने के प्रतिबिंब का भी उपयोग कर सकते हैं, जो इसे उत्सर्जित करने के बाद, जितना तेज़ लिखा है उतना तेज़ी से चलाएगा। आप एक अभिव्यक्ति वृक्ष भी उत्पन्न कर सकते हैं और इसे एक संकलित प्रतिनिधि में परिवर्तित कर सकते हैं। ये तकनीकें कुछ जटिल हैं, इसलिए आपको व्यापार-बंद का वजन करना होगा।

Answer 3

ठीक है, मेरे पास वास्तव में भयानक तरीका है जिससे आप इसे कर सकते हैं।

आप एक विधि लिख सकते हैं जो किसी विशेष प्रकार के लिए सभी गुणों को काम करने के लिए प्रतिबिंब (मेरे साथ भालू!) का उपयोग करता है, और उस प्रकार से गुणों को प्रतिलिपि बनाने के लिए एक प्रतिनिधि (प्रतिबिंब। प्रवेश) का उपयोग करता है। फिर यह सुनिश्चित करने के लिए एक अनाम प्रकार का उपयोग करें कि आपको केवल प्रतिलिपि प्रतिनिधि को एक बार बनाने की आवश्यकता है, इसलिए यह तेज़ है। आपका विधि तो इस प्रकार दिखाई देगा:

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

यहाँ बारीकियों:

• MagicCopier एक सामान्य प्रकार है और कॉपी ताकि आप स्पष्ट रूप से 'लक्ष्य' के प्रकार का उल्लेख कर सकते हैं एक सामान्य तरीका है, लेकिन परोक्ष "स्रोत" प्रकार निर्दिष्ट करें।
• यह एक प्रक्षेपण प्रारंभकर्ता उपयोग कर रहा है गुमनाम प्रकार

मुझे यकीन है कि क्या यह वास्तव में इसके लायक है नहीं कर रहा हूँ प्रारंभ करने में इस्तेमाल किया भाव से गुण के नाम अनुमान लगाने के लिए है, लेकिन यह काफी एक मजेदार विचार है ... मुझे इसे किसी भी तरह लागू करना पड़ सकता है :)

संपादित करें: MemberInitExpression के साथ हम इसे अभिव्यक्ति वृक्ष के साथ कर सकते हैं, जो इसे कोडडॉम से बहुत आसान बनाता है। इसे आज रात आज़माएं ...

संपादित करें: हो गया, और यह वास्तव में बहुत सरल कोड है।

/// <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(); 
     } 
    } 

और यह बुला:: यहाँ वर्ग है

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

Answer 4

आप फोन करने वाले केवल गुणों के साथ एक अनाम प्रकार के अपने स्वयं के ऑब्जेक्ट प्रदान करने दे सकते हैं वे की जरूरत है:

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});