c ++警告：Clang-Tidy：条件链中的重复分支

Question

你能帮我如何避免这段代码中的警告吗...

Warning: Clang-Tidy: Repeated branch in conditional chain

HubInterface *getFreeHub() {
        HubInterface *freeHub = nullptr;
        while (freeHub == nullptr) {
            for (auto hub : this->getHubList()) {
                if (freeHub == nullptr) {
                    freeHub = hub;
                } else if (hub->getStatus() == STATUS_READY &&
                           hub->getBusyNodes().size() < freeHub->getBusyNodes().size()) {
                    freeHub = hub;
                }
            }
            sleep(5);
        }
        return freeHub;
    }

CLion 提供的警告 IDE

Answer 1

警告是关于您在 if 和 else if 分支中做同样的事情，即 freeHub = hub;

if (freeHub == nullptr) {
    freeHub = hub;
} else if (hub->getStatus() == STATUS_READY &&
           hub->getBusyNodes().size() < freeHub->getBusyNodes().size()) {
    freeHub = hub; // same as above, what is the point?
}

我想你可以将它重写为一个条件，但它有点可怕：

if (freeHub == nullptr ||
    (hub->getStatus() == STATUS_READY &&
     hub->getBusyNodes().size() < freeHub->getBusyNodes().size())) {
    freeHub = hub;
}

c ++警告：Clang-Tidy：条件链中的重复分支

c++ Warning: Clang-Tidy: Repeated branch in conditional chain

c++

c++14

clion