Problèmes avec Chart

Bonjour, le problème d'aujourd'hui réside dans l'ajout de donnée dans un graphique.

Je travail avec l'acquisition de données a partir d'une source RS232 (8bits/1 stop bit/19200 bps) et lors de l'ajout de mes données dans le graphique, il met une erreur de pointeur, alors que lorsque je met en commentaires ces deux lignes (1 par graphique) le programme tourne nikel)

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 48 49 50 51 52 53 54 55 56

void __fastcall TFormMEC::CPortRxChar(TObject *Sender, int Count) { char *cRx = new char [Count]; CPort->Read(cRx, Count); int index = 0; do { if((cRx[index++] == CHAR(0xFF))&&(Count >= 5)) { int iVal1; int iVal2; double dVal; //Débit iVal1 = (int) cRx[index++]; iVal1 += 128; iVal2 = (int) cRx[index++]; iVal2 += 128; iVal2 *= 256; iVal2 += iVal1; dVal = (double)iVal2; //Ajout du debit dans le graphique //Erreur 1-> SerieDebit->AddXY(iTrame, dVal, "", clRed); //Pression iVal1 = (int) cRx[index++]; iVal1 += 128; iVal2 = (int) cRx[index++]; iVal2 += 128; iVal2 *= 256; iVal2 += iVal1; dVal = (double)iVal2; //Ajout du debit dans le graphique //Erreur 2-> SeriePression->AddXY(iTrame,iVal2, "", clRed); iTrame++; //Ajustement des echelles des graphiques if(iTrame > iEchelle) { ChartDebit->BottomAxis->Maximum = iTrame; ChartPression->BottomAxis->Maximum = iTrame; ChartDebit->BottomAxis->Minimum = iTrame - iEchelle; ChartPression->BottomAxis->Minimum = iTrame - iEchelle; if(!iEchelle) { ChartDebit->BottomAxis->Maximum = iTrame; ChartPression->BottomAxis->Maximum = iTrame; ChartDebit->BottomAxis->Minimum = 0; ChartPression->BottomAxis->Minimum = 0; } } else InitEchelle(); } else index++; } while(index < Count); }

---

Quelqu'un sais si les graphiques ont une fréquence de rafraichissement limite, ou alors ou est-ce que j'aurais fait mon erreur (aussi idiote soit-elle).

Merci

Salut,
Il n'y a pas de limite de temps ou quoique ce soit dans l'ajout de point. Au pire, il ne rafraichit pas le TChart tant qu'il n'a pas la main.

Je pense qu'il faut que tu mettes un point d'arrêt et que tu regardes la valeurs de iTrame.
Quel est l'erreur que tu as?
Quel est la valeur des variables avant l'erreur?
Si c'est une question de temps, en pas à pas, tu ne devrais pas avoir d'erreur. Est ce le cas?

en fait j'ai mis un tit label pour visualiser la valeur de dVal, elle varie de 0 à 65535 (0x00 à 0xFF)

le problème viendrait de la multiplication par 256 qui ne conviendrai pas.

l'erreur est une erreur de pointeur (je peux pas retester maintenant mais je le ferais des que possible)

Par contre, j'ai remarqué qu'avec ce code, j'utilisais 100% du CPU (un 2.80 GHz P4) ce qui me parrait énorme pour un simple bvoucle de calculs(je supose que les deux chart prennent énormément de ressources)

L'erreur que j'obtien est :

Citation:

---

Le projet MesureEnContinue.exe a provoqué une classe d'exception EInvalidPointer avec le message 'Opération de pointeur incorrecte'. Processus stoppé. Utilisez Pas-à-pas ou Exécuter pour continuer.

---

Pour quelques precisions, le programme scrute le port COM pour recevoir des trames de 8 octets (0xFF quatres octets utilisé, puis 3 octets non utilsé pour l'instant)
Il y a un nombre de 100 trames par secondes (un mode 200 trame par secondes est envisagé, mais la n'est pas le problème) donc on a au moins 8000 bits par secondes de transféré (1 bit start, 1 bit stop et 8 bits de donnees, le tout sur 8 octets)

Citation:

---

Envoyé par Galkir

en fait j'ai mis un tit label pour visualiser la valeur de dVal, elle varie de 0 à 65535 (0x00 à 0xFF)

le problème viendrait de la multiplication par 256 qui ne conviendrai pas.

l'erreur est une erreur de pointeur (je peux pas retester maintenant mais je le ferais des que possible)

Par contre, j'ai remarqué qu'avec ce code, j'utilisais 100% du CPU (un 2.80 GHz P4) ce qui me parrait énorme pour un simple bvoucle de calculs(je supose que les deux chart prennent énormément de ressources)

---

avec un tel code tu prendra 100% de ton uc meme si ta le tous dernier processeur.
Sa vien du fait que tu boucle un grand nombre de fois (je présume) ... combien vaut iCount?

Pour palier a ce prob met un Sleep(10); juste avant le while

Code:

---

1 2 3 4 5 6 7 8

else index++;   Sleep(10); } while(index < Count); }

---

Count (et non pas iCount) est le nombre d'octets présents dans le buffer du port (le nombre d'octets a lire), et il a une valeur de 8 (normalement) mais plus le graphique a de point, plus il prend du temps a calculer (même pour un 2800 MHz) et le buffer augmente, pour passer de 8 à 16 vers les 10 secondes, et à 24 de temps en temps.

Pour le Sleep(10) (ou tout autre valeur) ca fout la marde totale! donc solution non envisageable

Je vais simplifier ma demande (j'espere que c'est la clé du problème) :

J'ai deux octets (char1 et char2) en entrée, et je souhaite avoir un double en sortie. (char2 étant le LSB et char1 le MSB, cad double = char1*256 + char2 le tout allant de 0 à 65535 [0x0000 à 0xFFFF])

Comment faire cete conversion avec le moins d'instructions possibles?

Code:

---

1 2 3 4 5 6 7

iVal1 = (int) cRx[index++]; iVal1 += 128; iVal2 = (int) cRx[index++]; iVal2 += 128; iVal2 *= 256; iVal2 += iVal1; dVal1 = (double)iVal2;

---

Edit : l'utilisation de TFastLine améliore la rapidité de la fonction

moi avec des unsigned short j'avais fais comme sa :

Code:

---

1 2 3 4 5 6

unsigned short usVal; unsigned char ucChar1, ucChar2;     usVal = (unsigned short)(ucChar1<<8) | (unsigned short)ucChar2;

---

avec des double sa donnerais sa je pense (jai pas tester)

Code:

---

1 2 3 4 5

double dVal; char cChar1, cChar2;   dVal = (double)(cChar1<<8) | (double)cChar2;

---

voila, comment moi je vois la chose
En esperant que sa peut t'aider

Cordialement Baxter

Un conseil:
Si tu multiplies ta valeurs iVal2 par 256, vérifie bien qu'avant ta multiplication, elle est inférieur à 256 (256*256 = 65535), pour ne pas avoir de dépassement.

Sinon, pour la méthode:

Code:

---

1 2 3 4 5

int iVal1 = (int)cRx[index++] + 128; int iVal2 = (((int)cRx[index++] + 128)&0xFF )<<8;   iVal1 += iVal2;

---

NB: Il est aussi important de vérifier que l'index ne dépasse pas du tableau, d'où un problème de pointeur.

Citation:

---

Envoyé par bandit boy

Un conseil:
Si tu multiplies ta valeurs iVal2 par 256, vérifie bien qu'avant ta multiplication, elle est inférieur à 256 (256*256 = 65535), pour ne pas avoir de dépassement.


NB: Il est aussi important de vérifier que l'index ne dépasse pas du tableau, d'où un problème de pointeur.

---

Alors, a ma connaissance, un unsigned char (ou juste char) ne peu dépasser la valeur base 10 de 256.

en fait, ce problème survient quand je met "Application->ProcessMessage()" ou "Chart->Refresh()".
Si je ne met pas une de ces deux méthodes dans mon code, il passe nikel, sauf qu'au bout de 5-10 s, le premier chart s'arrete de s'afficher, et je commence a voir le buffer d'entré accumuler les données (au début que des trames de 8 octets, puis ensuite, 16, etc etc) et le processeur est utilisé a 100% (enfin au max), ce que je trouve bizar, et surttout ennuyeux, car par la suite, je suis ammené a avoir 4 chart dans la même fenêtre.

mais je vien de penser a un truc, est-ce qu'on peu avoir deux graph distinct dans un meme objet Chart? (comme avec TeeChart) ca éviterais peut-etre de traiter les deux objets actuels, et permettre au procs de moins raler.

comme je l'ai dis précédement, met un petit Sleep et ton utilisation CPU baisseras beaucoup

Cordialement Baxter

Avec Builder, il est possible d'avoir des dépassements. Je m'explique: il se peut que la variable soit ré-assigné en int pour pouvoir stocker la valeur(déjà vu), d'où l'utilité d'un test.

Peux tu faire un affichage dans des Tlabels des valeurs en direct et voir si elles cloches?

Pour le TChart, tu peux afficher d'autres courbes, ou affilier des composants aux TChart (composant enfant du TChart), donc au pire, avoir un TChart dans un TChart, mais pas 2 graphes distinct dans le même TChart à ma connaissance.

mais si je met un Sleep, il me bloque mon programme
Si j'ajoute une processMessage, il me fout en caraf le progrmme
faut-il que je passe par un thread et juste envoyer les données venant du buffer?

---

Les données sont cohérantes, souvent autour de 8000 elles ne depassent pas les 60k.

ben c sur, que si tu utilise un sleep il fo créer un thread qui fera cette gestion. ;)
Comme sa, le programme ne sera jamais freezer et ton utilisation CPU va chuter en fleche :)

bon, bah me reste plus qu'a potasser les thread, rdv a la prochaine expérience

ta un bon tutorial sur les thread dans la FAQ

J'ai pas fait attention dans la FAQ mais dans les tuto proposé par CGI j'ai trouvé ce qui devrais me convenir : http://chgi.developpez.com/thread/

Par contre, faut que je trouve comment on se sert de l'outils ComDataPacket car ca pourrais m'éviter pas mal de conditions, et donc améliorer la rapidité d'execution de ma fonction.

Edit : juste une question, comment récupérer la valeur (de 0x00 à 0xFF) d'un caractère AnsiString?

Alors, finalement, pas besoin de faire un thread spécialisé, un simple ComDataPacket suffit largement.

J'ai même pu réduire le code de façon assez intéréssante, ce qui me permet de ne plus avoir de problème de saturation de buffer (enfin j'en vois plus)

L'utilisation du processeur est toujours de 100% mais ce n'est pas grave (le logiciel est destiné a des poste de travail dédié a son utilisation (pas de programme qui tourne a coté)

Merci pour l'aide.

Pour info, voici mon code :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22

void __fastcall TFormMEC::CDPackPacket(TObject *Sender, const AnsiString Str) { char *cChar = new char [Str.Length()]; cChar = Str.c_str(); double dVal1, dVal2; dVal1 = (double)(cChar[1] << 8) + (double)cChar[0]; dVal2 = (double)(cChar[3] << 8) + (double)cChar[2]; //Ajustement des echelles des graphiques ChartDebit->BottomAxis->Maximum = iTrame; ChartPression->BottomAxis->Maximum = iTrame; if(iEchelle) { ChartDebit->BottomAxis->Minimum = iTrame - iEchelle; ChartPression->BottomAxis->Minimum = iTrame - iEchelle; } //Ajout du debit dans le graphique SerieDebit->AddXY(iTrame, dVal1); //Ajout du debit dans le graphique SeriePression->AddXY(iTrame,dVal2); iTrame++; }

---

Une erreur, ou plutot un problème persiste.

Sur ma fiche, j'y ai inséré deux composant Chart, avec chacun deux série de points.
Mais le problème est que au bout de quelques secondes d'acquisition et de traitement, le premier Chart ne se rafraichis plus.
Si j'ajoute la méthode ->Refresh(), je n'ai plus la main du tout, malgrès le problème résolu.
Si enfin j'ajoute Application->ProcessMessage(), j'ai la main, les graph tournent bien, mais le temps de traitement étant trop long, j'en arrive a avoir plusieurs secondes de retard entre le changement de données d'acquisitions et l'affichage dans les graphiques.

Y a-t-il une autre méhode que Refresh() qui soit moins gourmande, et qui me laisse la main pour agir sur ma fiche?

le thread :P

a part sa personnelement je voie pas

Je sais que le point est marqué résolu, mais il existe une astuce pour l'affichage des courbes dans un TChart.

Le fait que ca rame vient du problème du nombre de donnée dans la série.
L'idée est de charger un série en RAM et d'afficher uniquement la partie souhaiter.

Sinon, suivant comment se présente les données, il peut aussi être enregistrée de façon intelligente en supprimant les point non nécessaire au graphique (plusieurs point consécutifs à la même valeurs est inutile par exemple).

Alors, le problème est que je ne peux pas me permettre de supprimer de points (ils sont destiné à un traitement par un tableur, ou d'autres utilisations)

Pour ce qui est d'aficher que les points intéréssants, non plus je ne peux pas, vu que j'ai des radio button me permettant de choisir tous les points, les 3000 derniers ou les 6000 derniers.

Pour ce qui est de supprimer les points égaux, c'est possible, mais voyant mon graphique, je ne pense pas que ca me permette de gagner assez pour ne pas freez (ce sont des graphiques de débit et de pression entre une respirateur et un poumon)