Refactor दो विधियों है कि एक ही विधि

Question

में एक selectList उत्पन्न मैं निम्नलिखित दो तरीकों कि मेरे DB से डेटा प्राप्त है और एक आबादी वाले selectList वस्तु (एक "सभी" विकल्प मान सहित) लौटने कि मैं तो मेरे विचार पर पारित किया है। समस्या यह है कि वे अपवाद के लगभग समान हैं कि वे दोनों अलग-अलग भंडार वस्तुओं तक पहुंचते हैं और उनके पास अलग-अलग आईडी नाम हैं (StatusId और TeamId)। मुझे लगता है कि उन्हें एक ऐसी विधि में दोबारा करने का अवसर है जो रिपॉजिटरी को पैरामीटर के रूप में स्वीकार करता है और किसी भी तरह से आईडी नाम होना चाहिए, शायद प्रतिबिंब या किसी प्रकार का लैम्ब्डा अभिव्यक्ति का उपयोग करके, लेकिन मुझे नहीं पता कि कैसे यह प्राप्त करने के।

private SelectList GetStatusSelectList(int selectedStatusId) 
{ 
    List<MemberStatus> statusList = _memberStatusRepository.All().ToList(); 
    statusList.Insert(0, new MemberStatus {StatusId = 0, Name = "All"}); 
    var statusSelectList = new SelectList(statusList, "StatusId", "Name", selectedStatusId); 
    return statusSelectList; 
} 

private SelectList GetTeamSelectList(int selectedTeamId) 
{ 
    List<MemberTeam> teamList = _memberTeamRepository.All().ToList(); 
    teamList.Insert(0, new MemberTeam { TeamId = 0, Name = "All" }); 
    var teamSelectList = new SelectList(teamList, "TeamId", "Name", selectedTeamId); 
    return teamSelectList; 
} 

कैसे एक एकल विधि में इन refactor करने के लिए किसी को भी मदद आंकड़ा बाहर कर सकते हैं?

Answer 1

आप निम्नलिखित कोशिश कर सकते हैं:

private SelectList GetStatusSelectList(int selectedStatusId) 
{ 
    return GetGenericSelectList<MemberStatus>(selectedStatusId, _memberStatusRepository.All().ToList(), "StatusId"); 
} 

private SelectList GetTeamSelectList(int selectedTeamId) 
{ 
    return GetGenericSelectList<MemberTeam>(selectedTeamId, _memberTeamRepository.All().ToList(), "TeamId"); 
} 

private SelectList GetGenericSelectList<T>(int selectedTeamId, List<T> list, string idFieldName) where T : new() 
{ 
    var firstItem = new T(); 
    (firstItem as dynamic).Name = "All"; 
    var l = new List<T>(list); 
    l.Insert(0, firstItem); 
    return new SelectList(l, idFieldName, "Name", selectedTeamId); 
} 

यह समाधान आदर्श नहीं है और कुछ परंपराओं पर निर्भर करता है (उदाहरण के लिए अपने सभी आइटम Name संपत्ति होनी चाहिए)। हालांकि यह शुरू करने के लिए एक बुरा रास्ता नहीं लगता है। इसे संपत्ति नामों के बजाय अभिव्यक्तियों का उपयोग करके आगे बढ़ाया जा सकता है - जो संकलन समय जांच के साथ संपत्ति के नाम बदलने की अनुमति देगा।

Answer 2

खैर, यह सबसे सामान्य मैं के साथ आ सकता है, लेकिन यह आवश्यक होगा कि आपके MemberStatus और MemberTeamIIdentifiable है, जो मैं नहीं जानता कि लागू करता है, तो अपने मामले के लिए आवेदन कर सकते हैं। यदि ऐसा है, तो यह जाने का रास्ता होगा।

private SelectList GetList<T>(IRepository repository, int id, string name) 
    where T : IIdentifiable, new() 
{ 
    List<IIdentifiable> list = repository.All().ToList(); 
    list.Insert(0, new T() { Name = name, Id = id }); 
    var statusSelectList = new SelectList(list, "Id", "Name", id); 
} 

और इंटरफ़ेस कोड

interface IIdentifiable 
{ 
    int Id { get; set; } 
    string Name { get; set; } 
} 

Answer 3

मैं क्या देख से, एक भी विधि में इस पुनर्रचना के रास्ते में मुख्य बाधा दौड़ new MemberStatus और new MemberTeam कॉल, इसके अलावा में सही का निर्धारण करने के लिए कर रहे उपयोग करने के लिए भंडार।

एक सुरुचिपूर्ण समाधान के साथ आने के लिए, आपको थोड़ा और बुनियादी ढांचा कॉन्फ़िगर करने की आवश्यकता होगी - मूल रूप से आपको इस प्रकार के आधार पर सही भंडार को हल करने की आवश्यकता होगी और आप किसी प्रकार का कारखाना ऑब्जेक्ट इंस्टेंस बनाना चाहते हैं।

निम्नलिखित एक भी विधि में कोड refactor होगा, लेकिन (मेरी राय में) नहीं है अलग तरीकों की तुलना में किसी भी बेहतर आपके पास पहले से:

private SelectList GetSelectList<T>(int selectedId, Func<List<T>> repoAllFunc, Func<T> typeNewFunc, string idName) 
{ 
    List<T> list = repoAllFunc(); 
    list.Insert(0, typeNewFunc()); 
    var selectList = new SelectList(list, idName, "Name", selectedId); 
    return selectList; 
} 

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

var memberStatusSelectList = 
    GetSelectList<MemberStatus>(
     id, 
     () => _memberStatusRepository.All().ToList(), 
     () => new MemberStatus {StatusId = 0, Name = "All"}); 

Answer 4

यदि IRepository कुछ "विशेषताएं" जोड़ता है, तो आपको कुछ क्लीनर कोड मिल जाएगा।

All() के बजाय SingleRecordsWithAllRecord() विधि है जो पहले दो पंक्तियों को संभालती है। फिर भंडार अपने DataValueField और DataTextField परिभाषित करता है।

private SelectList GetSelectList(IRepository repo, int selectedId) 
{ 
    var selectListAll = repo.SingleRecordsWithAllRecord().ToList(); 

    return new SelectList(selectListAll, 
         repo.DataValueField, 
         repo.DataTextField, 
         selectedId); 
} 

Answer 5

आप एक छोटे इंटरफ़ेस पागल जा सकते हैं और निम्न करें:

using System; 
using System.Collections.Generic; 
using System.Linq; 

namespace ConsoleApplication3 
{ 

    public class MemberStatus : IDefault<MemberStatus> 
    { 
     public int StatusId { get; set; } 
     public string Name { get; set; } 

     public MemberStatus Default 
     { 
      get { return new MemberStatus() { StatusId = 0, Name = "All" }; } 
     } 

     public string IdName 
     { 
      get { return "StatusId"; } 
     } 
    } 

    public class MemberTeam : IDefault<MemberTeam> 
    { 
     public int TeamId { get; set; } 
     public string Name { get; set; } 

     public MemberTeam Default 
     { 
      get { return new MemberTeam() { TeamId = 0, Name = "All" }; } 
     } 

     public string IdName 
     { 
      get { return "TeamId"; } 
     } 
    } 

    public interface IDefault<T> 
    { 
     T Default { get; } 
     string IdName { get; } 
    } 

    public interface IRepository<T> 
    { 
     IEnumerable<T> All(); 
    } 

    public class MemberStatusRepository : IRepository<MemberStatus> 
    { 
     public IEnumerable<MemberStatus> All() 
     { 
      return new[] { 
       new MemberStatus(), 
       new MemberStatus() 
      }; 
     } 
    } 
    public class MemberTeamRepository : IRepository<MemberTeam> 
    { 
     public IEnumerable<MemberTeam> All() 
     { 
      return new[] { 
       new MemberTeam(), 
       new MemberTeam() 
      }; 
     } 
    } 

    public class DataAccessLayer 
    { 
     IRepository<MemberStatus> _memberStatusRepository; 
     IRepository<MemberTeam> _memberTeamRepository; 
     public DataAccessLayer() 
     { 
      _memberStatusRepository = new MemberStatusRepository(); 
      _memberTeamRepository = new MemberTeamRepository(); 
     } 


     public SelectList<TResult> GetTeamSelectList<TRepository, TResult>(TRepository repo, int selectedTeamId) 
      where TRepository : IRepository<TResult> 
      where TResult : IDefault<TResult>, new() 
     { 
      List<TResult> teamList = repo.All().ToList(); 
      var dummyobj = new TResult(); 
      teamList.Insert(0, dummyobj.Default); 
      var teamSelectList = new SelectList<TResult>(teamList, dummyobj.IdName, "Name", selectedTeamId); 
      return teamSelectList; 
     } 
    } 

    class Program 
    { 
     static void Main(string[] args) 
     { 
      var dal = new DataAccessLayer(); 
      SelectList<MemberStatus> results = dal.GetTeamSelectList<IRepository<MemberStatus>, MemberStatus>(new MemberStatusRepository(), 5); 
      Console.WriteLine(); 
      Console.Read(); 
     } 
    } 

    public class SelectList<TResult> 
    { 
     public SelectList(List<TResult> teamList, string p, string p_2, int selectedTeamId) 
     { 

     } 
    } 

} 

यह अच्छा होगा यदि आप एक अंतरफलक में स्थिर गुणों को परिभाषित सकता है, लेकिन जब से तुम मैं एक डमी वस्तु बनाने पर भरोसा नहीं कर सकते बजाय।