LINUX.ORG.RU
ФорумMobile

Приложение LOR для Android

 ,


21

19

В этой теме буду сообщать о прогрессе разработки мобильного приложения.

Репозиторий на Github

Последний залитый debug APK (если ссылка битая - собирайте из исходников)

Благодарю за материальную поддержку najlus, NeverLoved



Последнее исправление: mcgeek (всего исправлений: 3)

Ответ на: комментарий от upcFrost

Так как PreferenceFragment приложением не используется, сегодня дропну его из dev ветки. Поддержка 2.3 будет, раз тема поднимается не первый раз.

mcgeek
() автор топика
Ответ на: комментарий от mcgeek

«не удалось получить данные, проверьте подключение». Подключение точно есть т.к. тем же вафлем только что качал сам апк

upcFrost ★★★★★
()
Ответ на: комментарий от upcFrost

Если закрыть приложение, открыть и перейти сразу в трекер, что происходит?

mcgeek
() автор топика
Ответ на: комментарий от upcFrost

Действительно, если открыть любой другой раздел и перейти в трекер, то данные не подгружаются. Пофикшу.

mcgeek
() автор топика
Ответ на: комментарий от upcFrost

Пофикшено. Вылезла еще пара багов, надо проверить, не связано ли это с понижением версии API.

mcgeek
() автор топика
Ответ на: комментарий от mcgeek

А ты мог бы плюнуть на ожидание API и тупо парсить html? Почему все уверены, что это какое-то ужасное дело? Мобильными браузерами никто не пользуется? Такие же парсеры хэтээмэл, так же жрут память и акб. Изменится верстка сайта, и приложение перестанет работать? Ну так сделай набор типовых урл'ов сайта, чтобы они загружались при старте приложения из каталога на карте памяти, чтобы юзеры могли сами поправить. Приложение хотя бы писаться тогда начнет, а так, как сейчас, лучше выкинуть его из головы и забыть.

Virtuos86 ★★★★★
()
Ответ на: комментарий от yacuken

Да, так и есть. Но есть один момент. Он может пилить что угодно — PR принимает maxcom. Соответственно, если он не принимает их, по каким-то причинам, разработка стоит на месте; баста, карапузики.

Virtuos86 ★★★★★
()
Ответ на: комментарий от Deleted

А я не против. Поэтому каждый поступит в соответствии со своими хотелками и останется доволен. А если не останется, то пусть подумает, что он сделал для хип-хопа.

Virtuos86 ★★★★★
()
Ответ на: комментарий от mcgeek

Забьешь на клиент, то есть? Мудрое решение :-).

Virtuos86 ★★★★★
()
Ответ на: комментарий от Virtuos86

тут понимаешь, какая штука: макском не просто так обосрался pr принимать - первые варианты ломали очень много всего из-за того, что код лора нормально тестами не покрыт и безобидные изменения бабахали очень много где.

тч по уму нужно делать не сраный костыль в ввиде парсинга хтмл (который в рамках снижения нагрузки на сервер и зарезать можно), а сесть и покрыть код хотя бы процентов на 60 (учитывая, что макском не сильно парился трайкетчами и обработкой исключений - более чем достаточно) юнит-тестами. после этого - можно с ноги открывать дверь к макскому и либо требовать принятия пуллреквестов, либо требовать поднимать апи рядом с лором, натравливать его на ту же базу и не ковырять мозг постоянными мержами.

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

anonymous
()
Ответ на: комментарий от anonymous

Дельные мысли. Кто будет покрывать кодобазу тестами?

уж лучше бросить клич в толксах, чем хтмл парсить

Можно не парсить хтмл, можно строить DOM.

Virtuos86 ★★★★★
()
Ответ на: комментарий от anonymous

Вот тут ты прав, что-то там ломается, тесты проходят, а потом оказывается, что ты все поломал.

mcgeek
() автор топика
Ответ на: комментарий от hizel

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

а скала там или дельфи с 1С - монопенисуально по большему счёту

anonymous
()
Ответ на: комментарий от mcgeek

а можешь сделать и расшарить куда-нибудь кавереджрепорт для существующих тестов? нужно только подобрать грамотно тулзу, jacoco скорее всего много чего не подхватит, учитывая мешанину из вообще всего в коде

anonymous
()
Ответ на: комментарий от mcgeek

Пофикшена ошибка с загрузкой тем при скроллинге.

mcgeek
() автор топика

Анонс обновления

На данный момент в локальной ветке реализовано:

  • полноценное отображение списка новостей, галереи, трекера, разделов форума и тем в разделах. Вид списков в процессе переделывания на более компактный. Количество комментариев в трекере все еще не отображается, т.к. он использует API. Кстати, API можете закапывать.
  • иконки в панели навигации
  • прогрессбар при загрузке контента и отображение сообщений об ошибках (сети, парсера, пустой ответ) вместо списка
  • кнопка перехода комментариев заменена на иконку и перенесена наверх
  • добавлена кнопка обновления контента при нахождении внизу списка, чтобы не листать наверх
  • приложение больше не падает при повороте экрана
  • теги заменены на хештеги, но это, как и тему, можно будет менять в настройках
  • пофикшено большое количество багов

Скриншоты

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

mcgeek
() автор топика
Ответ на: Анонс обновления от mcgeek

Здорово уже то, что ты возобновил работу над клиентом. Из него уже можно срать в комменты? Undo/redo в текстовом поле отправления комментария — вот это было бы неплохо.

Virtuos86 ★★★★★
()
Ответ на: комментарий от Virtuos86

Функции, требующие авторизации, пока не разработаны. Авторизация и поддержка уведомлений будут реализованы в скором времени.

mcgeek
() автор топика
Ответ на: комментарий от mcgeek

Было бы неплохо. Тестировать как-то веселее будет, всё-таки ЛОР это форум, без комментариев ни куда.

Virtuos86 ★★★★★
()
Ответ на: комментарий от DeadEye

Итак, встречайте v.0.9.0!

Изменения по сравнению с анонсом (см. выше):

  • Более компактный интерфейс
  • Можно включить ночной режим в настройках
  • В трекере отображаются комментарии

Как это выглядит

Также, теперь можно следить за готовностью релизов на прямо на гитхабе.

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

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

Все, что касается авторизации, будет сделано в 1.1.0, который намечен на апрель. Поскольку мне лень искать ограничения по скору (с какого момента можно удалять сообщения и т.п.) просьба написать тут списочек, чтобы потом не писать лишних правил парсинга.

Свежий APK

najlus NeverLoved Virtuos86, кастаните еще кого забыл.

mcgeek
() автор топика
Ответ на: комментарий от mcgeek

Если вдруг кто захочет порадовать меня денежкой - кошельки в профиле. Спасибо :)

Сколько заплатить чтобы ты свернул все работы по АПИ и приложению?

Я серьезно!

anonymous
()
Ответ на: комментарий от mcgeek

maxcom, перелогинься

Ты раскрыл меня, теперь я должен тебя убить.

anonymous
()
Ответ на: Анонс обновления от mcgeek

Кстати, API можете закапывать

а я только собирался разгрести авгиевы конюшни и покрыть нормально код тестами, хе-хе. на гитхабе-то все твои наработки или что-то не коммитил?

vostrik ★★★☆
()
Ответ на: комментарий от vostrik

Коммитил все. Код в пуллреквестах надо пересмотреть, многого не хватает. Да и смысл ждать теперь API, если неизвестно когда их примут и прекрасно все парсится так?

mcgeek
() автор топика
Ответ на: комментарий от mcgeek

смысл ждать теперь API, если неизвестно когда их примут и прекрасно все парсится так?

нормальная авторизация, гораздо меньшая нагрузка на сервер, расширяемый и менее костыльный код. по поводу «примут» - я аноном где-то тут писал, что для нормального принятия коммитов должна быть хотя бы уверенность в том, что CI отлавливает p0 и p1 баги, после этого можно уже думать о приеме коммитов. в честь возвращения собирался как раз заняться этим

vostrik ★★★☆
()
Вы не можете добавлять комментарии в эту тему. Тема перемещена в архив.