- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
<form name='change_form'
enctype='multipart/form-data'
method='post'
onSubmit='if (!checkfilled(document.change_form,"name")) return false;
if (!checkfilled(document.change_form,"email")) return false;
if (!checkfilled(document.change_form,"text")) return false;
if (!checkfilled(document.change_form,"company")) return false;
if (!checkfilled(document.change_form,"phone")) return false;
if (!checkint(document.change_form,"city_code", -2147483648, 2147483648)) return false;
'
action='/request/?do=add'>
pee 01.03.2010 07:29 # +2
What-wha-what-what-what's it all about
Work-wa-work-work-work-wa-work it out
Let's turn this motherfuckin' party out (c) BeastieBoys
guest 01.03.2010 19:57 # 0
guest 01.03.2010 21:00 # −1
Во-вторых, реализация проверки формы, представленная здесь, также далека от оптимальной. Рациональнее было обойтись одной проверкой, в которой объединить все условия по логическому И.
Обобщая вышесказанное:
HTML:
JS:
Благодарю К.О. за неоценимую помощь в написании поста.
guest 01.03.2010 21:03 # 0
Да, в тему. Быстросамофиксы:
1) в HTML должно быть return checkThisOut()
2) в JS я-таки решил обойтись без if(...&&...&&...) и сделал цикл, ибо выходило много копипасты.
wwwguru 01.03.2010 21:08 # −1
1) 6 шт. return false вместо одного.
2) Неправильная запись if'ов
3) Проверка кода города в 32-битном диапазоне signed int - это мега ппц
4) Возможно что-то ещё )
Всю проверку можно выполнить в одном цикле с одним return false.
guest 01.03.2010 22:13 # 0
// тот самый guest