Проверяется текущий элемент является последним элементом множества

Question

Я пытаюсь написать функцию печати для набора в C++, и это то, что я писал:

void print_set(set<int> &s) 
{ 
    cout<<"{"; 
    for(auto it = s.begin() ; it!=s.end() ; ++it) 
    { 
     cout<<*it; 
     if(it!=(s.end()-1)) //shows error here 
      cout<<","; 
    } 
    cout<<"}"; 
} 

Но я получаю сообщение об ошибке. Как я могу проверить, является ли текущий элемент последним элементом или нет?

Answer 1

Могу ли я предложить альтернативный подход?

печати запятая перед тем каждый элемент, кроме первого:

void print_set(set<int> &s) 
{ 
    cout << "{"; 
    for(auto it = s.begin() ; it != s.end() ; ++it) 
    { 
     if(it != s.begin()) 
      cout << ", "; 
     cout << *it; 
    } 
    cout << "}"; 
} 

Answer 2

Просто проверить, равен ли следующий элемент конец:

auto next = it; 
++next; 
if (next != s.end()) 
    cout << ","; 

Answer 3

set «s целые числа не являются произвольным доступом, так что вы не можете выполнять арифметические операции над ними. Вместо этого используйте std::prev.

Answer 4

На установленных итераторах можно применять только операторы ++ и --. Добавление номера не определено. Вы можете сделать свой код работать так:

void print_set(set<int> &s) 
{ 
    cout<<"{"; 
    auto second_to_last = s.end(); 
    if (!s.empty()) { 
     second_to_last--; 
    } 
    for(auto it = s.begin() ; it!=s.end() ; ++it) 
    { 
     cout<<*it; 
     if(it!=second_to_last) { 
      cout<<", "; 
     } 
    } 
    cout<<"}"; 
} 

Что делает этот код, по существу, хранить итератор на второй до последнего элемента один раз, а затем сравнить элемент, который вы имеете с ним. Обратите внимание, что second_to_last не будет точным, если набор пуст, но код будет работать, как ожидалось.

Answer 5

Что делать в этом случае:

void print_set(const std::set<int>& s) 
{ 
    const char* sep = ""; 
    std::cout << "{"; 
    for (int e : s) 
    { 
     std::cout << sep << e; 
     sep = ", "; 
    } 
    std::cout << "}"; 
}