有没有更短的写法 `StringPtr ？ StringPtr：“空”`？

Question

我有这样的代码：

 std::wstringstream outstream;
 outstream << (prop.m_pwszOriginalVolumeName 
             ? prop.m_pwszOriginalVolumeName 
             : L"null") << L";"
           << (prop.m_pwszSnapshotDeviceObject 
             ? prop.m_pwszSnapshotDeviceObject 
             : L"null") << L";"
           << (prop.m_pwszOriginatingMachine 
             ? prop.m_pwszOriginatingMachine
             : L"null") << L";"
           << ... // some more strings here

有没有一种方法可以避免代码重复并且仍然拥有简洁的代码？

Answer 1

你可以定义一个小函数：

whatever_t strOrNull(whatever_t str) {
    return str ? str : L"null";
}

然后你的代码变成

std::wstringstream outstream;
outstream << strOrNull(prop.m_pwszOriginalVolumeName)   << L";"
          << strOrNull(prop.m_pwszSnapshotDeviceObject) << L";"
          << strOrNull(prop.m_pwszOriginatingMachine)   << L";"
          << ... // some more strings here

或者如果你想更简洁，你可以这样做（取决于 whatever_t 是什么；如果 wstringstream 已经有该类型的 operator<< 重载，这不起作用）：

wstringstream& operator<<(wstringstream& out, whatever_t str) {
    if (str)
        out << str;
    else
        out << L"null";

    return out;
}

然后你的代码变成

std::wstringstream outstream;
outstream << prop.m_pwszOriginalVolumeName   << L";"
          << prop.m_pwszSnapshotDeviceObject << L";"
          << prop.m_pwszOriginatingMachine   << L";"
          << ... // some more strings here

Answer 2

函数或 lambda：

auto foo = [](const wchar * p) { return p ? p : L"null;" };

outstream << foo(prop.m_pwszOriginalVolumeName) << L";"
          << foo(prop.m_pwszSnapshotDeviceObject) << L";"
          << ...etc...

Answer 3

其他的例子确实很好。还有另一种选择，但我不会推荐它（只是为了完整性而提及）。

GCC 有一个名为“带有省略操作数的条件”的扩展，它基本上看起来像这样：

x = a ?: b;

与（在像您这样的简单情况下，请参阅下面的详细信息）相同：

x = a ? a : b;

只是不太可移植。所以你可以写：

std::wstringstream outstream;
outstream << (prop.m_pwszOriginalVolumeName   ?: L"null") << L";"
          << (prop.m_pwszSnapshotDeviceObject ?: L"null") << L";"
          << (prop.m_pwszOriginatingMachine   ?: L"null") << L";"

但就像我说的，我不会推荐这个，我会使用像其他答案提到的辅助函数。

实际上，在某些情况下，它的执行方式与常规三元 if 不同，即 if 计算 a 具有副作用。从页面：

在这个简单的情况下，省略中间操作数的能力并不存在
特别有用。当第一个操作数变得有用时
确实或可能（如果它是宏参数）包含副作用。然后
在中间重复操作数会产生副作用
两次。省略中间操作数使用已经计算的值
不会因重新计算而产生不良影响。

请参阅http://gcc.gnu.org/onlinedocs/gcc/Conditionals.html

Answer 4

一个简单的函数就可以解决问题。

wchar_t* filterNullString(wchar_t* str)
{
  static wchar const* nullStr = L"null";
  return str ? str : nullStr;
}

std::wstringstream outstream;
outstream << filterNullString(prop.m_pwszOriginalVolumeName) << L";"
           << filterNullString(prop.m_pwszSnapshotDeviceObject)<< L";"
           << filterNullString(prop.m_pwszOriginatingMachine)<< L";" ;

Answer 5

您可以使用辅助函数：

const wchar_t *SafeOutput(const wchar_t *str)
{
    return str ? str : L"null";
}
// Analogous function for ANSI strings
...
outstream << SafeOutput(prop.m_pwszOriginalVolumeName)   << L";"
          << SafeOutput(prop.m_pwszSnapshotDeviceObject) << L";"
          << SafeOutput(prop.m_pwszOriginatingMachine)   << L";"
          << ... // more strings here