Конструктор хода не называется - PullRequest
0 голосов
/ 30 декабря 2018

Я реализую IntArray Class для изучения C ++.Я должен признать, что я еще не полностью понял r и lvalues ​​и переместить конструкторы.Я хотел попробовать, чтобы увидеть, работает ли мой код, но я не знаю, почему {IntArray array = IntArray(5);} не вызывает мой реализованный конструктор перемещения.Я думал, что это будет случай для этого.

#include "IntArray.h"
IntArray::IntArray()
    :data(nullptr), count(0), capacity(0) {std::cout << "Default Constructor called" << std::endl;}

IntArray::IntArray(int size)
    :data(new int[size]), count(size), capacity(size)  {std::cout << "Constructor called with size: " << size << std::endl;}

IntArray::~IntArray() {
    std::cout << "Destructor called" << std::endl;
    delete[] data; //rest is stack allocated and gets freed with end of scope
}

//COPY CONSTRUCT & ASSIGN OP
IntArray::IntArray(const IntArray& rhs) 
    :data(new int[rhs.count]), count(rhs.count), capacity(rhs.count) //warum nicht mit capacity? wir wollen doch eine exakte kopie?
{
    std::cout << "Copy Constructor called" << std::endl;
    std::copy(rhs.data, rhs.data + rhs.count, data); //pointer arithmetik?
}
IntArray& IntArray::operator=(const IntArray& rhs) {
    if (&rhs == this) //check for selfassign
        return *this;

    //if capacity of lhs is NOT big enough, reallocate new
    if (capacity < rhs.capacity) { 
        delete[] data;
        data = new int[rhs.count];
        capacity = rhs.count;
    }   
    count = rhs.count;
    std::copy(rhs.data, rhs.data + rhs.count, data);
    return *this;
}

//MOVE CONSTRUCT & ASSIGN OP
IntArray::IntArray(IntArray&& rhs)
    :data(rhs.data), count(rhs.count), capacity(rhs.capacity)
{
    std::cout << "Move Constructor called" << std::endl;

    rhs.data = nullptr;
    rhs.count = 0;
    rhs.capacity = 0;
}
IntArray& IntArray::operator=(IntArray&& rhs) {
    if (&rhs == this) //self assignment?
        return *this;
    std::cout << "Move assignment operator called" << std::endl;

    //steal
    delete[] data;
    data = rhs.data;
    count = rhs.count;
    capacity = rhs.capacity;

    //Reset old obj to prevent double freeing
    rhs.data = nullptr;
    rhs.count = 0;
    rhs.capacity = 0;

    return *this;
}

1 Ответ

0 голосов
/ 30 декабря 2018

То, что вы не видите конструкции перемещения или назначения перемещения, является просто результатом оптимизации!Пожалуйста, обратите внимание на «copy elision» https://en.cppreference.com/w/cpp/language/copy_elision

Если вы используете gcc, вы можете сказать компилятору не оптимизировать его:

g++ -O0 main.cpp -fno-elide-constructors

Теперь результат таков:

Constructor called with size: 5
Move Constructor called
Destructor called
Destructor called
...