Первым является корректный тип, и корректный в некотором строгом смысле. (Если Вы думаете о, размер никогда не может быть меньше, чем нуль.), Что предупреждение кажется мне одним из хороших кандидатов на то, чтобы быть проигнорированным, все же.

for (vector<int>::iterator it = polygon.begin(); it != polygon.end(); it++)
    sum += *it; 

Немного истории:

, Чтобы представить, отрицательно ли число или не компьютерное использование, которое укусил 'знак'. int тип данных со знаком, означающий его, может содержать положительные и отрицательные величины (о-2billion к 2 миллиардам). Unsigned может только сохранить положительные числа (и так как это не пропадает впустую немного на метаданных, это может сохранить больше: 0 к приблизительно 4 миллиардам).

std::vector::size() возвраты unsigned, поскольку, как вектор мог иметь отрицательную длину?

предупреждение говорит Вам, что правильный операнд Вашего оператора неравенства может содержать больше данных тогда левые.

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

Вызов к vector<T>::size() возвраты значение типа std::vector<T>::size_type, не интервал, неподписанный интервал или иначе.

Также обычно повторение по контейнеру в C++ сделано с помощью итераторы , как это.

std::vector<T>::iterator i = polygon.begin();
std::vector<T>::iterator end = polygon.end();

for(; i != end; i++){
    sum += *i;
}

, Где T является типом данных, Вы храните в векторе.

Или использование различных итеративных алгоритмов (std::transform, std::copy, std::fill, std::for_each и так далее).

Используйте size_t:

for (size_t i=0; i < polygon.size(); i++)

Заключение в кавычки Википедия :

stdlib.h и stddef.h заголовочные файлы определяют тип данных, названный size_t, который используется для представления размера объекта. Библиотечные функции, которые берут размеры, ожидают, что они будут типа size_t, и sizeof оператор оценивает к size_t.

фактический тип size_t зависим от платформы; частая ошибка состоит в том, чтобы принять size_t, совпадает с неподписанным интервалом, который может привести к программным ошибкам, особенно поскольку 64-разрядная архитектура становится более распространенной.

Я обычно использую BOOST_FOREACH:

#include <boost/foreach.hpp>

BOOST_FOREACH( vector_type::value_type& value, v ) {
    // do something with 'value'
}

Это работает над контейнерами STL, массивами, строками C-стиля, и т.д.

В конкретном случае в Вашем примере я использовал бы алгоритмы STL для выполнения этого.

#include <numeric> 

sum = std::accumulate( polygon.begin(), polygon.end(), 0 );

Для более общего, но все еще довольно простого случая, я пошел бы с:

#include <boost/lambda/lambda.hpp>
#include <boost/lambda/bind.hpp>

using namespace boost::lambda;
std::for_each( polygon.begin(), polygon.end(), sum += _1 );

Относительно ответа Йоханнеса Шауба:

for(std::vector<T*>::iterator it = v.begin(); it != v.end(); ++it) { 
...
}

Это может работать с некоторыми компиляторами, но не с gcc. Проблема здесь в том, является ли std :: vector :: iterator типом, переменной (членом) или функцией (методом). При использовании gcc мы получаем следующую ошибку:

In member function ‘void MyClass<T>::myMethod()’:
error: expected `;' before ‘it’
error: ‘it’ was not declared in this scope
In member function ‘void MyClass<T>::sort() [with T = MyClass]’:
instantiated from ‘void MyClass<T>::run() [with T = MyClass]’
instantiated from here
dependent-name ‘std::vector<T*,std::allocator<T*> >::iterator’ is parsed as a non-type, but instantiation yields a type
note: say ‘typename std::vector<T*,std::allocator<T*> >::iterator’ if a type is meant

Решение использует ключевое слово 'typename', как сказано:

typename std::vector<T*>::iterator it = v.begin();
for( ; it != v.end(); ++it) {
...