杀死无符号/签名的比较错误

时间:2010-04-28 21:03:47

标签: c++ comparison g++ suppress-warnings

一般来说,我想要无符号vs签名的警告。

然而,在这种特殊情况下,我希望它被压制;

std::vector<Blah> blahs;

for(int i = 0; i < blahs.size(); ++i) { ...

我想杀死这个比较。

谢谢!

(使用g ++)

1 个答案:

答案 0 :(得分:12)

你应该修复,而不是抑制。使用无符号类型:

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

您也可以使用unsigned,但size_t更合适(并且可能有不同的,更大的范围)。如果你只是使用i进行迭代而不需要在循环中使用它的值,那么请使用迭代器:

for (auto iter = blahs.begin(), end = blahs.end(); iter != end; ++iter)

如果您的编译器不支持auto,请将auto替换为T::iteratorT::const_iterator,其中T的类型为blahs。但是,如果您的编译器支持更完整的C ++ 11子集,请执行以下操作:

for (auto& element : blahs)

哪个最好。


严格来说,上述情况并非“正确”。它应该是:

typedef std::vector<Blah> blah_vec;
blah_vec blahs;

for (blah_vec::size_type i = 0; i < blahs.size(); ++i)

但这可能很冗长,而且我知道的每个实现都使用size_t作为size_type


如果由于某种原因你真的需要i的有符号整数类型,你必须施放:

// assumes size() will fit in an int
for (int i = 0; i < static_cast<int>(blahs.size()); ++i)

// assumes i will not be negative (so use an unsigned type!)
for (int i = 0; static_cast<size_t>(i) < blahs.size(); ++i)

// and the technically correct way, assuming i will not be negative
for (int i = 0; static_cast<blah_vec::size_type>(i) < blahs.size(); ++i)