Discussion :

[hatra]

Bonjour,

Ce post ce situe toujours dans le meme contexte que celui ci
http://www.developpez.net/forums/d12...e/#post6973976 c-a-d le coage du grep en c

alors voila.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
#include"my_grep.h"
 
int main(int argc,char** argv)
{
	t_parametre parametre;
 
	parseur_parametre(argc,argv,&parametre); 
	load_file(&parametre); // la probleme ce situe ici !!!
 
	return 0;
}

Code :

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

1
2
3
4
5
6
7
8
9
10
 
int read_file(t_parametre c_parametre,FILE *file) // t_parametre est une   
                                                                       //stucure
{
	char temp[1000];
	while(fgets(temp,1000,file)) // c'est i ici que sa plante ...
	{
		// va yavoir quelaue chose ici ....
	}
}

j'ai oublie de mettre ce code ...

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
 
int load_file(t_parametre *c_parametre)
{
	int i = 0;
	FILE* fichier = NULL;
	BOOL not_found = FALSE;
	// the name file(pattern) cannot to be in argv[0] 
	//c_parametre->str_cmd[2] = "C:\\Dev\\a_geter\\grep\\Debug\\toto.txt";
	for(i=1;c_parametre->str_cmd[i] != "\0";i+=1)
	{
 
		fichier = fopen(c_parametre->str_cmd[i],"r");
		if(!fichier)//if don't open or not found the pattern
		{
			//pattern don't match
			continue;
			//not_found = FALSE;
		}
		else
		{
			// partern founded
			read_file(c_parametre,*fichier);
			not_found = TRUE;
		}
 
	}
	if(not_found == FALSE)
	{
		My_Error();
		exit(1);
	}
	fclose(fichier);
 
	return(0);
}

[matafan]

Il faut passer fichier à read_file(), pas *fichier.
Et read_file() devrait attendre un t_parametre *, pas un t_parametre.

[hatra]

Pourquoi unpasser à read file() un fichier au lieu de *fichier ...?

[jouana]

Pour parametre il envoie bien une adresse avec '&'.

Ton if continue c'est un peu laid.

Faire un if fichier != null c'est un peu plus propre que de mettre un block vide.

Et dans ta boucle met != null sur le retour de fgets c'est un peu plus propre aussi.

[hatra]

heu j'ai remplace *fichier par fichier .... et la sa compile plus ...

Deplus j'ai un cannot convert from FILE* to FILE alors que je ne vois pas ou j'ai pue .... vouloir faire une convertion .

Code :

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

int read_file(t_parametre *c_parametre,FILE);

Code :

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

read_file(c_parametre,fichier);

[Invité]

Bonjour,

Si tu souhaites pouvoir passer un pointeur :

Code :

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

int read_file(t_parametre c_parametre,FILE *file)

Avec son utilisation :

Code :

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

1
2
3
FILE *fichier;
/* ... */
read_file(c_parametre, fichier);

Remarques :

• fclose(fichier); à mettre dans la boucle for, histoire de fermer tous les fichiers ouverts.
• Attention aux lignes
  

  Code :

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

  1
  2
  for(i=1;c_parametre->str_cmd[i] != "\0";i+=1)
  c_parametre->str_cmd[i] = "\0";

  "\0" représente une chaine de caractères. Donc pour assigner et comparer, on utilise strcpy et strcmp. Je t'invite à relire le message de jouana.