前进或移动或不前进；如何在类的使用情况下确定哪个是首选？

Question

class Test {
public:
    typedef std::set<std::pair<double, double>> DataSet;

    explicit Test(const DataSet&& d) {
        for (auto &itr : d) {
            std::cout << "value1 = " << itr.first << '\n';
            std::cout << "value2 = " << itr.second << '\n';
        }
    }

};

int main() {
    //using namespace util;
    try {
        // Forwarding
        Test obj( std::forward<Test::DataSet>(
            { 
              { 10.0, 20.0 },
              { 30.0, 40.0 },
              { 50.0, 60.0 }
            }
        ));

        std::cout << '\n';

        // Move
        Test obj2(std::move(Test::DataSet(
            {
              { 10.0, 20.0 },
              { 30.0, 40.0 },
              { 50.0, 60.0 }
            }
           )
        ));

    } catch (const std::exception& e) {
        std::cerr << e.what() << std::endl;
        return EXIT_FAILURE;
    }

    return EXIT_SUCCESS;
}

Answer 1

都没有。

Test::DataSet(/*...*/) 已经是一个可以绑定到右值引用的右值。 std::move 唯一要做的就是将左值转换为 xvalue（即右值），以便它可以绑定到右值引用，而左值不能。所以std::move 是多余的。

std::forward 仅根据其模板参数执行相同的条件。所以它也是多余的（充其量）。

const DataSet&& 作为参数没有意义。您需要DataSet&&，它只绑定到右值，或者const DataSet&，它绑定到右值和左值。

如果您想利用传递的引用可能是右值这一事实（即通过从其成员中移动），那么您需要对函数进行两次重载。一个采用DataSet&&，它使用右值参数调用，另一种采用const DataSet&，它使用左值参数调用。

在您的问题中 Test 函数的特定情况下，我看不出从成员中移动或对 DataSet 状态进行任何其他修改会有什么好处，所以我会选择只需将const DataSet& 作为参数，并且永远不要专门为该函数使用std::moveing 或std::forwarding。

Answer 2

[针对 cme​​ts 编辑的代码示例]
来自 ISO 核心指南的有用的 advice。这表明您应该更喜欢 (const T&)，但如果您真的需要速度，请特别添加 (T&&)。这样您就可以将小集合作为（L-Value）传递

const auto data = Test::DataSet{ 
          { 10.0, 20.0 },
          { 30.0, 40.0 },
          { 50.0, 60.0 }};
Test obj( data );

或（R值）

Test obj( Test::DataSet{ 
          { 10.0, 20.0 },
          { 30.0, 40.0 },
          { 50.0, 60.0 },
        });

添加额外的移动使第二种调用更有效，但建议仅在需要时使用这种类型的优化。您应该避免一个构造函数的语义与另一个构造函数不同，避免这种情况的最简单方法是只有一个构造函数。

就个人而言，即使经过 20 多年的 C++ 编程，我发现链接中的表格非常有用，我将其打印出来并贴在显示器上。