ムーブコンストラクタの定義にムーブ代入演算子を使用する際の書き方

Question

ムーブコンストラクタを定義する以下のようなコードを見かけたのですが、この書き方は正しいのでしょうか。

	MyClass(MyClass&& other) noexcept {
		*this = std::move(other);
	}

以下のようなケースで、不定値のptrをdeleteしてしまうことはないのでしょぅか。

class MyClass {
	int* ptr;
public:
	MyClass() : ptr(new int) {}
	~MyClass() { delete ptr; }
	MyClass(MyClass&& other) noexcept {
		*this = std::move(other);
	}
	MyClass& operator=(MyClass&& other) noexcept {
		if (this != &other) {
			delete ptr;
			ptr = other.ptr;
			other.ptr = nullptr;
		}
		return *this;
	}
};

Answer 1

不定値のptrをdeleteしてしまうことはないのでしょぅか。

んなこと気にするくらいならハナっから

std::unique_ptr<int> ptr;

ってやっとけばいいんじゃなくて?

Comments

[actorbug]

私自身がそう書くというわけではなくて、ネット上でたまたま見かけたコードがそうなっていたというだけです。
自分で書くなら、おっしゃる通りunique_ptrに頼ると思います。

Answer 2

質問者が懸念する通りです。

ムーブコンストラクタが起動した時点で ptr は初期化されていません。 delete に渡すポインタがデタラメなものになる可能性があります。

ptr を nullptr で明示的に初期化する必要があります。

Comments

[actorbug]

書き忘れていましたが、VisualStudio2019のデバッガで追うと、
コンストラクタに入った時点でnullptrに書き換わってしまい、
間違っているとの確証が得られなかったため質問しました。
ご回答、ありがとうございました。