Probleme avec la fonction rename()

Version imprimable

08/10/2005, 14h28
TheZenZen

[Resolu]Probleme avec la fonction rename()
Bonjour
Voila j'ai un petit probleme sur un petit bout de code.
Je fais une copie d'un fichier vers un autre et je veux supprimer l'ancienne version du fichier.
Seulement la fonction ne fait pas son office et ne renomme pas le fichier comme je voudrais. Je stocke les noms dans des chaines de caractères. J'ai essayé en utilisant des noms entre double quote, mais j'ai le meme résultat....
Code:

1 2 3 4 char tmp[10], name[20]; debug = rename (tmp, name);
L'affichage de debug m'indique -1.

Help please. Je précise je suis sur mac. Merci
:)
08/10/2005, 14h37
Emmanuel Delahaye

Re: Probleme avec la fonction rename()
Citation:
Envoyé par TheZenZen

Voila j'ai un petit probleme sur un petit bout de code.
Je fais une copie d'un fichier vers un autre et je veux supprimer l'ancienne version du fichier.
Seulement la fonction ne fait pas son office et ne renomme pas le fichier comme je voudrais. Je stocke les noms dans des chaines de caractères. J'ai essayé en utilisant des noms entre double quote, mais j'ai le meme résultat....

Code:

1 2 3 4 char tmp[10], name[20]; debug = rename (tmp, name);

L'affichage de debug m'indique -1.
Ben si il n'y a rien de défini dans tmp et name, ça va pas le faire...

Montre le code complet mais réduit au minimum compilable qui montre le défaut.

Bah j'ai bien évidement des données dans le tableau... Je ne pense pas que le probleme vient de la...
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 int fonction(char *name, int numero) { char tmp[10] = "temp"; char ligne[1000]; char * fin; int debug; FILE *input; FILE *output; output = fopen(name, "r"); input = fopen(tmp, "a+"); while(fin != NULL) { fin = fgets(ligne, 1000, output); fprintf(input, "%s", ligne); } fclose(output); remove (name); debug = rename (tmp, name); printf("%d\n", debug); return 0; }
Voila ce que ca donne en simplifier. Tout marche sauf la fonction rename...
:roll:

Citation:

Envoyé par TheZenZen

Voila ce que ca donne en simplifier. Tout marche sauf la fonction rename...
:roll:

Quels sont les mots que tu ne comprends pas dans

Citation:

Montre le code complet mais réduit au minimum compilable qui montre le défaut.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
Compiling: main.c
main.c: In function `fonction':
main.c:8: error: `FILE' undeclared (first use in this function)
main.c:8: error: (Each undeclared identifier is reported only once
main.c:8: error: for each function it appears in.)
main.c:8: error: `input' undeclared (first use in this function)
main.c:9: error: `output' undeclared (first use in this function)
main.c:11: warning: implicit declaration of function `fopen'
main.c:14: error: `NULL' undeclared (first use in this function)
main.c:16: warning: implicit declaration of function `fgets'
main.c:16: warning: assignment makes pointer from integer without a cast
main.c:17: warning: implicit declaration of function `fprintf'
main.c:19: warning: implicit declaration of function `fclose'
main.c:21: warning: implicit declaration of function `remove'
main.c:22: warning: implicit declaration of function `rename'
main.c:23: warning: implicit declaration of function `printf'
main.c: At top level:
main.c:1: warning: unused parameter 'numero'

A première vue :

pas de test après ouverture des fichiers
'name' devrait être l'entrée et 'tmp' la sortie... De meilleurs noms évitent de se tromper.
Pourquoi "a+" ? Qu'est-ce qui ne va pas avec "a" ?
Tous les fichiers ouverts ne sont pas fermés.
Le code retour de remove() n'est pas testé.

remove() d'un fichier non fermé invoque un comportement indéfini...

Quelque chose comme ça :

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
63
64
65
66
67
68
69
 
#include <stdio.h>
#include <assert.h>
 
static int fonction(char const *f_in, int numero)
{
    int err = 0;
    static char const f_out[] = "temp";
 
    if (f_in != NULL)
    {
        FILE *input = fopen(f_in, "r");
 
        if (input != NULL)
        {
            FILE *output= fopen(f_out, "w");
 
            if (output!=NULL)
            {
                unsigned long line_count=0;
                char ligne[1000];
 
                while(fgets(ligne, sizeof ligne, input) != NULL)
                {
                    line_count++;
                    fprintf(output, "%s", ligne);
                }
                fclose(output), output=NULL;
                printf("%lu line%s copied\n", line_count, line_count>1?"s":"");
            }
            else
            {
                err=1;
                perror(f_out);
            }
            fclose(input), input=NULL;
 
            assert(output==NULL);
            assert(input==NULL);
 
            if (!err)
            {
                int debug = remove(f_in)
                            ;
                printf("remove: %d\n", debug);
 
                debug = rename (f_out, f_in);
                printf("rename: %d\n", debug);
            }
        }
        else
        {
            err=1;
            perror(f_in);
        }
    }
    else
    {
        err=1;
    }
    return err;
}
 
int main (void)
{
    fonction("test.txt", 10);
 
    return 0;
}

08/10/2005, 15h12
TheZenZen

Désolé je n'avais pas compris que tu allais le compiler... :oops:
Néanmoins

Citation:

pas de test après ouverture des fichiers

Je n'en fais aucun, je vais essayer d'en faire, meme si je n'ai jamais eu d'erreur d'ouverture....

Citation:

'name' devrait être l'entrée et 'tmp' la sortie... De meilleurs nom évitent de se tromper.

J'utilise input et ouput, mais pour l'exemple j'avais modifier le code...

Citation:

Pourquoi "a+" ? Qu'est-ce qui ne va pas avec "a" ?

Merci je n'avais pas vu.

Citation:

Tous les fichiers ouverts ne sont pas fermés.

Mauvaise recopie, désolé.

Citation:

Le code retour de remove() n'est pas testé.

Je n'ai aucun probleme avec cette fonction, dans mon code elle me renvoie bien '0' donc no soucy la dessus....
08/10/2005, 15h39
Emmanuel Delahaye

Citation:

Envoyé par TheZenZen

Citation:

pas de test après ouverture des fichiers

Je n'en fais aucun, je vais essayer d'en faire, meme si je n'ai jamais eu d'erreur d'ouverture....

Comment tu le sais si tu ne fais pas le test ?

Citation:

Citation:

'name' devrait être l'entrée et 'tmp' la sortie... De meilleurs nom évitent de se tromper.

J'utilise input et ouput, mais pour l'exemple j'avais modifier le code...

Visiblement input et output sont inversés...

Citation:

Citation:

Le code retour de remove() n'est pas testé.

Je n'ai aucun probleme avec cette fonction, dans mon code elle me renvoie bien '0' donc no soucy la dessus....

Peut être...
08/10/2005, 15h59
TheZenZen

Bon je viens de réussir à obtenir ce que je voulais.
Ce que j'ai fait c'est recopier la chaine de caractère que je passais en argument dans une autre chaine, et la ca marche....
Je ne comprend pas puisque j'ouvre mes fichiers en utilisant la chaine de caractère que je passe en argument...

Bizarre :!: