Комментарии к итоговому заданию Вячеслава Григорьева

grigoriev.js

// Обработку событий настоятельно рекомендую делать 
	// внутри скрипта, а не навешиванием html-атрибутов
		function checkNum(x) {

			var n = document.getElementsByTagName("form")[0];

			if (isNaN(x.value)) {

        // в результате, если я ввож 139o (промахнулся по 0),
        // то стирается всё значение. Это нехорошо
        // Нужно удалять только последний символ:
        // x.value = x.value.substring(0, x.value.length-1)
				x.value = "";
			}

			if (x.value.length == x.maxLength) {
				var next = x.tabIndex;
				if (next < n.length) {
					n.elements[next].focus();
				}
			}

			var setYear = document.getElementsByTagName("input")[5].value;
			if (setYear && setYear.length == 4) {
				var d = new Date();
				var year = d.getFullYear();
				if (year > setYear) {
					x.value = "";
					x.focus();
				}
			}
		}

		function checkCaps(caps) {
			var cardOwner = document.getElementsByName("owner")[0];
			var f = document.getElementsByTagName("form")[0];
			cardOwner.value = caps.toUpperCase();

			if (cardOwner.value.length > 2) {
				var a = true;
				for (var i = 0; i < f.length; i++) {
					if (f.elements[i].value == "") {
						a = false
				        // В идеале здесь ещё можно добавлять визуальное
				        // выделение неправильно заполненным полям
					}
				}
			}

			if (a) {
				document.getElementsByTagName("div")[0].className += " valid";
				document.getElementsByName("submit")[0].removeAttribute("disabled");
			}
		}