Skip to content

Correction bug messages qui disparaissent#209

Open
A-Louarn wants to merge 9 commits into
ForumHFR:masterfrom
A-Louarn:master
Open

Correction bug messages qui disparaissent#209
A-Louarn wants to merge 9 commits into
ForumHFR:masterfrom
A-Louarn:master

Conversation

@A-Louarn

Copy link
Copy Markdown

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.

Refactoring: HTML parser instead of regexp. Should be easier to
maintain.
Bugfix: disappearing messages after moderation message
@Ayuget

Ayuget commented Sep 30, 2019

Copy link
Copy Markdown
Collaborator

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

Ayuget commented Sep 30, 2019

Copy link
Copy Markdown
Collaborator

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

Copy link
Copy Markdown
Author

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

Ayuget commented Sep 30, 2019

Copy link
Copy Markdown
Collaborator

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

Copy link
Copy Markdown
Author

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 😄

@A-Louarn

Copy link
Copy Markdown
Author

Les quotes & spoilers sont fixées 😉

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 😄

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants