Discussion :

[ickis]

Bonjour.

J'ai ecris un programme fournissant des statistiques a partir de données fournies dans un fichier.

Le prog se compile trés bien et marche, mais en affichant des resultat legerement faux....

Pour calculer les stat, je manipule des tableaux de double, et il semble qu'il y ai des décalages inexpliqués de lignes/colones dans mes tableaux.

par exemple j'ai un tableau tab[1000][21].
Lorsque je donne une valeur a tab[x][21] , la variable tab[x+1][0] se voit automatiquement attribué la meme valeur!

Quand je manipule tab[x][21] on dirait que le compilateur comprend tab[x+1][0]

De plus j'arrive a afficher la valeur tab[x][25] alors que mon tableau n est déclaré que jusqu'a [21].


J'ai déja passé pas mal de temps a relire, a re ecrire des parties du code mais en vin.

C'est ici que j'ai besoin de votre experience, que me conseillez vous de faire? Retaper le code en entier sans me servire de ce que j'ai déja fais? continuer a chercher comme ca?

ou si vous avez une idée d'ou peut venir ce \{@#|\]! de décalage dans les tableaux...

[Steph82]

Slt, faudrait que tu mettes une parti de ton code, car sinon ca va être dur de t'aider.

[ickis]

ah... en fait je crois que j'ai trouvé... la solution la plus logique est que un de mes tableau n'est pas assez grand ou que je me suis trompé de +1 ou -1 dans une boucle, resultat, la variable devant etre stoké en tab[x][21] est stoquée aprés en tab[x+1][0].

reste a trouver le bon tableau...

[Steph82]

Un conseil, fais 2 boucles imbriqués pour afficher les résultats de ton tableau, et affiches les. Cela te permttra de + facilement débugger ton tableau. Tu verras si les résultats sont incohérents, si ca bug par colonne,...
A+

[ickis]

C bon j suis arrivé!
Juste pour te montrer a quoi ca ressemble mon programme:

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
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
 
void use_file()
{
	for(int w=0; w<nbrdestat; w++)
	{
		for(int x=0; x<20; x++)
		{
 
			tab_stat[w][x] = - (tab_read[w][x+8]);
 
			if( tab_read[w][4] <= - (tab_stat[w][x]) )
			{
				tab_stat[w][x] = tab_stat[w][x] + tab_read[w][7];
			}
 
		}
 
		tab_stat[w][21] = tab_read[w][0];
	}
}
 
void comp_tab_rn()
{
 
 
	for (int k=0; k<10; k++)
	{
		tab_new_rn[k][21] = 9+k;
 
		for (int s=0; s<nbrdestat; s++)
		{
			if( tab_stat[s][21] == 9+k )
			{
				tab_new_rn[k][22] ++;
			}
		}
	}
 
	// mise en place des esperences
 
	for (int c=0; c<10; c++)
	{
		for (int s=0; s<nbrdestat; s++)
		{
			if( tab_stat[s][21] == 9+c )
			{
				for ( int v=0; v<20; v++)
				{
					tab_new_rn[c][v] = tab_new_rn[c][v] + tab_stat[s][v];
				}
			}
		}
	}
}

Faire du traitement de statistique c'est faire des boucles et encore des boucles!

[Steph82]

Bon ben si ca marche c bien alors Par contre un conseil, commente bien ton code, car si ce sont des stats, tu vas vite t'embrouiller si tu reprends le code plus tard.

[ickis]

ok.

Sinon a part ca, il est bo mon code non ?

enfin je veux dire pas trop bordélique?

[Steph82]

Ben c un code tout simple, y'a que des imbrications. c'est juste les commentaires qui manques car tu ne fais des traitements qu'a partir des tes indices.

[ickis]

des imbrications? C'est des algorythmes?

Moi c'est ce que j'aime bien faire, ca fait réflechire c cool...

genre ecrire des fonctions de tri, de classement, etc...

C des algo ca non?

[Steph82]

Les imbrication sont les encastrement que tu fais, par exemple:

Code :

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

1
2
3
4
5
6
7
8
9
10
if(...)
{
   if(...)
   {
      if(...)
      {
          ...
      }
   }
}

Ce que tu as fais, ce sont des boucles imbriquées. On ne peut pas parler d'algos à proprement parlé. Va faire un ptit tour ds la rubrique algorithmes, tu verras des exemples plus ou moins simple.

[Elijha]

Salut

Juste une petite remarque !

En C, si tu déclares un tableau de tab[1000][21], les valeurs des index du tableau doivent aller de 0 à 999 et 0 à 20. Sinon il risque d'avoir des dépassement en mémoire.
Exemple :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
int tab[1000][21] ;
void main(void)
  { int i, j ;
 
  // Init du tableau
  for(i=0; i<1000; i++)  // i prend les valeurs 0 à 999
    for(j=0; j<21; j++)  // j prend les valeurs 0 à 20
      tab[i][j] = 0 ;
 
  tab[0][21] = 10 ;  // Dépassement de mémoire (null pointer assigment)
  tab[0][20] = 10 ;  // Ok, on est dedans
  }

C'était juste une petite remarque.

Voila...

[Steph82]

Oui je l'avais pensé aussi, mais comme il avait mis "tab[x+1][0]", je pense qu'il le sait.

[Elijha]

Salut

Petite erreur de ma part ! Dans le code, j'ai mis que tab[0][21] = 10 impliquait un 'null pointer assigment' : faux, il attribut 10 à tab[1][0].
Il pourrait avoir un 'null pointer assigment' seulement dans le cas tab[999][21] = 10. Dans ce cas seul, on tape à l'exterieur du tableau en mémoire.

Je ne pourrais pas affirmer une erreur dans son code, car les déclarations de tableau n'y sont pas, mais on peut extrapoler que dans sa fonction 'void use_file()' la ligne tab_stat[w][21] = tab_read[w][0]; si tab_stat est déclaré

Code :

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

tab_stat[1000][21]

alors il aura une erreur; premièrement dans son algorithme, et deuxièmement un 'null pointer assigment' lorsque ces 2 boucles attendront la fin du tableau (ouf, j'y suis arrivé) :o .
Pour corrigé (s'il y a a corriger) sa déclaration doit être

Code :

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

tab_stat[1000][22] ;

Voila....

[gl]

Salut

Petite erreur de ma part ! Dans le code, j'ai mis que tab[0][21] = 10 impliquait un 'null pointer assigment' : faux, il attribut 10 à tab[1][0].
Il pourrait avoir un 'null pointer assigment' seulement dans le cas tab[999][21] = 10. Dans ce cas seul, on tape à l'exterieur du tableau en mémoire.

La consequence de ce type d'erreur va dependre du compilateur, des options de compilation et du systeme. Tu ne peux pas a priori etre certain de l'erreur que ca va generer.

[Elijha]

Salut

Exacte pour les options de compilations et mille excuses pour le 'null pointer assignment' (je viens de le tester).
Mais il y a des grandes chances que si l'index dépasse la taille total du tableau (dans notre cas 1000x21 cases mémoires), que le programme écrive à un endroit non définit.
Je travail sur micro-contrôleur, et une erreur de ce type est fatal à l'execution du programme (du moins à son bon fonctionnement).

Voila et merci de la précision.