Merci pour ces précieuses remarques (en fait, j'ai balancé un peu vite le code sans vraiment nettoyer). Dès que j'ai trouvé un endroit un peu plus approprié que mon blog pour publier ce code (en intégrant tes remarques), je refais signe.
++ yk Le 10/05/07, Jean-François<[EMAIL PROTECTED]> a écrit : > Yann : > > Voici donc le code que j'ai pondu pour me connecter aux serveurs de > > paiement électronique du Crédit Mutuel / CIC : > > > > http://strasslab.net/blog/index.php?2007/05/10/247-petit-intermede-technique > > Merci pour le code. > > Mes premières remarques, parfois cosmétiques : > > - Dans la méthode show, il y a : > > @data = @tpe + "*" + @date + "*" + @montant + "*" + @reference + > "*" + @texte_libre + "*" + @version + "*" + @lgue + "*" + @societe + "*" > > j'aurais tendance à préférer : > > @data = "[EMAIL PROTECTED]@[EMAIL PROTECTED]@[EMAIL PROTECTED]" > > (car on crée moins d'objets String, au détriment de la lisibilité) > > ou dans ce cas-ci : > > @data = [ @tpe, @date, @montant, @reference, @texte_libre, > @version, @lgue, @societe ].join("*") + "*" > > De plus @data ne sert pas dans la vue, donc une variable locale > suffit non ? data > > - Dans la méthode confirm : > Il y a deux fois : @mac = params[:MAC] > > et écrire @mac = params[:MAC].downcase > > ... permet d'avoir une conditionnelle plus claire je trouve : > > if @mac == hmac(@data) > > - S'il y a un render :text à la fin de la méthode #confirm, > pourquoi avoir des variables d'instance @tpe, @date... > alors que des variables locales suffisent ? > > - à quoi sert @result ? > > - je ne suis pas trop adepte du += > > @data += "..." > > pour moi, écrire @data << "..." économise un objet String > > - Dans le render :text il y a un Time.now.strftime("%d/%m/%Y:%H:%M:%S") > visiblement le même que dans #show, tu gagnerais à le > factoriser, pour n'avoir qu'un seul endroit pour modifier > le format de la date, si modification ultérieure il y a. > > - enfin si le formulaire a toujours la même forme, on peut > peut-être en faire un helper ? > et dans la vue on aurait un : > <%= cybermut_form(:tpe => @tpe, :reference => @reference, ...) %> > > un truc dans ce genre. > > Voilà pour l'instant, j'ai pas encore lu le pdf dans le Kit, mais > ça a l'air assez intéressant tout ça. > > -- Jean-François. > > -- > À la renverse. > > > > --~--~---------~--~----~------------~-------~--~----~ 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] -~----------~----~----~----~------~----~------~--~---
