Your stupidest coding goofs

[techno_race]

One time, I had to add 12 hours to the current hour and display it on screen in 12-hour time. My first instinct was, believe it or not, to do this:

var hour = new Date().getHours();
var hours;

switch(hour) {
	case 0:
		hours = 12;
	break;
	case 1:
		hours = 13;
	break;
	case 2:
		hours = 14;
	break;
	case 3:
		hours = 15;
	break;
	case 4:
		hours = 16;
	break;
	case 5:
		hours = 17;
	break;
	case 6:
		hours = 18;
	break;
	case 7:
		hours = 19;
	break;
	case 8:
		hours = 20;
	break;
	case 9:
		hours = 21;
	break;
	case 10:
		hours = 22;
	break;
	case 11:
		hours = 23;
	break;
	case 12:
		hours = 0;
	break;
	case 13:
		hours = 1;
	break;
	case 14:
		hours = 2;
	break;
	case 15:
		hours = 3;
	break;
	case 16:
		hours = 4;
	break;
	case 17:
		hours = 5;
	break;
	case 18:
		hours = 6;
	break;
	case 19:
		hours = 7;
	break;
	case 20:
		hours = 8;
	break;
	case 21:
		hours = 9;
	break;
	case 22:
		hours = 10;
	break;
	case 23:
		hours = 11;
	break;
	default:
		hours = "NaN";
}

...

Ack.

I am now using this:

var minutes = new Date().getMinutes();
var hour = new Date().getHours();
var hours = hour + 12;

if (hours > 23) {
	hours = hours - 12;
}

...

[techno_race]

Oh, dang it.

hours = hour

...

[Nile]

I unlinked() an array once.

[jscheuer1]

I have a terrible habit, that I think I'm finally starting to break, of forgetting to include the '[i]'s (highlighted below) or whatever the var is (usually it's 'i') when looping through something like:

for(var b = some_collection_or_array, i = 0; i < b.length; ++i)
if (b[i].something == 'some_string')
b[i].something_else = yet_another_thing;

I think the reason is that in my mind it should just be understood what I'm referring to, though computers rarely work that way.

I always catch the error(s) sooner (usually) or later, but if I had a nickel for every cumulative hour wasted on this, I'd probably be in a higher tax bracket.

[Nile]

Are you kidding? In that the i is my favorite part!

[rangana]

I'm always get pissed by this:

<script type="text/javascript">
window.onload=function()
{document.forms[0].onsubmit=function()
{var str=document.getElementById('email');
if (!(/^([a-z0-9])([\w\.\-\+])+([a-z0-9])\@(([\w\-]?)+\.)+([a-z]{2,4})$/i.test(str.value))){alert ("Please enter a valid email address.");
return false;}
}
return;
}
</script>
<form action="http://www.google.com">
<label>Email:</label><input type="text" id="email">
<input type="button" value="Submit">
</form>

Of course, the only mistake was the highlighted, which took me almost an hour to debug the script.

[jscheuer1]

Are you kidding? In that the i is my favorite part!

Mine too, but that doesn't prevent me from too often forgetting it in the evaluation phase of the loop. Though, as I say, I'm getting better at it.

[djr33]

I'd say I tend to type function(something(), and always forget that close parenthesis. Obviously that causes many problems.

[allahverdi]

Writing js after php:
(variables and almost ALWAYS forgot document.)

$myvalue = getElementById("thisis").value;

Writing PHP after js:

<?php
var $myvariable = $_POST['value'];
?>

[benslayton]

Last year I was redesigning our schools website. I had to use asp for the server side language... After that I would go to a C# class later on in the day. Then go home and program using php...

YOU CAN ONLY IMAGINE THE JUNK I WENT THROUGH...