Question

在指南支持库中有一个名为final_action的类（基本上是众所周知的ScopeGuard）。有两个独立的便利函数来生成这个模板类：

// finally() - convenience function to generate a final_action
template <class F>
inline final_action<F> finally(const F& f) noexcept
{
    return final_action<F>(f);
}

template <class F>
inline final_action<F> finally(F&& f) noexcept
{
    return final_action<F>(std::forward<F>(f));
}

（来源：https://github.com/Microsoft/GSL/blob/64a7dae4c6fb218a23b3d48db0eec56a3c4d5234/include/gsl/gsl_util#L71-L82）

第一个需要什么？如果我们只有第二个（使用转发，a.k.a。通用，引用）它不会做同样的事情吗？

Answer 1

让我们考虑完美转发版本：

使用右值调用时，它将返回final_action<F>(static_cast<F&&>(f))。
使用左值调用时，它将返回final_action<F&>(f)。

现在让我们考虑const F&重载：

当同时调用左值或右值时，它将返回final_action<F>(f)。

如您所见，存在重要差异：

将非const左值引用传递给finally将生成一个存储F&
将const左值引用传递给finally会产生一个存储F

live example on wandbox

我不确定为什么认为必须让const F&超载。

这是final_action：

的实施

template <class F>
class final_action
{
public:
    explicit final_action(F f) noexcept : f_(std::move(f)), invoke_(true) {}

    final_action(final_action&& other) noexcept 
        : f_(std::move(other.f_)), invoke_(other.invoke_)
    {
        other.invoke_ = false;
    }

    final_action(const final_action&) = delete;
    final_action& operator=(const final_action&) = delete;

    ~final_action() noexcept
    {
        if (invoke_) f_();
    }

private:
    F f_;
    bool invoke_;
};

除非我遗漏了某些内容，否则即时final_action<F&>并不合理，因为f_(std::move(f))无法编译。

live example on wandbox

所以我认为这应该是：

template <class F>
inline final_action<F> finally(F&& f) noexcept
{
    return final_action<std::decay_t<F>>(std::forward<F>(f));
}

最终，我认为GSL中finally的实现不正确/不理想（即冗余，代码重复）。

理解这种`const＆amp;'专业化的必要性

1 个答案: