c++ - 在使用指针之前检查 null

Question

大多数人都使用这样的指针...

if ( p != NULL ) {
  DoWhateverWithP();
}

但是，如果由于某种原因指针为空，则不会调用该函数。

我的问题是，不检查 NULL 是否更有益？显然，在安全关键系统上，这不是一个选项，但是如果程序在没有它的情况下仍然可以运行，那么您的程序在荣耀中崩溃比没有被调用的函数更明显。

关于第一个问题，你在使用指针之前总是检查 NULL 吗？

其次，假设您有一个将指针作为参数的函数，并且您在整个程序的多个指针上多次使用此函数。您是否发现在函数中测试 NULL 更有利(好处是您不必到处测试 NULL)，或者在调用函数之前在指针上测试(好处是调用函数没有开销)?

Answer 1

您的想法是正确的，NULL 指针通常会导致立即崩溃，但是不要忘记，如果您通过 NULL 指针对大型数组进行索引，您可能确实会获得有效的内存地址如果你的指数足够高。然后，您会遇到内存损坏或不正确的内存读取，这将更难定位。

每当我可以假设使用 NULL 调用函数是一个错误，这在生产代码中绝不应该发生，我更喜欢在函数中使用 ASSERT 保护，它只在调试版本中编译成实际代码，而不是检查否则为 NULL。

在我看来，一般来说，一个函数应该检查它的参数，而不是调用者。您应该始终假设您的调用者可能对检查有点草率，或者他们可能包含错误......

道德:检查被调用函数中的 NULL，或者通过一些抛出的 if() 语句，或者使用一些 ASSERT 构造(可能带有关于为什么会发生这种情况的明确消息)。还要检查调用者中的 NULL，但前提是调用者知道这种情况可能发生在正常的程序执行中，并采取相应的行动。