하나의 메서드에 SelectList를 생성하는 두 개의 메서드를 리 팩터 처리합니다.

Question

다음 두 메서드는 내 DB에서 데이터를 가져 와서 채워진 SelectList 개체 ("All"옵션 값 포함)를 반환하여 내보기로 전달합니다. 문제는 서로 다른 리포지토리 개체에 액세스하고 ID 이름 (StatusId 및 TeamId)이 다르다는 점과 거의 동일하다는 것입니다. 리파지토리를 매개 변수로 받아들이고 리다이렉션이나 람다 식을 사용하여 ID 이름이 무엇인지 파악하는 단일 메서드로 리팩토링 할 수있는 기회가 있다고 생각합니다. 이것을 달성하기 위해.

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

이러한 방법을 하나의 방법으로 리팩터링하는 방법을 알아낼 수 있습니까?

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 및 MemberTeam은 IIdentifiable을 구현해야하며, 귀하의 사례에 적용 할 수 있는지 여부는 알 수 없습니다. 그렇다면, 이것이 갈 길이 될 것입니다.

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 전화입니다 사용할 저장소.

우아한 솔루션을 내놓으려면 인프라를 좀 더 구성해야합니다. 기본적으로 유형에 따라 올바른 저장소를 해결해야하며 개체 인스턴스를 빌드하는 일종의 팩토리가 필요합니다.

다음은 하나의 메소드에 코드를 리팩토링, 그러나 (내 생각에)없는 것 별도의 방법보다 더 좋은 당신은 이미 :

는

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

그런 다음이처럼 호출 할 수

를

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) 
     { 

     } 
    } 

} 

당신이 인터페이스의 정적 속성을 정의 할 수 있다면 좋을 텐데,하지만 당신은 내가 더미 객체를 생성에 의존 할 수 없기 때문에 대신.