[баг или фича]Количество страниц темы в Top 10

top10.jsp стр 50:

        (стр. <a href="${fn:escapeXml(page_link)}">${message.pages+1}</a>)

   dto.setPages((int) Math.ceil(dto.getAnswers() / itemsPerPage));

Вот бага или фича это я не знаю :) но если поменять на

int tmp = dto.getAnswers();
dto.setPages(tmp/itemsPerPage+tmp%itemsPerPage);

Хотя может это и фича

>${message.pages+1}

Видать, все-таки фича. По-моему, если просто сделать ${message.pages}, то все будет хорошо.

нет. оно не будет писать последнюю страницу и ибо 201/50 = 40 =) В таком случае как щас, оно лажает только на кол-ве страниц кратном числу постов на странице ибо 201/50+1=40+1, но 200/50+1=41, что не верно. Этот неверный случай вылезает в 1/число-постов раз и на форуме на это может и можно забить (это не ко мне).

Соотв. правильный вариант (в посте выше я крупно налажал) надо n/50+(n%50!=0) в терминах C. Т.е. 201/50+(201%50!=0)=40+1=41, 200/50+(200%50!=0)=40+0=40.

так что это должно выглядеть как-то:

int tmp = dto.getAnswers()/itemsPerPage; 
if (tmp%itemsPerPage!=0) tmp++; 
dto.setPages(tmp); 

вот теперь правильно, хоть патч писать.

>ибо 201/50 = 40
Действительно, я не посмотрел, что там int-ы.

Ну, тогда можно сделать double'ы и не терять точности.

>if (tmp%itemsPerPage!=0) tmp++;
Ты хотел сказать if (dto.getAnswers()%itemsPerPage!=0) tmp++; ?

А то ты количество страниц на количество комментариев на странице поделил.

Итого, вариант 1:

dto.setPages((int) Math.ceil((double)dto.getAnswers() / itemsPerPage));

Или сразу itemsPerPage объявить double.

А если без Math.ceil'а, то:

int tmp = dto.getAnswers()/itemsPerPage;
if (dto.getAnswers()%itemsPerPage != 0) tmp++;
dto.setPages(tmp);

Так. Сейчас кто-нибудь fix for a patch for a commit пришлет...

А, ну да, еще же можно предусмотреть фантастический случай выхода в Top-10 темы с 0 комментариев, в которой без доп. проверок будет в обоих случаях «Стр. 0»...

fixed