Где в регулярном выражении ошибка?

Question

[Антон]

Здравствуйте!

Этот код возвращает false:

if (regex_match("[e-mail]", regex("/^[a-zA-Z0-9._-]+@[a-zA-Z0-9.-]+\\.[a-zA-Z]{2,4}$/")))
{
	return true;
}
else
{
	return false;
}

Где накосячил?

Answer 1

[Stalker_RED]

UPD: На самом деле, всё дело в лишних слешах "/regexp_here/"

Ниже идет неправильный ответ, и поиск истины в комментариях

Вместо \\. используйте \.
https://regex101.com/r/kY0uS0/2

Но вообще, это не серьезно же. Эта штука не пропустит кучу валидных адресов.


Примеры регулярок посолиднее
www.regular-expressions.info/email.html
stackoverflow.com/questions/46155/validate-email-a...

Вот вам "учебная" регулярка: \S+@\S+
По сути проверяется только наличие собачки и "каких-нибудь" символов перед и после нее.
https://regex101.com/r/cG7dW0/1

Comments

[Антон]

Не помогло.

Так и проект не серьезный. В данном случае этого хватит, ибо адреса будут примерно такие, как я показал в примере.

[Антон]

Учебный проект, где email адреса будут не настоящие, а демонстрационные.

[Антон]

Не работает. Притом я с регулярными выражениями не первый день знаком. На Qt, PHP или Ruby всегда все было супер. А сейчас бред какой-то.

Вот этот код выводит false:

if (regex_match("[e-mail]", regex("/\S+@\S+/gm")))
{
	return true;
}
else
{
	return false;
}

Вот этот код выводит true:

if (regex_match("subject", regex("(sub)(.*)")))
{
	return true;
}
else
{
	return false;
}

[Stalker_RED]

Антон: модификатор /g - служит для того, чтобы регулярка не останавливалась после первого найденного совпадения в строке. В данном случае он только для демонстрации примера.
/m - multiline - для многострочного текста, та-же фигня.

regex_match (Это же из boost?) сам по себе возвращает true/false, писать if(true) return true; совсем не обязательно.

К сожалению под рукой нет плюсов чтоб собрать пример.

[Антон]

Stalker_RED: ну то что вы написали мне известно и никак не относится к текущей проблеме. Эти регулярные выражения без проблем будут работать, например, в PHP, но в C++ тут явно что-то нужно еще. И я хз что именно.

[Stalker_RED]

Антон: А у нас всё работает! ©
coliru.stacked-crooked.com/a/cf722d5e3ee031b6

И да, двойной слеш пришлось вернуть. Это же плюсы, эскейп-последовательность, всё такое.

[Антон]

Stalker_RED: максимально приблизил к тому варианту, который был, чтобы понять, что не так было:

if (regex_match("[e-mail]", regex("^[a-zA-Z0-9._-]+@[a-zA-Z0-9.-]+\\.[a-zA-Z]{2,4}$")))
{
	return true;
}
else
{
	return false;
}

Ну, в общем, все прекрасно видно. Спасибо что не сдались и добились истинны :)

[AtomKrieg]

Антон: Почему вы пометили неправильный ответ как решение?

[Антон]

AtomKrieg: потому что человек потратил время на меня и в итоге помог.

[AtomKrieg]

Антон: А что делать следующему человеку, который найдет ваш вопрос по поиску? :)

[Stalker_RED]

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

Answer 2

[AtomKrieg]

if (regex_match("[e-mail]", regex("^[a-zA-Z0-9._-]+@[a-zA-Z0-9.-]+\\.[a-zA-Z]{2,4}$")))
		cout << "match" << endl;
	else
		cout << "no match" << endl;

Answer 3

[abcd0x00]

Где накосячил?

Пропустит
a@..aa

Answer 4

[Александр Латышев]

Вам надо заменить \\. на \.
И в качестве общей рекомендации: для упрощения стоит использовать спецсимволы, например так "/^[\w.-]+@[\w.-]+\\.[a-z]{2,4}$/i", где \w эквивалентно [a-zA-Z0-9_], а флаг "i" указывает, что следует игнорировать регистр символов при поиске совпадений.