Follow

Si quelqu'un a une idée pour refactorer ça, je suis preneuse. 😂
J'ai pris toute la matinée pour pondre ça… à un moment, j'hésitais à faire appel à un ami, dépitée.
Mais ça fonctionne ! \o/

· · Web · 3 · 0 · 0

@Alyve Ça ne me paraît pas si mal ça, pourquoi refactorer ? 🤔

@Alyve C'est bien du Kotlin ou mes yeux m'abuse...
Sinon je regarde demain mais je crois que tu dois avoir accès à de l'asynchrone pour remplacer les thread non?

@Alyve En l’état, ça se tient.

Peut-être créer un modèle (ou un trait) `Visit` qui serait en relation avec ton modèle `Thread`. Ça réduirait probablement la requête, et tu pourrais même déporter peut-être utiliser le Paginator natif. À voir si ça tient la route…

Au pire, une requête SQL native pour faire tout le ménage en une passe.

Deux petites améliorations gratos :

1) renommer `mostTrended` en `trended`.

2) remplacer `sortBy(/* things */)->reverse();` par `sortByDesc(/* things */);`.

Sign in to participate in the conversation
Hostux.social

This service is offered by hostux.net, visit our website to discover all the free services offered.
Beer, privacy and free software lovers. Join us!