Refactor İkisi, tek bir yöntem

Question

bir SelectList oluşturmak yöntemleri I Sonra Bence üzerine geçmek zaman DB veri almak ve (bir "tüm" seçeneği da dahil) bir doldurulan SelectList nesnesi geri aşağıdaki iki yöntem vardır. Sorun, bunların hem farklı depo nesnelerine erişimlerinin hem de farklı ID adlarına (StatusId ve TeamId) sahip olmaları istisnasıyla neredeyse aynı olmalarıdır. Ben bir parametre olarak depo kabul eder ve her nasılsa kimlik adı belki yansıması veya lambda ifadesi çeşit kullanarak, ne olması gerektiğini anlamadan tek yöntem haline bunları refactor için bir fırsat olduğunu düşünüyorum, ama oldukça bilmiyorum Bunu gerçekleştirmek için.

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

Bunların nasıl tek bir yöntemle yeniden düzenleneceğini anlamaya yardımcı olan var mı?

Answer 1

aşağıdaki deneyebilirsiniz:

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

Bu çözüm ideal değildir ve bazı sözleşmeler dayanır (örneğin tüm öğeleri Name özelliğine sahip olmalıdır). Ancak başlamak için kötü bir yol gibi görünüyor. Özellik isimleri yerine İfadeler kullanılarak daha fazla geliştirilebilir - bu özellik, derleme zamanı denetimi ile özellik adlarının değiştirilmesine izin verir.

Answer 2

Eh, bu şimdiye kadar gelebilir en genel, ama senin durumunda uygulayabileceğiniz eğer MemberStatus ve MemberTeam Bilmiyorum IIdentifiable uygulamak gerektirecektir. Eğer öyleyse, bu gitmek için bir yol olurdu.

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

Ve gördüğüm kadarıyla arayüz kodu

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

Answer 3

, tek bir yöntem içine bu üstlenmeden yolunda ana engeller hakkı belirlenmesine ek olarak, new MemberStatus ve new MemberTeam aramalar Kullanılacak depo.

Zarif bir çözüm bulmak için, biraz daha fazla altyapı yapılandırmanız gerekir - temel olarak türüne göre doğru depoyu çözmeniz gerekir ve bir çeşit fabrikada nesne örneğini oluşturmak istersiniz.

şu tek yöntem haline kodunu refactor ama (Bence) değil edecek ayrı yöntemlerden daha iyi zaten var:

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

Sonra böyle diyebiliriz:

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

Answer 4

IRepository bazı "özellikler" eklerse, bazı temiz kodlar alırsınız.

yerine All()

ilk iki satır işleyen bir SingleRecordsWithAllRecord() yöntemi vardır. Sonra depo DataValueField ve DataTextField kendi tanımlamak gerekir.

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

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

Answer 5

Deli biraz arayüze gidip yapın yapabilirsiniz:

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

     } 
    } 

} 

bir arayüzde statik özelliklerini tanımlamak eğer iyi olurdu, ancak ben bir kukla nesnesi oluşturarak güvenemez beri yerine.