Что я делаю не так в регулярке?

Question

[Александр Макаров]

Подскажите, люди добрые, что я делаю не так?

/^https:\/\/([\w]+\.)?avito.ru\/[\w-]+\/.+(?!(_[\d]{4,}))$/.test(x)

Мне надо проверить чтобы линк авито не содержал на конце _ и число, состоящее из 4 и более цифр
То есть при проверке

https://www.avito.ru/moskva/odezhda_obuv_aksessuary/magazin_pizham_kigurumi_na_lubyanke_56545

должно возвращать false
а при проверке https://www.avito.ru/moskva/kvartiry/snimu — true

Comments

[Сергей Сергей]

(?!_\d{4,}$)

[Александр Макаров]

Сергей Сергей,
/^https:\/\/([\w]+\.)?avito.ru\/[\w-]+\/.+(?!_\d{4,}$)/.test('https://www.avito.ru/moskva/kvartiry/snimu_1-k_kva...') - дает true, а должно false (

[Сергей Сергей]

[\w-]+\/.+ надо поправить на .+

[Александр Макаров]

Сергей Сергей,
/^https:\/\/([\w]+\.)?avito.ru\/.+(?!_\d{4,}$)/.test('https://www.avito.ru/moskva/kvartiry/snimu_1-k_kva...') = true (

[Сергей Сергей]

/^https:\/\/(?:\w+\.)?avito.ru\/(?:.(?!_\d{4,}))+$/

[Александр Макаров]

Сергей Сергей, Спасибо! Отметил бы решением, но в комментах неззя

Answer 1

[Сергей Соколов]

Может, не нужно проверять всю строку? Если все линки правильные-авитные, то проверять только окончание строки – можно реверснуть строку .split('').reverse().join('') и проверять только начало /^\d{4,}_/

Вот одной регуляркой под ваше требование:

var re = new RegExp('^https?://(\\w+\\.)?avito\\.ru/([^/]+/)+([a-z_]+(?!_\\d{4,}))$');

var t1 = 'https://www.avito.ru/moskva/odezhda_obuv_aksessuary/magazin_pizham_kigurumi_na_lubyanke_56545';
var t2 = 'https://www.avito.ru/moskva/kvartiry/snimu';

re.test(t1) // false
re.test(t2) // true

Comments

[Александр Макаров]

Круто, работает!

А зачем двойные обратные слэши? С одинарными тоже работает.
И еще, подскажите плиз, что значит запись ([^/]+/)+, я ее никак не пойму?

[Сергей Соколов]

[^/]+/ – всё-кроме-слеша 1 или несколько раз, и потом слеш. Например, "olo_lo/".
Всё это в скобках с плюсом – такая конструкция 1 или несколько раз: "ololo/trololo/"

Двойные обратные если регулярка как строка в конструктор new RegExp() идёт. В обычной строке '\d' даст просто символ 'd'

[Александр Макаров]

Сергей Соколов,

Первый вопрос понял, спасибо.
По второму, я попробовал через RegExp с одинарными слэшами - все работает.

[Сергей Соколов]

Александр Макаров,

var re1 = new RegExp('^\d+$');
var re2 = new RegExp('^\\d+$');
var str = '12345';

re1.test(str) // false
re2.test(str) // true

[Александр Макаров]

Сергей Соколов,
Понял!

Answer 2

[Сергей]

Попробуй сделать от противного.
^https:\/{2}(?:.*?\.)?avito\.ru[\/\w-]+(_\d{4,})$