C# EF 코드를 리팩토링하는 방법은 무엇입니까?

Question

EF를 사용하는 콘솔 응용 프로그램을 만들었습니다. 그것은 매우 잘 작동하지만 리팩터링 될 수 있다는 것을 알고 있습니다. 여기

CABEntities caContext = new CAEntities(); 
USEntities usContext = new USEntities(); 

var query = caContext.CA_companies 
    .Where(c => 
     c.coverage_status.Equals("active")) 
    .Select(c => new DatapullFields.Datapull 
    { 
     Ticker = c.ticker.Trim(), 
     Exchange = c.primary_exchange.Trim(), 
     CompanyName = c.company_name.Trim() 
    }); 

CreateCSVFile.WriteToCSVFile tf = new CreateCSVFile.WriteToCSVFile(query, "CA"); 
query = usContext.US_companies 
    .Where(c => 
     c.coverage_status.Equals("active_eq")) 
    .Select(c => new DatapullFields.Datapull 
    { 
     Ticker = c.ticker.Trim(), 
     Exchange = c.primary_exchange.Trim(), 
     CompanyName = c.company_name.Trim() 
    }); 
tf = CreateCSVFile.WriteToCSVFile(query, "US"); 

내가 편집 한 후 지금 가지고있는 내용은 다음과 같습니다 : 여기 내 코드는

public class Datapull 
{ 
    public string CompanyName { get; set; } 
    public string Exchange { get; set; } 
    public string Ticker { get; set; } 
} 

public interface ICountryEntity 
{ 
    string CompanyName { get; set; } 
    string coverage_status { get; set; } 
    string Exchange { get; set; } 
    string Ticker { get; set; } 
} 

public partial class CAEntity : Console.Interface.ICountryEntity 
{ 
} 

public partial class USEntity : Console.Interface.ICountryEntity 
{ 
} 

public static class ToDataPull 
{ 
    public static IEnumerable<T> ToDataPull<T>(this IEnumerable<T> entities) where T : ICountryEntity 
    { 
     return entities.Where(c => 
      c.coverage_status.Equals("active")) 
      .Select(c => new Console.DatapullFields.Datapull 
      { 
       Ticker = c.ticker.Trim(), 
       Exchange = c.primary_exchange.Trim(), 
       CompanyName = c.company_name.Trim() 
      }); 
    } 
} 

가 어떻게이 코드를 리팩토링 할 수 있습니까?

Answer 1

인터페이스 및 확장 방법을 사용합니다.
더 많은 코드이지만 앞으로 더 깨끗한 코드로 연결됩니다.

예.

public partial class CAEntity : ICountryEntity 
{ 
} 

public partial class USEntity : ICountryEntity 
{ 
} 

public interface ICountryEntity 
{ 
    string Ticker { get; set; } 
    string Exchange { get; set; } 
    string CompanyName { get; set; } 
    string coverage_status { get; set; } 
} 

//Extension Method must go in static class 
public static IEnumerable<T> ToDataPull<T>(this IEnumerable<T> entities) where T: ICountryEntity 
{ 
    return entities.Where(c => 
     c.coverage_status.Equals("active_eq")) 
     .Select(c => new DatapullFields.Datapull 
     { 
      Ticker = c.ticker.Trim(), 
      Exchange = c.primary_exchange.Trim(), 
      CompanyName = c.company_name.Trim() 
     }); 
} 

//Your modified code 
CABEntities caContext = new CAEntities(); 
USEntities usContext = new USEntities(); 

var query = caContext.CA_companies.ToDataPull(); 
CreateCSVFile.WriteToCSVFile tf = new CreateCSVFile.WriteToCSVFile(query, "CA"); 

query = usContext.US_companies.ToDataPull(); 
tf = CreateCSVFile.WriteToCSVFile(query, "US");