Rozbicie unique_ptr na frazy

Question

Czy da się jakoś podzielić unque_ptr aby od razu nie przypisywać obiektu? Nie chcę dublować kodu w programie więc chciałbym zadeklarować go sobie przed if'ami, potem w zależności od użytkownika odpowiednio go przypisać.

void Team::addToTheTeam()
{
	int choice;
	cout << "Who would you like to add?(1.Player 2.Employee): "; cin >> choice;
	//czy da sie utowrzyć tu unique_ptr a dopiero potem go przypisać żeby wsk było widoczne poza blokiem kodu?
	unique_ptr<Person>wsk;
	if (choice == 1)
	{
		wsk(new Player);	//<-w ten sposób
	}
	else if (choice == 2)
	{
		unique_ptr<Person>wsk(new Employee);
	}
	else
	{
		cout << "Bad choice. You're going to menu..." << endl;
		return;
	}
	wsk->addPerson();
	//todo
}

Wiem, że zwykłymi wskaźnikami bym tak zrobił jednak wolę ich unikać gdyż nie chcę później szukać wycieków.

Answer 1

Nie ma żadnego problemu. Wystarczy użyć funkcji reset.

unique_ptr<Klasa> wsk;
if (warunek) {
    wsk.reset(new Klasa());
}
else if (warunek2) {
    wsk.reset(new KlasaPotomna());
}
else {
    //error
}

 

Comments

Dzięki, wiecie dlaczego podczas takiej deklaracji wektora wskaźników wyskakuje błąd o treści: "próba odwołania do usuniętej funkcji?"

vector <unique_ptr <Person>>persons;

 

---

Prawdopodobnie podczas dodawania nowego elementu do wektora używasz konstruktora kopiującego zamiast przenoszącego.
Dla pewności pokaż deklarację klasy Person oraz użycie wektora persons.

---

Person.h

#pragma once
#include <iostream>

using namespace std;

class Person
{
public:
    int price = 0;
    string name;
    string surrname;
    int height = 0;
    int ratingOfThePerson = 0;

    virtual void addPerson();

    virtual void showPerson();

    virtual void readPerson(fstream& file);
};
 

Person.cpp

#include "Person.h"
#include <iostream>
#include <fstream>

using namespace std;

void Person::addPerson()
{
    cout << "What's the price?: "; cin >> this->price;
    cout << "What's the name?: "; cin >> this->name;
    cout << "What's the surrname?: "; cin >> this->surrname;
    cout << "Height?: "; cin >> this->height;
    cout << "What's the rating of the person?(1-10): "; cin >> this->ratingOfThePerson;
}

void Person:: showPerson()
{
    cout << "Price: " << this->price << endl;
    cout << "Name: " << this->name << endl;
    cout << "Surrname: " << this->surrname << endl;
    cout << "Height: " << this->height << endl;
    cout << "Rating: " << this->ratingOfThePerson << endl;
}

void Person::readPerson(fstream& file)
{
    cout << "Price: "; file >> this->price;
    cout << "Name: "; file >> this->name;
    cout << "Surrname: "; file >> this->surrname;
    cout << "Height: "; file >> this->height;
    cout << "Rating: "; file >> this->ratingOfThePerson;
}

 

Team.h

...

vector <unique_ptr <Person>>persons;

...

 

Team.cpp

void Team::addToTheTeamWsk()
{
    int choice;
    cout << "Who would you like to add?(1.Player 2.Employee): "; cin >> choice;
    unique_ptr<Person>wsk;
    if (choice == 1)
    {
        wsk.reset(new Player);
    }
    else if (choice == 2)
    {
        wsk.reset(new Employee);
    }
    else
    {
        cout << "Bad choice. You're going to menu..." << endl;
        return;
    }
    wsk->addPerson();
    persons.push_back(wsk);
}

void Team::showTeamWorkersWsk()
{
    if (persons.empty() == true)
    {
        cout << "There is no one in the team! You're going to menu..." << endl << endl;
        return;
    }
    for (auto it = persons.begin(); it < persons.end(); it++)
    {
        cout << "Number of player: " << numberOfPlayer++;
        (*it)->showPerson();
    }
}

 

---

Tak jak podejrzewałem.

W funkcji addToTheTeamWsk próbujesz wstawić kopię wskaźnika unique_ptr do wektora. Z definicji wskaźnik unique jest niekopiowalny, musisz go przenieść.

persons.push_back(std::move(wsk));

 

---

Nadal to samo. Z tego co widzę to problem tkwi w deklaracji bo jak zakomentuje tą linię z deklaracją wektora wskaźników program się kompiluje.

Answer 2

Jeśli chcesz przypisać później do unique_ptr to musisz użyć std::move + std::make_unique

Comments

Nie musi używać std::move, jeśli przypisuje to, co zwraca std::make_unique.