如果不为空，则为指针赋值

Question

我有一个整数指针作为默认函数参数。如果它不为空，我想分配一些值，也许每 200 行中的第 10 行。通过在每次分配之前进行标准检查，我的代码很容易变得庞大且难以阅读。 （我指的是源文件长度和可读性，而不是二进制大小。）

替换这个是好习惯吗：

// Previous statement
// Use to put empty line here
if(ptr)
    *ptr = val;
// Empty line also here
// Next statement

用这个：

// Previous statement
assignIfNotNull(ptr, val);
// Next statement

并把 if 放入函数中？

inline void assignIfNotNull(int *ptr, int val)
{
    if(ptr)
        *ptr = val;
}

现在，我似乎很细致，但这每次使用会节省 3 行。也许不是我编程风格中最糟糕的事情，这是我独自学习时即兴创作的。我想压制它，询问并跟上标准。 （我在任何地方都找不到这个。）

提前致谢。

Answer 1

你可以去：

T dummy;
if ( !ptr ) 
    ptr = &dummy;

一开始;然后你就可以放心地写*ptr = whatever;，而不需要空检查。

如果ptr 的原始值很重要，你可以调用参数orig_ptr 之类的。

Answer 2

C# null coalescing operator equivalent for c++ 几乎准确地提出了您的问题，但不要指望找到一些突破性的解决方案 - 三元运算符、模板和宏只有一些变化：

ptr = (ptr != null ? ptr : MY_DEFAULT)

//------------------- or as function
template<class T>
GetValueOrDefault(T* value, T* defaultValue) // or put your own types
{
    return value != null ? value : defaultValue;
}

ptr = GetValueOrDefault(ptr, MY_DEFAULT);

//------------------- or as macro

#define GET_VALUE_OR_DEFAULT(value, defaultValue) (value != null ? value : defaultValue)

ptr = GET_VALUE_OR_DEFAULT(ptr, MY_DEFAULT);

//------------------- or like this macro

#define VALUE_OR_DEFAULT(value, defaultValue) value = (value != null ? value : defaultValue)

VALUE_OR_DEFAULT(ptr, MY_DEFAULT);

---

但有几点你应该考虑：

1. 当问题不存在时不要想像 - 很多检查和重新初始化对于某些 API 是完全可行的 - 看看 WINAPI
2. 您确定要覆盖该变量吗？可能是你 可以重写您的函数或 API 以不覆盖它。

3. 您也可以考虑使用一些临时变量来代替 ptr - 解决不了问题，把它倒过来，但是 这可能会提高您的整体可理解性 功能和删除检查：

     char* tempPtr = ptr;
     retCode = RandomlyNullifyThePtr(&tempPtr); // RandomlyNullifyThePtr can nullify the ptr it gets ,
     // but it won't nullify the ptr in calling function.
   

4. 如果您在保存整个函数时遇到问题 良心那么直接的迹象表明它应该被重构 - 分成更小的部分，这样的检查可能会变成 不必要的：

     retCode = CompletePart1(ptr); // CompletePart1 can nullify the ptr it gets, 
     // but it won't nullify the ptr in calling function.
     ...
     retCode = CompletePart2(ptr);