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.

[hatra]

@Winjerome

j'ai pas mis de * à file parce que c'est ce que me recommandais matafan

merci de ton conseil pour flclose .. t'inquiete j'avais prevue de le faire ...

Puis apres pour "\0", j'ai bien verifier en pas à pasil comprend bien que j'ai une chaine de caractere avec le code asciii du '\0'

[Invité]

Il t'a conseillé de l'enlever ici :

Code :

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

read_file(c_parametre,*fichier);

Mais en aucun cas dans le prototype de la fonction.

Pour le reste je t'invite à consulter cette discussion. ('\0', un seul caractère est à différencier de "\0", chaine de caractères).

[hatra]

Merci, jsuis vraiment con ... l'avoir changer dans le prototype Meric bcp... je ne fait que des fautes de debutants ... et sa m'enerve!!

Parcontre je maintient que j'ai raison pour le "\0" parce que l'interpretteur considere que '\' comme un caractere speciaux qui invitera à n (saut de ligne ) ou autre ...
pour qu'il le prend pour un carractere dit normal ... il faut le doubler ....

"C:\\toto\\titi\\BTS_IRIS" => C:\toto\titi\BTS_IRIS

essaie dans ton compilateur : char temp[] = "bonjour" => char temp2[] = "bonjour\0".

Enfin par contre si j'ai tord ... et que d'apres mon experience (j'ai souvent tord) ..., je te devrais des escuses ... j'ai pas la science infuse .... mais j'ai besoin de depenser quelque argument(se forcer à avoir en confiance en moi )

[jouana]

Oui mais la c'est un caractère que tu vérifie pas une chaîne donc par soucis de propreté et de rigueur opte pour les simple quotte

Et les erreurs de débutant sont les erreurs que l'ont fait le plus souvant et qui prenne le plus de temps donc ne t'en veux pas trop, repaire les bien et gagne en automatisme pour les éviter au maximum