Преобразуйте кортеж из значений в кортеж из значений - PullRequest
Новая
       37

Преобразуйте кортеж из значений в кортеж из значений

1 голос
/ 15 апреля 2020

Извините, если я неправильно понимаю всю концепцию, но я пытаюсь сделать кортеж контейнером реальных объектов, где только при его уничтожении эти объекты go выйдут из области видимости.

В настоящее время у меня есть это: 

class MiniThread {
public:
    ~MiniThread() {
        if (m_thread) {
            if (m_thread->joinable())
                m_thread->join();

            delete m_thread;
        }
    }

    void join()
    {
        if (m_thread == nullptr)
            return;

        m_thread->join();
        m_thread = nullptr;
    }

    template<typename F, typename... Args>
    void run(F func, Args... args)
    {
        if (m_thread != nullptr)
            join();

        auto tuple = std::forward_as_tuple(args...);

        m_thread = new std::thread([=]() {
            __try
            {
                std::apply(func, tuple);
            }
            __except (CrashDump::GenerateDump(GetExceptionInformation()))
            {
                // TODO: log.
                exit(1);
            }
        });

        m_started = true;
    }

    bool started() const { return m_started; }

private:
    std::thread *m_thread = nullptr;
    bool m_started = false;
};


std::string getString()
{
    return std::string("sono");
}

int main()
{
    auto test = [&](std::string seila, const std::string& po, std::promise<int>* p)
    {
        std::cout << seila.c_str() << std::endl;
        std::cout << po.c_str() << std::endl;
        p->set_value(10);
    };

    std::promise<int> p;
    std::future<int> f;

    MiniThread thread;
    std::string hello = "hello";
    std::string seilapo = "seilapo";

    f = p.get_future();
    thread.run(test, getString(), "how are you", &p);

    thread.join();
    int ftest = f.get();
    std::cout << ftest << std::endl;
}

К тому времени, когда поток запущен, args уже не надежен. Они уже уничтожены. Поэтому мне было интересно, есть ли способ скопировать их в вызове потока по значению . Я сделал несколько попыток переместить аргументы variadi c в кортежи, но кортежи всегда отображаются с rvalues и все равно терпят неудачу.

1 Ответ

6 голосов
/ 15 апреля 2020

This: 

auto tuple = std::forward_as_tuple(args...);

Создает кортеж ссылок на args... Это и есть работа forward_as_tuple. Затем вы захватываете этот набор ссылок по значению: 

m_thread = new std::thread([=]{ /* ... */ });

Так что, как только ваши аргументы go выходят за рамки, вы только держитесь за ссылки на них ... и это будет зависать.

Но на самом деле вам ... вообще не нужен кортеж. Просто скопируйте сами аргументы: 

m_thread = std::thread([=]() {
    func(args...); // func and args, no tuple here
});

Также не пишите new thread - thread - это уже тип дескриптора, просто создайте его.

Выше копирует аргументы. Если вы хотите переместить их, то в C ++ 17, да, вам нужно иметь tuple и использовать std::apply. Но не forward_as_tuple ... просто make_tuple: 

m_thread = std::thread([func, args=std::make_tuple(std::move(args)...)]() mutable {
    std::apply(func, std::move(args));
});

В C ++ 20 вам больше не понадобится tuple, и вы можете написать расширение пакета: 

m_thread = std::thread([func, ...args=std::move(args)]() mutable {
    func(std::move(args)...);
});
...