Erreur incompréhensible une variable int change de valeur sans raison connue

Version imprimable

21/06/2010, 09h30
raphchar

Erreur incompréhensible une variable int change de valeur sans raison connue
Bonjour, j'ai un gros problème:

Fichiersize et FichierFactice sont des int;
is est un inputfilestream
Code:

1 2 3 4 5 cdata= new char[(Fichiersize+FichierFactice)]; data= new unsigned char[(Fichiersize+FichierFactice)]; is.read(cdata,Fichiersize); is.close();
A l'execution, au début du code FichierFactice=3, Fichier size=638988.
A la fin de ce bout de code FichierFactier = 56819744, affiché en rouge par le debuggeur, mais qu'est-ce qui cloche?
A ce stade FichierSize est correct et reste à 638988.

Quelqu'un peut-il m'aider? merci.

PS code debuggé et compilé sous VS2010
21/06/2010, 09h39
Furr

Les valeurs contenues dans FichierFactice changent aléatoirement ? Et sont différentes à chaque exécution ?

Vérifie que tu as bien initialisé la variable à une valeur précise, car un entier non initialisé prend n'importe quel valeur, ce qui peut expliquer le comportement que tu obtiens :P
21/06/2010, 09h45
Invité

Ou sinon un dépassement mémoire, qui écrirait en dehors d'un de tes tableaux et donc potentiellement dans la zone mémoire de ton int. Mais j'avoue que sur ces 4 lignes de code, je vois pas ce qui pourrait le provoquer.
21/06/2010, 09h47
fregolo52

Salut,

Ca sent l'écrasement mémoire !! Mais ça n'a pas l'air d'être ton code qui est en cause.
Quand tu exécutes pas à pas, regarde ce qu'il se passe en zone mémoire aux alentours de l'adresse de FichierFactice.

Après le read, essaies de voir avec la méthode gcount le nombre d'octet copiés.
21/06/2010, 10h10
raphchar

A priori

Code:

FichierFactice=9-(Fichiersize%9);

Cette ligne attribue une valeur correcte à FichierFactice.
Fait notable, le debbuger n'indique rien sur cdata et data un peu plus loin semble ne contenir que 4 caractères???
Il est en rouge

Quand à fichier factice il change à chaque fois

quant à gcount la valeur est bien la taille du fichier.
21/06/2010, 10h18
raphchar

Peut être les tableaux auraient une taille maximale.
21/06/2010, 10h44
white_tentacle

Peux tu poster un échantillon de code un peu plus gros ?

Notamment, avec les déclarations des variables en question, l'ensemble des opérations effectuées, etc.

Là il n'y a rien de choquant, le problème est probablement ailleurs.

Tout d'abord pour dire que j'ai remplacé le tableau par un vector et ca fonctionne.

Mais, bon je vroudrais aussi comprendre pourquoi ca ne fonctionnaint pas

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
 
int Fichiersize;
int FichierFactice=0; 
ifstream is; //Fichier à traiter
 
{ //Choix du fichier}
 
is.open(Fichier, ios::binary );//Ouverture du fichier à traiter
if(is.fail())
{
  logfile<<"Le fichier n'a pas pu etre charge"<<endl;
  return false;
}
is.seekg(0, ios::end);
Fichiersize= is.tellg();
is.seekg(0, ios::beg);
if(Fichiersize==-1)
{
  logfile<<"Le fichier est trop volumineux"<<endl;
  return false;
}
 
FichierFactice=9-(Fichiersize%9);
if(FichierFactice==9)FichierFactice=0;
 
//Depart du traitement
//Récupération du fichier d'origine
char *cdata;
#define TEST
#ifdef TEST
  vector<unsigned char> data;
#else
  unsigned char *data;
#endif
  cdata= new char[(Fichiersize+FichierFactice)];
#define TEST
  data.reserve(Fichiersize+FichierFactice);
#ifdef TEST
#else
  data= new unsigned char[(Fichiersize+FichierFactice)];
#endif
  is.read(cdata,Fichiersize);
  int a=is.gcount();
  cout<<a<<endl;
  is.close();
for(int i=0;i<FichierFactice;i++)
  cdata[Fichiersize+i]=0;
for(int i=0;i<Fichiersize+FichierFactice;i++)
{
  data[i]=(unsigned char)cdata[i];
}
delete [] cdata;

Voila, je ne sais pas si c'est suffisant
J'ai fais un define pour la nouvelle maéthode, il faut le supprimer pour avoir l'ancienne version

21/06/2010, 11h22
white_tentacle

Le code que tu as posté fonctionne (y compris la version avec les tableaux).

Question bête, ne serais-tu pas en train de débugger du code release ? Auquel cas, les valeurs données par le debugger peuvent être fausses.

Si tu fais un cout de ta valeur, la valeur n'est toujours pas bonne ?
21/06/2010, 12h48
raphchar

Ben si justement je debug en mode release
24/06/2010, 16h03
jblecanard

Citation:

Envoyé par raphchar

Ben si justement je debug en mode release

Toi, tu vas avoir des problèmes...

Il faut que tu débuggues en mode Debug, ça risque de fonctionner un poil mieux :mouarf:
25/06/2010, 13h20
raphchar

C'est juste que j'avais lu sur ce forum que des erreurs n'apparaissent pas en mode debug alors qu'en mode release... Du coup je compile toujours en mode release.
A l'origine mon erreur ne venait pas de ce changement "bizarre" mais la valeur incorrecte avait attirée mon attention.
25/06/2010, 17h34
JulienDuSud

Citation:

Envoyé par raphchar

C'est juste que j'avais lu sur ce forum que des erreurs n'apparaissent pas en mode debug alors qu'en mode release... Du coup je compile toujours en mode release.
A l'origine mon erreur ne venait pas de ce changement "bizarre" mais la valeur incorrecte avait attirée mon attention.

Non ça, c'est lorsque ton code contient *déjà* du code erroné et qui peut amener à des comportements indéfinis. Du coup ça fonctionne en debug (à cause de tout un tas de raisons que je ne vais pas lister ici) et ça ne fonctionne pas en release. D'où la définition de comportement indéfinis.

Bref, si ça fonctionne pas en mode release alors qu'en mode debug si, alors c'est qu'il y a un bug - à corriger - assez sévère ! D'où l'intérêt de retourner en mode debug et de débusquer le bug.
28/06/2010, 10h23
white_tentacle

Citation:

Envoyé par JulienDuSud

Bref, si ça fonctionne pas en mode release alors qu'en mode debug si, alors c'est qu'il y a un bug - à corriger - assez sévère ! D'où l'intérêt de retourner en mode debug et de débusquer le bug.

Le plus souvent, c'est quand même une variable non initialisée à zéro (le compilateur MS a la très mauvaise idée de les initialiser à 0 en debug, mais pas en release). Beaucoup plus rarement un volatile qui manque.