Words cannot describe my pain
-
http://xacc.files.wordpress.com/2008/10/helpme.png[^] :sigh: I think I need to give the office a crash course in R E F A C T O R I N G. :doh: Note: The #2 version is simply a copy/paste of the first with a bit of extra functionality. Honestly, I would not let anyone even see the code inside.
xacc.ide - now with TabsToSpaces support
IronScheme - 1.0 beta 1 - out now!
((lambda (x) `((lambda (x) ,x) ',x)) '`((lambda (x) ,x) ',x))Nice. Anyway I strongly doubt the developer who created such brilliant method names will agree about the need of refactoring. ;P Well, I forgot you're the boss! :-D
If the Lord God Almighty had consulted me before embarking upon the Creation, I would have recommended something simpler. -- Alfonso the Wise, 13th Century King of Castile.
This is going on my arrogant assumptions. You may have a superb reason why I'm completely wrong. -- Iain Clarke
[My articles] -
Nice. Anyway I strongly doubt the developer who created such brilliant method names will agree about the need of refactoring. ;P Well, I forgot you're the boss! :-D
If the Lord God Almighty had consulted me before embarking upon the Creation, I would have recommended something simpler. -- Alfonso the Wise, 13th Century King of Castile.
This is going on my arrogant assumptions. You may have a superb reason why I'm completely wrong. -- Iain Clarke
[My articles] -
CPallini wrote:
Well, I forgot you're the boss!
The author of the code is my boss (the IT Director) ... :~
xacc.ide - now with TabsToSpaces support
IronScheme - 1.0 beta 1 - out now!
((lambda (x) `((lambda (x) ,x) ',x)) '`((lambda (x) ,x) ',x))leppie wrote:
The author of the code is my boss (the IT Director) ...
Oh, don't worry, I think his/her skills are well above the average ones of the IT directors I ever had. :-D
If the Lord God Almighty had consulted me before embarking upon the Creation, I would have recommended something simpler. -- Alfonso the Wise, 13th Century King of Castile.
This is going on my arrogant assumptions. You may have a superb reason why I'm completely wrong. -- Iain Clarke
[My articles] -
http://xacc.files.wordpress.com/2008/10/helpme.png[^] :sigh: I think I need to give the office a crash course in R E F A C T O R I N G. :doh: Note: The #2 version is simply a copy/paste of the first with a bit of extra functionality. Honestly, I would not let anyone even see the code inside.
xacc.ide - now with TabsToSpaces support
IronScheme - 1.0 beta 1 - out now!
((lambda (x) `((lambda (x) ,x) ',x)) '`((lambda (x) ,x) ',x))Verb/Noun pair -- check! So what's the problem? :-D
-
CPallini wrote:
Well, I forgot you're the boss!
The author of the code is my boss (the IT Director) ... :~
xacc.ide - now with TabsToSpaces support
IronScheme - 1.0 beta 1 - out now!
((lambda (x) `((lambda (x) ,x) ',x)) '`((lambda (x) ,x) ',x))As a former boss of mine once told me: "People will be promoted to their levels of incompetence" And as I've seen firsthand, if management likes someone who is a dumb as a stump, they will be moved into a position where they can do the least damage.
I don't claim to be a know it all, for I know that I am not...
I usually have an answer though.
-
Verb/Noun pair -- check! So what's the problem? :-D
-
PIEBALDconsult wrote:
Verb/Noun pair -- check! So what's the problem?
And the copy paste is just good re-use of code, right? :laugh: Bill W
Just because code works, it doesn't mean that it is good code.
And orthogonal too. :-D
-
As a former boss of mine once told me: "People will be promoted to their levels of incompetence" And as I've seen firsthand, if management likes someone who is a dumb as a stump, they will be moved into a position where they can do the least damage.
I don't claim to be a know it all, for I know that I am not...
I usually have an answer though.
-
http://xacc.files.wordpress.com/2008/10/helpme.png[^] :sigh: I think I need to give the office a crash course in R E F A C T O R I N G. :doh: Note: The #2 version is simply a copy/paste of the first with a bit of extra functionality. Honestly, I would not let anyone even see the code inside.
xacc.ide - now with TabsToSpaces support
IronScheme - 1.0 beta 1 - out now!
((lambda (x) `((lambda (x) ,x) ',x)) '`((lambda (x) ,x) ',x))Am I seeing right? Does one of these 'functions' really have around 1500 lines ??? I normally consider more than 20/25 LOC bad (except it is a trivial switch statement or something of that kind)... And even worse, the 'programmer' who created this nightmare is your boss?? Poor you, get out of there as quick as possible (or at least prevent your boss from coding somehow)! Regards Thomas
-
Am I seeing right? Does one of these 'functions' really have around 1500 lines ??? I normally consider more than 20/25 LOC bad (except it is a trivial switch statement or something of that kind)... And even worse, the 'programmer' who created this nightmare is your boss?? Poor you, get out of there as quick as possible (or at least prevent your boss from coding somehow)! Regards Thomas
Seriously, 1467 lines worth of copy and pasting. I don't care if he/she's your boss, shoot them, or better yet, give me their id, I'll do it...
-
Nice. Anyway I strongly doubt the developer who created such brilliant method names will agree about the need of refactoring. ;P Well, I forgot you're the boss! :-D
If the Lord God Almighty had consulted me before embarking upon the Creation, I would have recommended something simpler. -- Alfonso the Wise, 13th Century King of Castile.
This is going on my arrogant assumptions. You may have a superb reason why I'm completely wrong. -- Iain Clarke
[My articles]lol, have fun :doh:
-
lol, have fun :doh:
Are you his boss? :rolleyes:
If the Lord God Almighty had consulted me before embarking upon the Creation, I would have recommended something simpler. -- Alfonso the Wise, 13th Century King of Castile.
This is going on my arrogant assumptions. You may have a superb reason why I'm completely wrong. -- Iain Clarke
[My articles]
