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.)

questionAnswers(3)

yourAnswerToTheQuestion