如何使用Move-emantics从类构造函数中重新分配资源？

Question

为了使我能够阅读一个二进制文件，我已经完成了此类。我想不复制我试图使用移动语义的临时性。但是此代码产生“访问违规错误”。

#include <fstream>
#include <iostream>

class myClass
{
public:
    std::istream& myStream;
    static myClass create(const char* path);
    myClass(myClass&&) noexcept = default;
    myClass(std::istream& input);
    myClass() = default;
};

myClass myClass::create(const char* path)
{
    std::ifstream input{ path, std::ios::binary | std::ios::in };
    if (!input.is_open()) throw std::runtime_error("Error - couldn't open file");
    return std::move(myClass(input));
}

myClass::myClass(std::istream& input) :
    myStream(input)
{

}
int main()
{
    const char* path = R"(file.bin)";
    myClass cr{myClass::create(path)};

    for (int i = 0; i < 10; ++i)
        cr.myStream.seekg(i);

    return 0;
}

Answer 1

临时销毁后，您将进入流。您需要将流存储在myClass对象中：

#include <fstream>
#include <iostream>

class myClass
{
public:
    std::istream& myStream;
    static myClass create(const char* path);
    myClass(myClass&& other) noexcept;
    myClass(std::istream& input);
    myClass() = default;
};

myClass myClass::create(const char* path)
{
    std::ifstream input{ path, std::ios::binary | std::ios::in };
    if (!input.is_open()) throw std::runtime_error("Error - couldn't open file");
    return std::move(myClass(input));
}

myClass::myClass(myClass&& other) noexcept :
    myStream(other.myStream)
    {
    other.myStream = std::cin;
}

myClass::myClass(std::istream& input) :
    myStream(input)
{

}
int main()
{
    const char* path = R"(file.bin)";
    myClass cr{myClass::create(path)};

    for (int i = 0; i < 10; ++i)
        cr.myStream.seekg(i);

    return 0;
}

Answer 2

问题是input in myClass :: create（）的寿命。正如Jensenmckenzie所提到的那样，它将在该功能的末尾被破坏，因此将传递给返回的myClass对象的 Reference 将是一个悬空的参考。 函数的唯一方法是存储一个具体

class myClass
{
    std::ifstream myStream;
    ...
};

保持myClass :: create（） 另一个std :: ifstream要传递给构造函数，它必须作为R值参考：

myClass::myClass(std::ifstream&& input): myStream(std::move(input)) {}

然后您可以编写create> create（）函数这样的功能：

myClass myClass::create(const char* path)
{
    std::ifstream input{ path, std::ios::binary | std::ios::in };
    if (!input.is_open()) throw std::runtime_error("Error - couldn't open file");
    return std::move(input);
}

请注意，您还可以写一个将路径名作为参数的构造函数：

myClass::myClass(const char *path):
        myStream(path, std::ios::binary | std::ios::in)
{
    if (!input.is_open())
        throw std::runtime_error("Error - couldn't open file");
}

然后您可以用main（）而写下此构造函数：

const char *path = ...;
myClass cr(path);

这根本不需要移动。