考虑以下代码有何问题:
class MyClass {
public:
string& getName() { return s; }
private:
string s;
};
...
MyClass* p = nullptr;
std::cout << p ? p->getName() : 0;
上述代码的最后一行会导致crash,原因在于,三目运算符返回的是一个左值,可以看作是一个临时变量,我们知道变量都有且只有一个类型,由于我们getName()的返回值是一个string,所以临时变量就是string类型,此时如果三目运算符条件为假,那么就将一个0值(nullptr)赋给了一个string,而这会导致程序崩溃。
这里我们的问题在于忽略了三目运算符返回的是一个左值(变量),并且让表达式在条件为真和假的时候返回的类型不兼容导致的。
此外我们要注意,三目运算符的优先级低于<<,因此最后一句会先执行std::cout << p
,然后返回的值作为三目运算符的条件(std::cout << p) ? p->getName() : 0
,这并非我们本意。正确的写法是将三目运算符括号括起来。
修正:
std::cout << (p ? p->getName() : "0");
输出0 1
,说明返回的是左值