在记录错误时取消引用ptr是错误的设计吗？

Question

在浏览一些我没有写的旧代码时，我注意到这样的事情（简体）

//switch case on a msg received from a queue,...
//... get_function returns msgtype and ptr
switch(msgtype)
//...
default:
{
    MYLOGGER<< "Unknown message" << (*ptr)->some_member_var <<"\r\n";
}

所以我想尝试将其更改为

        default:
{
    MYLOGGER<< "Unknown message...\r\n";
    MYLOGGER<< "..." << (*ptr)->some_member_var << "\r\n";
}

因为如果ptr是垃圾，我担心在登录之前就崩溃了？ 我在这个例子中是严厉的吗？ 通常，在记录错误时也应避免取消引用。

编辑：关于MYLOGGER，它是一个以此结尾的宏：具有重载运算符<<的logger类，它在dtror中执行ostream flush（），因此不需要endl。

Answer 1

如果当前代码崩溃，那么您的新代码也会崩溃。 更好的解决方案是在取消引用之前简单地检查nullptr，并在这种情况下打印一条特殊消息。

if(ptr == nullptr)
    MYLOGGER<<"invalid null message\r\n";;
else
    MYLOGGER<< "Unknown message " << (*ptr)->some_member_var <<"\r\n";

鉴于（根据您的评论）操作符<<已经执行了刷新，这是我们所能获得的安全，即使发生崩溃（也许ptr无效，但不是nullptr），我们很可能请参阅日志中的“未知消息”。

Answer 2

我将参考克里斯·拉特纳（Chris Lattner）的话， 每个C程序员应该了解未定义的行为 （谁比编译器作家更能谈论编译器的意外？）。

TL; DR版本是：未定义时，由于编译器的优化，因此没有时间安全性。 因此，您介绍的两个版本严格等效。

Answer 3

如果您不知道消息是什么，则不应尝试以任何方式解释其内容。 您可以记录ptr本身的值，但您实际上并不知道它指向的位置，因此请不要遵从它的意见。

Answer 4

您的记录器可能异步工作，或者使日志异步地持久化，在这种情况下，第二个代码不会比第一个有用。
无论如何，访问未检查的指针是未定义的行为，在这种情况下，可能会因为接收到未知消息而崩溃，这似乎有些苛刻。