Hello Guirec, C'est plutôt bien écrit, avec une intro et tout et tout, bravo.
Pour le contenu, j'ai par contre toussé sur le TokenGenerator. Du coup j'ai deux trois innocentes questions ouvertes. La classe TokenGenerator n'illustre-t-elle pas un cas "comiquement dégénéré"? J'ai l'impression que tu échanges juste un SecureRandom.urlsafe_base64 par un TokenGenerator.new.call . J'ai du mal à voir où on y gagne vraiment. Au final, si tu perds de vue que ta variable est un token_generator, tu es perdu. Le nom de méthode "call" n'apporte pas beaucoup d'information en utilisant le "duck-typing" (dans ton exemple, "call" est vraiment surchargé). Est-ce que tu juges que urlsafe_base64 contient trop de détails? On dirait que tu essayes d'avoir deux niveaux à chaque fois: un niveau élémentaire avec des considérations spécifiques détaillées (atomiques?) et un niveau de "groupage" qui combine ces responsabilités (moléculaire?). Si tel est le cas, penses tu que cette approche puisse avoir des inconvénients? Cheers, --Lucas Le 27 juillet 2014 20:56, Tim <[email protected]> a écrit : > Salut > > Je pense que tu as écris présidence au lieu de persistance (dans le > dialogue final) > > J ai une question.: Est ce que si on va au bout du concept il ne faudrait > pas creer un UserValidator ? Est ce que c est au model de se valider lui > meme ? > Le 27 juil. 2014 20:36, "Guirec Corbel" <[email protected]> a écrit > : > > Bonjour, >> >> Je viens de publier un article sur le SRP : http://t.co/UY8LTE7T0w. >> >> N'hésitez pas à m'envoyer vos commentaires. >> >> -- >> -- >> 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] >> --- Vous recevez ce message, car vous êtes abonné au groupe Google >> Groupes Railsfrance. >> Pour vous désabonner de ce groupe et ne plus recevoir d'e-mails le >> concernant, envoyez un e-mail à l'adresse railsfrance+unsubscribe@ >> googlegroups.com. >> Pour plus d'options, visitez le site https://groups.google.com/d/optout . >> > -- > -- > 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] > --- > Vous recevez ce message, car vous êtes abonné au groupe Google Groupes > "Railsfrance". > Pour vous désabonner de ce groupe et ne plus recevoir d'e-mails le > concernant, envoyez un e-mail à l'adresse > [email protected]. > Pour obtenir davantage d'options, consultez la page > https://groups.google.com/d/optout. > -- -- 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] --- Vous recevez ce message, car vous êtes abonné au groupe Google Groupes Railsfrance. Pour vous désabonner de ce groupe et ne plus recevoir d'e-mails le concernant, envoyez un e-mail à l'adresse [email protected]. Pour plus d'options, visitez le site https://groups.google.com/d/optout .
