所以我有这样的东西(C++03(:
class MyClass
{
// ...
}
class something
{
private:
std::vector<MyClass*> container;
// ...
}
// cmdarg can be anything, negative int too...
void something::foo(const std::string& cmdarg)
{
const int res = std::stoi(cmdarg);
if (res >= 0 && static_cast<std::vector<MyClass*>::size_type>(res) < this->container.size())
{
// ...
}
}
我想问一下从int
到std::vector<MyClass*>::size_type
的转换是否有效。res >= 0
说它是非负数,所以我想转换成另一个非负数是可以的。
我的问题是,如果我写
if (res >= 0 && res < container.size())
由于与有符号和无符号整数类型进行比较,我收到警告。
我上面的代码(完整的代码(编译后似乎可以工作,但我不确定。
谢谢。
您的代码看起来有点太完美了。
分解:
const int res = std::stoi(cmdarg);
if (res >= 0 && static_cast<std::vector<MyClass*>::size_type>(res) < this->container.size())
用于检查零以下的if语句很好。就我个人而言,我会这样写:
if (res < 0)
{
std::cerr << "Negative number " << res <<" given for ..., a positive was expected" << std::endl;
return -1;
}
这就引出了演员阵容:
auto unsigned_res = static_cast<std::vector<MyClass*>::size_type>(res);
但是,size_type这个向量总是size_t
,因为它使用std::allocator
。在代码审查中,为了可读性,我要求将其更改为:
auto unsigned_res = static_cast<std::size_t>(res);
最后,你可以很好地比较一下:
if (unsiged_res < container.size())
// Do something
请注意,我提到了比较和演员阵容,因为这需要按顺序进行。除此之外,当std::stoi
失败时,您还需要一些异常处理,请参阅文档
有关如何正确处理已签名/未签名的更多详细信息,我可以在ithare上推荐这篇文章。