> Peut on faire ceci plus joliment ? > if params[:ll].blank? > @ll = "41.14, -73.99" > else > @ll = params[:ll] > end
Salut. Pour moi, c'est TOUT ce bloc qui n'a rien à faire dans un contrôleur. Pour moi une action de contrôleur doit juste récupérer les paramètres, les envoyer à un modèle d'un seul coup et initialiser une ou deux variables globales maximum à partir du dit modèle. Ce n'est pas au contrôleur d'effectuer une logique quelconque sur les paramètres reçus, c'est au modèle de le faire. De plus, je ne parle même pas du fait qu'un modèle est plus facile à unit-tester qu'un contrôleur Rails. Avant Rails 3, il était relativement compliqué d'utiliser des modèles non reliés à une table de données encore moins de les mapper à un formulaire. Cela a donné de mauvaises habitudes aux développeurs et des actions de contrôleurs énormes bourrées de manipulation de paramètres et de if params[:truc] then @pouet =... else ... du vrai code spaghetti ! Depuis cela à changé. J'ai d'ailleurs fait une petite gem "helper" toute bête : https://github.com/novagile/basic_active_model Nicolas. -- Vous avez reçu ce message, car vous êtes abonné au groupe "Railsfrance" de Google Groups. Pour transmettre des messages à ce groupe, envoyez un e-mail à l'adresse [email protected] Pour résilier votre abonnement envoyez un e-mail à l'adresse [email protected]
