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

Question

[Ivan Sharapenkov]

Во всех приведенных вызовах возвращает false. А должно true). Помогите разобраться.
Моя логика:
^1? - в начале строки либо 1 либо ничего.
\s? -дальше, возможно, есть пробел
\((?=...\)) - открывающая скобка, если дальше есть три символа и закрывающая скобка
...\)?\s?-? - три символа, и, возможно, закрывающая скобка, пробел, дефис.
Дальше регулярное выражение работает правильно.
Где ошибка?

function telephoneCheck(str) {
  return /^1?\s?\((?=...\))...\)?\s?-?...-?\s?....$/.test(str);
}



telephoneCheck("555-555-5555"); // true
telephoneCheck("5555555555"); // true
telephoneCheck("1 555-555-5555"); // true
telephoneCheck("1 555 555 5555"); // true

/*
Also true (it works)
[phone]
[phone]
1(555)555-5555
*/

/*
False
(6505552368)
0 [phone]
-1 [phone]
10 [phone]
555)-555-5555
etc.
*/

Comments

[Kovalsky]

Что именно вы проверяете, какому формату должна соответствовать строка чтобы быть телефоном?

[Ivan Sharapenkov]

Никита Полевой:
Разве это не очевидно из регулярного выражения??
Единственные варианты, которые неправильно идентифицируются я указал.

telephoneCheck("555-555-5555");
telephoneCheck("5555555555");
telephoneCheck("1 555-555-5555");
telephoneCheck("1 555 555 5555");

То есть варианты в которых код сотового оператора не взят в скобки.

[Kovalsky]

sM0kfyz:

Разве это не очевидно из регулярного выражения??

Эээ, серьёзно, это должно быть очевидно из неправильной регулярки?) Ну да, очевидно, но не то что вы хотели, а то что у вас получилось) Заметьте, остальные то же самое спрашивают - "что именно должна проверять эта штука".

К тому же строчки telephoneCheck("555-555-5555"); // true никак не намекают на то, что они неправильные, там true справа написано.

Answer 1

[Igor Koch]

Попробуйте так:

^1?\s?\(?[0-9]{3}\)?[\-|\s]?[0-9]{3}[\-|\s]?[0-9]{4}$

https://regex101.com/r/bQcUVo/2

[\-|\s]? - минус или пробел
[0-9]{3} - 3 числа от 0 до 9

Comments

[Ivan Sharapenkov]

А если пропустить случайно одну скобочку, то Ваш код выдает true. Но это же неправильно)

[Igor Koch]

sM0kfyz: Позвонить по нему без скобки можно но если очень хочется, то можно и так https://regex101.com/r/bQcUVo/3

[Ivan Sharapenkov]

Спасибо, первый день изучаю, не знал, что можно использовать () таким образом.

Answer 2

[Stalker_RED]

Это, конечно, тоже фигня, но уже работает:
^1?\s?\d{3}(\s|\-)?\d{3}(\s|\-)?\d{4}https://regex101.com/r/f8iqFa/1

Уточните что именно должна проверять эта штука. Уверены ли вы, что хотите отсеять все телефоны, в которых потеряна скобочка, например?

Comments

[Ivan Sharapenkov]

Да. Нету скобочки - false. Несколько пробелов|дефисов - false.
Ваше выражение не проверяется номера со скобками.

[Stalker_RED]

sM0kfyz: я писал под эти вот:

telephoneCheck("555-555-5555"); // true
telephoneCheck("5555555555"); // true
telephoneCheck("1 555-555-5555"); // true
telephoneCheck("1 555 555 5555"); // true

[Stalker_RED]

sM0kfyz: Какой смысл проверять все эти скобочки и дефисы, если на самом деле все что вам нужно - это 10-11 цифр. Кстати, бывают еще добавочные номера, и запись в виде 5-555-FCK-YEAH.

Вот здесь в комментах неплохой срачик на тему номеров с выводом золотого правила:
Нужно просто контролировать на наличие минимальной длины.