Использование авто в шлейфы C++

Question

Я получаю предупреждение подписал/без знака рассогласования для следующего кода:

auto n = a.size(); 
for (auto i = 0; i < n; i++) { 
} 

Проблема заключается в том, что, назначая от 0 до i становится int, а не size_t. Так что лучше:

size_t n = a.size(); 
for (size_t i = 0; i < n; i++) { 
} 

или это:

auto n = a.size(); 
for (size_t i = 0; i < n; i++) { 
} 

или, может быть, у вас есть лучшее решение? Мне нравится первый, потому что бит более согласован, он просто использует size_t, а не size_t и auto для этой же цели. на основе цикл

Answer 1

Диапазона может быть чист раствором:

for (const auto& i : a) 
{ 

} 

Здесь i является const ссылкой на элемент контейнера a.

В противном случае, если вам нужен индекс, или если вы не хотите перебирать весь диапазон, вы можете получить его с помощью decltype(a.size()).

for (decltype(a.size()) i = 0; i < a.size(); ++i) { 
} 

Answer 2

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

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

auto n = a.size(); 
for (decltype(n) i = 0; i < n; i++) { 
} 

Таким образом, вы связать i и n типов всегда совпадает друг с другом.

Answer 3

Если вы использовали правильный литерал, все будет нормально: 0U. auto видит литерал типа int, так что это тип i. Добавьте U, и вместо этого вы увидите неподписанный int literal. В противном случае вы бы хотели использовать decltype, как предложили другие, тем более, что sizeof (size_t) может быть больше sizeof (int) (он находится в Windows, OS X и т. Д., Если он работает в режиме с 64-разрядным интервалом).

Answer 4

for(auto n = a.size(), i = 0; i != n; ++i) { 
} 

... возможно, самое чистое решение, если вам нужно получить доступ к индексу, а также к фактическому элементу.

Update:

for(auto n = a.size(), i = n*0; i != n; ++i) { 
} 

Было бы обходной путь для Richard Smiths комментарий, хотя это не выглядит, что чистый больше.

Answer 5

Для обсуждения:

auto n = a.size(); 
for (auto i = n-n; i<n; ++i) { 
} 

Обратите внимание, что для типов меньше int, результат вычитания расширяется до int (так называемого целочисленного продвижения).

Answer 6

Попытки быть константным-правильно, когда это возможно, я обычно пишу:

const auto n(a.size()); 
for (auto i = decltype(n){0}; i < n; ++i) 
{ 
} 

Это не очень краткое, но это понятно, что вы хотите переменный инициализируются 0 типа n «s (и nconst).

Answer 7

Вот простое решение для очистки &, чтобы оно работало.

for(auto i: a) 
{ 
}