Проверка формы на js

[strange_atom]

Доброго времени суток,

Есть форма (всю не пишу, ибо большая)

<form name="corp_calc" method="post" action="mail.php">

в ней следующая конструкция с полем (его пытаюсь проверить)

<div>
	<div class="err err1">Вы не указали Ваше имя</div>
	<label for="corp_result_fio">Ваше имя<span class="red"> *</span></label>
	<input type="text" name="fio" id="corp_result_fio" />
</div>

<div class="err err1"> - скрыт (display: none;)

submit следующий:

<input class="send_result" type="submit" value="Отправить заявку" onsubmit="checkForm();" />

пробую написать проверку обязательных полей

function checkForm() {
	if(document.corp_calc.fio.value == null) {
		document.getElementsByClassName("err1").style.display = "block";
		document.corp_calc.fio.focus();
		return false;
	}
	return true;
}

почему-то всегда выполняется mail.php

где я ошибся?

[sigma77]

strange_atom,

Внимательнее с типами данных.

http://jsfiddle.net/an5BG/2/

[strange_atom]

sigma77

так тоже не получается

function checkForm() {
	var valueFio = document.corp_calc.fio.value;
	if(!valueFio) {
		document.getElementsByClassName("err1").style.display = "block";
		document.corp_calc.fio.focus();
		return false;
	}
	return true;
}

[strange_atom]

разобрался (пишу решение, если кому-то пригодится)

1) "Внимательнее с типами данных" - спасибо, sigma77

2) нужно было вызывать функцию не инпутом, а формой

<form name="corp_calc" method="post" action="mail.php" onsubmit="return checkForm();">

3) getElementsByClassName не хотел применять свойство .style.display = "block" - заменил классы на id

в итоге js пока что такой:

function checkForm() {
	if(!document.corp_calc.fio.value) {
		document.getElementById("err1").style.display = "block";
		document.corp_calc.fio.focus();
		return false;
	}
	return true;
}

пошел разбираться дальше

Edited October 16, 2012 by strange_atom