|
Thanks for that!
12,000 line of code in one stored procedure - I don't think that pachyderm goes far enough in describing how horrendous that must have been.
David Days wrote: no consistency in anything - that's probably the worst part, although anyone who writes a 12,000 line stored procedure is unlikely to be very well organised.
Glad they listened to you.
“That which can be asserted without evidence, can be dismissed without evidence.”
― Christopher Hitchens
|
|
|
|
|
Sounds more it has been written by some consultant company who deliver something and then run away. Close to Martians but much more greedy. Know this too good and I absolutely feel with you.
If I cannot understand how something works within half a day of studying it, I write it new. This isn't always easy or possible but if I can, I do it. Saves me and the company a lot of trouble and reduces risk.
|
|
|
|
|
BruceN wrote: So bizarre, so obtuse you just can't see the answer?
I've been with the same company for 20 years now, and I've seen this many times. You know the something even worse? When you realize that you wrote the bizarre, obtuse code in question...
|
|
|
|
|
|
Congratulations, you've learned a new skill...Real-World Programming. Sorta like Real-Time Programming, except someone else wrote all the code, nothing ever works, so it doesn't matter if the tasks finish within the allotted time slot.
Any Real-World programmer knows that return 97 means error 96 with one argument, what's the problem? And yes, that print routine should have been one base class with 23 levels of inheritance piled on, sloppy programming there, sure to bring down your cyclomatic complexity score.
Set some realistic goals and all will be well. Myself, I'm working on getting that cyclomatic complexity score to overflow and crash Visual Studio.
|
|
|
|
|
Bruce,
You have to be the Yoda! What they did is done. You "get" to clean it up.
You "get" to use that brain to figure out what brainless people were doing.
You get to be the white hat, and the hero!
Everyday, just make something better... And you will leave a legacy!
In this case, the code is the stimulus. And the most amazing part of being human,
is we get to CHOOSE our response to stimuli... We can get all upset. Lose hope/faith.
Cry that it isn't fair.
Or, we can take some of these examples, put them aside for a future article/book/training
material (as in... If you do this while working for me, you will be fired, and I will publicly
post what you did!).
And then clean it up. The hardest thing is to be the Jedi that only makes the changes
that are required while making things a little better each time. (Most programmers see horrible
stuff and want to re-write all of it. That is far easier than understanding illogical code, and
fixing it with the smallest, best change possible).
Finally... Think of ways that would prevent this in the future. Code Reviews!!!
Start having them. Stop the creation of bad code NOW.
When I approach it this way, I find it empowering.
Otherwise, if your perspective is off, one could reduce fixing "well written" code to "All I do is fix bugs or implement STUPID business changes that they should have put in there already". Which might be accurate, but does nothing to answer the question "How can I become Happier?"
|
|
|
|
|
Rather than rant, you could just do like me and my coworker did once -- named an award of coding shame after the former engineer and passed it around for years whenever we caught each other, or ourselves, doing something so stupid.
We can program with only 1's, but if all you've got are zeros, you've got nothing.
|
|
|
|
|
I feel for you. Probably the worst code I ever worked on was a real-time program where the apparently nonsensical sequence of events had a real-world impact that was not immediately obvious.
Fletcher Glenn
|
|
|
|
|
Try a project written by a sucksession of contractors all of whom codded differently badly with simple aspx pages with 2k+ line code behinds. The contractors are long gone. Then anyone with even passing knowledge of how the application works doesn't have time to waste on explaining fertilizer ( think organic) to you. Can you get this done by tomorrow?
|
|
|
|
|
Hmmm... If you know what the program's supposed to do, then I'd advise you to delete it and write a new one... by the time you figure out how this crap works, you might as well have done that anyway.
|
|
|
|
|
You didn't say how legacy "legacy" is, but there are always logical reasons for these things.
The first reason is that someone was making changes and didn't get around to making the change everywhere.
The meaning of the magic code 97 can be found on a note pasted under the system operator's keyboard.
The print routine call is a masterpiece of software engineering that allows you to call different routines with the same code.
You have a remarkably stable database to be able to stand firm for a thousand years. I would be grateful.
The toga might be allowed on some casual Fridays. I would check into it.
|
|
|
|
|
That's my code you're complaining about. Seriously though, I've run into code like this and you're almost better off rewriting from scratch when it's this bad.
|
|
|
|
|
Sounds like the problems I face on a day to day basis...
What I try to do is first of all figure out what is required of me. Then, very carefully inject what I need to do somewhere that looks appropriate. The key is to move things around but keep what is does and when it does it.
"Program testing can be used to show the presence of bugs, but never to show their absence."
<< please vote!! >></div>
|
|
|
|
|
|
Moving from "old school" GWT to UI Binder is not "getting current".
Elephant elephant elephant, sunshine sunshine sunshine
|
|
|
|
|
It can be... I mean, trying to use UIBinder with generic classes kinda feels like getting a large amount of current shunted through your skull...
|
|
|
|
|
I'm shocked that you would say such a thing.
|
|
|
|
|
Sorry, thinking about GWT just got me a little amped up... I couldn't resist.
|
|
|
|
|
I would have upvolted that, only if you two had difference in your potential.
You have just been Sharapova'd.
|
|
|
|
|
You don't upvolt things. I guess you can't be a transformer then, oh well.
I'm not very funny. Sorry.
|
|
|
|
|
|
|
|
that's the idea report it there, the rest of us kick the cr@p outta it till it goes away
|
|
|
|
|