Where is my move assignment?!

[etrusks]

Hi, I was having this exercise where I had to implement simple

default constructor
constructor
copy constructor
copy assignment
move constructor
move assignment
destructor

for a simple class and in those function bodies just cout << "Where I'm at\n";

#include <iostream>

using namespace std;

class X {
	int val{0};
public:
	X(int x) : val{ x } { cout << "X constructor\n"; }
	X() : val{ 0 } { cout << "X default constructor\n"; };

	X(const X& c) : val{ c.val } { cout << "copy constructor\n"; }
	X& operator=(const X& c) { val = c.val; return *this; cout << "copy assignment\n"; }

	X(X&& a) : val{ a.val } { cout << "\tmove constructor\n"; }
	X& operator=(X&& a) { val = a.val; return *this; cout << "\tmove assignment\n"; }

	~X() { cout << "~Destructor()\n"; }

};

int main() {

	X x1{10};   //constructor
	X x2;       //default constructor

	x2 = x1;    //nothing???

}

Why x2 = x1; line does produce "copy assignment" output?

[TheHardew]

move assignment works for temporaries, i.e. variables which are not bound to names or pointers, like X(). Also you should include <algorithm> and use val{std::move(a.val)} in you move constructor to ensure you actually move it.

[ne555]

	X& operator=(const X& c)
	{
		val = c.val;
		return *this;
		cout << "copy assignment\n"; //¿you wonder why you don't see this?
	}

[etrusks]

:D All right, yea my mistake as always. Thank you guys :)

[Peter87]

TheHardew wrote:

Also you should include <algorithm> and use val{std::move(a.val)} in you move constructor to ensure you actually move it.

The header you should include is <utility>. There is a std::move in <algortihm> too but it's not the one that is used here.

[etrusks]

Tnx for this good update man! Checked out both of them.