Occurences dans un tableau

**sneakz** · 22/06/2009, 13h22

Bonjour,

Soit 5 fichiers placés sous /tmp, portant les noms suivants : J0, J1, J2, J3, J4, J5
Dans chaque fichier est écrit les valeurs suivantes :
18
12

Voici le code source qui calcule le nombre d'occurences d'un entier saisi par l'utilisateur :

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
 
#include <stdlib.h>
#include <stdio.h>
 
#define SIZE 4
 
int main()
{
	int j, cpt, number1, number2;
	int obs; 
	int tab[4];
	int smax = sizeof tab/ sizeof * tab;
	char filename[3];
	FILE *file; 
 
	for(j = 0; j <= SIZE; j++) 
	{
		sprintf(filename,"/tmp/J%d",j);
		if ((file = fopen ( filename, "r" )) == NULL)
		{
			perror ("Impossible d'ouvrir le fichier.");
		}
		else
		{
			while ( fscanf ( file, "%d %d", &number1, &number2 ) == 2) 
			tab[j] = number2;
			printf("tab[%d] = %d\n", j, tab[j]);
			fclose (file);
		}
	}
	printf("Numéro à compter ? ");
	scanf("%d",&cpt);
 
	/* compteur */
	obs = 0;
	for(j = 0; j <= smax; j++) 
	{
		printf("tab[%d] = %d\n", j, tab[j]);
		if(tab[j] == cpt) 
		{
			obs = obs + 1;
		}
	}
	printf("Total : %d\n", obs);
	return EXIT_SUCCESS;
}

La sortie est la suivante :

tab[0] = 12
tab[1] = 12
tab[2] = 12
tab[3] = 12
tab[4] = 12
Numéro à compter ? 12
tab[0] = 877277040
tab[1] = 0
tab[2] = 12
tab[3] = 12
tab[4] = 12
Total : 3

Pourquoi mon tableau est-il modifié ?

Merci bien

**Trap D** · 22/06/2009, 13h25

Si tu as 5 fichiers il faut faire SIZE = 5

Il faut faire j < SIZE et j < smax car comme tu le vois tu as un tableau de 4 valeurs (int tab[4]) et tu en affiches 5 (de 0 à 4 ça fait 5 !!)

Invité(e) · 22/06/2009, 13h30

Bonjour,

De plus, tu n'a pas assez de place dans filename.

Code :

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

1
2
3
4
/* trois caractères pour filename (soit 2 caractères plus le '\0' final */
char filename[3];
/* Ici, on écrit au moins 8 caractères */
sprintf(filename,"/tmp/J%d",j);

Agrandi filename. Donne lui une grande taille (genre 64 octets) pour être tranquille.