Discussion :

[copirate]

J' utilise dans un controleur:

find(:all, :conditions=>["nom= :nom or prenom= :prenom", params])

J' ai une vue avec un formulaire de recherche qui renvoie vers une action du controleur dans admin, mais je n'arrive à rédiger le code pour params et renvoyer le contenu de la table de hachage vers ma requête find.

de plus est-il possible d'avoir des conditions en or et and.

Merci

[titoumimi]

Bonjour et bienvenue à toi

Tu devrai plutôt essayer sous cette forme :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
 
mon_nom = "toto"
mon_premom = "tata"
 
MonModele.find(:all, :conditions=>["nom= ? OR prenom= ?", mon_nom, mon_premom])

[ghost-x]

Il y a aussi une autre façon d'écrire la requéte (ni pire ni meilleur), je la préfère
parce que plus facilement lisible:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

 MonModel.find_all_by_nom_and_prenom(nom, prenom)

[SekYo]

Me semblait avoir lu, je sais plus où, que l'utilisation des finders dynamiques étaient plutot déconseillée, ils sont assez couteux en terme de performance (vu qu'ils sont générés a la volée via un missing_method ou truc du genre de mémoire).

[Zfred]

Me semblait avoir lu, je sais plus où, que l'utilisation des finders dynamiques étaient plutot déconseillée, ils sont assez couteux en terme de performance

Peux tu retrouver la source de cette info ?
Au contraire je trouve que la solution du "find_all_by_nom_and_prenom" est trés propre.
Ca donne une bonne lisibilité du code.
Ensuite, c'est completement dans le sens de Rails : Plutot charger le modele, et faire des controleurs simples, lisibles et concis.
Autre avantage, si le finder doit etre réutilisé, il est déja factorisé dans le modele, c'est plutot bon ça !

[Mookie]

J'ai également lu que les finders dynamiques sont moins performants :

http://antoniocangiano.com/2007/02/1...rformance-tips

[SekYo]

Je l'avais lu au même endroit, merci.

Par contre Zfred, ca t'empeche pas de charger ton modele, simplement il vaut mieux redéfinir une methode a la mano dans ton model ( par exemple find_all_by_fullname ) dans laquelle tu appelles find(:conditions =>... ) plutot que d'utiliser le find_all_by_nom_and_prenom ( qui est effectivement lisible par contre, je suis bien d'accord )