Ayuget / Redface

Android client for HFR's forum
Apache License 2.0
42 stars 21 forks source link

Correction bug messages qui disparaissent #209

Open A-Louarn opened 4 years ago

A-Louarn commented 4 years ago

J'ai commencé à regarder pour corriger les posts qui disparaissent (souvent après un message de la modération).

J'ai corrigé le problème mais le rendu des posts est assez moche, et je ne suis pas certain de comment le réparer.

Le crawler de post a été passé des regexps à un parser HTML (JSoup) qui devrait être plus simple à maintenir et poser moins de problème avec les agencements non standards du forums.

Ayuget commented 4 years ago

Merci, je vais étudier ça le week-end prochain en rentrant de vacances.

Je n'avais pas utilisé jSoup volontairement car c'était vraiment moins performant que les regex a l'époque. Mais t'as raison il faudrait que je refasse des bench

Ayuget commented 4 years ago

Car c'est sûr qu'utiliser des Regex c'est en soit tout pourri. Sur desktop je ne me serais jamais posé la question.

De mémoire quand j'ai commencé à coder l'app il y a 4 ans, c'était assez flagrant la différence de perf entre jSoup et les regex

A-Louarn commented 4 years ago

Je n'ai pas l'impression d'avoir des ralentissements notables, mais il est possible que ce soit le cas. Si c'est trop génant, il est possible d'utiliser un parser SAX au lieu de DOM, ça devrait donner un boost de performance à moindres frais (et AMHA être presque aussi rapide que les regexp).

Dans tous les cas, ça serait une bonne idée de faire des bench, oui. Si tu sais comment faire, ça m'intéresse.

Ayuget commented 4 years ago

Tu peux tester ça avec JMH c'est pas super compliqué à mettre en place.

Merci pour ton travail en tout cas ça fait super plaisir !!!

A-Louarn commented 4 years ago

Je regarderai JMH quand j'aurai du temps.

De rien, c'est toujours sympa de bosser sur une appli que j'utilise tous les jours :smile:

A-Louarn commented 4 years ago

Les quotes & spoilers sont fixées :wink:

J'ai eu un bug marrant en corrigeant les quotes : celles créées avec un "[quote]" n'étaient pas fermées et donc tout le resté de la page était dans un bloc de quote :smile:

Reste plus qu'à supprimer les signatures des messages et ce devrait être prêt pour la prod.