问个笨笨的问题

ashhyc

简要程序:
IDMusicPerformance  * lpdmperf = NULL;

if(lpdmperf)
{
  if(FAILED(lpdmperf->stop(xxxx))
       return(0);
}

if(lpdmperf)
{
  if(S_OK != (lpdmperf->release())
       return(0);
}

请问if(lpdmperf)这句话还有必要放上吗?看书上这么写,有些表示怀疑,呵呵... [em5]

伊拉虾伊

有必要阿，如果lpdmperf为null 后面的操作……

伊拉虾伊

另外 if(lpdmperf)  最好写成 if ( lpdmperf != NULL )

ashhyc

但是FAILED(lpdmperf->stop(xxxx)，或则S_OK检测不到错误的吗？

伊拉虾伊

说实话，我没有对HRESULT的工作过程研究过

但在程序设计的时候最好保证一个对象存在再去对它进行操作，否则很容易出错，深的我也不懂了，改天研究一下，如果你搞明白了也麻烦po出来给偶看看哈~

上面的FAILED(lpdmperf->stop(xxxx)，括号里面的应该先执行，所以如果不对lpdmperf进行判断，出错就会在此了，都还没执行到FAILED(。。。。。。)，我是这么理解的

logos

你想想拿空指针来调用东西是什么结果？

ashhyc

对呀，我的问题实质就是这个啊！拿空指针来调用东西，不知道是在运行时出错，还是被FAILED拦截而返回失败。。。但我想的和伊啦虾相反了，我认为FAILED会捕获到这个错误哩。。。虽然我试过，但仍然不感肯定，不知道能不能给个确定的答案啊。。。

lights

漏掉什么代码了吧！
上面那些，只有IDMusicPerformance  * lpdmperf = NULL;会执行

lights

还是你只想问空指针执行么？
那是绝对要验证的，不过空指针调用只会产生内存无法访问而已。编译不会报错。
FAILED是个带参的宏，实际可以理解为函数，FAILED会返回逻辑真或假！
该宏的定义非常搞笑，你在编译器里可以查到，我从没用过。
他在FAILED()括号内的整数值，或函数返回值，小于零时返回true,否则false.

它没有任何检验空指针的能力，除非你用的sdk里的该宏的定义与我不同。

  if(FAILED(lpdmperf->stop(xxxx))
       return(0);

写漂亮点应该这样。
HRESULT hr;
hr = lpdmperf->stop(xxxx);
if(hr<0)return 0;

或者if(lpdmperf->stop(xxxx)<0)return 0;

游戏战士

有必要 ,保险嘛