Discussion :

[aperdemak]

Bonjour, je cherche à écrire un programme qui copierait un fichier assez lourd. Si j'utilise un seul char* le programme crache donc j'ai fait une boulce, mais le problème c'est que les fichier font pas exactement la même taille au final et je comprend pas pourquoi.(même si le mp3 marche)
Si quelqu'un pouvait m'aider j'en serais reconnaissant. Merci d'avance.

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
#include <fstream>
#include <iostream>
#include <windows.h>
 
using namespace std;
 
int main(int argc, char** argv)
{
  fstream fichier("a.mp3", ios::in |ios::out | ios::binary);
  fstream fichier2("b.mp3", ios::in |ios::out | ios::binary | ios::trunc);
  if(fichier)
    {
      fichier.seekg(0, ios::end);
      int size = fichier.tellg();
      fichier.seekg(0, ios::beg); 
      int i= size/5000;
      int r= size%5000;
      int j=0;
      while(j<=i)
	{
	  char contenu[5000];
	  fichier.read(contenu, 5000);
	  fichier2.write(contenu, 5000); 
	  j++;
	}
      if(r!=0)
	{
	   char contenu[r];
	  fichier.read(contenu, r);
	  fichier2.write(contenu, r); 
	}
      fichier.close();
      fichier2.close();
    }
  else
    cerr << "Error: " << strerror(errno);
 
  Sleep(5000);
  return 0;
}

[Bousk]

Salut,

char contenu[r]; n'est pas une syntaxe valide en C++ mais un VLA C.
while(j<=i) me parait faux et la boucle devrait être sur < uniquement. Je suppose que le fichier destination est plus important que l'original donc ?
Une boucle for serait aussi plus maline, ne serait ce que pour ne pas avoir un j inutile qui traîne dans la portée par la suite.

[aperdemak]

mon dieu que je suis bète merci bien

[koala01]

Salut,

Déjà, tu devrais abandonner l'idée d'utiliser un char * pour la représentation de tes données. Si tu veux manipuler un tableau de taille fixe (connue à la compilation), utilise std::array, et si tu veux un tableau de taille dynamique (connue à l'exécution), utilise std::vector.

En effet, tu as déjà un problème au niveau de char contenu[r]; : comme r n'est connu qu'à l'exécution, tu essayes d'utiliser une technique qui n'existe qu'en C (et qui est refusée en C++) appelée les VLA. D'ailleurs, il me semble que la norme C est revenue sur ses pas en ce qui concerne cette fonctionnalité, et que la dernière norme ne les accepte plus...

Mais, pour répondre à ta question je dirai simplement "vérifie ta logique", entre autres, au niveau de ta condition d'arrêt pour ta boucle while... Je n'en dirai pas plus, histoire de te laisser chercher un peu, mais, essaye de suivre ta logique pour, mettons, size() = 15255, et tu trouveras surement ta solutions

[aperdemak]

merci bien pour ces précisions, je connaissais pas vla.

[Bousk]

C'est encore expérimental, mais copier un fichier serait mieux via un appel à copy_file, ou via les API spécifiques à chaque système, ou Boost qui les encapsule etc.