今天看到有一个网友在群里发了下面这样一段代码,说这段代码运行起来时不时出现Segmentation fault错误,但是找不到原因。这段代码的功能非常简单,从头到尾打印vector中的每个元素(vector的规模较小,不会超过int的表示范围)。看起来好像没什么问题?为什么程序会崩溃呢?

void printVector(vector<int> &s)
{
    for (int i = 0; i <= s.size() - 1; i++) {
        cout << s[i] << endl;
    }
}

这段程序出bug的原因在于s.size() - 1的值。

查一下C plus plus网站可以知道,size()的返回值是一个无符号整型。当传入的s为空时,s.size() 返回0,0 - 1的值转换成无符号整型是0xFFFF即十进制的4294967295,i的值是0,显然是小于4294967295,因此判断条件成立,进入for循环,打印s[0]。但是访问这个内存是非法的,因此程序报Segmentation fault错误。

这个bug是因为对vector的使用不熟悉造成的,挺简单,要注意!