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

Question

[gregorypetrov]

/[A-Za-zа-яА-ЯёЁ0-9._-]+@[A-Za-zа-яА-ЯёЁ0-9._-]+\.[A-Za-zа-яА-ЯёЁ]{2,4}/

<?php

$email = "mail@сайт.рф";

if (!preg_match("/[A-Za-zа-яА-ЯёЁ0-9._-]+@[A-Za-zа-яА-ЯёЁ0-9._-]+\.[A-Za-zа-яА-ЯёЁ]{2,4}/", $email)) {
    echo "ERROR";
} else {
    echo "Ok";
}

Если убрать а-яА-ЯёЁ , то латинские эмэйлы пропускает, т.е. в принципе регулярка верная, но почему-то не работает для кириллицы при добавлении а-яА-ЯёЁ в правило.

Comments

[Антон]

удобнее валидировать так
filter_var($email, FILTER_VALIDATE_EMAIL);

Answer 1

[neol]

У вас там unicode, для корректной работы с которым к шаблону нужно добавить флаг u, чтобы получилось как-то так

<?php

$email = "mail@сайт.рф";

if (!preg_match("/[A-Za-zа-яА-ЯёЁ0-9._-]+@[A-Za-zа-яА-ЯёЁ0-9._-]+\.[A-Za-zа-яА-ЯёЁ]{2,4}/u", $email)) {
    echo "ERROR";
} else {
    echo "Ok";
}

А если выпить кофе и окончательно проснуться, то получится что-то типа

if (!preg_match("/[\w._-]+@[\w._-]+\.[\w]{2,4}/u", $email)) {

Answer 2

[vreitech]

потому что punycode.
например, то, что вы видите как пример.рф, в доменной зоне у регистратора прописано как xn--e1afmkfd.xn--p1ai. с наскоку как минимум не удовлетворяет вашему [A-Za-zа-яА-ЯёЁ]{2,4}.
как правильно - зависит от задачи. если пропускать домены с кириллическими именами - исправьте то, что пометил, сняв ограничение на длину и добавив в список валидных символов цифры и знак дефиса. если валидировать любые e-mail адреса - оставить только проверку на символ @ и наличие как минимум одного символа в локальной и доменной частях e-mail.

Comments

[gregorypetrov]

Но скрипт из моего примера (специально для теста) должен ведь отрабатывать, ведь там $email = "mail@сайт.рф"; а не пуникод. Но не пропускает. Какая-то принципиальная ошибка.

[vreitech]

gregorypetrov, так-то да, вроде как должен. может конкретная версия php, установленная у вас, не умеет корректно в регулярках работать с диапазонами русских символов? могу предложить для теста разбить выражение и строку для проверки на несколько подстрок и попроверять их отдельно, чтобы выяснить, какая части исходной регулярки не работает так, как ожидается.

Answer 3

[4iloveg]

Показывает что ок. https://regex101.com/r/FGyuHK/1