Не удается создать объект при создании пользовательского конструктора копирования - PullRequest
Новая
       40

Не удается создать объект при создании пользовательского конструктора копирования

1 голос
/ 02 ноября 2019

Этот код работал перед попыткой добавить конструктор копирования. 

#include <iostream>
#include <string.h>

using namespace std;

class Laptop
{
    public:
        string brand;
        string model;
        int ram;
        int storage;

        Laptop(string brand, string model, int ram, int storage)
        {
            this->brand = brand;
            this->model = model;
            this->ram = ram;
            this->storage = storage;
        }
        Laptop(Laptop &x)
        {
            this->brand = x.brand;
            this->model = x.model;
            this->ram = x.ram;
            this->storage = x.storage;
        }
        void displayData()
        {
            cout << brand << " " << model << ": RAM = " << ram << "GB, Storage = " << storage << "GB" << endl;        
        }
};

int main()
{
    Laptop myLaptop = Laptop("Asus", "Zenbook", 16, 2048);
    Laptop copyOriginal = Laptop(myLaptop);

    copyOriginal.displayData();

    return 0;
}

Приведенный выше код не работает, он работает только при создании myLaptop и oldLaptop с использованием этого синтаксиса: 

Laptop myLaptop("Asus", "Zenbook", 16, 2048);
Laptop copyOriginal(myLaptop);

1 Ответ

0 голосов
/ 02 ноября 2019

Вы не можете привязывать непостоянную ссылку к временному объекту, и, например, в этом объявлении правая сторона является временным объектом 

Laptop myLaptop = Laptop("Asus", "Zenbook", 16, 2048);

Объявите копию как 

    Laptop( const Laptop &x)
    {
        this->brand = x.brand;
        this->model = x.model;
        this->ram = x.ram;
        this->storage = x.storage;
    }

Или добавьте конструктор перемещения.

Учтите, что вам нужно включить заголовок <string>, а не заголовок <string.h>. 

#include <string>

Ваш класс может выглядеть следующим образом 

#include <iostream>
#include <string>
#include <utility>

using namespace std;

class Laptop
{
    public:
        string brand;
        string model;
        int ram;
        int storage;

        Laptop( const string &brand, const string &model, int ram, int storage)
            : brand( brand ), model( model ), ram( ram ), storage( storage )
        {
        }


        Laptop( const Laptop &x)
            : brand( x.brand ), model( x.model ), ram( x.ram ), storage( x.storage )
        {
        }

        Laptop( Laptop &&x)
            : brand( std::move( x.brand ) ), model( std::move( x.model ) ), ram( x.ram ), storage( x.storage )
        {
        }

        void displayData() const
        {
            cout << brand << " " << model << ": RAM = " << ram << "GB, Storage = " << storage << "GB" << endl;        
        }
};
//...
...