|
It is not "agile" to document every decision you make when coding, meaning that the original dev probably had some reasons for building things the way they are. If there's an improvement to be made, I wanna know - it might teach me something and prevent me from making the same mistake in the future.
..but changing code that isn't broken needs a damned good reason - since any change has the potential to introduce new (and subtle) bugs.
Bastard Programmer from Hell
If you can't read my code, try converting it here[^]
"If you just follow the bacon Eddy, wherever it leads you, then you won't have to think about politics." -- Some Bell.
|
|
|
|
|
If the code works, and you're not tasked to work on it, don't "optimize" it. In fact, don't even reformat it. See if you can get that guy fired.
".45 ACP - because shooting twice is just silly" - JSOP, 2010 ----- You can never have too much ammo - unless you're swimming, or on fire. - JSOP, 2010 ----- When you pry the gun from my cold dead hands, be careful - the barrel will be very hot. - JSOP, 2013
|
|
|
|
|
#realJSOP wrote: See if you can get that guy fired. That is insane! If you get him fired, they'll give you MORE of his code!
|
|
|
|
|
If it works, I won't touch it.
".45 ACP - because shooting twice is just silly" - JSOP, 2010 ----- You can never have too much ammo - unless you're swimming, or on fire. - JSOP, 2010 ----- When you pry the gun from my cold dead hands, be careful - the barrel will be very hot. - JSOP, 2013
|
|
|
|
|
Code can be as complicated and convoluted as you like, as long as it's adequately documented.
The problem is that the worst offenders seem to not believe in documenting code.
I wanna be a eunuchs developer! Pass me a bread knife!
|
|
|
|
|
Super Lloyd wrote: Did it happen to you that a coworker refactored some (large amount?) of your code
Thankfully, I'm the only coder here for most of the last 20 years so I don't have that to worry about.
My task of the week has been putting lipstick on a pig...new interface/graphics/theme/icons for 94 forms in a legacy project. My designer made it difficult and chose a diagonal gradient and not all controls have transparency. When I get this one done, I have around 70 'add-on modules/utilities' that need rebranding. What I really need is an intern!
"Go forth into the source" - Neal Morse
|
|
|
|
|
Not exactly, but I've once talked to a co-worker about his mess of code both convoluted and hard to debug. According to him, it's simpler this way. Turned out he learned to code in the 60s and pretty much stuck with what he learned for half a century now.
|
|
|
|
|
* Yes, There are multiple points of view a developer can look at a task (or a job)
* If you gives the **beeeep*** about loosing your position (job) your point of view
is the correct to take (personal risk).
* If this job is probably the only developer job you will get for the next 6 months, you have to use your Emotional Intelligence forging a way threw the difficult organizational culture (politics) and accept the risk you will get the blame for accepting/following the work some other person forced you using.
sometimes a task you get assignd to you just in order to protect someone else with better score
in the organization political structure and you have no chance surviving it....
|
|
|
|
|
Not so much code, but documentation I wrote. They did a lot of pointless wordsmithing - changed fonts, removed page headers, linked sentences into compound sentences using semicolons, introduced spelling errors, and removed paragraphs in places. A section they added didn't adhere to the formatting in the rest of the document at all.
Fortunately this document lives in source control. When I needed to update the document recently, I could go back to my original. I added their new section, reformatting it so that it looked like the rest. I did adopt a couple of their wording changes.
Software Zen: delete this;
|
|
|
|
|
I had asked a less experienced programmer to add a new feature to an existing javascript module.
Some code in that module was building a fairly complicated DOM element for rendering into a page.
They change the logic from safe, DOM manipulation into a massive string concatenation because it was easier for them to read!
I had them test with some data that included HTML special characters, typical of our data domain. BARF!
Sometimes simpler means "not to spec", especially when security is involved.
Thank goodness for source control.
|
|
|
|
|
I often found that obfuscution is not needed to keep me out of my own code!
CQ de W5ALT
Walt Fair, Jr., P. E.
Comport Computing
Specializing in Technical Engineering Software
|
|
|
|
|
Haha!
|
|
|
|
|
A good friend of mine, after 7 years of medical school and training has been fired for one minor indiscretion. He slept with a patient and can no longer work in the profession.
What a waste of time, effort, training and money. He's still paying on his student loans. This just goes to show that one minor mistake can ruin your life. Thoughts for him and his family. He really is a great guy and a brilliant Veterinarian.
Give me coffee to change the things I can and wine to accept the things I cannot!
JaxCoder.com
|
|
|
|
|
Lesley says I posted that one back in '09 ... The Lounge[^]
The oldies are the best ones!
Sent from my Amstrad PC 1640
Never throw anything away, Griff
Bad command or file name. Bad, bad command! Sit! Stay! Staaaay...
AntiTwitter: @DalekDave is now a follower!
|
|
|
|
|
Ah thanks, tired and eyesight about shot. (Worked with sheet rock last few hours)
Give me coffee to change the things I can and wine to accept the things I cannot!
JaxCoder.com
|
|
|
|
|
|
Wait a minuyte. He{s a veterinarian and slept with a patient. Sounds pretty kinky to me!
CQ de W5ALT
Walt Fair, Jr., P. E.
Comport Computing
Specializing in Technical Engineering Software
|
|
|
|
|
|
The U of A, huh? I'll go have a word with them.
|
|
|
|
|
They have got it all wrong again, it's just a boomerang thrown by the Aussies
|
|
|
|
|
Nah it's not.
It's a cheap copy of a boomerang. All the ones that we make return to sender. The counterfeiting outfit doesn't understand the importance of aerofoil design.
|
|
|
|
|
|
What a likeness! I must take up religion immediately!
(Or get better at photoshop)
Sent from my Amstrad PC 1640
Never throw anything away, Griff
Bad command or file name. Bad, bad command! Sit! Stay! Staaaay...
AntiTwitter: @DalekDave is now a follower!
|
|
|
|
|
Super Lloyd wrote: I give you Jesus on a potato chips! That page makes me wish the Spanish Inquisition were still around.
The page owners could expect them.
I wanna be a eunuchs developer! Pass me a bread knife!
|
|
|
|
|