Почему валидация сработала не так как задумывалась?

dollar, извините
но Вы не правы
Вы не весь вопрос прочитали?
зачем смотреть весь код?
я в описании вопроса написала - речь о проверке поля Email / телефон
а это одна строка

olya_097, я прочитал весь вопрос. Вы не указали, как именно должна осуществляться валидация, то есть какая задумка. Вы не описали, что вы пытались делать и что именно не получилось, то есть в чем конкретно ваша проблема. Хотя бы нужно было описать, что не так, т.е. что ожидалось и что получилось. Вы не выделили проблемный участок кода в пару строк, а вместо этого просто даете ссылку на весь код - мол, пусть каждый сам ищет. В общем, вы не поставили себя на место того, кто будет читать вопрос и пытаться его понять, либо просто поленились оформить - оба варианты попадают в категорию "не красиво". А разобраться в вашем коде, конечно, не очень сложно для эксперта, но сопоставимо по сложности с заданием, которые здесь запрещены, т.е. вы по сути просите сделать за вас работу, а не решить вашу проблему.

Вообще правильно формулировать вопросы - это своего рода искусство, так что у меня нет претензий. Но порой формулировка из разряда "Почему интернет не работает, ведь я только вчера компьютер купил?", и в таких случаях я не могу удержаться от комментария.

dollar, частично согласна с вами
Но какое же это задание если нужно поправить только одну строку?
конкретизировала в вопросе

olya_097, так что же вы нагуглить не можете, раз не задание? Я вот по первой же ссылке нашел регэксп, чуть подправил и вот

/^([a-zA-Z0-9_\.\-])+\@(([a-zA-Z0-9\-])+\.)+([a-zA-Z0-9]{2,4})|((\+)?[0-9]+)+$/

like-a-boss, dollar, отчасти вы правы, но ведь и сами понимаете что иногда сформулировать вопрос очень трудно (в силу ряда субъективных причин). Этот вопрос родился у автора с моей подачи. По ее вопросам хорошо видно, что она не может идентифицировать саму проблему (не хватает знаний и опыта), потому и вывалила комом. Даже в таком случае я бы ей ответил, разжевав и дав более правильный (в долгосрочной перспективе) вариант реализации ну или намек на него, но все никак не могу приспособиться писать код на терефоне, а до компа доберусь не скоро)))

Роман, понимаю. Просто это задача по отладке. Даже если не учитывать правила и оформление, и взяться-таки за выполнение этой задачи (отладки), нужно понимать проблему. Отрывок из ТЗ в виде требований к функционалу помог бы.

Но задача есть задача. Тем более по отладке. Мало того, что мне это не интересно, так я еще и более правильным считаю намекнуть на улучшение навыков. В частности, научиться отлаживать (искать ошибки, не парадоксы) - жизненно важно для программиста. А автор вроде как позиционирует себя программистом. Отсюда и весь этот флуд. Я ни на кого не нападаю и не пытаюсь оскорбить, - наоборот, указываю как раз на ошибки. Правда, не в скрипте.

dollar, тут согласен с Вами полностью. Но мне кажется что автор только учится. При этом она делает это достаточно активно и очень трудолюбива, но во многих вопросах пока еще плавает. За усидчивость и трудолюбие всегда делаю ей небольшие поблажки)))

Не пытаться унифицировать функцию валидации

категорически не согласен))))

Роман, а я категорически с вами не согласен. Для новичка, да и в целом: в программировании чем проще, тем лучше, тем быстрее интерпретируешь в голове, тем быстрее выявишь проблему. Усложнять можно, когда ты можешь это делать машинально(а не сидеть днями над такими вот проблемами), и так же машинально в голове выполнять пробегаясь по коду, написанному N времени назад.
Да и вообще от этого жейквери слёзки наворачиваются ;(

like-a-boss, про jquery согласен. Про автоматизм тоже. Но конкретно про валидацию нет)))
Не унифицированная валидация добавляет в программу кучу функций, каждая со своей , плохо читаемой логикой из-за обилия if-ов, проверок на типы и кучи || и &&. Лучше один раз разобраться с унификацией этого вопроса чем потом часами выискивать ошибку в коде (наподобие представленного автором)
ЗЫ: естественно, что унифицировать данный процесс надо несколько иначе, чем это делается у автора.

Роман, Согласен