HowTo: Цикл foreach - будьте аккуратны, Qt понимает только простые типы

[xintrea]

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

Код

C++ (Qt)
QList< QMap<QString, QString> > branch;

Мне нужно было пробежать по списку QList, и распечатать каждую пару ключ-значение для QMap.

Заглянув в документацию, я написал следующий код:

Код

C++ (Qt)
 // Пробегается QList
 foreach(QMap<QString, QString> current_branch, branch)
 {
  qDebug() << "Branch data:";
 
  // Пробегается QMap, для извлечения пар ключ-значение
  foreach(QString field_name, current_branch.keys())
   qDebug() << field_name << ":" << current_branch.value(field_name);
 }

Как оказалось, скомпилировать данный код невозможно. Выдается следующая ошибка:

error: macro "Q_FOREACH" passed 3 arguments, but takes just 2

То есть, в строке "foreach(QMap<QString, QString> current_branch, branch)" запятая между QString воспринимается как запятая для разделения параметров, хотя на деле является запятой в описании типа.

Решение в такой ситуации следующее - описать новый тип, который "вберёт" в себя тип, в котором есть запятая, и в цикле foreach использовать этот новый тип. Вот так:

Код

C++ (Qt)
 // Пробегается QList
 typedef QMap<QString, QString> branch_type;
 foreach(branch_type current_branch, branch)
 {
  qDebug() << "Branch data:";
 
  // Пробегается QMap, для извлечения пар ключ-значение
  foreach(QString field_name, current_branch.keys())
   qDebug() << field_name << ":" << current_branch.value(field_name);
 }

[Авварон]

Мок не имеет отношения к форычу. Это чистые макросы/темплейты.

[xintrea]

Мок не имеет отношения к форычу. Это чистые макросы/темплейты.

Очень даже имеет, ибо использование foreach всеравно проходит через MOC - "Q_FOREACH. This macro is available even when no_keywords is specified using the .pro file's CONFIG variable." Обработкой .pro занимается qmake, используя MOC для преобразования/дополнения кода, и то, что foreach обрабатывается только C/С++ - шными механизмами макросов и темплейтов, целиком и полностью лежит на MOC. Тролли могли бы написать и более корректное раскрытие нестандартного ключевого слова foreach.

[Vass]

Добавлю, что это касается вообще всех Qt'шных макросов, например, на собственном опыте убедился, что тип с запятой не принимается в Q_DECLARE_METATYPE.

[Авварон]

xintrea
Интересно, а как же тогда форыч работает в cmake, где сорцы для мока указываются руками? И даже если нет ни 1го мокнутого файла, то форыч работает. Хз, откуда вы взяли про связь мока и форыча - смотрите объявление макросов, там нет никаких заглушек, которые бы вставлял мок (да и куда он их будет вставлять??)

[xintrea]

Интересно, а как же тогда foreach работает в cmake, где сорцы для MOC указываются руками? И даже если нет ни одного MOC-файла, то foreach работает. Хз, откуда вы взяли про связь MOC и foreach - смотрите объявление макросов, там нет никаких заглушек, которые бы вставлял MOC (да и куда он их будет вставлять??)

Скажу по-другому.

В данный момент foreach трансформируется в C/C++ код только макросами, входящими в состав Qt, от этого и проблема. По хорошему, цикл foreach должен обрабатывать MOC, причем так, чтобы небыло спотыканий на запятых, входящих в описание типа используемой переменной.

Согласен, что название топика неудачное, ща подумаю на что исправить.

[Авварон]

Учитывайте, что moc не может "править" сорцы, в отличие от препроцессора - он же фактически изменяет .cpp, тогда как moc создает свой файл и в нем срет. Так что это технологически сложнее.

[zenden]

Код

C++ (Qt)
#define HAS_COMMAS(...) __VA_ARGS__
 
QList< QMap<QString, QString> > branch;
 
foreach(HAS_COMMAS(QMap<QString, QString> current_branch), branch)
{
 //....
}
 

проверял под mingw

[ритт]

Добавлю, что это касается вообще всех Qt'шных макросов, например, на собственном опыте убедился, что тип с запятой не принимается в Q_DECLARE_METATYPE.

добавлю, что это вообще не касается Qt'шных макросов, но макросов как таковых. для препроцессора С это норма и в книгах часто можно встретить упоминание об этом...
перекладывать данную задачу на мок - имхо, бред.

и просто не могу пройти мимо
> foreach(branch_type current_branch, branch) // больше сущностей хороших и всяких (а также ненужных)!
найдите отличия с
> foreach(const branch_type &current_branch, branch)

[Авварон]

Константин
Ты меня уже приучил почти:)

[blood_shadow]

оператор continue коректно работает внутри qt-шного foreach?

[Пантер]

Да.

[artiom_kh]

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

Код

C++ (Qt)
QList< QMap<QString, QString> > branch;

Мне нужно было пробежать по списку QList, и распечатать каждую пару ключ-значение для QMap.

Заглянув в документацию, я написал следующий код:

Код

C++ (Qt)
 // Пробегается QList
 foreach(QMap<QString, QString> current_branch, branch)
 {
  qDebug() << "Branch data:";
 
  // Пробегается QMap, для извлечения пар ключ-значение
  foreach(QString field_name, current_branch.keys())
   qDebug() << field_name << ":" << current_branch.value(field_name);
 }

Как оказалось, скомпилировать данный код невозможно. Выдается следующая ошибка:

error: macro "Q_FOREACH" passed 3 arguments, but takes just 2

То есть, в строке "foreach(QMap<QString, QString> current_branch, branch)" запятая между QString воспринимается как запятая для разделения параметров, хотя на деле является запятой в описании типа.

Решение в такой ситуации следующее - описать новый тип, который "вберёт" в себя тип, в котором есть запятая, и в цикле foreach использовать этот новый тип. Вот так:

Код

C++ (Qt)
 // Пробегается QList
 typedef QMap<QString, QString> branch_type;
 foreach(branch_type current_branch, branch)
 {
  qDebug() << "Branch data:";
 
  // Пробегается QMap, для извлечения пар ключ-значение
  foreach(QString field_name, current_branch.keys())
   qDebug() << field_name << ":" << current_branch.value(field_name);
 }

а вот так:

Код

C++ (Qt)
QList< QMap<QString, QString> > branch;
 
 // Пробегается QList
 foreach(const QMap<QString, QString>& current_branch, branch) {}
 

?

[mutineer]

Удивительное рядом - если начать наконец-то читать документацию (а не что вы там делаете с ней), то там можно найти такие строки

За исключением типа данных, содержащего запятую (например, QPair<int, int>), переменная, используемая для перебора элементов контейнера может быть определена внутри выражения foreach:

[Bepec]

некропостеры блин, успокойтесь