[keen]

сравните эти два сематически идентичных куска кода и задумайтесь.

// 1й
	    if ( document.getElementById('login').value == '' ) {document.getElementById('login').className = 'err';}
	    else {document.getElementById('login').className = 'inp';}
	    if ( document.getElementById('pass').value == '' ) {document.getElementById('pass').className = 'err';}
	    else {document.getElementById('pass').className = 'inp';}

// 2й
	        var el = document.getElementById('login'),
				ep = document.getElementById('pass');
				
			el.className = (el.value == '') ? 'err' : 'inp';
			ep.className = (ep.value == '') ? 'err' : 'inp';

и так - весь код..