Kiedy używać Move Constructors / Assignments
Szukałem, ale nie mogę znaleźć odpowiedzi na pytanie „Kiedy” z nich korzystać. Po prostu słyszę, że to dobrze, ponieważ oszczędza mi dodatkową kopię. Poszukałem go w każdej klasie, jaką miałem, ale trochę nie miało to sensu dla niektórych klas: S Czytałem niezliczone samouczki na temat LValues i RValues i std :: move vs. std :: copy vs. memcpy vs. memmove, itp. I nawet czytać na throw (), ale nie jestem pewien, kiedy z niego korzystać.
Mój kod wygląda tak:
struct Point
{
int X, Y;
Point();
Point(int x, int y);
~Point();
//All my other operators here..
};
Następnie mam tablicę klas podobną do (RAII sorta thing):
class PA
{
private:
std::vector<Point> PointsList;
public:
PA();
//Variadic Template constructor here..
~PA();
//Operators here..
};
Czy powinienem używać konstruktora ruchu i konstruktora kopii? Miałem go w klasie Point, ale to było dziwne, więc go usunąłem. Potem miałem go w klasie PA, ale pomyślałem, że nic nie da, więc też go usunąłem. Następnie w mojej klasie bitmap kompilator skarżył się na posiadanie wskaźników, ale bez przeciążenia, więc:
//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);
}
Czy to jest poprawne? Czy zrobiłem coś złego lub złego? Czy potrzebuję throw ()? Czy operatorzy przypisań i przeniesień powinni być tacy sami? Czy muszę przydzielić kopię? Ahh tak wiele pytań: c Ostatnie forum, o które prosiłem, nie mogło odpowiedzieć na wszystkie pytania, więc byłem zdezorientowany. Na koniec powinienem użyć unique_ptr dla Bajtów? (Który jest tablicą bajtów / pikseli.)