Есть ли более элегантный способ сделать следующее в C++ 11?

Question

Я хочу избежать предупреждений о сравнении между unsigned int и подписанным int.

Я использую петли, такие как:

for (int i =0 ; i < vec.size(); i++) { 
// do something 
} 

, который дает эти предупреждения (что само по себе хорошо, я хочу, высокий уровень предупреждений).

Я думал об изменении его на следующее:

for (auto i = vec.size()*0; i < vec.size(); i++) { 
// do something 
} 

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

Answer 1

Использование диапазона на основе цикл

for (auto& i : vec) 
{ 
    cout << *i; 
} 

использовать итераторы.

for (auto it = vec.begin(), end_it = vec.end(); it != end_it; ++it) 
{ 
    cout << *it; 
} 

Если вы хотите счетчик, используйте std::vector<T>::size_type или просто size_t, если вы ленивы.

Answer 2

Поскольку вопрос начинается с фразой:

«Я хочу, чтобы избежать предупреждений о сравнении между неподписанными INT и подписанных междами»

Я предположил, что vec.size() возвращает unsigned int.

Если тип, возвращаемый vec.size(), может быть сохранен в unsigned int, то вы можете использовать следующее.

for (auto i = 0u; i < vec.size(); i++) { 
     // do something 
} 

В противном случае используйте соответствующий тип или проверьте другие ответы.

Если vec имеет тип std::vector<T> (который не был упомянут в вопросе, следовательно, выше коде) вы можете также использовать цикл for_each (который также не был упомянут)

std::for_each(vec.begin(), vec.end(), [](T t){ /* do something with t */ }); 
std::for_each(vec.begin(), vec.end(), [](T & t){ /* do something with t */}); 
std::for_each(vec.begin(), vec.end(), [](const T & t){ /* do something with t */ }); 

Это может быть более удобно, если у вас есть ходить по векторным элементам с некоторыми уже существующими функциональными-объектами (или методы или функцию, не являющиеся членами)

void f(int x) { std::cout << "f: " << x << "\n"; } 
void g(int x) { std::cout << "g: " << x << "\n"; } 
... 
std::vector<int> vec{1, 2, 3}; 
std::for_each(vec.begin(), vec.end(), std::bind(f, std::placeholders::_1)); //calls f(int) 
std::for_each(vec.begin(), vec.end(), std::bind(g, std::placeholders::_1)); //calls g(int) 

более удобно, еще потому вы можете держать фантазии обратного вызова и установить его в соответствующую функцию в соответствии с потребностями

std::function<void(int)> callback = std::bind(f, std::placeholders::_1); 
if (userWantsG()) 
    callback = std::bind(g, std::placeholders::_1); 
std::for_each(vec.begin(), vec.end(), callback); //calls g or f 

Answer 3

Или даже петли по убыванию:

auto i = vec.size(); 
while (i --> 0) 
    // ... 

Answer 4

Если это вариант, предпочитают стандартной библиотеки примитивов высшего порядка из <algorithm>, такие как (не ограничиваясь ими):

• std::accumulate
• std::copy
• std::transform

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