Déjà ce genre de code devrait se trouver dans un controller.
Ensuite le if elsif is_a? c'est immonde, en Ruby on évite au maximum de discriminer les objets en fonction de leur classe on utilise plutôt du polymorphisme.
Et à la rigueur quand on a pas le choix on utilise la structure case: case obj
when Integer
# blah
when Array
# blah
Et pour finir même en oubliant les points précédents cette méthode pourrait être très fortement factorisée.
Et encore là j'ai juste pris deux modèles au hasard, mais je pourrais continuer comme ça un moment.
Enfin bref tout plein de code pourri qui laisse supposer qu'ils ont appris Rails sur le site du zéro cet été. On pousserai quelque chose comme ça dans mon dépôt ce serait git revert tout de suite.
Et avant que tu me dise "c'est libre contribue" sache que je compte le faire dès ce soir ;)
[^] # Re: Doués
Posté par Jean B . En réponse au journal Diaspora is real. Évalué à 10.
Déjà ce genre de code devrait se trouver dans un controller.
Ensuite le if elsif is_a? c'est immonde, en Ruby on évite au maximum de discriminer les objets en fonction de leur classe on utilise plutôt du polymorphisme.
Et à la rigueur quand on a pas le choix on utilise la structure case:
case obj
when Integer
# blah
when Array
# blah
Et pour finir même en oubliant les points précédents cette méthode pourrait être très fortement factorisée.
Autre chose: http://github.com/diaspora/diaspora/blob/master/app/models/p(...)
La bibliothèque standard de Ruby fournit une classe URI::HTTP pour faire ce genre de chose.
Plus l'assignation en dur de http://localhost:3000/ . Rails fournit un système d'environnement c'est pas pour les chiens.
Et encore là j'ai juste pris deux modèles au hasard, mais je pourrais continuer comme ça un moment.
Enfin bref tout plein de code pourri qui laisse supposer qu'ils ont appris Rails sur le site du zéro cet été. On pousserai quelque chose comme ça dans mon dépôt ce serait git revert tout de suite.
Et avant que tu me dise "c'est libre contribue" sache que je compte le faire dès ce soir ;)