Вопрос

У меня есть макрос foreach, который я часто использую в C++, который работает для большинства контейнеров STL:

#define foreach(var, container) \
  for(typeof((container).begin()) var = (container).begin(); \
      var != (container).end(); \
      ++var)

(Обратите внимание, что typeof — это расширение gcc.) Оно используется следующим образом:

std::vector< Blorgus > blorgi = ...;
foreach(blorgus, blorgi) {
  blorgus->draw();
}

Я хотел бы сделать что-то подобное, которое перебирает значения карты.Назовите это, возможно, «foreach_value».Поэтому вместо того, чтобы писать

foreach(pair, mymap) {
  pair->second->foo();
}

я бы написал

foreach_value(v, mymap) {
  v.foo();
}

Я не могу придумать макрос, который бы это сделал, потому что он требует объявления двух переменных:итератор и переменная значения ('v', выше).Я не знаю, как это сделать в инициализаторе цикла for, даже используя расширения gcc.Я мог бы объявить его непосредственно перед вызовом foreach_value, но тогда он будет конфликтовать с другими экземплярами макроса foreach_value в той же области видимости.Если бы я мог присоединить текущий номер строки к имени переменной итератора, это сработало бы, но я не знаю, как это сделать.

Это было полезно?

Решение

Вы можете сделать это, используя два цикла.Первый объявляет итератор с именем, которое является функцией переменной контейнера (и вы можете сделать это более уродливым, если беспокоитесь о конфликтах с вашим собственным кодом).Второй объявляет переменную значения.

#define ci(container) container ## iter
#define foreach_value(var, container) \
    for (typeof((container).begin()) ci(container) = container.begin(); \
         ci(container) != container.end(); ) \
        for (typeof(ci(container)->second)* var = &ci(container)->second; \
             ci(container) != container.end(); \
             (++ci(container) != container.end()) ? \
                 (var = &ci(container)->second) : var)

При использовании того же условия завершения цикла внешний цикл происходит только один раз (и, если вам повезет, он будет оптимизирован).Кроме того, вы избегаете вызова ->секунды на итераторе, если карта пуста.По той же причине используется тернарный оператор приращения внутреннего цикла;в конце мы просто оставляем последнее значение var, так как на него больше не будут ссылаться.

Вы могли бы встроить ci(container), но я думаю, что это сделает макрос более читабельным.

Другие советы

Вы бы искали BOOST_FOREACH - они уже сделали всю работу за вас!

Если вы действительно хотите свернуть свой собственный, вы можете объявить блок в любом месте C ++, что решает вашу проблему с помощью вашего промежуточного хранения ITR-> Second ...

// Valid C++ code (which does nothing useful)
{
  int a = 21; // Which could be storage of your value type
}
// a out of scope here
{ 
  int a = 32; // Does not conflict with a above
}

СТЛ трансформировать функция также делает нечто подобное.

Аргументы (по порядку):

  1. Входной итератор, обозначающий начало контейнера.
  2. Входной итератор, обозначающий конец контейнера.
  3. Выходной итератор, определяющий, куда поместить выходные данные (для преобразования на месте, аналогичного for-each, просто передайте входной итератор в # 1)
  4. Унарная функция (объект функции), выполняемая над каждым элементом.

В качестве очень простого примера вы можете написать каждый символ в строке с заглавной буквы:

#include <iostream>
#include <string>
#include <algorithm>
#include <cctype>

int main(int argc, char* argv[]) {
    std::string s("my lowercase string");
    std::transform(s.begin(), s.end(), s.begin(), toupper);
    std::cout << s << std::endl; // "MY LOWERCASE STRING"
}

В качестве альтернативы также существует накапливать функция, которая позволяет сохранять некоторые значения между вызовами объекта функции. накапливать не изменяет данные во входном контейнере, как в случае с трансформировать.

Вы думали об использовании Увеличить библиотеки?У них есть foreach макрос реализован что, вероятно, более надежно, чем все, что вы напишете...и есть также transform_iterator который, казалось бы, можно использовать для выполнения второй части извлечения того, что вы хотите.

К сожалению, я не могу вам сказать точно как использовать его, потому что я недостаточно знаю C++ :) Этот поиск Google дает несколько многообещающих ответов: comp.lang.c++.модерируемый, Вариант использования Boost Transform_iterator.

Boost::For_each — безусловно, лучший выбор.Самое приятное то, что они на самом деле дают вам макрос BOOST_FOREACH(), который вы затем можете обернуть и #определить так, как вы действительно хотели бы назвать его в своем коде.Большинство людей выберут старый добрый «foreach», но в других магазинах могут быть другие стандарты кодирования, так что это соответствует такому мышлению.В Boost также есть много других полезных вещей для разработчиков C++!Определенно стоит использовать.

Я создал небольшой помощник Foreach.h с несколькими вариантами foreach(), включая те, которые работают как с локальными переменными, так и с указателями, а также дополнительную версию, защищенную от удаления элементов из цикла.Итак, код, использующий мои макросы, выглядит красиво и уютно вот так:

#include <cstdio>
#include <vector>
#include "foreach.h"

int main()
{
    // make int vector and fill it
    vector<int> k;
    for (int i=0; i<10; ++i) k.push_back(i);

    // show what the upper loop filled
    foreach_ (it, k) printf("%i ",(*it));
    printf("\n");

    // show all of the data, but get rid of 4
    // http://en.wikipedia.org/wiki/Tetraphobia :)
    foreachdel_ (it, k)
    {
        if (*it == 4) it=k.erase(it);
        printf("%i ",(*it));
    }
    printf("\n");

    return 0;
}

выход:

0 1 2 3 4 5 6 7 8 9
0 1 2 3 5 6 7 8 9

Мой Foreach.h предоставляет следующие макросы:

  • foreach() — обычный foreach для указателей
  • foreach_() — обычный foreach для локальных переменных
  • foreachdel() — версия foreach с проверками на удаление внутри цикла, версия указателя
  • foreachdel_() — версия foreach с проверками на удаление внутри цикла, версия локальной переменной

Мне они определенно помогают, надеюсь, они также сделают вашу жизнь немного проще :)

В этом вопросе есть две части.Вам нужно каким-то образом (1) сгенерировать итератор (или, скорее, итерируемую последовательность) над вашей картой. ценности (не ключи) и (2) использовать макрос для выполнения итерации без большого количества шаблонов.

Самое чистое решение — использовать Диапазон повышения Адаптер для части (1) и Повышение Foreach по части (2).Вам не нужно писать макрос или реализовывать итератор самостоятельно.

#include <map>
#include <string>
#include <boost/range/adaptor/map.hpp>
#include <boost/foreach.hpp>

int main()
{
    // Sample data
    std::map<int, std::string> myMap ;
    myMap[0] = "Zero" ;
    myMap[10] = "Ten" ;
    myMap[20] = "Twenty" ;

    // Loop over map values
    BOOST_FOREACH( std::string text, myMap | boost::adaptors::map_values )
    {
        std::cout << text << " " ;
    }
}
// Output:
// Zero Ten Twenty

Вы можете определить класс шаблона, который принимает тип mymap в качестве параметра шаблона и действует как итератор над значениями, перегружая * и ->.

#define foreach(var, container) for (typeof((container).begin()) var = (container).begin(); var != (container).end(); ++var)

В C++ нет typeof...как у вас это компилируется?(это, конечно, не портативный)

я реализовал свой собственный foreach_value на основе Boost foreach код:

#include <boost/preprocessor/cat.hpp>
#define MUNZEKONZA_FOREACH_IN_MAP_ID(x)  BOOST_PP_CAT(x, __LINE__)

namespace munzekonza {
namespace foreach_in_map_private {
inline bool set_false(bool& b) {
  b = false;
  return false;
}

}
}

#define MUNZEKONZA_FOREACH_VALUE(value, map)                                  \
for(auto MUNZEKONZA_FOREACH_IN_MAP_ID(_foreach_in_map_it) = map.begin();      \
        MUNZEKONZA_FOREACH_IN_MAP_ID(_foreach_in_map_it) != map.end();)       \
for(bool MUNZEKONZA_FOREACH_IN_MAP_ID(_foreach_in_map_continue) = true;       \
      MUNZEKONZA_FOREACH_IN_MAP_ID(_foreach_in_map_continue) &&               \
      MUNZEKONZA_FOREACH_IN_MAP_ID(_foreach_in_map_it) != map.end();          \
      (MUNZEKONZA_FOREACH_IN_MAP_ID(_foreach_in_map_continue)) ?              \
        ((void)++MUNZEKONZA_FOREACH_IN_MAP_ID(_foreach_in_map_it)) :          \
        (void)0)                                                              \
  if( munzekonza::foreach_in_map_private::set_false(                          \
          MUNZEKONZA_FOREACH_IN_MAP_ID(_foreach_in_map_continue))) {} else    \
  for( value = MUNZEKONZA_FOREACH_IN_MAP_ID(_foreach_in_map_it)->second;      \
        !MUNZEKONZA_FOREACH_IN_MAP_ID(_foreach_in_map_continue);              \
        MUNZEKONZA_FOREACH_IN_MAP_ID(_foreach_in_map_continue) = true)        

Например, вы можете использовать его в своем коде следующим образом:

#define MUNZEKONZA_FOREACH_VALUE foreach_value

std::map<int, std::string> mymap;
// populate the map ...

foreach_value( const std::string& value, mymap ) {
  // do something with value
}

// change value
foreach_value( std::string& value, mymap ) {
  value = "hey";
}
#define zforeach(var, container) for(auto var = (container).begin(); var != (container).end(); ++var)

нет typeof(), поэтому вы можете использовать это:

decltype((container).begin()) var 
decltype(container)::iterator var
Лицензировано под: CC-BY-SA с атрибуция
Не связан с StackOverflow
scroll top