Probleme avec la fonction rename()

**TheZenZen** · 08/10/2005, 14h28

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 :

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

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

**Emmanuel Delahaye** · 08/10/2005, 14h37

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 :

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

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.

**TheZenZen** · 08/10/2005, 14h46

Bah j'ai bien évidement des données dans le tableau... Je ne pense pas que le probleme vient de la...

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
 
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...

**Emmanuel Delahaye** · 08/10/2005, 14h59

Envoyé par TheZenZen

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

Quels sont les mots que tu ne comprends pas dans

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

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
 
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 :

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
 
#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;
}

**TheZenZen** · 08/10/2005, 15h12

Désolé je n'avais pas compris que tu allais le compiler...

Néanmoins

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....

'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...

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

Merci je n'avais pas vu.

Tous les fichiers ouverts ne sont pas fermés.

Mauvaise recopie, désolé.

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....

**Emmanuel Delahaye** · 08/10/2005, 15h39

Envoyé par TheZenZen

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 ?

'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...

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...

**TheZenZen** · 08/10/2005, 15h59

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

Probleme avec la fonction rename()

C

Discussions similaires

Partager

Partager