Как можно сократить код?

Question

[lucky4]

public async Task<IList<WishlistItemDto>> GetAllAsync(PageInfo pageInfo)
        {
            if (pageInfo.Page <= 0 || pageInfo.ItemsPerPage <= 0)
            {
                pageInfo.Page = 1;
                pageInfo.ItemsPerPage = 1;
            }

            int pageSize = 10;
            int pageNumber = (pageInfo.Page - 1) * pageInfo.ItemsPerPage;

            var items = await _wishListItemRepository.GetAllAsync(pageNumber, pageSize);

            var reversedList = (List<WishlistItemDto>)_mapper.Map<IList<WishlistItemDto>>(items);
            reversedList.Reverse();

            return reversedList;
        }

Мне не нравится вот это переменная reversedList. Как ее можно сразу в return загнать?

Comments

[Василий Банников]

if (pageInfo.Page <= 0 || pageInfo.ItemsPerPage <= 0)
            {
                pageInfo.Page = 1;
                pageInfo.ItemsPerPage = 1;
            }

Кажется тут у тебя баг. Должно быть два отдельных ифа.

Answer 1

[Василий Банников]

Сократить код не получится, тк он и так очень компактный
Номер страницы и размер страницы в PageInfo есть смысл контролировать снаружи, например в конструкторе или свойствах PageInfo
Offset и Limit имеет смысл вынести наружу - в екстеншены или в методы PageInfo

public async Task<IList<WishlistItemDto>> GetAllAsync(PageInfo pageInfo)
        {
            var items = await _wishListItemRepository.GetAllAsync(pageInfo.Offset(), pageInfo.Limit());
            return _mapper.Map<IList<WishlistItemDto>>(items).Reverse().ToList();
        }

Comments

[lucky4]

Спасибо за замечания с offset/limit, учту.
А вот по поводу метода Reverse(), его нельзя c IList использовать, только с List =(

[Василий Банников]

lucky4, Есть Reverse на IEnumerable :)
Делать даункаст - это ещё больший бред. Мапь тогда сразу в List