Регулярные выражения, что не так с $?

Question

[sophiko]

Здравствуйте,
выражениями нечасто пользуюсь.
Написала код для проверки введенного телефона (согласно заданию 2 варианта формата:
а) [phone];
б) 123 12312345;)
Код:

function checkPhone(input){
  var patt = /^\+7\s\d{3}\s\d{7}|^\d{3}\s\d{7}/;

  var res = patt.test(input);
  return res;
};
//А это тест для проверки и что должно выдавать:
checkPhone('[phone]');//true
checkPhone('123 12312345'); // true
checkPhone('123 12312345aaa'); // false

В последнем выдает ошибочно true.
Нашла оператор $ конца строки, но когда записываю:
/^\+7\s\d{3}\s\d{7}$|^\d{3}\s\d{7}$/
мой браузер мне пишет все три теста false.
Подскажите, пожалуйста, что я неправильно делаю?

Answer 1

[kompi]

12312345 - всего 8 цифр, а у вас в регулярке d{7}, сооствественно с d{7}$ - все будут false;
Вариант по-короче: /^(\+7\s)?\d{3}\s\d{8}$/

Comments

[sophiko]

Спасибо, когда долго изучаешь регекспы, то можно и доизучаться).

Answer 2

[Евгений Петров]

Очевидно же, что цифр в российском телефонном номере не может быть 8... И не нужно забывать, что человек может ввести пробел, где не надо.

function checkPhone(input){
	var result = /^\s*(?:\+7\s*)?(?:\d\s*?){10}$/.test(input);

	console.log(result);
	return result;
};

//А это тест для проверки и что должно выдавать:
checkPhone('[phone]'); // true
checkPhone('123 1234567'); // true
checkPhone('123 1234567aaa'); // false

Comments

[sophiko]

Про пробел не очень поняла, я когда тестила вставляла пробелы в начале в середине и в конце -- он понимает, что это не тот шаблон. Выдает false.

[Евгений Петров]

@sophiko а я не понял вашу историю с пробелами:)
Я говорю про то, что кто-то может ввести вот так:
+7 (926) 678 - 1234

Такие ситуации нужно отслеживать. Тогда:

var result = /^\s*(?:\+7\s*)?\(*\s*\d{3}\s*\)\s*(?:\d\s*-*\s*){7}$/.test(input);

[Евгений Петров]

@sophiko подвожу итог — вы написали плохое выражение, которое сломается на любом пробеле, любом символе из допустимых при вводе номера.

[sophiko]

Ну не сердитесь вы так), возможно, что и плохое, надо разобраться как улучшить ситуацию. Тот формат который вы указали: +7 (926) 678 - 1234 (кстати, как вам удалось поменять цвет в комменте?) по идее не должен вводиться, потому что в плейсхолдере инпута прописано: "телефон в формате [phone] или 812 7776644". Но с другой стороны может действительно возникнуть ситуация, когда человек случайно введет пробел, а ему в ответ - у вас неправильный формат - и у человека будет фрустрация. Тогда- да, надо переделать.
Не совсем поняла слова "сломается на любом пробеле" можете на тесте показать, как оно сломается?

[Евгений Петров]

@sophiko есть такой тэг <code>тут пишем код</code>

Теперь по поводу «сломается» — вот ваш паттерн
/^\+7\s\d{3}\s\d{7}$|^\d{3}\s\d{7}$/

Берём номер
[phone]

И всё, приплыли, хотя 2 пробела не так просто заметить. Человек может просто скопировать текст из документа, со страницы.
Вы перекладываете на пользователя дополнительную работу просто потому, что нет желаний или знаний написать нормальную обработку данных.

Вы можете использовать событие Input, чтобы запретить вводить ненужные символы. Вот пример — попробуйте ввести что-то, кроме вещественного числа (группа цифр, точка, группа цифр).

Вы можете написать более качественную проверку и регулярное выражение будет учитывать разнообразные ситуации и позволит пользователю не чувствовать себя дураком.

Но вы останавливаетесь на паттерне, который может сломать лишний пробел, хотя написать вместо пробела \s* или \s+ никто не мешает. У меня складывается ощущение, что вы делаете как попроще для вас. В результате получается очередной кривой велосипед.

[sophiko]

Поняла вашу идею - конечно, согласна). За интересную ссылку спасибо. Тогда не совсем понимаю зачем в вашем выражении встречается эта штука (?: ) - пассивная группа или группа без захвата, везде по-разному называется ?

[sophiko]

Но вот кстати я протестировала ваш регексп - чет не проходит правильный формат указанный в инпуте.

[sophiko]

А там просто звездочки нет после \). Все работает.

[Евгений Петров]

@sophiko потому что у второй скобки пропущен *. Вот так верно

var result = /^\s*(?:\+7\s*)?\(*\s*\d{3}\s*\)*\s*(?:\d\s*-*\s*){7}$/.test(input);

Эта штука ?: зовётся «non capturing group». Поставил для того, чтобы не включать куски номера в результат без надобности.

[sophiko]

@Petroveg благодарю))