Discussion :

[virtuadrack]

Bonjour,

Je voudrais creer un programme similaire a la commande "grep" de unix que je voudrais appeller "newgrep" ou :

Code :

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

newgrep trucachercher fichier.c

Imprimera tout les resultats dans la console et comptera les lignes trouvees.

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
 
# include <stdio.h>
 
int main ( int argc, char * argv [ ] )
{
  FILE * fp;
  char buffer [ 81 ];
 
  if(argc != 3){
    printf("Usage : newgrep searchstr searchfile [-o outputfile]\n");
   }
 
  else {
    if ((fp = fopen(argv[2],"r")) == NULL) {
      printf ("\n Cannot open file specified.\n");
      exit(1);
     }
 
    while ( fgets ( buffer, 81, fp ) ) {
      fputs ( buffer, stdout );
    }
  }
  return 0;
}

Pour l'instant le code ne fait que lire un fichier et imprimer le resultat dans la console. Je ne sais pas comment modifier :

Code :

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

1
2
3
4
 
    while ( fgets ( buffer, 81, fp ) ) {
      fputs ( buffer, stdout );
    }

De telle maniere a ce que le program interprete argv[1](trucachercher) en tant que chaine de caractere a chercher dans le fichier specifiee...

[Gruik]

Salut,

Premierement faut savoir ce que tu veux faire exactement, car grep ne permet pas de chercher une chaine dans un fichier, ça permet de faire des recherches de chaines définies par des expressions regulieres (plusieurs syntaxes sont d'ailleurs proposées) dans un ou plusieurs fichiers ou sur l'entrée standard.
C'est comme printf, ça sert pas à afficher une chaine, ça sert à afficher quelquechose suivant une chaine de format.

Admettons que tu veuilles afficher les lignes où apparait exactement la chaine recherchée, tu pourrais utiliser strstr pour rechercher une chaine dans un autre.
Un simple fgets ne suffira pas car tu pourras pas faire une recherche fiable sur des fichiers contenant des lignes de plus de sizeof buffer caractères. Un moyen serait d'utiliser realloc pour allouer et reallouer le buffer d'entrée (çe ne serait plus un tableau de char et il faudrait maintenir une variable contenant la taille de ce buffer).

[virtuadrack]

Salut,

Premierement faut savoir ce que tu veux faire exactement, car grep ne permet pas de chercher une chaine dans un fichier, ça permet de faire des recherches de chaines définies par des expressions regulieres (plusieurs syntaxes sont d'ailleurs proposées) dans un ou plusieurs fichiers ou sur l'entrée standard.
C'est comme printf, ça sert pas à afficher une chaine, ça sert à afficher quelquechose suivant une chaine de format.

Admettons que tu veuilles afficher les lignes où apparait exactement la chaine recherchée, tu pourrais utiliser strstr pour rechercher une chaine dans un autre.
Un simple fgets ne suffira pas car tu pourras pas faire une recherche fiable sur des fichiers contenant des lignes de plus de sizeof buffer caractères. Un moyen serait d'utiliser realloc pour allouer et reallouer le buffer d'entrée (çe ne serait plus un tableau de char et il faudrait maintenir une variable contenant la taille de ce buffer).

C'est ce que je veux faire, mais pas aussi sophistiquee que "grep" d'Unix.
A ce que j'ai compris:
Il faut donner une variable au buffer de facon a ce qu'on lui realloue une memoire suffisante apres chaque recherche.

Code :

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

1
2
3
4
5
6
7
char buffer[search];
 
buffer = malloc(81);
 
strcpy(buffer, search);
 
buffer = realloc(buffer, sizeof(search));

La lecture se fera donc :

Code :

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

1
2
3
4
5
6
 
    while ( fgets ( buffer, 81, fp ) ) {
      strstr( buffer, argv[1]);   // trouver argv[1] dans buffer (ce qui a ete lu)
      fputs ( buffer, stdout );   // imprimer le nouveau contenu de buffer ??
 
    }

Ou est l'erreur ?

[Emmanuel Delahaye]

Code :

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

1
2
3
4
5
6
 
    while ( fgets ( buffer, 81, fp ) ) {
      strstr( buffer, argv[1]);   // trouver argv[1] dans buffer (ce qui a ete lu)
      fputs ( buffer, stdout );   // imprimer le nouveau contenu de buffer ??
 
    }

Ou est l'erreur ?

Il faut lire la doc de strstr() et s'intéresser à son retour...

Ton mécanisme de réallocation est plus que douteux... Le tableau ne doit jamais déborder.

[virtuadrack]

Ajouter un if a regle le probleme. En principe avec ca :

Code :

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

1
2
3
4
5
6
 
    while ( fgets ( buffer, sizeof(buffer), fp ) ) {
      if(strstr( buffer, argv[1])){
       fputs ( buffer, stdout );}
 
    }

Le resultat est bon, toute les lignes recherche sont affichees. Mais est ce que c'est efficace ? (efficient)

Comment faire pour afficher les resultats en majuscules ? J'ai essaye avec toupper() de cette maniere:

Code :

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

fputs(toupper(buffer), stdout);

Mais ca na pas l'air de plaire au compiler.

Autre question, si je veux compter les lignes de resutats et de chaine caracteres trouvees, je procede comment ?

Merci

PS: pour la reallocation, j'ai prefere donner a 'buffer' 250, les lignes ne depasseront pas cette taille.

[souviron34]

LA PROGRAMMATION N'EST PAS DE METTRE DES INSTRUCTIONS AU HASARD ET D'ESPERER QUE CA MARCHE !!!!


Lis la doc de toupper avant de demander de l'aide.... Et tu verras que tu fais n'importe quoi..

D'autre part, ton test de fgets n'est pa la meilleur manière d'écrire, mais plutôt

Code :

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

while ( fgets(...) != NULL )

[Emmanuel Delahaye]

Ajouter un if a regle le probleme. En principe avec ca :

Code :

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

1
2
3
4
5
6
 
    while ( fgets ( buffer, sizeof(buffer), fp ) ) {
      if(strstr( buffer, argv[1])){
       fputs ( buffer, stdout );}
 
    }

Le resultat est bon, toute les lignes recherche sont affichees. Mais est ce que c'est efficace ? (efficient)

C'est pas très efficace. Un automate traitant les caractères un par un devrait êre meilleur, mais c'est un peu plus complexe. Il y a aussi le problèmes des mots 'coupés' (sur 2 lignes...)

Comment faire pour afficher les resultats en majuscules ? J'ai essaye avec toupper() de cette maniere:

Code :

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

fputs(toupper(buffer), stdout);

Mais ca na pas l'air de plaire au compiler.

toupper(), comme l'indique la doc que tu aurais du lire au lieu de faire n'importe quoi, ne traite qu'un caractère, pas une chaine. Je te laisse écrire la fonction de traitement de chaine.

Autre question, si je veux compter les lignes de resutats et de chaine caracteres trouvees, je procede comment ?

Tu définis un compteur initialisé à 0, puis tu l'incrémentes de 1 à chaque fois que la condition correspondante est vérifiée. (C'était trop dur ! Je me suis pété 3 neurones en cherchant ça !)