Discussion :

[DakM]

Bonjour,

Lorsque je connecte 3 clients, ou plus, ma classe qui correspond a un client change un de ses membre sans aucune raison
Pour faire court,

Code :

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

1
2
3
4
5
6
class Session {
        public:
                inline const int GetSocket() const { return sock; }
        private:
                int sock;
}

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
 
//[... plus de code]
        ConnectedClients = Collection->AllSessions(); // copie par reference des clients
// de type: std::map<int, Session*>
 
        if(Collection->Count() == 3)
        {
            std::cout << "I have now 3 clients connected, let's echo their socket number !"<<std::endl;
            ConnectedClients = Collection->AllSessions();
 
            for(auto itr = ConnectedClients.begin(); itr != ConnectedClients.end(); ++itr)
            {
                Target = itr->second; // de type Session*
                sock = itr->first;
                std::cout << "Key: " << sock << " & Session*->sock = " << Target->GetSocket() << std::endl;
            }
        }
 
        for(auto itr = ConnectedClients.begin(); itr != ConnectedClients.end(); ++itr)
        {
            Target = itr->second;
            sock = itr->first;
            std::cout << "Loop over : " << Target->GetSocket() << std::endl;
 
//[... plus de code]

Donc voila, entre la deuxieme boucle et la troisieme, la valeur de Target->GetSocket() change, voici un screenshot pour mieux comprendre...



edit:
J'ai ajouté en plus dans le code un std::cout sur l'adresse de Target pour voir si elle ne changait pas en cours de route, non elle est toujours bonne. Donc c'est bien le int sock; qui change d'un coup... !

Evidemment, le reste du programme plante salement à cause de ca..

Si vous avez des idées, je suis preneur!
merci, nico

[Trademark]

Salut,

qui a t'il à la fin de la deuxième boucle ?

[Bousk]

La valeur qui est affichée me fait beaucoup penser à une variable non initialisée.
Serait-il possible que le 3° client soit supprimé de la liste avant d'en recréer un ?
Peut-être un constructeur par copie incorrect ?

[DakM]

Regarde la premiere boucle, elle lit les valeurs des clients également !
Et elle marche tres bien, en revanche la 2e boucle, paf...
Je comprend pas du tout comment c'est possible !

Les valeurs dans le constructeur sont bonnes, et meme juste apres, elles sont encore bonnes, quand le 3e client arrivé, voila ce qu'il se passe:

main thread
- lock mutex
- creer un objet session, le memoriser
- ajouter le socket au desripteur de fichier
- lacher le mutex
thread de reception
- on verifie si il y a des clients, et aussi si select > 0
- la boucle donnée dans le premier post

voici un screenshot avec les adresses:

[Bousk]

Si tu es sous Visual, passe en debug et utilise un DataBreakpoint
Si tu es sous autre chose, il y a peut-être aussi un outil correspondant.
Recherche dans tout le code où est modifié cette variable et debug à l'ancienne.

[DakM]

je vien s de lancer un debug, et la y avait aucun changement etrange de variable <_<


Et sans le debogage, ben vous avez l'image, ca foire...