Thoughtbot's Style Guide for Ruby about intermission-assignments HOT 5 CLOSED

Well, none of it really made me jump out of my chair. I do think it's really important to have a set style, especially when so many things in Ruby are optional. The thing that did make me grumble a little bit was the "don't misspell" thing, I don't see why that was necessary, as if some developers misspell things on purpose as a personal style choice... (I misspell things a lot, but not on purpose, I usually correct it when it's pointed out)

from intermission-assignments.

There are definitely a few rules I had not been following like using detect over find and adding a trailing comma but I agree with Kristina none of it seemed unreasonable and it is good to pick a style and stick with it.

from intermission-assignments.

I agree with Kristina and Michelle that nothing really stood out as something I'd hadn't seen before (or something that Hound pointed out). I think in general it's important to get used to following some type of style guide. Presumably the job you'll get will have one, and if you plan to contribute to any large open source project (like Rails) they will definitely not merge your stuff in if it doesn't follow their guides.

from intermission-assignments.

This style guide makes me happy because it's everything I already do :) I have always hated ternary operators because I think they are hard to read, and I feel validated that they don't use them either. The protected over private is interesting. I didn't know you could do that. I like being able to follow a style guide so that code can be consistent not only throughout one project, but throughout all of them. Especially while working at a company, that way when reading someone else's code you will likely be able to follow it easier.

from intermission-assignments.

There was a lot I agreed with on the Style Guide and a few I disagree with. For example, I don't see problems using ternary operators for simple if statements. I also think find & find_all are more descriptive than detect & select. Also, we were taught to use private instead of protected. I have actually never seen protected until now.

from intermission-assignments.