Welcome to Code Chatter, your go-to series for conversational coding insights. What makes this series of questions different from all the others? Well, truth be told, not much, but they're still thought-provoking and fun. Join us as we explore the coding world, one witty question at a time.
What's your secret 'code pet peeve' that, while not critical, drives you crazy?
Follow the DEVteam for more discussions and online camaraderie!
Latest comments (36)
I have 3 that really get to me:
inconsistent code formatting. i’m fine adopting different patterns or standards, but that pattern should be consistent throughout the codebase.
when someone says “we’ll circle back and fix this later”…in my experience…no we won’t. unless this is a prototype and the team understands that this code will be thrown away, let’s do it correctly now.
duplicate code. it may be easier for you in the moment to copy a chunk of code to get something working, but it causes so much more pain later. invest a little time now rather having to pay interest on that time later.
👏😁
Exceptions-as-control-flow, few things get me upset faster.
No comments, and when (in Python), not all of the import statements are at the top.
It’s code-ish, but I ran into this doing a review with one of our juniors today. She was working on the mobile side of a site, and had the hamburger menu on the left…that’s not a problem. The problem is, the menu opened from the right. It’s so minor as to be pointless, but it drives me crazy to see that…
Well to be fair if the spec she was given didn't specify what the hamburger button was supposed to do, she could have made it randomly open at the top/bottom/left/right and it would have been just as correct 😉
She wasn’t give a spec for it, and I didn’t say anything about it…I suffered in silence… 😂
Long, long functions. More than 2 levels of loop nesting. More than 2 levels of conditionals nesting.
Not splitting long functions is like writing an email without paragraphs.
🪓
Line length limit standards predicated on screens narrower than the narrowest screens we have today (not moved with the times). Aka PEP8.
Upper-case SQL keywords. It looks like someone's crazy uncle sent out another email forward and the illegibility gives me a headache.
For me, it's over optimizing code to the point that whilst short and elegant, it is not readable at a glance for those that might not live in the code base every day.