string类find函数返回值判定

代码示例

#include<iostream>
#include<cstring>
using namespace std;
int main()
{
static const size_t npos = -;
string s = "Alice Bob Charlie";
size_t position;
position = s.find("none");
//position = s.find("Bob");
if (position >= )
cout << "Found! position is : " << position << endl;
else
cout << "Not found!" << endl;
}

现象&后果

程序运行结果输出"Found! position is : 18446744073709551615",但实际上所找的字符串"none"并不存在于字符串s中。

Bug分析

程序的目的是,在源字符串s中查找目的字符串,若找到,则显示"Found",并返回目标子串在源字符串中的位置;反之,若未找到,则返回"Not found"。string.find在未找到时会返回string::npos。

在C++中常量npos是这样定义的:

  1. static const size_t npos = -1;

即常量npos定义的值为-1. 但又因为npos 的类型size_t是无符号整数类型,所以npos实际上是一个正数,并且是size_t类型的最大值。

上述代码中,把find函数返回的值赋给size_t类型的变量position,而size_t类型的变量position是永远大于等于0,所以即使find返回npos,if条件也为true。

正确的做法是在if条件中直接用npos作比较。

正确代码

#include<iostream>
#include<cstring>
using namespace std;
int main()
{
static const size_t npos = -;
string s = "Alice Bob Charlie";
size_t position;
position = s.find("none");
if (position !=string::npos)
cout << "Found! position is : " << position << endl;
else
cout << "Not found!" << endl;
}
04-15 18:26