यह संभव एक भी विधि

Question

में इस refactor करने के लिए मैं तरीकों कि इन दोनों की तरह लग रही का एक समूह है है। तरीकों के बीच बहुत कम परिवर्तन के साथ मुझे लगता है कि ऐसा करने का एक बेहतर तरीका होना चाहिए, लेकिन मैं किसी के बारे में नहीं सोच सकता। मैं वीएस 2005 का उपयोग कर रहा हूं।

Answer 1

लगभग 3 तरीके कैसे हैं?

public void SourceInfo_Get()  
{ 
    SendGet(pFBlock.SourceInfo); 
} 

public void SourceAvailable_Get() 
{ 
    SendGet(pFBlock.SourceAvailable); 
} 

private void SendGet(Object obj) { 
    MethodInfo mi = obj.GetType().GetMethod("SendGet"); 
    if (mi != null) 
    { 
     ParameterInfo[] piArr = mi.GetParameters(); 
     if (piArr.Length == 0) 
     { 
      mi.Invoke(obj, new object[0]); 
     } 
    } 
} 

यहां विचार एक सहायक विधि जोड़ने के लिए है जिसे आप पैरामीटर पास कर सकते हैं। फिर आप अपने कोड को भारी रूप से छोटा करने के लिए अपने अन्य तरीकों से सहायक विधि का उपयोग कर सकते हैं।

Answer 2

आप इस

private void Source_Get(Object source) 
    { 
     MethodInfo mi = source.GetType().GetMethod("SendGet"); 
     if (mi != null) 
     { 
      ParameterInfo[] piArr = mi.GetParameters(); 
      if (piArr.Length == 0) 
      { 
       mi.Invoke(source, new object[0]); 
      } 
     } 
    } 

public void SourceInfo_Get() 
    { 
     Source_Get(pFBlock.SourceInfo); 
    } 

public void SourceAvailable_Get() 
    { 
     Source_Get(pFBlock.SourceAvailable) 
    } 

Answer 3

public void SourceAvailable_Get() 
{ 
    CallMethod(pFBlock.SourceAvailable); 
} 

private void CallMethod(object source, String methodName = "SendGet") 
{ 
    MethodInfo mi = source.GetType().GetMethod(methodName); 
    if (mi != null) 
    { 
     if (mi.GetParameters().Length == 0) 
     { 
      mi.Invoke(source, new object[0]); 
     } 
    } 
} 

या हम गतिशील का उपयोग की तरह नकली कोड कुछ refactor सकता है ...

public void CallSendGet(object obj) 
{ 
    obj.AsDynamic().SendGet(); 
} 

Answer 4

public void Source_Get(string memberName) 
    { 
     object member = pFBlock.GetMember(memberName); 
     MethodInfo mi = member.GetType().GetMethod("SendGet"); 
     if (mi != null) 
     { 
      ParameterInfo[] piArr = mi.GetParameters(); 
      if (piArr.Length == 0) 
      { 
       mi.Invoke(member, new object[0]); 
      } 
     } 
    } 

कभी कभी अलग कार्यों कोड अधिक पठनीय हैं। आपको जितना संभव हो सके अपने कार्यों को मर्ज नहीं करना चाहिए।

Answer 5

इसे आजमाएं। टाइप नहीं किया गया।

private void MasterGet(PFBlock pFBlock, string propertyName) 
{ 
    Type t = pFBlock.GetType(); 

    // Cycle through the properties. 
    foreach (PropertyInfo p in t.GetProperties()) 
    { 
     if(p.Name == propertyName) 
     { 
      MethodInfo mi = p.GetType().GetMethod("SendGet"); 
       if (mi != null) 
       { 
       ParameterInfo[] piArr = mi.GetParameters(); 
       if (piArr.Length == 0) 
        { 
         mi.Invoke(p, new object[0]); 
        } 
       } 
     } 
    } 
}