Печать списков с запятыми C ++

Комбинация C++ 11 лямбд и макрос:

#define INFIX_PRINTER(os, sep)([&]()->decltype(os)&{static int f=1;os<<(f?(f=0,""):sep);return os;})()

Использование:

for(const auto& k: keywords)
    INFIX_PRINTER(out, ", ") << k;

Я бы выбрал что-то вроде этого, простое решение, которое должно работать для всех итераторов.

int maxele = maxele = v.size() - 1;
for ( cur = v.begin() , i = 0; i < maxele ; ++i)
{
    std::cout << *cur++ << " , ";
}
if ( maxele >= 0 )
{
  std::cout << *cur << std::endl;
}

Вы можете использовать цикл do, переписать условие цикла для первой итерации и использовать оператор замыкания && и тот факт, что действительный поток является true.

auto iter = keywords.begin();
if ( ! keywords.empty() ) do {
    out << * iter;
} while ( ++ iter != keywords.end() && out << ", " );
out << endl;

Я думаю, это должно работать

while (iter != keywords.end( ))
{

    out << *iter;
    iter++ ;
    if (iter != keywords.end( )) out << ", ";
}

Есть небольшая проблема с оператором ++, который вы используете.

Вы можете попробовать:

if (++iter != keywords.end())
    out << ", ";
iter--;

Таким образом, ++ будет оцениваться до сравнения итератора с keywords.end().

Попробуйте следующее:

typedef  std::vector<std::string>   Container;
typedef Container::const_iterator   CIter;
Container   data;

// Now fill the container.


// Now print the container.
// The advantage of this technique is that ther is no extra test during the loop.
// There is only one additional test !test.empty() done at the beginning.
if (!data.empty())
{
    std::cout << data[0];
    for(CIter loop = data.begin() + 1; loop != data.end(); ++loop)
    {
        std::cout << "," << *loop;
    }
}

Что-то вроде этого?

while (iter != keywords.end())
{
 out << *iter;
 iter++;
 if (iter != keywords.end()) cout << ", ";
}

Мой типичный метод использования разделителей (на любом языке ) - это использовать промежуточный цикл. Код C ++ будет выглядеть так:

for (;;) {
   std::cout << *iter;
   if (++iter == keywords.end()) break;
   std::cout << ",";
}

(примечание: дополнительная проверка if необходима перед циклом, если ключевые слова могут быть пустыми)

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

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

const char *padding = "";
for (auto iter = keywords.begin(); iter != keywords.end(); ++iter) {
    out << padding << *iter;
    padding = ", "
}

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

В качестве альтернативы вы должны иметь возможность создать свой собственный поток, который поддерживает текущее состояние строки (до endl) и помещает запятые в соответствующие места.

РЕДАКТИРОВАТЬ: Вы также можете использовать цикл с промежуточной проверкой, предложенный T.E.D. Это будет примерно так:

if(!keywords.empty())
{
    auto iter = keywords.begin();
    while(true)
    {
        out << *iter;
        ++iter;
        if(iter == keywords.end())
        {
            break;
        }
        else
        {
            out << ", ";
        }
    }
}

Я сначала упомянул метод «напечатать первый элемент перед циклом», потому что он сохраняет тело цикла очень простым, но любой из подходов работает нормально.

Поскольку все решили сделать это с помощью циклов while, я приведу пример с циклами for.

for (iter = keywords.begin(); iter != keywords.end(); iter++) {
  if (iter != keywords.begin()) cout << ", ";
  cout << *iter;
}

Может быть так ..

bool bFirst = true;
for (auto curr = keywords.begin();  curr != keywords.end(); ++curr) {
   std::cout << (bFirst ? "" : ", ") << *curr;
   bFirst = false;
}