Правильно ли составлено регулярное выражение на PHP?

Question

[Дмитрий]

Здравствуйте. Нужно проверить правильность заполнения поля формы с номером телефона.
Составил регулярное выражение. Делаю это впервые. Подскажите правильно ли составлено оно:

if(!preg_match("/[+][3][8][0][(][0-9]{2}[)][0-9]{3}[-][0-9]{2}[-][0-9]{2}$/i",$phone)) {
		$errorPhone .= "Введите корректный номер телефона<br>";
	}

Сам номер телефона - должна быть маска +380(__)___-__-__ только числа, длина до 20 символов.

Answer 1

[Алексей Уколов]

• Делать символьные классы из одного символа совершенно излишне. С другой стороны, тогда вам придётся экранировать специальные символы (скобки, плюс). Такая запись (с экранированием) более привычна тем, кто знаком с регулярками, но может быть сложнее для новичков. Однако, я всё же рекомендую привыкать к обратным слэшам - скоро вы перестанете их замечать.
  
• Символьный класс [0-9] можно заменить на сокращение \d. На мой взгляд, так выглядит чище.
  
• Если уж ставите ограничение на конец строки ($), то и начало (^) стоит поставить.
  
• Учитывая, что у вас в строке вообще нет букв, ни к чему добавлять модификатор i
  

Результат:
/^\+380\(\d{2}\)\d{3}-\d{2}-\d{2}$/

Answer 2

[Александр]

https://regex101.com/r/lCsKL2/1

Основная ваша ошибка - не экранированные символы "+", "(" и ")". Ну и каждый отдельный символ брать в квадратные кавычки не обязательно. Вместо "[0-9]" можно использовать "\d"

Comments

[Алексей Уколов]

Как раз внутри символьных классов экранировать не нужно, так что это не ошибка.

Answer 3

[Андрей]

Не нужно оборачивать одиночные символы в квадратные скобки. Можно писать их просто так. Для символов со специальным значением (плюс и скобки) нужно использовать экранирование обратным слэшем. Чтобы не плодить по два слэша, используйте строку с одинарными кавычками. Тире, если не внутри квадратных скобок, спецсимволом не является и экранировать его не нужно.

Итого:

preg_match('/^\+380\([0-9]{2}\)[0-9]{3}-[0-9]{2}-[0-9]{2}$/i', $phone)

Если в каком-то месте могут быть пробелы, можно добавить туда \s*. Если пробелы могут быть где угодно, проще сначала заменить пробел на пустую строку по всему значению.

Answer 4

[Saboteur]

Ваша регулярка рабочая, но нагромождена излишеством. Можно проще:

if(!preg_match("/+380\(\d{2})\d{3}-\d{2}-\d{2}$/i",$phone)) {
    $errorPhone .= "Введите корректный номер телефона<br>";
  }