코드 리뷰 스페셜리스트 역할 프롬프트 3

주요 버그는 반복문의 종료 조건입니다.

for (int i = start; i < end - 1; i++)

i < end - 1이므로 end - 1 인덱스를 제외합니다. 그 결과 마지막 페이지뿐 아니라 모든 페이지에서 최대 pageSize - 1개만 반환됩니다.

또한 마지막 페이지에 pageSize보다 적은 항목만 남아 있으면 items[i] 접근 중 예외가 발생할 수 있습니다.

수정 예시는 다음과 같습니다.

public List<string> GetPageItems(
    List<string> items,
    int page,
    int pageSize)
{
    if (items == null)
        throw new ArgumentNullException(nameof(items));

    if (page < 0)
        throw new ArgumentOutOfRangeException(nameof(page));

    if (pageSize <= 0)
        throw new ArgumentOutOfRangeException(nameof(pageSize));

    int start = checked(page * pageSize);

    if (start >= items.Count)
        return new List<string>();

    int count = Math.Min(pageSize, items.Count - start);

    return items.GetRange(start, count);
}

LINQ를 사용하면 더 간결하게 작성할 수도 있습니다.

return items
    .Skip(page * pageSize)
    .Take(pageSize)
    .ToList();

추가로 검토할 사항:

• 현재 구현은 page가 0부터 시작한다고 가정합니다. 1부터 시작하는 페이지 번호라면 page - 1을 사용해야 합니다.
• log는 생성만 되고 사용되지 않습니다. 필요하다면 StringBuilder를 사용하고, 불필요하면 제거하는 것이 좋습니다.
• page * pageSize의 정수 오버플로 가능성을 고려해야 합니다. 위 예시에서는 checked로 이를 감지합니다.