c++ - 三元条件结果的方法

Question

在三元条件的结果中使用方法是否被认为是不好的风格？类似的东西:

(node->parent()->left() == node ? node->parent()->left()
                                : node->parent()->right()).reset();

我知道代码风格可能是主观的，但我仍然希望我能得到一些建议，就像使用它一样，或者更确切地说，写一些像这样的东西:

if (node–>parent()–>left() == node) {
    node->parent()->left().reset()
} else {
    node–>parent()->right().reset()
}

(来自评论)关于:

auto& childUnq = node->parent()->left() == node ? node->parent()->left()
                                                : node->parent()->right();
childUnq.reset();

Answer 1

我强烈推荐另一种选择:

auto& left = node->parent()->left();
auto& right = node->parent()->right();
auto& pick = (left==node) ? left : right;
pick.reset();

通过引入简单的名称，很清楚发生了什么。三元运算符本身并不坏，但挑战在于仅在简单的上下文中使用它。