Передача права собственности на уже определенный std::unique_ptr члену объекта класса

Я пытаюсь передать право собственности на уже определенный std::unique_ptr члену объекта класса:

#include <iostream>
#include "memory"


// base class
class base
{
public:
    void setVal(int val)
    {
        m_val = val;
    }

    int getVal()
    {
        std::cout << "base val = " <<m_val<<std::endl;
        return m_val;
    }

private:
    int m_val = 0;
};

// derived class
class derived1: public base
{
private:
};

// derived class
class derived2: public base
{
public:

private:
};


class user
{
public:

    user(std::unique_ptr<base> d)
    {
//        m_class = static_cast<std::unique_ptr<base>>(d.get());
//        m_class = std::make_unique<base>();
        m_class = std::move(d);
    }
    user(std::unique_ptr<derived1> d)
    {
//        m_class = static_cast<std::unique_ptr<derived1>>(d.get());
//        m_class = std::make_unique<derived2>();
        m_class = std::move(d);

    }
    user(std::unique_ptr<derived2> d)
    {
//        m_class = static_cast<std::unique_ptr<derived2>>(d.get());
//        m_class = std::make_unique<derived2>();
        m_class = std::move(d);
    }

    void setVal(int val)
    {
        m_class->setVal(val);
    }

    int getVal()
    {
        return m_class->getVal();
    }

private:
    std::unique_ptr<base> m_class;

};

int main()
{
    std::unique_ptr<base> b = std::make_unique<base>();
    std::unique_ptr<base> d1 = std::make_unique<derived1>();
    std::unique_ptr<base> d2 = std::make_unique<derived2>();

    user ud1(std::move(d1));
    user ud2(std::move(d2));

    ud2.setVal(90);
    ud1.setVal(10);

    ud2.getVal();
    ud1.getVal();

    return 0;
}

Кажется, я дважды нажал указатель; впервые передаем std::movestd::unique_ptr и d1 конструктору объекта d2; во второй раз это userstd::move или d1 в переменную-член d2.

Дважды m_class указатель не кажется интуитивно понятным, чтобы в конечном итоге передать право владения std::move и d1 m_class. Правильно ли я делаю, или есть правильный/лучший способ сделать это? Спасибо!!

Стоит ли изучать PHP в 2023-2024 годах?
Стоит ли изучать PHP в 2023-2024 годах?
Привет всем, сегодня я хочу высказать свои соображения по поводу вопроса, который я уже много раз получал в своем сообществе: "Стоит ли изучать PHP в...
Поведение ключевого слова "this" в стрелочной функции в сравнении с нормальной функцией
Поведение ключевого слова "this" в стрелочной функции в сравнении с нормальной функцией
В JavaScript одним из самых запутанных понятий является поведение ключевого слова "this" в стрелочной и обычной функциях.
Приемы CSS-макетирования - floats и Flexbox
Приемы CSS-макетирования - floats и Flexbox
Здравствуйте, друзья-студенты! Готовы совершенствовать свои навыки веб-дизайна? Сегодня в нашем путешествии мы рассмотрим приемы CSS-верстки - в...
Тестирование функциональных ngrx-эффектов в Angular 16 с помощью Jest
В системе управления состояниями ngrx, совместимой с Angular 16, появились функциональные эффекты. Это здорово и делает код определенно легче для...
Концепция локализации и ее применение в приложениях React ⚡️
Концепция локализации и ее применение в приложениях React ⚡️
Локализация - это процесс адаптации приложения к различным языкам и культурным требованиям. Это позволяет пользователям получить опыт, соответствующий...
Пользовательский скаляр GraphQL
Пользовательский скаляр GraphQL
Листовые узлы системы типов GraphQL называются скалярами. Достигнув скалярного типа, невозможно спуститься дальше по иерархии типов. Скалярный тип...
1
0
51
1
Перейти к ответу Данный вопрос помечен как решенный

Ответы 1

Ответ принят как подходящий

Правильно ли я делаю, или есть правильный/лучший способ сделать это?

Правильно переместить его дважды, поскольку вы переносите значение между тремя разными переменными.

Это тоже не проблема. std::move — это приведение во время компиляции. Вызов его дважды не влияет на производительность.

Однако...

Вам следует использовать списки инициализаторов участников.

user(std::unique_ptr<base> d)
    : m_class{ std::move(d) }
{
}

И эти конструкторы лишние. Удалите их. Вам не нужно учитывать все возможные производные типы, все они могут неявно конвертироваться в std::unique_ptr<base>.

    user(std::unique_ptr<derived1> d)
    {
//        m_class = static_cast<std::unique_ptr<derived1>>(d.get());
//        m_class = std::make_unique<derived2>();
        m_class = std::move(d);

    }
    user(std::unique_ptr<derived2> d)
    {
//        m_class = static_cast<std::unique_ptr<derived2>>(d.get());
//        m_class = std::make_unique<derived2>();
        m_class = std::move(d);
    }

Единственная проблема заключается в том, что если конструктор получит точку как базовый класс, станет ли член в конечном итоге базовым классом, а не производным?

Nick X Tsui 05.09.2024 18:00

Спасибо !!@Дрю Дорманн

Nick X Tsui 05.09.2024 18:04

@NickXTsui нет, член всегда останется тем классом, в котором он был создан (именно поэтому необходимы виртуальные деструкторы), но вы все равно всегда сохраняете его как std::unique_ptr<base> m_class;, поэтому дополнительные конструкторы не нужны.

Sir Nate 05.09.2024 18:51

Спасибо @SirNate!!

Nick X Tsui 05.09.2024 19:00

Другие вопросы по теме