Когда использовать Move Constructors / Assignments
Я искал, но не могу найти ответ на "Когда" использовать их. Я просто продолжаю слышать, что это хорошо, потому что это экономит мне эту дополнительную копию. Я использовал это в каждом классе, который у меня был, но кое-что из этого не имело смысла для некоторых классов: S Я прочитал бесчисленные учебники по LValues и RValues и std :: move против std :: copy vs. memcpy против memmove и т. д. И даже читайте о throw (), но я также не уверен, когда это использовать.
Мой код выглядит так:
struct Point
{
int X, Y;
Point();
Point(int x, int y);
~Point();
//All my other operators here..
};
Затем у меня есть массив классов такого типа (RAII sorta вещь):
class PA
{
private:
std::vector<Point> PointsList;
public:
PA();
//Variadic Template constructor here..
~PA();
//Operators here..
};
Должен ли я использовать конструктор перемещения и копирования конструктор? У меня было это в Классе Пункта, но это было странно, таким образом я удалил это. Потом у меня было это в классе PA, но я думал, что это ничего не сделает, поэтому я тоже удалил это. Затем в моем классе растровых изображений мой компилятор жаловался на наличие членов-указателей, но без перегрузки, поэтому я сделал:
//Copy Con:
BMPS::BMPS(const BMPS& Bmp) : Bytes(((Bmp.width * Bmp.height) != 0) ? new RGB[Bmp.width * Bmp.height] : nullptr), width(Bmp.width), height(Bmp.height), size(Bmp.size), DC(0), Image(0)
{
std::copy(Bmp.Bytes, Bmp.Bytes + (width * height), Bytes);
BMInfo = Bmp.BMInfo;
bFHeader = Bmp.bFHeader;
}
//Move Con:
BMPS::BMPS(BMPS&& Bmp) : Bytes(nullptr), width(Bmp.width), height(Bmp.height), size(Bmp.size), DC(0), Image(0)
{
Bmp.Swap(*this);
Bmp.Bytes = nullptr;
}
//Assignment:
BMPS& BMPS::operator = (BMPS Bmp)
{
Bmp.Swap(*this);
return *this;
}
//Not sure if I need Copy Assignment?
//Move Assignment:
BMPS& BMPS::operator = (BMPS&& Bmp)
{
this->Swap(Bmp);
return *this;
}
//Swap function (Member vs. Non-member?)
void BMPS::Swap(BMPS& Bmp) //throw()
{
//I was told I should put using std::swap instead here.. for some ADL thing.
//But I always learned that using is bad in headers.
std::swap(Bytes, Bmp.Bytes);
std::swap(BMInfo, Bmp.BMInfo);
std::swap(width, Bmp.width);
std::swap(height, Bmp.height);
std::swap(size, Bmp.size);
std::swap(bFHeader, Bmp.bFHeader);
}
Это правильно? Я сделал что-то плохое или неправильное? Мне нужно бросить ()? Должны ли мои операторы присваивания и перемещения быть такими же? Нужно ли мне задание на копирование? Ах, так много вопросов: c Последний форум, на котором я спрашивал, не мог ответить на все вопросы, поэтому я остался в замешательстве. Наконец, я должен использовать unique_ptr для байтов? (Который является массивом байтов / пикселей.)