डुप्लिकेट कोड को कैसे खत्म करें?

Question

मेरे पास निम्नलिखित कोड हैं और मैं इसे इस तरह लिखना चाहता हूं कि मेरे पास कोडों का न्यूनतम डुप्लिकेशन है।

if (Categories != null) 
{ 
    bool flag=false; 
    foreach (dynamic usableCat in Category.LoadForProject(project.ID)) 
    { 
     foreach (dynamic catRow in Categories) 
     { 
      if (usableCat.ID == catRow.ID) 
       flag = true;        
     } 
     if (!flag) 
     { 
      int id = usableCat.ID; 
      Category resolution = Category.Load(id); 
      resolution.Delete(Services.UserServices.User); 
     } 
    } 
} 
if (Priorities != null) 
{ 
    bool flag = false; 
    foreach (dynamic usableCat in Priority.LoadForProject(project.ID)) 
    { 
     foreach (dynamic catRow in Priorities) 
     { 
      if (usableCat.ID == catRow.ID) 
       flag = true; 
     } 
     if (!flag) 
     { 
      int id = usableCat.ID; 
      Priority resolution = Priority.Load(id); 
      resolution.Delete(Services.UserServices.User); 
     } 
    } 
} 

कृपया ध्यान दें कि श्रेणी और प्राथमिकता में एक सामान्य आधार प्रकार या इंटरफ़ेस नहीं है जिसमें आईडी शामिल है।

Answer 1

मुझे एक वैकल्पिक दृष्टिकोण सुझाने दें: flag बात बाहर बाँटे के बजाय, LINQ का उपयोग झंडा पाश के लिए जरूरत दूर करने के लिए:

if (Categories != null) 
{ 
    foreach (var usableCat in Category.LoadForProject(project.ID)) 
    { 
     if (!Categories.Any(row => usableCat.ID == row.ID)) 
      Category.Load(usableCat.ID).Delete(Services.UserServices.User); 
    } 
} 
if (Priorities != null) 
{ 
    foreach (var usablePri in Priority.LoadForProject(project.ID)) 
    { 
     if (!Priorities.Any(row => usablePri.ID == row.ID)) 
      Priority.Load(usablePri.ID).Delete(Services.UserServices.User); 
    } 
} 

Answer 2

void DeleteUsable<Ttype>(IEnumerable<Ttype> usables, IEnumerable<Ttype> collection, Func<int, Ttype> load) 
{ 
    bool flag = false; 
    foreach (dynamic usableCat in usables) 
       { 
        foreach (dynamic catRow in collection) 
        { 
         if (usableCat.ID == catRow.ID) 
          flag = true; 
        } 
        if (!flag) 
        { 
         int id = usableCat.ID; 
         Ttype resolution = load(id); 
         resolution.Delete(Services.UserServices.User); 
        } 
       } 
} 

संपादित करें: कॉल यह:

if (Categories != null) 
    DeleteUsable(Category.LoadForProject(project.ID), Categories, Categoriy.Load); 
if (Priorities != null) 
    DeleteUsables(Priority.LoadForProject(project.ID), Priorities, Priority.Load); 

Answer 3

मैं इस तरह एक विधि की सलाह देते हैं (जब से तुम गतिशील प्रकार की पहुंच है):

void DeleteUsables(dynamic usablesResource, dynamic usablesCatalog) 
{ 
    bool flag = false; 
    foreach (dynamic usableCat in usablesCatalog.LoadForProject(project.ID)) 
    { 
     foreach (dynamic catRow in usablesResource) 
     { 
      if (usableCat.ID == catRow.ID) 
       flag = true;        
     } 
     if (!flag) 
     { 
      int id = usableCat.ID; 
      dynamic resolution = usablesCatalog.Load(id); 
      resolution.Delete(Services.UserServices.User); 
     } 
    } 

} 

है जिसे आप कहेंगे इस तरह:

if (Categories != null) 
{ 
    DeleteUsables(Categories, Category) 
} 
if (Priorities != null) 
{ 
    DeleteUsables(Priorities, Priority) 
}