既要判断指针非空，又要判断指针所指成员的值，这种怎么写方便些

[sk609_king]

经常遇到这种情况：

class A
{
public:
	bool m_bVar;
};

void Function()
{
	A *p;
	// ...
	if(p == NULL)
	{
		// 失败处理
	}
	else
	{
		if(!p->m_bVar)
		{
			// 失败处理
		}
		else
		{
			// 成功处理
		}
	}
}

如果直接判断p->m_bVar很可能会出现非法访问因为p有可能为NULL
只能像上面这样写
但是

// 失败处理

这个经常又有很多操作，把代码复制一份感觉很笨，有么有什么好的格式可以解决这个问题？

[zilaishuichina]

 if(p != NULL && p->m_bVar)
{
    // 成功处理
}
else
{
    // 失败处理
}

[sk609_king]

引用 1 楼 zilaishuichina 的回复:

 if(p != NULL && p->m_bVar)
{
    // 成功处理
}
else
{
    // 失败处理
}

为什么判断 p != NULL 为true后 p->m_bVar就不再判断了？ &&这个运算符难道不是要把前后两个值都得出来才行吗？ 如果写成

if(p->m_bVar && p != NULL)

就挂了

[zilaishuichina]

引用 2 楼 sk609_king 的回复:

[quote=引用 1 楼 zilaishuichina 的回复:]

 if(p != NULL && p->m_bVar)
{
    // 成功处理
}
else
{
    // 失败处理
}

为什么判断 p != NULL 为true后 p->m_bVar就不再判断了？ &&这个运算符难道不是要把前后两个值都得出来才行吗？ 如果写成

if(p->m_bVar && p != NULL)

就挂了[/quote] if(p != NULL && p->m_bVar) 先判断p != NULL，如果为真，再判断下一个条件， 如果不为真，则整个if必定为假，不需要再判断下一个条件

[paschen]

if(p && p->m_bVar)
{
     // 成功
}
else
{
     // 失败
}

[幻夢之葉]

|| &&符合短路求值规则 || 如果第一个操作数为true，那么整个表达式必定为true，则后续不再求解 && 如果第一个操作数为false，那么整个表达式必定为false，则后续不再求解

[ooolinux]

逻辑运算符&&和||具有短路效应，比如 a && b 当表达式a为0（false），不再计算表达式b的值 a || b 当表达式a为非0（true），不再计算表达式b的值

[ooolinux]

引用 6 楼 u010165006 的回复:

逻辑运算符&&和||具有短路效应，比如 a && b 当表达式a为0（false），不再计算表达式b的值 a || b 当表达式a为非0（true），不再计算表达式b的值

如果表达式b是函数，短路的时候也是不调用的，比如： 0&&printf("hello");

[ooolinux]

参考： 《C语言点滴》 《C陷阱与缺陷》

[自信男孩]

先判断p是否为NULL，然后在p!= NULL后再判断p->m_bVar

[啊哈moment]

[赵4老师]

搜“逻辑短路”