Discussion :

[Jero13]

Bonjour à tous,

Voilà mon problème : j'ai une fonction qui parcours un fichier et qui stocke au fur et à mesure les caractères dans un tableau en indiquant le nombre de fois que le caractère est rencontré dans le fichier et retourne à la fin le contenu de ce tableau. Cette fonction se trouve dans un fichier test.c.

Je veux faire appel à cette fonction à partir du fichier main.c de mon application.

Voici le code du fichier test.c

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
 
#include <stdio.h>
#include <stdlib.h>
 
#include "test.h"
 
int freq_carac(char *nom_fichier) {
	// Déclaration des variables
	int i;
	char car_ascii;
	int tab_carac[127];
	FILE *fp;
 
	// Contrôle de l'existance du fichier txt
	if ((fp = fopen(nom_fichier, "r")) == NULL) {
		//fprintf("Le fichier n'existe pas \n");
		exit(1);
	}
	else
	{
		// Si le fichier existe, ouverture du fichier en mode lecture
		fp = fopen(nom_fichier, "r");
 
		// Initialisation du tableau stockant les caractères
		for (i=31; i < 127; i++)
			tab_carac[i] = 0;
 
              	while(1)
             	{
			// Récupération du code ascii du caractère lu
                	car_ascii = fgetc(fp);
 
			printf("%d", car_ascii);
 
			// Test si on est à la fin du fichier
                	if(feof(fp))
                 	{
                    		break;
                	}
 
			// Stockage du caractère dans le tableau
             		if(car_ascii > 31 && car_ascii < 127)
                		tab_carac[car_ascii]++;
		}
 
		// Erreur à corriger
		return tab_carac;
 
		// Fermeture du fichier
		fclose(fp);
        }
 }

Dois-je utiliser des pointeurs pour accèder au contenu de mon tableau ou pas ? si oui, comment dois-je m'y prendre ? merci de votre aide

[Skyounet]

Donc ta fonction retourne un tableau de int d'ou le prototype
int* freq_carac(char *nom_fichier)

Par contre si tu retourne tab[127] tu retourne l'adresse d'une variable local
Donc il faut y aller a coup de malloc et free
En gros

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
int *test(void) {
    int *tab = malloc(10*sizeof(int)):
    if(tab == NULL) exit(-1);
    tab[0]=0;
    return tab;
}
 
int main(void) {
    int *t=test();
    printf("%d\n", t[0])):
    free(t);
    return 0;
}

[AxldenieD]

je vais tenter de répondre en esperant ne pas dire de bêtises :p

Quand tu déclare :

Code :

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

1
2
 
int tab_carac[127];

tu as déjà un pointeur, quand tu déclare un tableau comme ça en fait tu déclare un pointeur mais c'est implicite.

tu aurai tout aussi bien pu faire :

Code :

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

1
2
3
 
int *tab_carac=NULL;
tab_carac=(int*)malloc(sizeof(int)*127);

ça aurai été la même chose,

donc pour renvoyer ce pointeur il faut changer le prototype de ta fonction par :

Code :

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

1
2
 
int *freq_carac(char *nom_fichier)

et là tu peux le renvoyer avec :

Code :

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

1
2
 
return tab_carac;

à savoir qu'il faut fermer ton fichier avant le return sinon ton code de fermeture de fichier ne sera pas executé

Voila j'espère ne pas avoir répondu de travers car je ne suis pas (encore? :p) un pro du C
Et si c'est le cas promis j'arrête de dire des bêtises et laissera des gens plus "pro" que moi répondre :p

[Skyounet]

Code :

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

1
2
 
int tab_carac[127];

tu as déjà un pointeur, quand tu déclare un tableau comme ça en fait tu déclare un pointeur mais c'est implicite.

Oui mais si il retourne cette adresse elle ne sera plus valable en dehors de la fonction (d'ailleurs warning du compilo si celui ci bien reglé)

tu aurai tout aussi bien pu faire :

Code :

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

1
2
3
 
int *tab_carac=NULL;
tab_carac=(int*)malloc(sizeof(int)*127);

Pas besoin du cast
tab_carac=malloc(sizeof(int)*127);

[Jero13]

Merci pour votre aide, j'ai suivi vos conseils et ça fonctionne. Merci

[AxldenieD]

merci de m'avoir repris

[Emmanuel Delahaye]

je vais tenter de répondre en esperant ne pas dire de bêtises :p

Ahem...

Quand tu déclare :

Code :

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

1
2
 
int tab_carac[127];

tu as déjà un pointeur,

Ah ? Où ça ? Je peux le modifier ?

http://emmanuel-delahaye.developpez.....htm#pointeurs

quand tu déclare un tableau comme ça en fait tu déclare un pointeur mais c'est implicite.

Un pointeur implicite ? C'est fort !

tu aurai tout aussi bien pu faire :

Code :

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

1
2
3
 
int *tab_carac=NULL;
tab_carac=(int*)malloc(sizeof(int)*127);

Une façon compliquée d'écrire
int *tab_carac = malloc (sizeof *tab * 127);

ça aurai été la même chose,

Ben non. La principale différence est que dans un cas, le tableau est crée dans la mémoire automatique et donc detruit dès qu'on quitte la fonction. Dans le deuxième cas, la mémoire est crée dans la mémoire allouée, qui existe jusqu'à appel de free(). C'est très différent.

Maintenant, le fait qu'un pointeur initialisé avec l'adresse d'un tableau puisse s'utiliser avec la notation de type indice p[i] au lieu de la notation pointeur *(p + i) est une facilité offerte par le langage C. C'est tout ce qu'on peut dire.

donc pour renvoyer ce pointeur il faut changer le prototype de ta fonction par :

C'est pas le pointeur qu'on renvoit, c'est sa valeur (et on la 'retourne' !), c'est à dire l'adresse du bloc alloué...

Code :

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

1
2
 
int *freq_carac(char *nom_fichier)

et là tu peux le renvoyer avec :

Code :

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

1
2
 
return tab_carac;

Ok... Revoir la terminologie...

[AxldenieD]

Bon okay, message bien reçu
(/me aimerai bien que sa prof de C lise ce topic :p)