Discussion :

[atum25]

Bonjour,

Voila je cherche a crée un objet dans un nouveau thread, cette objet contient des méthodes, qui relèvent des informations d'un poste de travail.
JE cherche ensuite à afficher ces données dans un mémo.
Mais comme j'ai environ 100 threads, lorsqu'il arrive a la méthodes affiche(), il mélange les informations, ou alors affiche une parti d'un, puis un autre puis la fin du 1ere.

Je voudrais pouvoir le forcer à exécuter la fonction affiche d'un seul coup afin de n'afficher que les informations correspondantes a une seul machine.

[cob59]

Utilise un mutex pour restreindre à 1 seul thread à la fois les appels à affiche().
Si affiche() est une méthode, tu ajoutes un mutex dans tes attributs de classe et tu le verrouilles le temps de l'appel à affiche(). Sinon, même principe avec une variable globale que tu verrouilles au début de ta fonction.

[atum25]

Merci de ta reponse,
C'est ce que j'éssai de fair, mais cela ne fonctionne pas correctement (surement me suis-je tromper dans la mise en place du mutex.)

dans mon constructeur de l'objet j'ai placer :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 ghMutex = CreateMutex(
        NULL,              // default security attributes
        FALSE,             // initially not owned
        NULL);             // unnamed mutex
 
    if (ghMutex == NULL)
    {
        //cout<<"CreateMutex error: %d\n", GetLastError();
        Form1->Memo1->Lines->Add("mutex error");
        //return 1;
    }
    Affiche();

puis dans la fonction affiche :

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
 
       DWORD dwCount=0, dwWaitResult; 
       AnsiString Info;
       AnsiString Ip,Mac,Nom;
       switch (dwWaitResult)
       {
           // The thread got ownership of the mutex
           case WAIT_OBJECT_0:
               __try {
                          Ip = CAddIp;
                          Mac = CAddMac;
                          Nom = CNomNetBios;
                          Info = Nom + "   " + Ip + "   " + Mac ;
                          Form1->Memo1->Lines->Add(Info);
                }
 
                __finally {
                    // Release ownership of the mutex object
                    if (! ReleaseMutex(ghMutex)) 
                    { 
                        // Handle error.
                       // Form1->Memo1->Lines->Add("handle error");
                    } 
                } 
                break; 
 
            // The thread got ownership of an abandoned mutex
            // The database is in an indeterminate state
            case WAIT_ABANDONED: 
                Form1->Memo1->Lines->Add("erreur");
          }

[cob59]

A mon avis tu as copié-collé un code sans trop réfléchir.
Tu crées un mutex ghMutex mais tu ne le verrouilles pas au début de affiche()
Sauf erreur, je crois qu'il faudrait un WaitForSingleObject().

[atum25]

Effectivement, je n'avais pas compris cela :/

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
void PosteDeTravail::Affiche()
{
 
       AnsiString Info;
       AnsiString Ip,Mac,Nom;
       WaitForSingleObject(g_hMutex,INFINITE) ;
       __try
        {
           Ip = CAddIp;
           Mac = CAddMac;
           Nom = CNomNetBios;
           Info = Nom + "   " + Ip + "   " + Mac ;
           Form1->Memo1->Lines->Add(Info);
        }
       __finally
        {
           ReleaseMutex(g_hMutex);
        }
}

J'ai donc fait un WaitSingleForObjet, mais cela ne fonctionne toujours pas mieu .Je joint une copie d'ecran de ce que j'obtiens,
Je devrais avoir sur chaque ligne "Nom + Addresse ip + Adresse MAc"

exemple : Poste_CTRL 192.168.2.8 00:11:29...
Pour chaque poste qui repond au ping

[cob59]

Est-ce que chaque thread possède une instance de PosteDeTravail ?
Si oui, ça signifie que tu as N instances de PosteDeTravail, qui gèrent N mutex vers 1 ressource unique (ton logger/afficheur), donc tes mutex en l'état ne servent à rien. Essaie plutôt d'allouer un mutex en static dans Affiche() :

Code :

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

1
2
3
4
5
6
7
8
9
 
void Affiche()
{
static HANDLE ghMutex = CreateMutex(NULL, FALSE, NULL);
WaitForSingleObject(ghMutex, INFINITE) ;
 
...
 
}