vortexwolf / 2ch-Browser

Source code of the 2ch Browser Android application
https://play.google.com/store/apps/details?id=com.vortexwolf.chan
MIT License
86 stars 16 forks source link

(Не) работает уведомление о количестве новых постов #115

Closed nbmrjuhneibkr closed 7 years ago

nbmrjuhneibkr commented 7 years ago

В версии 1.5.28 и ниже при открытии ранее посещенного треда (из закладок, например) на экране выводилось сообщение с количеством новых постов. В версии 1.6.0 этого нет, при открытии тред обновляется незаметно, и единственная возможность найти новые посты - читать тред и пытаться вспомнить, каких постов не было в прошлый раз.

nbmrjuhneibkr commented 7 years ago

Хм, а вот только что все же сработало уведомление в одном из тредов ("нет новых сообщений"). Может быть, с уведомлениями все в порядке, а проблема в том, что данные о предыдущих посещениях тредов хранятся в кэше не так долго, как раньше?

nbmrjuhneibkr commented 7 years ago

После еще нескольких часов использования могу с уверенностью сказать, что уведомления действительно появляются реже, чем в старых версиях. Похоже, что треды пропадают из кэша гораздо быстрее. Это серьезная проблема.

nbmrjuhneibkr commented 7 years ago

Или еще одно возможное объяснение: уведомления появляются только при открытии тредов, уже посещенных в течение текущего сеанса работы приложения. А для всех тредов, которые открывались раньше - не работают. В любом случае, это надо фиксить.

kolokolovg commented 7 years ago

Переписывайте сетевую часть как я хотел но не стал. В нормальном режиме одновременно не больше 5 тредов стоит отслеживать.
Воркер пул асинхронный, чекает дельта джейсон треда и сохряняет в бдшку. Если тред ее открывался долго воркер мрет по ттл. Уведомление отдельным воркером. Чекаем бд на новые записи

kolokolovg commented 7 years ago

А насчет новых постов хешмапа хватит. Номер треда последний прочитаный пост

Quireg commented 7 years ago

Проверил по коду - всё так и должно быть. Автообновление треда не работает если зайти с вкладки History\Favorites\Opened tabs. Работает для ранее просмотренных тредов если заходить через общий список тредов. Проверьте если у вас тоже так. Предлагаю оставить как есть.