|
You seemed to have ignored the reader; and context ... as long as the formatting is automated, it's not OCD.
"Before entering on an understanding, I have meditated for a long time, and have foreseen what might happen. It is not genius which reveals to me suddenly, secretly, what I have to say or to do in a circumstance unexpected by other people; it is reflection, it is meditation." - Napoleon I
|
|
|
|
|
THIS!
To err is human to really elephant it up you need a computer
|
|
|
|
|
Curious, for context.
First, are the coding standards well-documented/understood and followed by everyone?
Second, do you have "allowable exceptions"?
Personally, you have coding standards for a reason. And they should be defaulted in a modern editor, and shared.
I have worked on 20+ year old code. Even a bad coding standard beats a lack of any standard.
Properly formatted (and consistently formatted) code is less error prone, and easier to read/spot bugs, from my experience.
But exceptions should be allowed when the add to the readability/understanding of the code.
|
|
|
|
|
Is your style guide that precise? Or is this just the manager's personal preference?
Either way, it's a quick fix and is not at all reflective of your skill as a developer. When I get that kind of comment, I count it all good, because the reviewer had to try really hard to find things to fix before pushing my code.
|
|
|
|
|
Cpichols, very good point well made. IMHO, when it comes to formatting developers spend far too much time and effort (this thread is a good example) shouting at each other "I'm right and you're wrong!" If a reviewer raises a pedantic issue concerning formatting, just suck it up, fix it and move on. You probably have more important things that really are worth arguing about.
|
|
|
|
|
Maybe I'm way off base on this, but when it comes to style I ask 2 questions:
Does it affect the functionality? Both ways would compile exactly the same, so no.
Does it make the code harder/easier to read and understand? Any dev confused by the curly braces or lack there of needs to find a new line of work.
|
|
|
|
|
20 years developer here. One of the most annoying thing for me is lack of coding standard and inconsistency around the code.
That just not make any sense for me, just make reading, understanding and following the code more difficult.
When I'm coding I think every time "what if it's me reading this code in 2 years?".
So - for me - bad formatting is very very annoying, because I want my brain to focus on the functionality and not to spend energy jumping around differents brackets style.
As example I found very difficult to read a IF without brackets, or 1-liner (like your case).
The worst case is when I need to read code written by the smart guy that have 12 logical steps in one line (like 200 chars long line, making calls to 2/3 functions) that I could split in 8 lines with good variable names explaining what's happening there... soooo easy to follow and understand, debug, etc. But wait... usually that code comes from the dev that have very low quality and a lot of bugs, as they never test/debug the code, because they think that "it works for sure". And it's pretty obvious they cannot debug their code, as it's a multi step singleline with a high cyclomatic complexity... that could be impossible to do!
So in my experience I have associated bad coding styles to low-quality-developer. And as today I have never meet an exception to this my "personal" rule.
So just stick to your company coding style, if doesn't exist, make one, otherwise you cannot complain. Many IDE have linter, editorconfig or similar.
But to be honest, your manager is not "micro-managing" you, but just saying his opinion and giving you a feedback. That's the goal of a code review, is it?
Usually developer have poor communication skills, this I think is a good example of it.
|
|
|
|
|
Uniformity in code is simply for facilitating skimming through it.
I think they call it K&R convention when you follow the function header with a newline and then a bracket '{'?
Anyways I didn't before and I reformed. That and another convention about never going past 80 columns in any line so you never have to use the horizontal scroll bar (and something else, I think the ol' camel case thing)
But to this day all my quick and dirty functions or variables start with my initials. I clean it up later to make it less personal (especially the really dirty names). But it almost seems religious for most C/C++ programmers that we continue to use as much short hand in our naming conventions as possible (nobody wants carpal tunnel syndrome, especially if you have to quickly debug in 'vi')
|
|
|
|
|
Intervention[^]
"I have no idea what I did, but I'm taking full credit for it." - ThisOldTony
"Common sense is so rare these days, it should be classified as a super power" - Random T-shirt
AntiTwitter: @DalekDave is now a follower!
|
|
|
|
|
Not clicking on that. Got a synopsis?
|
|
|
|
|
Yeah - itβs on the Internet you talking monkey!
If you can't laugh at yourself - ask me and I will do it for you.
|
|
|
|
|
PIEBALDconsult wrote: Got a synopsis?
A girl named Danae peeks into a sewage drain pipe. She calls out for someone named Lars. Then a small craft exits the drain pipe and hovers next to the little girl. Danae then laments about all the stupidity she sees in the world today. A small blue-skinned being responds by insulting the human race and calling all humans dumb monkeys. The alien says they have tried to help humanity. Danae was curious how the aliens have helped. She simply inquiries "What did you do?" to which the alien responds "We gave you the internet!".
The scene ends with the girl Danae lowering her head and sulking away. The comic strip is designed to lead the reader towards the conclusion that the internet is the source of the problems we face today.
|
|
|
|
|
It's a Nonsequitor cartoon.
|
|
|
|
|
If OG posts a link it's ok to click
"Life should not be a journey to the grave with the intention of arriving safely in a pretty and well-preserved body, but rather to skid in broadside in a cloud of smoke, thoroughly used up, totally worn out, and loudly proclaiming βWow! What a Ride!" - Hunter S Thompson - RIP
|
|
|
|
|
|
I need a sobbing react.
To err is human. Fortune favors the monsters.
|
|
|
|
|
| π
Did you ever see history portrayed as an old man with a wise brow and pulseless heart, weighing all things in the balance of reason?
Is not rather the genius of history like an eternal, imploring maiden, full of fire, with a burning heart and flaming soul, humanly warm and humanly beautiful?
--Zachris Topelius
|
|
|
|
|
Of all their various ideas, restricting knowledge to vetted individuals is far from the church's worst one.
People that need the instruction "Do not iron this garment while wearing" are a prime example of a cohort whose access should be limited.
Think how much faster and easier driving would be if certain people never held a license. 
|
|
|
|
|
Wordle 298 5/6
β¬β¬β¬β¬β¬
β¬β¬π©β¬β¬
β¬π©π©β¬β¬
π©π©π©β¬π©
π©π©π©π©π©
|
|
|
|
|
Wordle 298 5/6
β¬β¬β¬β¬π¨
β¬π¨β¬β¬π¨
π©β¬π©β¬β¬
π©π©π©β¬π¨
π©π©π©π©π©
|
|
|
|
|
Wordle 298 4/6
π¨β¬β¬β¬π¨
β¬β¬β¬β¬β¬
β¬π¨β¬π¨β¬
π©π©π©π©π©
Software rusts. Simon Stephenson, ca 1994. So does this signature. me, 2012
|
|
|
|
|
4 again:
Wordle 298 4/6
β¬β¬β¬β¬β¬
🟨🟨β¬β¬β¬
β¬🟨🟨🟨🟨
🟩🟩🟩🟩🟩 tThe 3rd try results were a surprise, and it took me a few minutes to work out the 4th ...
"I have no idea what I did, but I'm taking full credit for it." - ThisOldTony
"Common sense is so rare these days, it should be classified as a super power" - Random T-shirt
AntiTwitter: @DalekDave is now a follower!
|
|
|
|
|
4/6
β¬β¬β¬β¬β¬
β¬β¬π©π©β¬
β¬π©π©π©π©
π©π©π©π©π©
GCS/GE d--(d) s-/+ a C+++ U+++ P-- L+@ E-- W+++ N+ o+ K- w+++ O? M-- V? PS+ PE Y+ PGP t+ 5? X R+++ tv-- b+(+++) DI+++ D++ G e++ h--- r+++ y+++* Weapons extension: ma- k++ F+2 X
|
|
|
|
|
Wordle 298 6/6
β¬π¨β¬π©β¬
β¬β¬β¬β¬β¬
β¬β¬π¨β¬β¬
β¬β¬π©π©β¬
β¬β¬π©π©π©
π©π©π©π©π©
Phew, hard one!
|
|
|
|
|
Impressive it says
Wordle 298 3/6*
β¬β¬β¬π¨β¬
π©β¬π©β¬β¬
π©π©π©π©π©
"It is easy to decipher extraterrestrial signals after deciphering Javascript and VB6 themselves.", ISanti[ ^]
|
|
|
|
General 
News 
Suggestion 
Question 
Bug 
Answer 
Joke 
Praise 
Rant 
Admin 
Submit an article or tip
