Repeater에 대한 페이지 매김 지원

Question

Repeater에서 Pagination Support를 사용하려고 할 때 PagedDataSource가 나를 구출했습니다. 나는 다음과 같은 방법을 만들었으며 함정이 있는지 또는 개선의 기회가 있는지 여부를 여러분 모두와 나누고 싶습니다.

다음

그것이,

/// <summary> 
/// Create pagination for Repeater 
/// </summary> 
/// <param name="context">HttpContext</param> 
/// <param name="obj">System.Collections.IEnumerable</param> 
/// <param name="rptr">Repeater Control</param> 
/// <param name="pgSize">How many records in each page</param> 
/// <returns>Pagination String</returns> 
public static String pagination(HttpContext context,Object obj,Repeater rptr ,int pgSize) 
{ 
    String rtn = String.Empty; 

    int curpage = 0; 

    PagedDataSource pds = new PagedDataSource(); 
    pds.DataSource=(System.Collections.IEnumerable)obj; 
    pds.AllowPaging = true; 
    pds.PageSize = pgSize; 

    if (context.Request.QueryString["page"] != null) 
    { 
     curpage = Convert.ToInt32(context.Request.QueryString["page"]); 
    } 
    else 
    { 
     curpage = 1; 
    } 

    pds.CurrentPageIndex = curpage - 1; 

    if (!pds.IsFirstPage) 
    { 
     rtn = "<a href='?page=" + (curpage - 1).ToString() + "'>Prev</a>&nbsp;"; 
    } 

    if (curpage == 1 && pds.DataSourceCount > pds.PageSize) 
     rtn = "1"; 
    else if (pds.DataSourceCount == 0) 
     rtn = "No data to display"; 
    else if (curpage > 1 && pds.DataSourceCount > pds.PageSize) 
     rtn = rtn + "<a href='?page=1'>1</a>&nbsp;"; 

    for (int i = 2; i <= pds.PageCount; i++) 
    { 
     if (i == curpage) 
      rtn = rtn + "&nbsp;" + i.ToString(); 
     else 
      rtn = rtn + "&nbsp;<a href='?page=" + i.ToString() + "'>" + i.ToString() + "</a>"; 
    } 

    if (!pds.IsLastPage) 
    { 
     rtn += "&nbsp;<a href='?page=" + (curpage + 1).ToString() + ">Next</a>"; 
    } 

    rptr.DataSource = pds; 
    rptr.DataBind(); 

    return rtn; 
} 

Answer 1

나는 가장 좋은 방법의 측면에서 일부를 볼 수 있지만 구조는 꽤 소리. 그러나 다음과 같이 제안 할 수 있습니다.

1) 코드 가드를 제안합니다.

2) 인라인 formating (+ 연산자를 사용하여 문자열을 작성)은 String.Format() 메소드보다 비용이 많이 듭니다.

3) 코드 변환을 유발할 수 있으므로 Convert.ToInt32()를 사용하지 마십시오. 더 안전하기 때문에 Int32.TryParse()를 사용하십시오.

4) 또한 URL 라우팅이나 URL 재 작성의 개념을보고 URL을 좀 더 친숙하게 만들 수 있습니다.