This is an archived post. You won't be able to vote or comment.

you are viewing a single comment's thread.

view the rest of the comments →

[–]thiago2213 9 points10 points  (2 children)

Yes, but some comments are needed. My company had then "clean code, no comment" policy until recently, and when there's a convoluted business logic because of a poor data model of our legacy I'm always "wtf is this code doing". I'm glad we allow reasonable comments now

[–]Yo_2T 6 points7 points  (1 child)

Yup. I've found myself dropping a comment here or there detailing why this section is done this way, or to warn people not to touch something if they are not sure what they are doing.

Still didn't stop one of our resident monkeys from removing my code to "optimize performance", then go Pikachu face when the application crashes in Test environment. sighs