|
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[ ^]
|
|
|
|
|
Can't work out what the Black Square could have been on go 4
"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
|
|
|
|
|
Wordle 298 5/6*
β¬β¬β¬β¬π¨
π¨β¬β¬β¬β¬
β¬π¨π¨π¨β¬
π©π©π©β¬π©
π©π©π©π©π©
Happiness will never come to those who fail to appreciate what they already have. -Anon
|
|
|
|
|
Wordle 298 3/6
β¬β¬β¬β¬β¬
β¬β¬π©π¨π©
π©π©π©π©π©
|
|
|
|
|
Which wordle are you guys playing?
|
|
|
|
|
Wordle at the New York Times website
|
|
|
|
|
I have a client that writes their own software.
They hired me last year to write a tool for them.
They're not a software company, but their services require these tools.
They've got a couple of websites and a mobile app for Android and iPhone.
It's all mostly updated to the latest versions of .NET with Angular front-ends and all hosted in Azure.
I've seen some docker files too, as well as npm, yarn, Karma, Protractor... All the tools that are, or once were, the popular framework of the day.
One app can have three projects (a back-end service, front-end for customers and front-end for employees), or just the one, kind of depending on when or why it was written.
And it's all written by one guy (who is also the CTO and a real kind of stereotypical programmer, apparently).
Now that one guy got sick, like not coming to work and also not able to do anything for months kind of sick.
And they called me in to take over his work.
It took the guy about two weeks to give me admin rights to everything.
And from that point onward I'll have to figure it out.
string documentation = null;
I'm currently looking for passwords in configs that I now have access to (after I found the configs to begin with).
Just created a whole new database for some multi-tenant application that really needed a new tenant ASAP.
This took me hours as opposed to minutes it would take the original guy.
I also just found out this is one of those three-project applications and I have to add another language to it/them.
Still don't know how to run any of these projects locally
Taking it one baby step at a time afraid I'll break something at every step...
Cool client though, very nice people, and understanding that at this point I can't make any guarantees.
I'm also kind of impressed this guy wrote all this and judging from a few projects is trying new things.
Still, the whole situation feels kind of weird.
And yes, they knew this was a risk, actually wanted to talk to me about being back-up sooner, but didn't get around to it.
|
|
|
|