Discussion :

[DrakaSAN]

Bonjour a tous et a toutes

Je suis chargé de programmer un petit soft pour "nettoyer" un fichier texte d informations inutiles.
Jusque la rien de bien compliqué, le problème se pose au moment de l exécution

Le fichier de sortie se retrouve bien avec les bonnes valeurs, cependant, Windows (7 home) ferme de force le programme car la boucle dans laquelle s execute le nettoyage est répétée trop de fois (protection contre les boucles infinies) bien avant la fin de mon fichier d entrée

Y a t il moyen de forcer windows a laisser le programme s executer ou faut il que je fasse autrement?

Merci beaucoup

(Au passage, est ce que dans la ligne
while (feof(in)==0)
j utilise correctement feof(FILE*) ?)

[Neckara]

Bonjour,

Testes-tu aussi la présence d'erreur? (ferror si je ne me trompe pas)
En effet si tu ne gère pas les erreurs, feof renverra toujours false et tu entreras dans une boucle infinie.

[Bktero]

(Au passage, est ce que dans la ligne
while (feof(in)==0)
j utilise correctement feof(FILE*) ?)

Pas vraiment : http://c.developpez.com/faq/index.ph...eneral#ES_feof

[DrakaSAN]

Effectivement je ne testait pas ferror

Je viens de le rajouter derrière mes lectures et écritures de fichiers, avec un perror pour l affichage.

Après test, windows ferme mon programme avant l affichage de perror.

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
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
int main()
{
    //Variables
    FILE* in;
    FILE * out;
    char cache [50]={0};
    int videur=0;
 
    //Recuperer le nom du fichier d entrée
    fscanf(stdin, "%s", cache);
 
    //Vider le tampon
    while ((videur = getchar ()) != '\n' && videur != EOF);
 
    //Ouvrir le fichier d adresse e mail
    in=fopen(cache, "r");
    if (in==NULL)
    {
        fprintf(stdout, "Couldn t open %s", cache);
        return -1;
    }
 
    //Recuperer le nom du fichier de sortie
    fscanf(stdin, "%s", cache);
 
    //Vider le tampon
    while ((videur = getchar ()) != '\n' && videur != EOF);
 
    //Ouvrir le fichier nettoyé
    out=fopen(cache, "w");
    if (out==NULL)
    {
        fprintf(stdout, "Couldn t open %s", cache);
        return -1;
    }
 
//!\/!\/!\/!\/!\/!\/!\/!\/!\/!\/!\/!\/!\/!\/!\/!\/!\/!\
 
    //Nettoyer le fichier
    while (feof(in)==0)
    {
        if (ferror(in))
        {
            perror("in: ");
            return -1;
        }
 
        //Lire le champs
        fscanf(in, "%s", cache);
 
        //Si le champ est une adresse e mail
        if (cache[strlen(cache)-4]=='.'||cache[strlen(cache)-3]=='.')
        {
            //L ecrire dans le fichier de sortie
            fprintf(out, "%s\n", cache);
            if (ferror(out))
            {
                perror("out: ");
                return -1;
            }
        }
    }
 
//!\/!\/!\/!\/!\/!\/!\/!\/!\/!\/!\/!\/!\/!\/!\/!\/!\/!\
 
    fprintf(stdout, "\nend\n");
    return 0;
}

Développé en C sous Windows-CodeBlocks

Sachant que mes fichiers d entrée contiennent plus d 1 millions de lignes avec 5 a 8 champs par lignes, je persiste sur un conflit avec les boucles infinie
(Mon fichiers de sortie fait 280k lignes, avec 1champ par ligne)

EDIT:
Pour détecter la fin du fichier sans feof, il faut donc que j utilise les retours de fscanf?

EDIT2: Je vérifie maintenant via feof(in) && rets!=EOF avec rets=fscanf
Toujours les mêmes problèmes

[Bktero]

Ta boucle ne reprend pas vraiment la structure proposée dans la FAQ. En fait, tu dois faire un scanf et si celui-ci échoue, déterminer ce qu'il s'est passé. Scanf peut rater parce que qu'il ne rencontre pas le format attendu (tu cherches une chaine, normalement s'il y a quelque chose à lire, ça passe), ou il n'y a rien à lire ou pas assez (fin du fichier avec feof) ou alors impossible de lire le fichier (on est au delà d'un problème de contenu non adapté, et on regarde ferror).

Est-ce que tu as mis un compteur pour voir le nombre d'itération qu'il fait ? Savoir si c'est toujours la même chose, quelque soit le fichier.

[Médinoc]

Qu'est-ce que c'est que cette histoire de protection contre les boucles infinies? Je ne trouve pas ça sur MSDN.

Par contre, je vois un gros potentiel de débordement de tampon si n'importe quelle ligne du fichier fait plus de 49 caractères. Et si ça arrive, ça peut écraser les pointeurs in et out, ce qui ferait planter les fonctions qui les utilisent...

[Mat.M]

je suis entièrement d'accord avec Medinoc , la probabilité de dépasser ton char cache[50] est fortement probable ce qui te donne un plantage...