Discussion :

[MrScriptX]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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();
    }
}

[bacelar]

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

[MrScriptX]

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

Désole d'être aussi nul, mais j'y arrive toujours pas :/

[bacelar]

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.

Regarde les contraintes sur le premier paramètre indiquées dans la documentation.

[MrScriptX]

Il faut l*'argument en referrence? Je comprends pas...

[bacelar]

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

[MrScriptX]

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 &

[bacelar]

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.

[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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[bacelar]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[MrScriptX]

Merci pour ton aide, j'ai trouver une solution

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[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".

[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)....