QRegExp, два подвыражения, предваряющихся известными строками

[Smogg]

Ниче не понимаю...

вот такая строчка:
https://oauth.vk.com/blank.html#access_token=17a6ab0190a8ad75c2f3dffa713c3708c6943dc1562cdee85252f33a8fdabdbc803d33e7aa841bc4e9b70&expires_in=86400&user_id=14266994

как из нее получить два Qstring'a - sAccess_token и sExpires_in?

Код:

QRegExp rx;
rx.setPattern("(?:access_token=(\\w+))(?:expires_in=(\\d+))");

if (y = rx.indexIn(s) ) {	
	sAccess_token = rx.cap(1);
	sExpires_in = rx.cap(2);
	}

не работает

[kambala]

Код

C++ (Qt)
QRegExp rx("access_token=([a-z\\d]+)&expires_in=(\\d+)");
rx.setMinimal(true);

это если порядок параметров гарантирован. и то, тут можно и обычным поиском QString::indexOf() обойтись.

если же порядок параметров неизвестен (например expires_in идет перед access_token и между ними еще что-нибудь внутри), то лучше выполнить два отдельных поиска (опять же, хоть через QString::indexOf()).

[thechicho]

QRegExp rx; rx.setMinimal(true); rx.setCaseSensitivity(Qt::CaseInsensitive);
rx.setPattern("access_token=([^&]+).*&expires_in=([^&]+).*$");

и как сказал kambala лучше выполнять два отдельных поиска.

to kambala, через QString::indexOf() как это будет выглядеть?

[Bepec]

Две строки. Поиск первого вхождения, поиск второго вхождения, вырезание результатов.

Всё просто.

PS я бы indexOf использовал. Не люблю регЭкспы.

[thechicho]

Верес, код в студию.

[Bepec]

Код:

QString tmp = "https://oauth.vk.com/blank.html#access_token=17a6ab0190a8ad75c2f3dffa713c3708c6943dc1562cdee85252f33a8fdabdbc803d33e7aa841bc4e9b70&expires_in=86400&user_id=14266994";

int i1 = (tmp.indexOf("token="));
int i2 = (tmp.indexOf("&expires_in="));
int i3 = (tmp.indexOf("&user_id="));

QString sAccess_token = tmp.mid(i+6, i2);
QString sExpires_in = tmp.mid(i2+12, i3);

При желании можно ужать в две строки и без промежуточных переменных.

PS тот кто начнёт говорить про "гибкость" и "расширяемость" - огурец.

[thechicho]

tmp.mid(i+6, i2);
tmp.mid(i2+12, i3);
отжег

[Igors]

Код:

QString tmp = "https://oauth.vk.com/blank.html#access_token=17a6ab0190a8ad75c2f3dffa713c3708c6943dc1562cdee85252f33a8fdabdbc803d33e7aa841bc4e9b70&expires_in=86400&user_id=14266994";

int i1 = (tmp.indexOf("token="));
int i2 = (tmp.indexOf("&expires_in="));
int i3 = (tmp.indexOf("&user_id="));

QString sAccess_token = tmp.mid(i+6, i2);
QString sExpires_in = tmp.mid(i2+12, i3);

При желании можно ужать в две строки и без промежуточных переменных.

PS тот кто начнёт говорить про "гибкость" и "расширяемость" - огурец.

Это больше подходит для тех кто лепит 6 и 12 прямо в код (убивать надо), попутал парвметры mid, втулил нафиг не нужные скобки и никак не позаботился об обработке ошибок.

[Bepec]

thechicho, Igors - специально для вас я написал это в браузере, по памяти, максимально быстро, дабы передать примерное представление о коде.

PS переделка по умному займёт минуты полторы (90 секунд).  С отсутствием магических 6/12. Попробуйте - у вас должно получиться Займёт правда 4 строки.

PPS умение или религия не позволяет вам понять, что это пример кода?  

update: Igors религия не позволяет. Это я уже знаю. Так что последний вопрос к thechicho.

[kambala]

2Bepec: искать вместе с амперсандом не надо (вдруг во входной строке порядок параметров будет другой).

ну, и как уже сказали выше, магические числа — это нехорошо, лучше искомые строки засунуть в переменные и прибавлять их length()/qstrlen(). если показываешь пример, то показывай правильный, тем более что он бы занял не намного больше времени.

PS переделка по умному займёт минуты полторы (90 секунд).

[thechicho]

мне просто интересно было как это выглядит с QString::indexOf(). чтобы сравнить - может действительно так проще, чем регулярками парсить

[Bepec]

Ну уж извините, не было у меня тогда большого желания писать верный код. Было желание показать примерный алгоритм.

PS как то помню сравнивал RegExp с indexOf - цель была парсинг параметров по разделителю. RegExp был медленнее чем indexOf почему то.

[thechicho]

QRegExp может и программу "подвесить навеки", если регулярку кривую написать. но все равно на мой взгляд лучше для парсинга, чем строковые (QString) функции.
если нужна супер-мега производительность, наверное Qt не лучший вариант для решения задачи.

[Smogg]

PS я бы indexOf использовал. Не люблю регЭкспы.

Я тоже Но в данном случае правильнее и масштабируемее через регулярности, безотносительно моих симпатий(

rx.setPattern("access_token=([^&]+).*&expires_in=([^&]+).*$");

Что тут происходит пo моему разумению... :

• Отсутствие ^ в начале паттерна говорит, что перед первым соответствие регэкспу может быть что и сколько угодно.
• access_token=
  Потом прямо вписанный access_token= означает именно такой порядок символов.
• ([^&]+)
  Дальше открываются скобки и они означают начало нового экземпляра поиска по регэкспу и поэтому можно опять использовать ^?
  Как прочитал и запомнил, в квадратных скобках может быть перечисление без пробелов вариантов вхождений или интервалы через тире. Амперсанд же по справке не относится к служебным символам. Что & тут означает?
  + - как минимум одно вхождение предыдущего выражения.
• Потом очередное подвыражение: .*&expires_in
  каковое означает вхождение любого символа не важно сколько раз, пока не найдется очередной plaintext &expires_in, который начинается с &, использованный совсем в другом смысле в квадратных скобках...
  
• ([^&]+)
  дальше не понятный мне дзен в скобка
• .*$
  есть ли необходимость явно указывать, что до конца строки может быть еще что-то?

Еще один вопрос:
Как складывать в QStringList несколько соответствий одному и тому же регекспу?

[tester64]

Если очерёдность известна, то например так:

Код

C++ (Qt)
QRegExp rx(".+(?:access_token=([0-9a-f]+)(?:&|$)).*(?:expires_in=(\\d+)(?:&|$)).*");
if (rx.exactMatch(s)) { sAccess_token = rx.cap(1); sExpires_in = rx.cap(2); }