Почему javascript проваливает проверку когда много условий?

Question

[Николай]

И снова здравствуйте :)
имеем элементарный скриптик проверки заполненности формы:

function car_validate(form) {
	valid = true;
	if (form.var1.value.length < 2) {
		document.getElementById('var1_e').innerHTML='err1';
		valid = false;
	}
	if (form.var2.value == 0) {
		document.getElementById('var2_e').innerHTML='err2';
		valid = false;
	}
	if (form.var3.value == 0) {
		document.getElementById('var3_e').innerHTML='err3';
		valid = false;
	}
	if (form.var4.value == 0) {
		document.getElementById('var4_e').innerHTML='err4';
		valid = false;
	}
	return valid;
}

который несмотря на незаполненные поля пропускает (проще говоря - не работает)
Но если убрать любое одно условие - то он начинает работать :)

Может кто подскажет в чём дело? Или кинется готовым вариантом проверки на именно на JS

P.S. да, да, я в курсе про jQuery, и пр., но не настолько большой сайт, что бы подключать что-то ради проверки одной формы + сторонние ссылки не хотелось бы подключать (используется SSL для доступа к сайту)

Comments

[Виталий Киренков]

проверьте мой коммент...

Answer 1

[GM2mars]

Проверьте сравниваемые значения одинакового ли типа. Если в форме только числа, то желательно явно указывать parseInt(form.var4.value)==0

Comments

[GM2mars]

(upd) parseInt(value, 10) - указываем систему десятиричного счисления, дабы исключить ошибки

Answer 2

[Виталий Киренков]

Ну для начала объявите переменную:
var valid = true;

И вы в каждом условии переменную переопределяете. Соответственно, если у вас все первые были false, а последняя true, то и результат будет - true.

Comments

[GM2mars]

Я тоже сначала подумал про переопределение переменной, но возможно достаточно одного valid=false и уже форма не валидна. Так что в данном случае переопределение неважно, так как она в условия не принимает значение true;

[Виталий Киренков]

У вас length только в первой проверке, в этом я думаю дело =)

[Николай]

Дак длина нужна у первого поля. У остальных нужно значение)

Answer 3

[Сергей Краснодемский]

Переходите на светлую сторону Html5, с его "required" и типами полей :)

Comments

[Николай]

Хороший вариант, но поддерживается не всеми браузерами..

Answer 4

[sasha]

мне кажется скрипт просто падает в виду того что вы просите у него свойства у объекта который не объявлен

Comments

[Николай]

Как это? Без одного условия же все работает..

Answer 5

[Алексей Павлов]

Если у вас в элементах var2-var4 записаны числа, то тип данных всё равно строка, и для сравнения с нулём нужно преобразовать строку в число. Либо как сказал @GM2mars, только в его примере лучше указать вторым параметром систему счисления: parseInt(form.var4.value, 10) === 0, потому как в старых браузерах числа, начинающиеся с нуля ("012"), считались восьмеричным числом. Либо просто указать +form.var4.value === 0, где унарный плюс переведёт строку в число.
А само присвоение valid = false в данном случае корректно.
Но я обычно делаю так:

function validate(form) {
  var errors = [];
  if (form.var1.value.length < 2) errors.push('length');
  if (+form.var2.value == 0) errors.push('var2');
  if (+form.var3.value == 0) errors.push('var3');
  if (+form.var4.value == 0) errors.push('var4');
  showErrors(errors);
  return errors.length == 0;
}
function showErrors(errors) {
  foreach (error in errors) {
    switch (errors[error]) {
      case 'length': document.getElementById('var1_e').innerHTML='err1'; break;
      case 'var2': document.getElementById('var2_e').innerHTML='err2'; break;
      case 'var3': document.getElementById('var3_e').innerHTML='err3'; break;
      case 'var4': document.getElementById('var4_e').innerHTML='err4'; break;
    }
  }
}

Такой подход позволяет разделить собственно проверку значений и обработку ошибок.

Comments

[Николай]

Ваш пример не работает..
А KEdit вообще ругается
ReferenceError: invalid assignment left-hand side: (on column 24)
на строчку:
"if (+form.var2.value = 0) errors.push('var2');"

[Алексей Павлов]

а, точно. нужно два раза знак равенства. +form.var2.value == 0. Исправил в ответе.
Надо же, пропустил детскую ошибку...
Кстати, именно поэтому многие программисты пишут проверку вторым способом из этих двух:

if (a == 0) doWork();
if (0 == a) doWork();

В первом случае если случайно вместо == поставить =, то будет работающий код, но работающий неверно. А во втором случае ошибка произойдёт.

[Николай]

Буду знать, спасибо!
Теперь ругается
SyntaxError: missing ; before statement: (on column 29)
на строчку:
foreach (error in errors) {