|
|
Llasus wrote: I'd prefer a one letter name or a "temp" instead of this one.
calling it crap makes more sense because then it is obvious to someone else that it is crap and then they can make a post about it on here and feel all big about themselves - if he had given it a sensible name then maybe CurtD wouldn't have noticed what a load of crap it was and wouldn't have fixed it!!
|
|
|
|
|
|
sadly enough, often all the documentation one gets. 
|
|
|
|
|
CurtD wrote: something_to_which_to_point
Are you concerned by the nomenclature of this variable? Perhaps he was attempting to desist hardcoding in the parameter and using a constant type for better readability?
Vasudevan Deepak Kumar
Personal Homepage
Tech Gossips
A pessimist sees only the dark side of the clouds, and mopes; a philosopher sees both sides, and shrugs; an optimist doesn't see the clouds at all - he's walking on them. --Leonard Louis Levinson
|
|
|
|
|
Years ago, another developer told me about a special coding horror... he had an developer that falled in love and so he called for the next module all his variables with the name of his beloved... from Susi001 to Susixxx  Now, try to read such a program ...
|
|
|
|
|
Daniel Ch. Bloch (MCSD, MCAD, MCTS) wrote: Years ago, another developer told me about a special coding horror... he had an developer that falled in love and so he called for the next module all his variables with the name of his beloved... from Susi001 to Susixxx Now, try to read such a program ...
In the really geeky days, I knew a guy who named all his vars after Star Trek characters. "Spock1", "Kirk42".
|
|
|
|
|
I used to know a sys admin who named all the hardware after Star Trek things. A server was called StarBase1 and it had printers attached to it called DeepSpace1 and so on. The reason he did this - he was called Pete Warr and we had the habit of using the first 4 characters of the surname and the first initial as user names so he was warrp (mind you, he was luckier than Kenny Cockburn, he really had problems with his login).
|
|
|
|
|
A couple of sys admin i know have named their server room "the shire" all the *nix server names have a LOTR theme to them and the windows servers are named after the sith. eg Darth Maul.
|
|
|
|
|
"Real programmers just throw a bunch of 1s and 0s at the computer to see what sticks" - Pete O'Hanlon
|
|
|
|
|
My personal favourite is from some old Foxpro code I saw which had a global variable called:
OB1KNOBI (Obi-wan Kenobi...)
(Only 8 chars long of course...)
Some (very) old support utilities I have seen had swear words as variable names, which caused a couple of problems when an error occurs...
A big DOS error box appears showing 'Variable "swear word" not found' !
|
|
|
|
|
An old favourite of mine was a piece of commercial software code I ran across in college. The programmer had a particular routine which was used to handle error exceptions, close any open files, etc. and terminate. He decided to mark the routine with the label HELL. Why? So that whenever he detected that something had gone wrong he could write GOTO...
Clive Pottinger
Victoria, BC
modified on Tuesday, December 18, 2007 4:48:12 PM
|
|
|
|
|
That is good.
"Real programmers just throw a bunch of 1s and 0s at the computer to see what sticks" - Pete O'Hanlon
|
|
|
|
|
Very cool.
|
|
|
|
|
| Reminds me of a former co-worker who named all his programs POORLY. (I don't mean he chose bad names, I mean *each* of his programs had the same name ... POORLY.) I asked him why he did so. He replied that he liked it when he saw the computer display "COMPILING POORLY" when he submitted his programs....
;^)
'droid
|
|
|
|
|
The absolut hardes thing I found was when I searched for a bug in a Module written in some kind of PL/I. I tried to understand, why a special line of code does not work, using 'true' and 'false' Boolean statements... now ... my forerunner was able du redefine true and false so that true was 0 and false was -1 ... incredible!
|
|
|
|
|
In my first Fortran program I managed to redefine a constant 3 (written as a digit '3' in the source) to something else (by passing it to a subroutine that changed its argument) and then the program would crash at the DO statement which should have executed 3 times. I never figured it out, needed a senior colleague to find it for me.
|
|
|
|
|
My favorite was a guy who used to contract at our company. He liked to write his code using variable names like tt (for Transaction Type), and pt (for Process Type, not Point). I happen to understand what he was writing as I somewhat knew the system he was working on, but what about another developer who was coming in fresh? They're not going to have a clue without asking, or a lot of time analyzing it. Even better, he liked to rewrite other people's code to fit in with his "better" design.
There's nothing worse than someone "knows" they know better than you, but clearly doesn't.
Kyosa Jamie Nordmeyer - Taekwondo Yi (2nd) Dan
Portland, Oregon, USA
|
|
|
|
|
Sounds like a clever chap to me.... My kind of plan.... Guarenteed employment for life (or the life of the software )
|
|
|
|
|
Some years ago I had to convert a quickbasic program to VB6. The original developer had a variable called drol (for diameter of the roll). However in my (and his) native language, dutch, this also means turd.
Mark C Hagers
New Media Ventures
Amersfoort, the Netherlands
|
|
|
|
|
Does anyone see anything stupid in this code? Here's a hint: it isn't the most efficient logic I've seen.
void CrapClass::reverse_points()
{
CPoint *pt, *npt;
CList <CPoint *, CPoint *> list;
POSITION next, prev;
next = m_point_list.GetHeadPosition();
while (next != NULL)
{
pt = m_point_list.GetNext(next);
npt = new CPoint(*pt);
list.AddTail(npt);
}
delete_points();
prev = list.GetTailPosition();
while (prev != NULL)
{
pt = list.GetPrev(prev);
npt = new CPoint(*pt);
m_point_list.AddTail(npt);
}
while (!list.IsEmpty())
delete list.RemoveHead();
}
modified on Thursday, December 13, 2007 6:52:30 PM
|
|
|
|
|
That code is bad. I love the use of a completely pointless copy the first time round.
|
|
|
|
|
Pete O'Hanlon wrote: Forum:Coding Horrors
Subject:Re: How NOT to reverse a list
Sender:Pete O'Hanlon
Date:Friday, December 14, 2007 4:49:00 PM
That code is bad. I love the use of a completely pointless copy the first time round.
I'm thinking he wanted a really fresh copy -- nothing like data where the bits are still warm. Yummy.
|
|
|
|
|
CurtD wrote: nothing like data where the bits are still warm
Souldn't he have done it twice then - just in case the first wasn't heated through warmly enough.
|
|
|
|
|
well the class has been named CrapClass...was the developer in question trying to write bad code??
but i guess u must have renamed it urself... 
modified on Friday, December 14, 2007 9:57:28 AM
|
|
|
|
General 
News 
Suggestion 
Question 
Bug 
Answer 
Joke 
Praise 
Rant 
Admin 
Submit an article or tip
