Discussion :

[Stopher]

Salut ,
voila mon soucis , je gere un programme qui recupere des codes barres via le port serie , le principe est le suivant :
le main lance un thread lecture qui est bloqué , dans l'attente d'un evenement sur le handle du port serie !
puis recupere les données et les affiches ... voila le code de ce thread :

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
 
void __fastcall serie::Execute()
{
nbr_octet_a_lire = 1;
char c;
terminer = false;
while (!terminer)
{
if (!ReadFile(Form1->hCom,&Buffer_lecture,nbr_octet_a_lire,&nbr_octet_lu,&Form1->ovl_lecture))
         {
         WaitForSingleObject(Form1->ovl_lecture.hEvent,INFINITE);
 
         if(GetLastError() == ERROR_IO_PENDING) //lecture mise en attente pas l'os
          {
            if (GetOverlappedResult(Form1->hCom,&Form1->ovl_lecture,&nbr_octet_lu,FALSE))
             {
             if (nbr_octet_lu!=0)
                {
                Synchronize(lecture_buffer);
                }
             }
           }
         }
         else
         {
         ShowMessage("erreur_lecture port");
         }
} //Fin du While
 
}
//---------------------------------------------------------------------------
void __fastcall serie::lecture_buffer()
{
Form1->Memo1->Text=Form1->Memo1->Text + Buffer_lecture;
}

le probleme est le suivant :
je dois initialiser le nombre d'octet à lire pour le readfile() , seulement les codes barres recuperé sont de 2 sortes l'un composé de 8octets , l'autre de 6 ... lorsque j'initialise nbr_octet_a_lire à 8 , tout les codes barres composés de 8 octets , passes sans probleme , mais si je passe un code barre de 6octets , rien ne se passe , il faut que je passe une seconde fois le code barre pour qu'il y ai 8 octets .... comment faire pour ne pas avoir à choisir le nombre d'octet à lire ?ou alors ne pas attendre que le buffer ateigne la valeur entrée pour le recuperer ??

Je ne sais pas si j'ai était assez clair , mais mon probleme n'est pas evident à expliquer !
Merci d'avance pour vos posts !

[Grulim]

2 questions:
Pourquoi lis-tu en asynchrone ?
Utilises-tu la fonction SetCommTimeouts (si oui, montre ces parametres) ?

[Stopher]

je li en asynchrone , pour la simple et bonne raison que ca m'est imposé ... j'ai deja fait le programme en synchrone , qui fonctionnait tres bien , seulement mon chef de projet voulait que ce soit un thread qui gere separement la lecture avec gestion d'evenement .... voila .

oui j'utilise des SetComTimeouts , voila les parametres :

Code :

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

1
2
3
4
5
6
 
COMMTIMEOUTS comout = {0}; // pas de timeout
 
.....
 
SetCommTimeouts(hCom,&comout);

y aurait il une erreur la dedans ?

[Aurelien.Regat-Barrel]

Code :

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

1
2
3
if (!ReadFile(Form1->hCom,&Buffer_lecture,nbr_octet_a_lire,&nbr_octet_lu,&Form1->ovl_lecture)) 
         { 
         WaitForSingleObject(Form1->ovl_lecture.hEvent,INFINITE);

Moui, mais utilisé comme àa ça revient à faire du synchrone avec de l'asynchrone. Il faudrait tester la valeur de retour de WaitForSingleObject au passage aussi.
Pas besoin d'être asynchrone si t'as un thread à part. Un simple ReadFile bloquant simplifie ton code:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
if (!ReadFile(Form1->hCom,&Buffer_lecture,nbr_octet_a_lire,&nbr_octet_lu,NULL)) 
{ 
    if (nbr_octet_lu!=0) 
    { 
        Synchronize(lecture_buffer); 
    } // et sinon ?
} 
else 
{ 
    ShowMessage("erreur_lecture port"); 
}

[Stopher]

d'acord ,tout d'abord , merci pour vos posts ,
Mais si je n'utilise pas WaitForSingleObject() , la boucle infinie du thread tournerat sans arret , à l'attente d'un evenement sur le port , je pense donc qu'il est preferable de garder cette fonction . Je vais rajouter le test de la valeure de retour de WaitForSingleObject( ) ...
Mais au niveau du nombre d'octets à lire .... faut il mettre 1 pour qu'il les lises 1par 1 ou directement 8 ? afin que les codes B de 8 ou 8 octets puissent etre lue ! :

[Aurelien.Regat-Barrel]

ReadIntervalTimeout
Maximum time allowed to elapse between the arrival of two bytes on the communications line, in milliseconds. During a ReadFile operation, the time period begins when the first byte is received. If the interval between the arrival of any two bytes exceeds this amount, the ReadFile operation is completed and any buffered data is returned. A value of zero indicates that interval time-outs are not used.

Spécifie une valeur de timeout correcte et t'auras plus de problème, tu pourras mettre 1024 et ça marchera.

Mais si je n'utilise pas WaitForSingleObject() , la boucle infinie du thread tournerat sans arret , à l'attente d'un evenement sur le port , je pense donc qu'il est preferable de garder cette fonction . Je vais rajouter le test de la valeure de retour de WaitForSingleObject( ) ...

Sauf que dans ton exemple tu utilises INFINITE. Mais même, WaitCommEvent remplit à peu près la même fonctionnalité.

Un peu de lecture:
http://msdn.microsoft.com/library/en-us/dnfiles/html/msdn_serial.asp
http://c.developpez.com/faq/vc/?page...WithSerialPort

[Stopher]

Probleme resolu , un grand merci à tous !
8)

[Aurelien.Regat-Barrel]

C'était quoi alors ?

[Stopher]

je ne sais pas si c'est la solution ideale , mais dans tous les cas j'ai pu la tester avec une bonne 50ene de codes barres , et aucun bug n'est survenu , aucun ralentissement meme avec plussieur saisies par seconde tous fonctionnai vraiment bien ! !
voila le code utilisé :

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
 
void __fastcall serie::Execute()
{
nbr_octet_a_lire = 1;
char c;
terminer = false;
while (!terminer)
{
if (!ReadFile(Form1->hCom,&Buffer_lecture,nbr_octet_a_lire,&nbr_octet_lu,&Form1->ovl_lecture))
         {
         WaitForSingleObject(Form1->ovl_lecture.hEvent,INFINITE);
 
         if(GetLastError() == ERROR_IO_PENDING) //lecture mise en attente pas l'os
          {
            if (GetOverlappedResult(Form1->hCom,&Form1->ovl_lecture,&nbr_octet_lu,FALSE))
             {
             if (nbr_octet_lu!=0)
                {
                Synchronize(lecture_buffer);
                }
             }
           }
         }
         else
         {
          if (GetOverlappedResult(Form1->hCom,&Form1->ovl_lecture,&nbr_octet_lu,FALSE))
             {
             if (nbr_octet_lu!=0)
                {
                Synchronize(lecture_buffer);
                }
             }
         }
} //Fin du While

je fait un test du readfile qui me renvoie 1 ou 0 or dans mon ancien code , je ne traite qu'un seul de ces cas! la solution est la , j'ai remarqué , que lorsque je passé un code barre , le if (!read ...... etait validé une seul fois , puis ceului ci renvoyait sa valeur inverse (1) autant de foi qu'il restait de caracteres à lire , j'ai donc exploité ma decouverte en ajoutant un "else" apres le "if (!readfile ....." et à l'interieur de ce "else" j'effectue une lecture sur le buffer .... et la champagne , je m'apercoit que cela fonctionne tres bien meme apres des tests de torture pour trouver un bug , tout allé pour le mieux !!!!!
le code posté n'est pas tout à fait ceului utilisé actuellement , ayant oublié de sauvegarder mes données sur ma clée usb .... mais ca y ressemble fortement !!!