Error C2664 lors d'implantation de Thread

Version imprimable

Bonjour a tous,
J'ai un léger problème, enfin plutôt gros problème en faite:( Je travaille sur Qt Creator et je suis entrain de faire une messagerie. Le problème c'est que lors de l'implémentation de thread j' ai cette erreur de compilation:

Code:

erreur : C2664: 'std::tuple<void (__cdecl Window::* )(Server),Window *,Server>::tuple(std::tuple<void (__cdecl Window::* )(Server),Window *,Server> &&)'ÿ: impossible de convertir l'argument 1 de 'void (__cdecl Window::* )(Server)' en 'std::allocator_arg_t'

Bref, j'y comprend rien. Donc je m'en remet a vous. Voila le reste du code.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 #include "window.h" Window::Window(): QWidget() { //code pas important do { //nothing }while(config.getLaunch() == false); Server myServer(config.getPort(), config.getBroadcast()); std::thread serverThread(&Window::ListenToClients, this, myServer);//Ligne qui est en faute } void Window::Config() { config.exec(); } void Window::ListenToClients(Server server) { for(int i = 0; i < 100; i++) { server.ListenNewConnections(); } }

25/10/2016, 11h27
bacelar

RTFM. :aie:
http://www.cplusplus.com/reference/t...thread/thread/
25/10/2016, 13h23
MrScriptX

Citation:

Envoyé par bacelar

RTFM. :aie:
http://www.cplusplus.com/reference/t...thread/thread/

Désole d'être aussi nul, mais j'y arrive toujours pas :/
25/10/2016, 14h21
bacelar

Citation:

Désole d'être aussi nul,

T'es pas nul, c'est assez tricky, même dans la documentation. ;)
Mais les réponses sont dedans. :aie:

Regarde les contraintes sur le premier paramètre indiquées dans la documentation.
25/10/2016, 14h35
MrScriptX

Il faut l*'argument en referrence? Je comprends pas...
25/10/2016, 18h13
bacelar

Est-ce que la classe "Window" est copy-constructible, ou mouvable ?
25/10/2016, 18h21
MrScriptX

Citation:

Envoyé par bacelar

Est-ce que la classe "Window" est copy-constructible, ou mouvable ?

Je pense que oui, une classe est copy-constructible. Donc il faut que j'enlève le &
25/10/2016, 18h30
bacelar

Citation:

Je pense que oui, une classe est copy-constructible.

Une "QWidget", c'est de l'IHM. Une bout d'IHM copy-constructible, j'en doute, ça serait une grosse peau de banane.
25/10/2016, 19h47
MrScriptX

Mais je ne fais pas passer window par référence ? J'ai essayé avec std::ref, sa n'a rien donné

EDIT:

J'ai supprimer le this du code

Code:

std::thread serverThread(&Window::ListenToClients, myServer);

Mais j'ai toujours une erreur même si celle-ci est différente:

Code:

erreur : C2661: 'std::tuple<void (__cdecl Window::* )(Server),Server>::tuple'ÿ: aucune fonction surchargée ne nécessite 2 arguments

Je n'ai qu'un VS2012 sans Qt sous la main mais le code suivant compile :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 #include "stdafx.h" #include <thread> #include <iostream> class Server{ public: void ListenNewConnections(){}; }; class Window{ public: Window(); void ListenToClients(Server server); }; void Window::ListenToClients(Server server) { for(int i = 0; i < 100; i++) { server.ListenNewConnections(); } } Window::Window() { Server myServer; std::thread serverThread(&Window::ListenToClients, this, myServer); } int _tmain(int argc, _TCHAR* argv[]) { Window wiwi; return 0; }
A l'exécution, c'est plus "funky", en mode Debug, car le runtime qui appelle le destructeur de serverThread à la sortie de "Window::Window()", il a repéré que le thread n'a pas été "joiné" et t'envoie bouler comme un gros sale que je suis.

C'est donc une "propriété" de vos classes Serveur ou Window ou de leurs ancêtres qui empêche la compilation, ou une différence d'implémentation de std::thread.

Merci pour ton aide, j'ai trouver une solution

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
#include "window.h"
 
Window::Window(): QWidget()
{
    setFixedSize(600, 400);
 
    m_start = new QPushButton("Start Server");
    m_shut  = new QPushButton("Shut Down");
    QTextEdit *log = new QTextEdit;
 
    log->setReadOnly(true);
    log->setFont(QFont("Courier"));
    log->setLineWrapMode(QTextEdit::NoWrap);
 
    QHBoxLayout *footer = new QHBoxLayout();
    footer->addWidget(m_start);
    footer->addWidget(m_shut);
 
    QGridLayout *window = new QGridLayout();
    window->addWidget(log, 0, 1);
    window->addLayout(footer, 1, 0, 1, 2);
 
    setLayout(window);
 
    QObject::connect(m_start, SIGNAL(clicked()), this, SLOT(Config()));
    QObject::connect(m_shut, SIGNAL(clicked()), this, SLOT(close()));
 
    Server myServer(1111);
    std::thread serverThread([this, &myServer]()
    {
      Window::ListenToClients(myServer);
    });
 
}
 
void Window::Config()
{
    config.exec();
}
 
void Window::ListenToClients(Server &myServer)
{
    for(int i = 0; i < 100; i++)
    {
        myServer.ListenNewConnections();
    }
}

En revanche le programme crash a l'execution. Apparemment quelque chose ne va pas avec

Code:

1
2
3
4
std::thread serverThread([this, &myServer]()
    {
      Window::ListenToClients(myServer);
    });

27/10/2016, 20h08
bacelar

Passage par expression lambda, pourquoi pas.

Mais le problème indiqué au runtime reste le même.
Ton destructeur de "std::thread" va être appelé à la fin de "Window::Window(): QWidget()", comme dans mon code.
Les "std::thread", c'est comme les autres objets en C++, c'est à nous de gérer leur cycle de vie et leur "ownership".
28/10/2016, 09h03
stephane78l

Bonjour tu aurais pu aussi utiliser une classe derivant de QThread pour ton thread ou un QThread et un objet derivant de QObject(une des deux manieres de faire du multithreading avec Qt)....