|
Ah, variable is surely of type MagnitudeNumber, yes? With a value of -50? It is both less than 24 (because it's negative) and its magnitude is greater than 26.
The comparison operator obviously checks both the signed and unsigned number, and returns true if either satisfy the condition.
|
|
|
|
|
Uhhhh... sure! 
Just kidding. The programmer at the time just probably forgot that they checked the same variable twice. Maybe it was supposed to be another variable. I have no clue what the intention was, but its still active in the program. I don't have the heart to comment it out because of how funny the goof is. 
|
|
|
|
|
int main (){
long variable = -1;
long var24 = 24;
unsigned long var26 = 26;
if( variable < var24 && (variable > var26)){
printf("%ld, is a magic number, %lu, %ld, %ld", variable, variable, var24, var26);
}
return 0;
}
I was just wondering if this could come up somewhere in the code and still be funny.
-JD
|
|
|
|
|
it could be a 'typo' for an Or
|
|
|
|
|
That was my thoughts exactly. However, there are many other instances like the example I posted. The worse I have seen are many goto statements.
|
|
|
|
|
if (this == null)
{
Environment.Exit(-1);
}
I cannot remember the exact code block, but it was something along those lines. And yes, somehow, that condition happened. And 'this' was the actual keyword, not some redefinition of it. I still don't know how that one happened.
I think computer viruses should count as life. I think it says something about human nature that the only form of life we have created so far is purely destructive. We've created life in our own image.
Stephen Hawking
|
|
|
|
|
your object was finalized before the time, probably it had a unmanaged reference, when it had no more managed references, the GC collected it...
Forgeting to pin a object is really a PITA, especially when it's the first time you are doing P/Invoke 
(i was speaking of me, not sure if this is your case)
I'm brazilian and english (well, human languages in general) aren't my best skill, so, sorry by my english. (if you want we can speak in C# or VB.Net =p)
|
|
|
|
|
This wasn't my code, it was from one of my friend's projects. I cannot remember what the actual class did, but most of the projects code files were naive (and buggy) implementations of classes in the framework. I did show him the documentation, and his reply was "Cool! How did you know this existed?". Me: 
I think computer viruses should count as life. I think it says something about human nature that the only form of life we have created so far is purely destructive. We've created life in our own image.
Stephen Hawking
|
|
|
|
|
PEBKAC 
I'm brazilian and english (well, human languages in general) aren't my best skill, so, sorry by my english. (if you want we can speak in C# or VB.Net =p)
|
|
|
|
|
I think it was more of an ID-Ten-T error. Either that or a BrainNotFoundException.
He has decided to become an English Major with Anthropology and History minors.
I think computer viruses should count as life. I think it says something about human nature that the only form of life we have created so far is purely destructive. We've created life in our own image.
Stephen Hawking
|
|
|
|
|
well, can't say i haven't found such cases...
there was a guy on my class who left to become a surgeon... I hope I never have to be operated by him 
I'm brazilian and english (well, human languages in general) aren't my best skill, so, sorry by my english. (if you want we can speak in C# or VB.Net =p)
|
|
|
|
|
Zac Greve wrote: Environment.Exit(-1);
Way back in the 1980's, I was using Sun 3 workstations running UNIX. We had FrameMaker, back before Adobe got ahold of it. On UNIX, there is a command called "strings" that attempts to extract all the ASCII strings from a file. If you ran it over FrameMaker.exe, one of the strings you got was...
"Evacuate now? In our moment of triumph? I think you overestimate their chances."
There is no way to tell what error case causes this message to print, or just what happens right after, but I really want to know.
|
|
|
|
|
Which language is that? Java?
|
|
|
|
|
A professor of Computer Science -- a world-renowned expert on PL/I -- once wrote the following:
if ((i != 1) || (i != 2))
...
else
...
(Yes, the above is in C. I can't remember the PL/I syntax.)
He spent most of a day trying to figure out why the "else" branch was never executed. A graduate programming assistant had to point it out to him.
(This message is programming <i>you</i> in ways you cannot detect. Be afraid.)
|
|
|
|
|
if (my $thisScriptHasBecomeSkynet) {
die;
}
|
|
|
|
|
When I first saw this comment, I thought it was ridiculous, but now I see a lot of beauty in the f actuality and irony of it. Here it is:
// Note that there are 1024 milliseconds in a second
|
|
|
|
|
You mean aside from this[^]?
At one place I worked, we were under deadline and a coworker was going to take some time off. He said his part of the project was done and tested. He gave me a copy to integrate into my code and went on vacation.
I installed it and it promptly fell on its face. Looking through the code, it was a complete mess. I ended up taking a hatchet to it, but one routine caught my eye.
It was supposed to center some text on a line. Instead of taking the length of the line, subtracting the length of the string, then dividing by two to get the number of spaces to print before the string, he instead created a string of spaces for the length of the line, overlaid the string at the beginning, and then iteratively added spaces at the beginning and removed spaces at the end until they were equal or the leading spaces was longer than the trailing spaces.
It was the only routine that worked and I left it in place to illustrate the type of code I had to deal with.
I still tried to cover for him, but since rewriting his code impacted the development of my code, when the manager came by breathing fire because I was falling behind, I had to show his work to her. When he came by after his vacation I showed him my changes, he said, "Yes, that's my code." I snarled internally that this bozo couldn't even recognize his code and how I had changed it (chopped 1/3 or more of it out) and then I advised him to take a tube of KY Jelly with him to his meeting with our supervisor because I knew she was going to ream him out.
Psychosis at 10
Film at 11
Those who do not remember the past, are doomed to repeat it.
Those who do not remember the past, cannot build upon it.
|
|
|
|
|
It wasn't funny at the time, but any VB.Net code that looked like this is hilarious in retrospect:
try
catch
end try
Or even worse:
On Error Resume Next
The developer's excuse? "My code doesn't crash!" Yeah. It also didn't work.
|
|
|
|
|
|
That's downright scary. I insist that people use C# so they don't get into bad habits like that.
|
|
|
|
|
Hahaha, level: advanced, and spent all day on it. So many levels of humor in that article...mind blown.
|
|
|
|
|
in FORTH:
Begin ...
if
( swap )
...
until
then
The parentheses allow you to leave the compiler, so the swap changes the addresses on the stack so the straddled begin and if statements worked.
|
|
|
|
|
|
I'm brazilian and english (well, human languages in general) aren't my best skill, so, sorry by my english. (if you want we can speak in C# or VB.Net =p)
|
|
|
|
|
This is going back many years, and from memory it went something like this:
- (bool)is_even(int value)
{
if (value == 0)
{
return true;
}
else if (value == 2)
{
return true;
}
else if (value == 4)
{
return true;
}
...
else if (value == 12)
{
return true;
}
return false;
}
And so it went to 12 and I guess for the context in which it was used it apparently was a high enough number. You could of course replace this method using a simple modulus operation.
What's also funny is that I saw this code in a printout on the wall of an engineer, which had been there for a couple years. A short time after, another engineer, new to the group, found this method still existing in another part of the code base (apparently just copy-pasted from its original location). The code base was very large.
Lesson learned is if you find bad code somewhere do a global search to insure it is not duplicated anywhere else.
|
|
|
|
|
Haha I am sure i have read this in a book or programming tutorial as an example for bad code.
|
|
|
|
General 
News 
Suggestion 
Question 
Bug 
Answer 
Joke 
Praise 
Rant 
Admin 
Submit an article or tip
