在这种情况下使用多个条件运算符是个好主意吗？

Question

我刚刚在关于条件运算符的 Wikipedia 文章中看到了这段代码：

Vehicle new_vehicle = arg == 'B' ? bus      :
                      arg == 'A' ? airplane :
                      arg == 'T' ? train    :
                      arg == 'C' ? car      :
                      arg == 'H' ? horse    :
                      feet;

我稍微改变了代码，但想法是一样的。您认为条件运算符的这种使用可以接受吗？它比 if-else 结构简洁得多，并且使用 switch 肯定会带来一系列全新的错误机会（有人会失败吗？）。另外，if-elses 和 switch 不能用作 R 值，因此您必须先创建变量，对其进行初始化，然后根据需要进行分配。

我真的很喜欢这个，但我想知道其他人的想法。

但格式化是必不可少的。

编辑：我还是喜欢这个。但我理解那些说“switch 语句就是为此而创建的”的人。好吧，也许是这样。但是，如果条件是返回 bool 的函数调用怎么办？或者还有一百万个你无法打开的其他东西。

你们换恋人真的想让我相信一个巨大的 if-else 链更好吗？是的，不知道如何使用条件运算符的程序员不会理解这一点。他们应该学习如何使用它。这并不神秘。

Answer 1

我已经多次使用过这种类型的结构。只要它的格式很好（即不是全部在一行上，使其不可读），我不认为它有问题。

Answer 2

我会使用开关，因为这就是它的设计目的。是的，存在出现错误的风险，但是这种嵌套条件块被其他程序员误解的风险要高得多。

Answer 3

这是条件运算符使用的一个很好的例子。我在 C++、Java 和 Perl 中一直以这种方式使用它。

Answer 4

它不仅没有任何问题，而且以最简洁、最清晰的方式传达了操作的意图。

用 if else 或 switch 结构替换需要

"new_vehicle =  "

在每个实例中重复该片段，这要求读者阅读它的每个重复实例以确保它在每个实例中实际上都是相同的。

Answer 5

我喜欢它。它类似于 if-else-if 梯子，只是更简洁。

Answer 6

字符常量周围有很多空格，这使得阅读起来有点困难。我会将比较括起来：（并且可能会移动行中的最后一个值。）

Vehicle new_vehicle = (arg == 'B') ? bus      :
                      (arg == 'A') ? airplane :
                      (arg == 'T') ? train    :
                      (arg == 'C') ? car      :
                      (arg == 'H') ? horse    :
                                     feet;

现在看起来很棒。

Answer 7

条件运算符版本干净、简单，任何了解 C 或 C++ 的人都会立即明白发生了什么。它的另一个优点是它立即返回一个值，这意味着它可以放入初始化中（就像这个例子一样）。

switch 语句会更加笨拙。它需要声明变量然后初始化，如果可以避免的话，这通常是一个坏主意。这将需要更多的打字，并且会有更多的地方让错误潜入。它不会那么清楚，因为有必要查看每个案例以查看它是否说了类似 new_vehicle = foo; 的内容。中断；。

如果您只想在这里进行查找，那么有条件版本就很好，因为它可以立即显示正在发生的情况。如果它会发生多次，请考虑将其放入一个函数中，以便在发生任何变化时只有一个位置可以更新（例如，“R”代表马车或“L”代表直升机）。

Answer 8

纯粹是一种风格选择。对于像您在这里展示的小数据集，只要您的编程团队没有被这样的事情困扰，那么在我的书中就可以了。

Answer 9

切换更清晰，而且可能更有效。如果我在代码审查中看到这样的代码，我会担心。另外，这是“条件运算符”——它是三元运算符的一个实例（尽管目前是 C 和 C++ 中唯一的一个）。

Answer 10

Vehicle new_vehicle = getVehicleByType(arg);

Vehicle getVehicleByType(char arg){
  if (arg == 'B') return bus;
  if (arg == 'A') return airplane;
  if (arg == 'C') return car;
  if (arg == 'T') return train;
  if (arg == 'H') return horse;
  return feet;
}

我更喜欢这个。嵌套条件很聪明，但我认为这几乎同样简洁，而且不太可能让未来的读者感到困惑。抱歉，如果语法不正确，我现在没有做太多 C 工作。

编辑：修复了注释中指出的返回类型遗漏。谢谢！

编辑：顺便说一句，我对你的版本并不感到害怕。当我看到它时，我并没有惊呼“WTF”或“OMG”。我只是更喜欢我的:)

Answer 11

我不是特别关心它。

它并没有真正购买任何东西，或者使任何东西变得更清晰，而且这是运算符的非常非标准的用法。

看起来主要的优点是它有点聪明。我会避免聪明，除非有一个很好的（外部）理由要聪明。

Answer 12

我倾向于使用 switch 语句，因为编译器会捕获重复的情况。可以说，在这个例子中这不是一个问题，但如果列表变得非常长并且由几个不同的人处理，那么很容易添加重复项并且没有意识到这一点。

Answer 13

仔细阅读 维基百科文章的 C++ 部分 。它明确列出了一些使用 ?: 运算符是唯一选项的情况，并且不能用 if/else 或 switch 替代。

另一方面，我不会仅仅因为它看起来更漂亮而使用它。

Answer 14

我以前从未见过这样写的东西。虽然它很聪明且格式良好，但这似乎是使用字典/哈希表的绝佳机会（假设 Vehicle 是一个枚举，这一点尚不清楚）。

Answer 15

有些人已经提到了使用 std::map 或其他关联数组类型来完成这项工作的可能性。只要您只在一个地方（或几个地方）执行此操作，您可能会考虑仅使用普通数组或向量：取决于

Vehicle vehicles[CHAR_MAX];

// Initialization    
std::fill_n(vehicles, CHAR_MAX, feet);
vehicles['A'] = airplane;
vehicles['B'] = bus;
vehicles['C'] = car;
vehicles['H'] = horse;
vehicles['T'] = train;

// Use
Vehicle new_vehicle = vehicles[arg];

您需要/使用的表（存储相同类型的对象）的数量，以及所包含对象的大小（在本例中为车辆），这可以是 std::map 的完美合理替代方案。如果您要创建很多表，或者每个对象都非常大，那么 std::map 会成为更合理的替代方案。

当您使用 std::map （或 unordered_map 等）时，您将使用更多代码来节省数据存储。这会起到相反的作用——但只要 Vehicle 很小（比如 4 字节），像上面这样的一个表通常会占用大约半千字节。很难准确猜测特定编译器的 std::map 代码有多大，但看起来通常会大于一半一千字节，因此如果您只创建一个这样的表，std::map 可能会造成净损失。

当然，如果你知道你只处理字母作为输入，你可以大大减少表的大小：

template <class T>
class letter_table { 
    static const int range = 'Z' - 'A';

    T table[range];
public:
    // ...
    T operator[](int index) {
        index -= 'A';
        assert(index<range);
        return table[index];
    }
};

在示例中，这将给出一个大约 100 字节的表——你可以创建相当数量的表。 std::map 空间中的 100 字节表通常会占用。

Answer 16

纯粹实用：

加号： 三元序列更加灵活，可以用来避免switch的限制，可以使用其他运算符（例如<=、>） =) 或任何其他测试，包括字符串比较。

x = IsEven(arg) ?  0 : 
    (arg < 0)   ? -1 : 1; // or whatever

此外，如果交换机是性能瓶颈并且概率不均匀，则可以强制首先执行最可能的测试（由于未评估未选择路径的保证）。

马马虎虎
与 switch 语句不同，顺序很重要（除非您坚持 ==）。这可能是一个优势，但在其他方面与 switch 类似，当维护者不熟悉这个概念或匆忙时可能会产生误导。

许多开发人员可能会回避，因为他们不确定细节（将评估哪些术语，运算符的正确性好吗？） - 但是，如果您的开发人员池无法掌握一个很好的示例，您可能会禁止三元运算符无法解决的问题。

减号
它不像 switch 那样常见，因此优化器可能不会以相同的方式对待它。我们知道优化器会为开关选择最合适的实现（表、二分搜索、比较序列或它们的任意组合）。优化器无法重新排列我们的评估顺序，并且不太可能支持此处的表查找。

需要良好的格式才能轻松识别（排列“？”和“：”） - 当您使用制表符时很糟糕。

美学

我喜欢它的精确性和简洁性，接近数学符号。然而，这也可能被用来对付它。这在代码审查中可能会让人大吃一惊，因为它不太常见而且更脆弱。

Answer 17

只是为了进行比较，在 C++0x 中，您可以在不使用条件运算符或外联函数的情况下获得表达式：

Vehicle new_vehicle = [&]() -> Vehicle {
    if (arg == 'B') return bus;
    if (arg == 'A') return airplane;
    if (arg == 'T') return train;
    if (arg == 'C') return car;
    if (arg == 'H') return horse;
    return feet;
}();

但实际上并没有更好。

Answer 18

怎么样：

enum Vehicle { bus = 'B', airplane = 'A', train, car = 'C', horse = 'H', feet = 'F' };
...
new_vehicle = arg;

:-)，顺便说一句。

Answer 19

我认为它对于编写代码的人很有用，但对于审阅者来说很难理解，

“保持简单，伙计”

Answer 20

在我看来，由于示例的简单性，您所做的事情是可以接受的。如果您对每种情况都做更多的事情，这种类型的构造可能会很快变得混乱。出于这个原因，我更喜欢使用 switch 甚至嵌套 if then elses （如果没有太多情况），格式如下：

  if (A) {
      //Do A stuff
  }
  else if (B) {
      //Do B stuff
  }
  else if (C) {
      //Do C stuff
  }
  else {
      //Do default stuff
  }

这是关于代码的可读性，这有助于代码的可维护性。我从来都不是条件运算符的超级粉丝，因为我不喜欢在一行上看到多个表达式。在调试器中单步执行代码时，条件运算符可能很难遵循。代码越简单，就越容易专注于代码的作用。

Answer 21

我们中的许多人都习惯了各种报告工具中的 Iif 函数或 Excel 中的 If 函数，我们基本上需要使用不太明确的 Iif(arg="B";"Bus";Iif(arg="A";飞机；“脚”））。相比之下，我更喜欢你的样品:)
我个人会使用 if-elses，但我对你的示例不会有任何问题。