Erreur List Index Out Of Bound

Bonjours, jai une application multi thread qui fonctionne niquel dans la plupart des cas ..

Mais la j'ai laiser tourner toute la nuit (c'est une appli qui doit fonctionner 24h/24) jai eu plusieur Dizaine d'erreur du stule Index Out Of Band (XXX) (xxx etais preque a chaque fois different).
Etant doner que le poste sur lequelle le programme n'a pas builder, jai eseyer de reproduire l'erreur sur mon pc, et jai reusi une fois et il ma mis l'ereur juste apres le Application->Run sa me semble bizare

Le programme fonctionne nikel pendant des heures, mais meme avec ces ereur je clike que ok et sa repart.

Un probleme peut etre lié et que certain thread sarete il me semble ... malgré le fait qu'il y ai une boucle infini dans le execute du thread ... je ne vois vraiment plus comment debugger tous sa :s

Si quelqu'un pouvais m'aider s'il vous plait :)

Cordialement Baxter

Pour information mon programme fais la supervision d'un automate.

Jai un thread qui lit et qui ecrie des info dans l'automate a partir de tableau que jai aloué dinamiquement (il sont de la bonne taille)
Jai plusieur autre thread qui accéde a ces tableau pour changer la couleur de boutons, afficher des valeurs ou afficher des message, ou encore de géré le tracage de courbe TChart lefacement de ces courbes et lexportation impression de ces courbes.

voila

Au vu de ton erreur, tu dois essayer d'accéder à un index qui n'existe pas dans une liste.

Comme ca, c'est pas évident de t'aider sans avoir de code

ben j'aimerais bien donner du code ... mais il ma pas montré ou exactement il a planter ...

de temps en temps sa saconpagne d'un message d'ereur du type ...

Citation:

---

Acces violation at adress 00440C63 in module 'project.exe'
Read of address 00000014

---

je me tape la tete contre un mur, avant sa fonctionnais nikel

Et etant doner que mes lecture dans les list son ciclique, si je fesais la dedans quelque chose que javais pas le droit il devrais me faire une erreur a chaque fois ... mais la c ponctuelle et completement aléatoire

Comment génère tu l'index pour accéder à tes listes.

Moi j'ai déjà eu se problème ou l'index que je génèrais n'existait pas dans ma liste et ca me fesait cette erreur, et comme toi de façon aléatoire.

Si je comprend bien, tu n'a pas réussi a identifier quelle ligne provoque l'erreur.

C'est par là qu'il faudrais commencer

tu ne pourrais par faire un affichage de ton index a chaque fois que tu veux acéder à tes liste, ca te permettrait de voir s'il est correct ou non.

voici les deux fonction qui accede au donnée du tableau en s'aidant d'une liste :

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

//--------------------------------------------------------------------------- int AccesDataRead(int iVal) { int iIndex = -1; int iReturn;   //Fonction retourant la valeur du MW iVal ou -1 si le MW n'est pas lue   iIndex = Form1->slListMWRead->IndexOf(IntToStr(iVal)); if(iIndex != -1) iReturn = Form1->TabValeurIndex[iIndex]; else iReturn = -1;   return iReturn; } //--------------------------------------------------------------------------- int AccesDataRead(int iVal, int iBit) { int iValRetourner = -1;   iValRetourner = AccesDataRead(iVal);   if(iValRetourner != -1) { if((iValRetourner>>iBit)& 0x01) return 1; else return 0; } else return -1; }

---

Bon en eseyant dafficher lindex auquele on accede, il a fais le acces data read etc a l'index 145
Ma list et mon tableau contienne 385 lignes ... donc normalement c bon :s

Tu as essayé de balayer l'ensemble des valeurs de ton tableaux pour voir s'il le lisait en entier.

Pourquoi tu n'utilise pas uniquement des listes au lieu de tableaux.

Cela serait beaucoup plus simple pour l'allocation.
Tu crée un élément liste et tu l'ajoute dans ta liste. tu fais cela pour chaque élément que tu veux ajouter dans la liste. (pour toi un élément liste, c'est un pointeur d'entier mais tu peux te créer une classe qui va bien pour stocker ton entier, moi c'est ce que je fais).

Ben, jai des tableau de ansistring et de int

pour les Ansistring je peut utiliser des TStringList mais pas pour les int a moin de frequement faitr InToStr et StrToInt ...

Ce que javais penser pour les message par exemple
pour le moment jai


- un tableau qui dis le mot du mess
- un tableau qui dis le bit dans ce mot
- un tableau qui contien le message
- et deux tableu qui sont mis a jour pour gérer les fronts sur ces message pour les afficher que une fois

EN regardant un peu le TList jaimerais bien savoir si il est possible de faire que une seul list contenant des structure regroupant toute ces info ?

pour la gestion de mes message et de mes alarmes jai remplacer 6 tableaux par un seul

Code:

---

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

class clasMessageAla { public : clasMessageAla() { this->iMots = 0; this->iBit = 0; this->asMessage = ""; this->iAncienEtat = 0; this->iNouvelEtat = 0; this->Couleur = clBlack; } int iMots; int iBit; AnsiString asMessage; int iAncienEtat; int iNouvelEtat; TColor Couleur; };

---

Dans le .h du principale :

Code:

---

1 2 3 4 5 6

//DECLARATION DES TABLEAU POUR LE STOCKAGE DES MESSAGE clasMessageAla *TabMessage;   //DECLARATION DES TABLEAU POUR LE STOCKAGE DES ALARMES clasMessageAla *TabAlarme;

---

Apres pour lalocation je fais comme sa :

Code:

---

1 2 3

TabAlarme = new clasMessageAla [ListTmp->Count]; TabMessage = new clasMessageAla [ListTmp->Count];

---

ou ListTmp est une StringList contenant tous les message mot bit couleur etc stocker en resource jai fais des fonction qui stoke ces valeur dans le on crate


Est ce que cette maniere de travailler est propre ?

mais bon, je ne suis pas persuader que mon prob vien de la ... jai mis des try catch dans chacun de mes thread sa me permetra de localiser plus précisement je pense

tu peux mettre ce que tu veux dans un TList
Tu n'est pas obligé que chaque élément soit du même type.

Moi j'utilise TList dans mes logiciels avec plusieurs type.

Tu te crée une classe qui va bien et qui contient toutes les infos dont tu as besoin.

Ensuite tu crée un pointeur sur un objet de cette classe et c'est lui que tu ajoute dans ta TList.

Pour y accéder, il te suffit ensuite de l'extraire de la liste.

Sinon pour tes int, moi je le fais sans intToStr et StrToInt

Je me suis créé une classe toute simple pour stocker un int

Code:

---

1 2 3 4 5 6 7 8 9

class TInt { public : int Valeur;   // Constructeur TInt(); };

---

Ensuite lorsque tu veux ajouter des int dans une liste, je fais ceci
Pour l'exemple je fais une suite de 1 à 10

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

for(int i = 1; i <= 10; i++) { // Je crée l'élément TInt *Entier = new TInt();   // Je renseigne la valeur de l'entier Entier->Valeur = i;   // Je l'ajoute dans la liste Liste->Add(Entier); }

---

Après pour y accéder, c'est très simple

Code:

---

1 2 3 4 5

// J'extrait le 3ème élément (Index 2 car les index commence à 0) TInt *Entier = (TInt *)(Liste->Items[2]);   ShowMessage("l'élément d'index 2 est : " + AnsiString(Entier->Valeur));

---

Bien sur, il ne faut pas oublié de déclarer ta liste

Code:

---

1 2	TList Liste = new TList();

---

Donc pour ton exemple avec ta classe Message, cela irait très bien

il te suffit de déclarer une liste comme je t'ai montrer plus haut
Et ensuite a chaque fois que tu crée un élément de type clasMessageAla *, tu l'ajoute dans ta liste.

Pas besoin d'allocation puisqu'elle se fait automatiquement à chaque fois que tu crée ton élément et que tu l'ajoute dans la liste

ben si je le fais avec une TList et ma class message, si je ve accéder a quelque chose je suis obliger de le sortir

si je ve modifier quelque chose, je dois le sortir, puis modifier, supprimer l'ancien du TList puis Insert la version modifier, c pas plus lourd ?

Non tu n'as pas à supprimer quand tu modifies, vu que c'est un pointeur que tu stocks dans la liste, tu accède directement à l'élément et donc tu le modifies.

Exemple (Je reprend ma liste d'entiers de tout à l'heure)
On va retrancher 1 à tous les entier de la liste

Code:

---

1 2 3 4 5 6 7 8 9

for(int i = 0; i < Liste->Count; i++) { // Je récupère mon élément à l'index i TInt *Entier = (TInt *)(Liste->Items[i]);   // Je retranche 1 à sa valeur Entier->Valeur = (Entier->Valeur - 1); }

---

Et voilà ma liste est modifié, et je n'ai pas enlever mon Entier de ma liste et remis l'entier modifié, cela se fait directement

ah oui, javais oublier sa, merci beaucoup

jai réduit le nombre de tableau de pal mal

il m'en reste plus que deux maintenant, celui ou je stoke ce ke je lit et celui ou je stoke ce que je dois ecrire, suis en train deseyer danaliser pour les remplacer par des list ausi, mais sa entrenne pas mal de modif, vue que javais fais pas mal de fonction qui travaillais avec un tableau ..

C'est sur, j'ai fait exactement la même chose que toi au début, mais tu verras que par la suite, les listes sont quand même plus facile a manipuler.

voila jai plus aucun tableau , a par celui que le TTcpClient et resoi mais la il n'y a pas le choix.


Donc pour les valeur que je lie jai créé une clase TCoupleInt qui contien 2 int.

Par exemple pour les valeurs que je lit et que j'ecrie Valeur1 contien La num du MW, Valeur2 contient la valeur du MW.

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

class TCoupleInt { public : int Valeur1; int Valeur2;   TCoupleInt() { this->Valeur1 = 0; this->Valeur2 = 0; } };

---

le tous stocker dans une TList

voila
Derniere question pour rendre la memoire il faut juste faire delete List; ou delete chaque Items de la list séperement vue que j'ai fais un new pour chaque?

Salut !

Séparément puis :

Code:

---

1 2 3

Liste->Clear(); delete Liste;

---

A plus !

ok merci
donc jai fais comme sa :

Code:

---

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

while(iCpt < ListValeurEcriture->Count ) { delete ListValeurEcriture->Items[iCpt]; iCpt++; } ListValeurEcriture->Clear(); delete ListValeurEcriture; iCpt = 0;   while(iCpt < ListValeurLecture->Count ) { delete ListValeurLecture->Items[iCpt]; iCpt++; } ListValeurLecture->Clear(); delete ListValeurLecture; iCpt = 0;

---

je fais comme sa pour chaque list

c'est bien ou il y a plus simple ?

Pour la destruction moi je fais comme ca

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

for(int i = (ListeValeurEcriture->Count - 1); i >= 0; i--) { TCoupleInt *CoupleInt = (TCoupleInt *)(ListeValeurEcriture->Items[i]);   delete CoupleInt; CoupleInt = NULL; }   ListeValeurEcriture->Clear(); delete ListeValeurEcriture; ListeValeurEcriture = NULL;   for(int i = (ListeValeurLecture->Count - 1); i >= 0; i--) { TCoupleInt *CoupleInt = (TCoupleInt *)(ListeValeurLecture->Items[i]);   delete CoupleInt; CoupleInt = NULL; }   ListeValeurLecture->Clear(); delete ListeValeurLecture; ListeValeurLecture = NULL;

---

tu peux utiliser un for puisque tu connais le nombre d'éléments contenu dans ta liste.

bon, ben jai toujours des erreur d'accé memoire, ou de list out of band, malgré le fait que je sois sur que j'accede jamais a des valeur eroné ...

Est ce que c'est possible que sa vienne du fait que je trace trois série dans un TChart dans un thread ?
Parceque c'est quand je trace cette courbe que sa le fait le plus souvent

voila le code mais comme dis je n'en suis pas sur, quand jai l'erreur il ne me montre pas ou sa c'est produit il m'ouvre juste la fenetre avec les ligne assebleur mais je ne sais pas ou sa se produit :(

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

AnsiString asTmp = "";   Form1->bPremPression = true; Sleep(Form1->iTpsAttente); while(Form1->bFinThread) { try { while(AccesDataRead(4251,0) && Form1->bFinThread) { Form1->TimerSecu->Enabled = true; Form1->iPF1 = 1; if(Form1->bPremPression == true) { asTmp = TimeToStr(Now()) + "\r" + DateToStr(Now()); Form27->Series1->AddXY(Now(), AccesDataRead(4110), asTmp, clRed); Form27->Series2->AddXY(Now(), AccesDataRead(4111), asTmp, clBlue); Form27->Series3->AddXY(Now(), AccesDataRead(4112), asTmp, clYellow); Form1->bPremPression = false; } else { Form27->Series1->AddXY(Now(), AccesDataRead(4110), TimeToStr(Now()), clRed); Form27->Series2->AddXY(Now(), AccesDataRead(4111), TimeToStr(Now()), clBlue); Form27->Series3->AddXY(Now(), AccesDataRead(4112), TimeToStr(Now()), clYellow); } Sleep(Form1->iEchantillonage); } Form27->LabelTrace->Visible = false; Form1->TimerSecu->Enabled = false; Form1->iTimerSecu = 0; Form1->bPremPression = true; Sleep(Form1->iTpsSleepThreadSecond); Form1->iPF1 = 0; } catch(...) { Form1->iListAla->Insert(0, "ERREUR PENDANT LE THREAD PF1"); Form1->iListAla->SaveToFile(Form1->RepAppli+"ListAlarme.txt"); ShowMessage("PF1"); } } }

---

javais meme eseyer de metre un try catch mais il ne rentre pas dedans :s

Je ne sais plus si tu peux faire afficher des ShowMessage dans un Thread car tu pourrais en mettre à certains endroit pour pouvoir repérer à quelle ligne ca plante.

Sinon tu pourrais juste pour voir te créé une sort de Log qui te note ce qu'il fait, tu verrais ainsi ou cela s'arrête.

Je l'avais fait moi aussi pour identifier une erreur que je ne trouvais pas et cela m'a bien aider. Bien sur, cela va ralentir ton appli mais c'est juste pour tester, après tu peux l'enlever.

ce qui m'enerve c que c'est trop aleatoire pour etre localiser précisement :s

Et le pire c'est que la plupart du temps sa ne plante pas l'appli c'est juste un showmessage avec logo ereur a coté, clike sur ok et tous continue comme si de rien été, bien que parfois apres la form de la courbe ne souvre plus, ou des thread s'arrete ... parfois meme juste en cliquan sur un bouton donc le code est Form->Show sa me met une erreur d'accée memoire


GRRRRRR je desespere, en plus sur certain pc sa le fais moin que sur d'autre

Le point positif est que pour le moment grace a vous jai pu rendre le prog plus propre a laide des class et des list

Ce n'est pas que c'est pas propre avec des tableaux mais c'est beaucoup plus chiant à gérer car il faut allouer l'espace alors qu'avec les listes, il s'alloue à la création de l'objet.

Pour tes erreurs, il faurais arriver à identifier précisément la ligne qui plante, et c'est là que c'est difficile.

ben, moi perso ce que je comprend pas c'est que ces lignes sont traité tres regulierement toute les 100ms envirron mais parfois pendant plusieur heure jai aucune erreur ... dans ma ptite tete si c une ligne qui provoque ce défaut il devrais apparaitre a chaque fois non ?


pour l'allocation des Tlist ben faut kand meme alouer a chaque fois les class lol donc le resultat est le meme non ? :p mais je préfére quand meme avec les TList c'est plus pro :p

Oui avec les liste, tu alloues un objet de la classe, mais c'est beaucoup plus simple de rajouter des éléments que dans un tableau, car pour ton tableau, il faut allouer de l'espace supplémentaire pour ton tableau, alors que pour la liste non, tu crée ton objet et tu l'ajoute dans la liste, Pour moi cela semble plus simple.

Sinon pour ton erreur, je ne vois pas comment faire, sans savoir quelle ligne plante. Si quelqu'un à une idée

je pense avoir résolue mon prob

En fait jai 9 thread qui accéde au donnée.
Chaque thread est dans une boucle infini avec un sleep a la fin de la boucle.

Javais ce sleep a 10 ms.
La jai paser ce sleep a 100ms, et jai plus aucun message du week end ... vous pensez que sa peut venir de la ?

Je ne sais pas du tout, cela peut dépendre de la façon dont tes Thread accèdent au données. Peut être que tes thread accèdaient au même élément en même temps. L'accès en lecture vérouille l'accès il me semble mais je suis pas sur

Nan c'est pas possible qu'il accédais au meme élement, a la meme List oui, mais plusieur thread n'accede pas au meme items de la list. Chaque Thread a ces items auquel il accéde de maniere reguliere pour metre a jour l'affichage des differente vue, tracer des courbes, imprimer les courbes les exporter etc

Bon a priori cela fonctionne avec un sleep de 100 ms au lieu de 10ms, je touche du bois pour que sa soit sa.
De toute maniére le thread de communication avec l'automate mes environ 200ms a rafraichir les 450 mots, donc tous ce qui mimporte c'est de rester en desous des 200ms :)

Voili voilou, je vous tien au courant