创建对象的副本以进行多重比较是一种很好的做法

Creating a copy of the object for multiple comparison is a good practice?

本文关键字：一种 很好 比较 副本 创建对象      更新时间：2023-10-16

>假设一次需要多个比较：

enum usermode
{
    active,
    standingby,
    inactive,
    dead,
    // many other modes....
};
class A
{
public:
    usermode mode;
};

函数继承指向类 A 的指针（ptr 指向 A）

方法一：

if( ptr->mode == active || ptr->mode == standingby || ptr->mode == inactive || ptr->mode == dead ...//etc )
{
    //do something
}

方法B： 

usermode cmpmode = ptr->mode;
if( cmpmode == active || cmpmode == standingby || cmpmode == inactive || cmpmode == dead ...//etc )
{
    //do something
}

这样做是一种好的做法吗？

在这种情况下，它只是减少了if语句表达式中的字符数量，因此无论哪种都有效且最具可读性。

但是，在检查函数的返回值的情况下，我会将其填充到变量中并检查，即

auto ret = SomePossiblyExpensiveOperation();
if(ret != whatever && ret < something) {
    //...
}

从表面上看，你的if陈述，我会这样做：

switch(ptr->mode) {
    case active:
    case standingby:
    case inactive:
    case dead:
        ptr->whatever();
        break;
    // other cases
    default:
        // something else
        break:
}

如果这是一个常见的检查，你可能想为它做一个特例，即

usermode any_mode = active | standingby | inactive | dead;
//...
if(ptr->mode & any_mode) {
    // whatever...
}

由于 mode 只是类的成员变量，任何像样的编译器都会为这两种实现生成相同的代码。但是，通常，如果使用方法而不是字段（尤其是具有副作用的方法），则这两种方法并不等效。

函数调用与局部变量中有更多详细信息

如果您的类有一个直接处理这个问题的方法，可能会更好

bool A::IsSomeState() const
{
  return mode == active || mode == standingby; // Etc
}
相关文章：