타입을 잃지 않고 일반화

Question

이 두 메서드를 하나의 일반화로 병합해야합니다. IQueryable은 IEnumerable을 구현하고, IOrderedQueryable은 IOrderedEnumerable을 구현하며, 첫 번째 메소드는 두 번째가 존재하면 쓸모없는 것처럼 보입니다. 일부 엔티티 프레임 워크는 이유로 그러나 두 번째는 서버 측 쿼리 번역 LINQ (- 투 - SQL)

private static IQueryable<T> Pag<T>(IOrderedQueryable<T> data, int totalRows, int reqLength, int pageNum) 
    { 
     if (reqLength == 0) 
      return data; 
     else 
     { 
      int skipRows = reqLength * (pageNum - 1); 
      if (skipRows >= totalRows) 
       skipRows = 0; 
      int diff = totalRows - skipRows; 
      return data.Skip(skipRows).Take(diff > reqLength ? reqLength : diff); 
     } 
    } 

    private static IEnumerable<T> Pag<T>(IOrderedEnumerable<T> data, int totalRows, int reqLength, int pageNum) 
    { 
     if (reqLength == 0) 
      return data; 
     else 
     { 
      int skipRows = reqLength * (pageNum - 1); 
      if (skipRows >= totalRows) 
       skipRows = 0; 
      int diff = totalRows - skipRows; 
      return data.Skip(skipRows).Take(diff > reqLength ? reqLength : diff); 
     } 
    } 

이 방법은 DRY 규칙을 깨고 그것을 몹시 짜증나 나누기.

Answer 1

코드가 동일하게 보이더라도 같은 유형에서 작동하지 않으며 제네릭도 사용할 수 없기 때문에 where 절이 여러 개의 지정된 값을 결합 할 수 있습니다.

당신이 할 수있는 것은 모두 계산 논리를 배제하는 것입니다. 그러나 이것이 정말로 더 잘 읽고 유지 보수가 가능하다면 당신의 태도에 달려 있습니다.

private static bool TryCalculateRange(int totalCount, int pageLength, int page, out int skip, out int take) 
{ 
    skip = 0; 
    take = 0; 

    if(pageLength <= 0) 
     return false; 

    skip = pageLength * (page - 1); 

    if(skip >= totalCount) 
     skip = 0; 

    take = totalCount - skip; 

    if(take > pageLength) 
     take = pageLength; 

    return true; 
} 

public static IQueryable<T> Pag<T>(IOrderedQueryable<T> data, int totalRows, int reqLength, int pageNum) 
{ 
    int skip; 
    int take; 

    return TryCalculateRange(totalRows, reqLength, pageNum, out skip, out take)) 
      ? data.Skip(skip).Take(take) 
      : data; 
} 

public static IEnumerable<T> Pag<T>(IOrderedEnumerable<T> data, int totalRows, int reqLength, int pageNum) 
{ 
    int skip; 
    int take; 

    return TryCalculateRange(totalRows, reqLength, pageNum, out skip, out take)) 
      ? data.Skip(skip).Take(take) 
      : data; 
    } 
}