Chercher et supprimer

Version imprimable

Bonjour a tous,

J'ai ecit un code qui est sensé copier un txt en enlevant les ligne qui ont plus de 5 fois le même caracteres.
Le fichier de sortie se creer mais rien ne s'écrit de dans je ne sais pas pourquoi.

Voila le code:

Code:

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
53
54
55
56
57
58
59
60
#include <cstdlib>
#include <iostream>
#include <string>
#include <stdio.h>
 
using namespace std;
 
int main(int argc, char *argv[])
{
    //Variable
    FILE *file = NULL;
    FILE *file_clean = NULL;
    char ligne[256];
    char caractere;
    int suppr;
    char nom_clean;
 
    file=fopen(argv[1],"r+");
 
    if (file == NULL)
    {
            printf("[INFO]Echec de l'ouverture de %s\n\n",argv[1]);
            system("PAUSE");
            return 0;
    }
 
 
 
    file_clean=fopen("Dico_cleaned.txt","w");
 
    if (file_clean == NULL)
    {
            printf("[INFO]Echec de la creation de Dico_cleaned\n\n");
    }
 
    while (fgets(ligne,256,file) != NULL)
    {
 
          for ( int i=0;i<26;i++)
          {
              caractere = ligne[i];
              for ( int j=0;j<26;j++)
              {
                   if (caractere==ligne[j])
                   {
                            suppr = suppr+1;              
                   }
              }
 
              if (suppr < 6 )
              {
                   fprintf(file_clean,"%s\n",ligne);
              }
          }
 
    }
 
    system("PAUSE");
    return EXIT_SUCCESS;
}

Ton code :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 while (fgets(ligne,256,file) != NULL) { for ( int i=0;i<26;i++) { caractere = ligne[i]; for ( int j=0;j<26;j++) { if (caractere==ligne[j]) { suppr = suppr+1; } } if (suppr < 6 ) { fprintf(file_clean,"%s\n",ligne); } } }
Si je ne me trompe pas, tu stocke les 256 premiers caractères dans "ligne", et après tu vérifie tu vérifie s'il existe plus que 5 caractères semblables, tu supprimes ses 256 caractères.
voilà les choses que j'aurais vérifié :

* Si dans les 256 premiers caractères, j'ai 5 caractères qui se ressemblent. Dans ce cas c'est normal que tu as un fichier vide.

* Le nombre de caractères pas ligne, pour voir si c'est effectivement 256 ou non.

* Dans ton code, tu ne dépasse pas les 256 premiers caractères, dans à mon avis tu ne parcoure pas tout le texte.

J'espère que ça va t'aider. Bon courage.

24/06/2009, 12h08
Squaks78360

Avec gets en théorie je prend tout les caractere possible jusqu'au \n donc il prend la ligne entiere

puis la boucle while lui fais prendre la ligne d'apres

mes lignes font 26 caractere

je ne vois vraiment pas ce qui cloche
merci quand meme de ton aide
24/06/2009, 12h16
Lucien63

Ta variable suppr n'est jamais initialisée.
ni au départ, ni quand tu change de lettre.
24/06/2009, 12h36
Squaks78360

A oui effectivement sa marche merci beaucoup
24/06/2009, 13h25
Squaks78360

Ha un nouveau probleme: le fichier cleaned contient le fichier normal en 3 ou 4exemplaires et ecrit n'importe comment dans tous les sens
24/06/2009, 13h49
Lucien63

Il faut fermer tes fichiers après utilisation !

Heu je les fermes

j'ai remanier le code :

Code:

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
53
54
55
56
57
58
59
60
61
62
#include <cstdlib>
#include <iostream>
#include <string>
#include <stdio.h>
 
using namespace std;
 
int main(int argc, char *argv[])
{
    //Variable
    FILE *file = NULL;
    FILE *file_clean = NULL;
    char ligne[256];
    int suppr=0;
    char nom_clean;
    //Ouverture du fichier choisi
    file=fopen(argv[1],"r");
 
    if (file == NULL)
    {
            printf("[INFO]Echec de l'ouverture de %s\n\n",argv[1]);
            system("PAUSE");
            return 0;
    }
    //creation du fichier clean
    file_clean=fopen("Dico_cleaned.txt","w+");
 
    if (file_clean == NULL)
    {
            printf("[INFO]Echec de la creation de Dico_cleaned\n\n");
            system("PAUSE");
            return 0;
    }
 
    while (fgets(ligne,256,file) != NULL)
    {
          for ( int i=0;i<26;i++)
          {
              suppr = 0;
 
              for ( int j=0;j<26;j++)
              {
                   if (ligne[i]==ligne[j])
                   {
                            suppr = suppr+1;              
                   }
              }
 
              if (suppr < 6)
              {
                   fprintf(file_clean,"%s",ligne);
                   printf("%s",ligne);   
              }
          }
 
    }
    fclose(file_clean);
    fclose(file);
 
    system("PAUSE");
    return EXIT_SUCCESS;
}

24/06/2009, 13h53
Goten

Ou alors utiliser des structures C++, ici notamment fstream :). Qui t'éviterais bien des désagréments.
24/06/2009, 18h03
fmr130

Je pense que cela provient de l'ouverture du fichier en écriture "w+"

met simplement "w" sa devrait passé car w+ te met en mode mise a jour dc ça parait logique que sa fasse n'imorte quoi ...
24/06/2009, 19h21
3DArchi

Ceci est du code C. Tu trouveras de l'aide plus pertinente dans le forum C.
Si tu souhaites le faire en C++, alors consulte les entrées sur les flux, essaies avec et revient sur le forum C++ si tu n'y arrives pas ;)

C'est bon j'ai fais comme sa

Code:

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
53
54
55
56
57
58
 
int main(int argc, char *argv[2], int Nbre)
{
    FILE *file = NULL;
    FILE *file_clean = NULL;
    char ligne[256];
    char caractere;
    int suppr;
    bool write;
 
    file=fopen(argv[1],"r+");
 
    if (file == NULL)
    {
            printf("[INFO]Echec de l'ouverture de %s\n\n",argv[1]);
            system("PAUSE");
            return 0;
    }
 
 
 
    file_clean=fopen("Dico_cleaned.txt","w");
 
    if (file_clean == NULL)
    {
            printf("[INFO]Echec de la creation de Dico_cleaned\n\n");
    }
 
    while (fgets(ligne,256,file) != NULL)
    {
          write=true;
          for ( int i=0;i<26;i++)
          {
              suppr=0;
              caractere = ligne[i];
              for ( int j=0;j<26;j++)
              {
                   if (caractere==ligne[j])
                   {
                            suppr = suppr+1;
                   }
              }
 
              if (suppr > 6 )
              {
                  write=false;
              }
          }
          if (write)
          {
          fprintf(file_clean,"%s",ligne);
          }    
    }
 
    fclose(file);
    fclose(file_clean);
    return EXIT_SUCCESS;
}

Code:

file=fopen(argv[1],"r+");

Il n'y a aucune raison d'ouvrir le fichier en "r+", le mode "r" convient.

Code:

for ( int j=0;j<26;j++)...

pourquoi partir de 0 ? les caractères d'indice inférieurs à i+1 ont déja été analysés.
Pourquoi 26 (alors que le tableau de stockage fait 256) ? La fin de la chaîne est aisément détectable par la présence du '\0' ou du '\n'

Pourquoi continuer à analyser la ligne si write == false ? On sait déjà qu'elle va être éliminée.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 write=true; for ( int i=0;ligne[i] != '\0' && write;i++) { suppr=0; caractere = ligne[i]; for ( int j = i+1 ; ligne[j] != '\0' && write ; j++) { if (caractere==ligne[j]) { suppr = suppr+1; write = suppr <= 6 ; } } }
(Code non testé)