Discussion :

[Premium]

Bonjour,

j'ai écrit un programme permettant de copier un fichier dans un autre.
Le soucis que j'ai, c'est que la copie n'a pas lieu.

Voici mon code :

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
#include <stdio.h>
#include <stdlib.h>
 
int copy(FILE *fdi, FILE *fdo)
{
  char buf[BUFSIZ];
  unsigned int lus;
  lus = fwrite(buf, sizeof(char), BUFSIZ, fdi);
  if(fread(buf, lus, 1, fdo) != lus)
    return 1;
  return 0;
}
 
int main(int argc, char *argv[])
{
  FILE *src, *dst;
  if(argc != 3)
    {
      fprintf(stderr,"Usage : test src dest\n");
      return EXIT_FAILURE;
    }
  if((src = fopen(argv[1],"r")) == NULL)
    {
      fprintf(stderr,"erreur ouverture du fichier\n");
      return EXIT_FAILURE;
    }
  if((dst = fopen(argv[2],"w")) == NULL)
    {
      fclose(src);
      fprintf(stderr,"erreur ouverture du fichier\n");
      return EXIT_FAILURE;
    }
  if(copy(src, dst) == 1)
    {
      fclose(src);
      fclose(dst);
      return EXIT_FAILURE;
    }
  fclose(src);
  fclose(dst);
  return EXIT_SUCCESS;
}

Si quelqu'un pouvait m'indiquer ce qui cloche ?

Merci

[Emmanuel Delahaye]

j'ai écrit un programme permettant de copier un fichier dans un autre.
Le soucis que j'ai, c'est que la copie n'a pas lieu.

Voici mon code :

Code :

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

1
2
 lus = fwrite(buf, sizeof(char), BUFSIZ, fdi);
  if(fread(buf, lus, 1, fdo) != lus)

C'est sûr que si tu inverses fread() et fwrite(), ça va pas le faire... Pour s'en rendre compte, il suffisait d'afficher la valeur de 'lus'. Il faut apprendre à déboguer son code. Une erreur si simple ne mérite peut être pas un post sur un forum, surtout de la part d'un "membre éprouvé"... Un archi-débutant, OK...

D'autre part, le code est bien sûr incomplet (il ne sait pas traiter les fichiers > BUFSIZ octets)

Et je rappelle que sizeof(char) vaut 1 par définition...

[Pouic]

surtout de la part d'un "membre éprouvé"... Un archi-débutant, OK...

Tu sais bien que le "grade" se mesure au nombre de post, pas à leur valeur...

[mehdi.berra]

Je crois qu'il te serait beaucoup plus facile pour copier des 2fichiers d'utiliser une simple boucle for ou tu initialisera les 2 pointeurs qui seront bien sur tous les deux pointer sur les debuts de fichiers

A mon humble avis ,c'est beaucoup plus simple et rapide.

[Trap D]

Je crois qu'il te serait beaucoup plus facile pour copier des 2fichiers d'utiliser une simple boucle for ou tu initialisera les 2 pointeurs qui seront bien sur tous les deux pointer sur les debuts de fichiers

A mon humble avis ,c'est beaucoup plus simple et rapide.

Ça veut dire quoi exactement ce que tu écris

[mehdi.berra]

Si tu comprend pas ce que je dis ,je ne peux malheuresement pour toi pas te faire de dessin

Apres toi c'est toi l'expert confirmé pas moi

Allez concentre toi et lis dououououcement ,mot apres mot,aaaaallez tu va y arriver,voioioioioioilaaaaa

[zooro]

Ça veut dire quoi exactement ce que tu écris

J'allais poser la même question, mais tu m'as devancé

[mehdi.berra]

Le mec veut copier un fichier dans un autre

Ce que je propose est simple:
une boucle for qui aura comme parametre les 2pointeurs,un pointeurs du 1er fichiers plein et l autre du fichier vide

Le bloc de la boucle for contiendra un strcpy et il faudra mettre une boucle_while qui controlera la fin de copie du fichier

C'est bon ou pas?????

[Trap D]

Ben par exemple j'ai pas compris s'il fallait initialiser les pointeurs dans la boucle ou à l'extérieur, j'ai une petite idée, mais un débutant, j'en doute un peu

[edit] je vois pas ce que vient faire ici un strcpyn ni une boucle while pour contôler la fin de fichier [/edit]

[zooro]

C'est sûr que si tu inverses fread() et fwrite(), ça va pas le faire...

C'est vrai qu'en remettant tout ça à l'endroit, ça ne pourra qu'améliorer les chose

Par ailleurs, je suis tombé là-dessus en cherchant autre chose tout à l'heure : FAQ C - Copier un fichier