Comments on: The Rails Way http://c.matallin.com/2009/04/the-rails-way/ geek, blogger, entrepreneur and web designer. Thu, 26 Nov 2009 15:38:22 +0100 http://wordpress.org/?v=2.8.4 hourly 1 By: Rails Rumble ‘09 - Carlos Matallín http://c.matallin.com/2009/04/the-rails-way/#comment-1675 Rails Rumble ‘09 - Carlos Matallín Mon, 12 Oct 2009 03:05:06 +0000 http://c.matallin.com/?p=527#comment-1675 [...] really felt like taking part in such an event while commiting (nice word, huh?) one of my new year’s resolutions, so I proposed this to Felipe Talavera one of my partners in the startup we cofounded almost a year [...] [...] really felt like taking part in such an event while commiting (nice word, huh?) one of my new year’s resolutions, so I proposed this to Felipe Talavera one of my partners in the startup we cofounded almost a year [...]

]]> By: loki http://c.matallin.com/2009/04/the-rails-way/#comment-797 loki Wed, 15 Apr 2009 13:46:47 +0000 http://c.matallin.com/?p=527#comment-797 I think <a href="http://oreilly.com/catalog/9780596516178/toc.html" rel="nofollow"> The Ruby Programming Language</a> is a "must read". Everybody will tell you to have the <a href="http://www.amazon.com/Rails-Way-Addison-Wesley-Professional-Ruby/dp/0321445619" rel="nofollow"> The Rails Way </a> so I will point you to something completely different, but as valuable as anything else. called <a href="http://www.amazon.com/Pragmatic-Programmer-Journeyman-Master/dp/020161622X/ref=sr_1_1?ie=UTF8&s=books&qid=1239802895&sr=1-1" rel="nofollow">The Pragmatic Programmer</a>. For the authentication process in your Rails apps, all the cool kids are switching to <a href="http://github.com/binarylogic/authlogic/tree/master" rel="nofollow"> authlogic </a> because of the better and more understandable code base. I have not wisdom but I'm damn hot. You can't have it all. ^_^ I think The Ruby Programming Language is a “must read”. Everybody will tell you to have the The Rails Way so I will point you to something completely different, but as valuable as anything else. called The Pragmatic Programmer.

For the authentication process in your Rails apps, all the cool kids are switching to authlogic because of the better and more understandable code base.

I have not wisdom but I’m damn hot. You can’t have it all. ^_^

]]> By: Carlos Matallín http://c.matallin.com/2009/04/the-rails-way/#comment-795 Carlos Matallín Wed, 15 Apr 2009 11:46:42 +0000 http://c.matallin.com/?p=527#comment-795 It makes a lot of sense. I didn’t modified t because it came with the restful_authentication plugin and I thought if I changed something the user authentication wouln’t work. Anyway I will refactor it, thanks for the advise. It makes a lot of sense. I didn’t modified t because it came with the restful_authentication plugin and I thought if I changed something the user authentication wouln’t work. Anyway I will refactor it, thanks for the advise.

]]> By: Carlos Matallín http://c.matallin.com/2009/04/the-rails-way/#comment-793 Carlos Matallín Wed, 15 Apr 2009 11:43:21 +0000 http://c.matallin.com/?p=527#comment-793 Hi loki, first of all I will add that link as soon as I end writting this comment. Take for granted my intention is not leave their credit behind and I have also thanked them so much in the mailing list. I just skipped it, sorry ;) About getting to grips with ruby I am reading the "Agile Web Development with Rails" and every document I think is worth to spend some minutes to read (some examples http://delicious.com/matallo/rails) but I will be so grateful if somebody more experienced could give me advise about some resources to help my rails learning. No way I think I will wake up one morning knowing rails! Anyway, most of the code comes with the restful_authentication plugin, but I think I will have to refactor it a bit and adapt it for my needs. Thanks for your wisdom. Hi loki, first of all I will add that link as soon as I end writting this comment. Take for granted my intention is not leave their credit behind and I have also thanked them so much in the mailing list. I just skipped it, sorry ;)

About getting to grips with ruby I am reading the “Agile Web Development with Rails” and every document I think is worth to spend some minutes to read (some examples http://delicious.com/matallo/rails) but I will be so grateful if somebody more experienced could give me advise about some resources to help my rails learning. No way I think I will wake up one morning knowing rails!

Anyway, most of the code comes with the restful_authentication plugin, but I think I will have to refactor it a bit and adapt it for my needs.

Thanks for your wisdom.

]]> By: dagi3d http://c.matallin.com/2009/04/the-rails-way/#comment-792 dagi3d Wed, 15 Apr 2009 09:56:08 +0000 http://c.matallin.com/?p=527#comment-792 hi, I think there is one point where you could refactor your code a little bit. You are using this piece of code to check everything went ok while recording the user: <code> success = @user && @user.save if success && @user.errors.empty? </code> why not just use the #save method? It has no sense to check if the @user variable is a nil or not because it will *never* be because you have instantiated it just one line above. Eventually, when the #save method returns a true value, it means that everything went fine and there are no errors, so it's a little bit redundant to check if the errors list is empty or not: <code> if @user.save ... </code> hi,
I think there is one point where you could refactor your code a little bit. You are using this piece of code to check everything went ok while recording the user:

success = @user && @user.save
if success && @user.errors.empty?

why not just use the #save method? It has no sense to check if the @user variable is a nil or not because it will *never* be because you have instantiated it just one line above.
Eventually, when the #save method returns a true value, it means that everything went fine and there are no errors, so it’s a little bit redundant to check if the errors list is empty or not:


if @user.save
...

]]> By: loki http://c.matallin.com/2009/04/the-rails-way/#comment-791 loki Wed, 15 Apr 2009 09:12:12 +0000 http://c.matallin.com/?p=527#comment-791 In my humble opinion, it would be a good idea to get to grips with ruby before writing too much Rails. Your code shows clear signs of strong type language habits (are you sure you want to be so defensive on your create method?). The gap between C and Ruby it's quite big, so don't be fooled thinking it will happen overnight. Another thing related to codding, and with everything else, is love. In this, case love for the awesome community Ruby and Rails have. Don't be afraid to show your love for the people who really guide you to the process your article is talking about (not me BTW), like the folks on the ror-es list, instead of linking the same articles from Obie, Jamis, etc that everybody has already read or is going to do so anyway. In my humble opinion, it would be a good idea to get to grips with ruby before writing too much Rails. Your code shows clear signs of strong type language habits (are you sure you want to be so defensive on your create method?). The gap between C and Ruby it’s quite big, so don’t be fooled thinking it will happen overnight.

Another thing related to codding, and with everything else, is love. In this, case love for the awesome community Ruby and Rails have. Don’t be afraid to show your love for the people who really guide you to the process your article is talking about (not me BTW), like the folks on the ror-es list, instead of linking the same articles from Obie, Jamis, etc that everybody has already read or is going to do so anyway.

]]>