Нашли или выдавили из себя код, который нельзя назвать нормальным,
на который без улыбки не взглянешь?
Не торопитесь его удалять или рефакторить, — запостите его на
говнокод.ру, посмеёмся вместе!
function validate() {
var at = document.getElementById("email").value.indexOf("@");
var age = document.getElementById("age").value;
var fname = document.getElementById("fname").value;
submitOK = "true";
if (fname.length > 10) {
alert("Имя не должно содержать более 10 знаков");
submitOK = "false";
}
if (isNaN(age) || age < 1 || age > 100) {
alert("Возраст должен быть от 1 до 100");
submitOK = "false";
}
if (at == -1) {
alert("Неправильный e-mail");
submitOK = "false";
}
if (submitOK == "false") {
return false;
}
}
submitOK объявить как "var submitOK=true;", чтоб принадлежал только функции. Присваивать ему булевы значения (а не строковые), а в конце вместо последней проверки поставить "return submitOK;". Да и вообще, если используется "alert", то лучше сразу делать "return false"! А то представьте себе: у вас 3 ошибки и последовательно появятся 3 сообщения, которые еще и закрывать надо, будет раздражать.
Действительно, говнокод.
Абдурахмангаджи (15 букв) и Вильгельмина (12 букв) стало быть не люди?
Абдурахмангаджи = Абдурах...
Вильгельмина = Вильгел...
Действительно, говнокод.
Лучше собрать все сообщения об ошибках и показать одним алертом!
не хватает веток else