C++ 中delete空指针

今天写了一些代码给同事review的时候，同事指出我有一个地方对空指针指针的判断是不必要的。记录在此作为提醒。

 

局部的代码是这样的

if(fail) continue; delete matches; matches = 0; return context->getItemFactory()->createBoolean(true, context); } if(matches != 0) { delete matches; matches = 0; }

 

这里matches是一个指针，很明显在下面的delete maches代码前面我去判断了matches是否为空，事实上这是不必要的，delete操作符已经对指针是否为空做过判断了（参见：C++ Primer第三版340页）。

 

实际上g++中一个delete的实现是这样的：

_GLIBCXX_WEAK_DEFINITION void operator delete(void* ptr) throw () { if (ptr) std::free(ptr); }

我们可以看到delete已经判断过指针是否为空了。