fstream qui pète un plomb ?

fstream qui pète un plomb ? - C++ - Programmation

Marsh Posté le 27-11-2010 à 22:31:31    

Bonsoir \o/
 
Bon, voilà, j'ai un drôle de problème avec la libraire fstream, mon fichier se découpe comme ça (animation pour un jeu) sign(x),pos(x),sign(y),pos(y),frame_index,xscale,yscale,layer.
Lorsque je lis toute ces valeurs avec mon code, bah au bout de la 9ème frame il pète un plomb (frame_index a rien à voir avec la frame de l'animation), voilà mon code :  
 

Code :
  1. double open_animation(char* file)
  2. {
  3.     int filesize;
  4.     int framecount;
  5.     int i;
  6.     unsigned short temp_xabs;
  7.     unsigned short temp_yabs;
  8.     int temp_x;
  9.     int temp_y;
  10.     int binpos;
  11.     unsigned char temp_sign;
  12.     char temp_xscale;
  13.     char temp_yscale;
  14.     unsigned short temp_frame_index;
  15.     unsigned char temp_layer;
  16.     ifstream f(file,ios::binary);
  17.     if (!f.is_open()) cerr << "!!";
  18.     f.seekg(0,ios::end);
  19.     filesize = f.tellg();
  20.     framecount = (filesize/11)-10;
  21.     f.seekg(0,ios::beg);
  22.     resize(framecount);
  24.       i=0;
  25.       binpos=0;
  26.       while (binpos<filesize-1)
  27.     {
  28.       //  f.seekg(binpos);
  29.         f.read((char*) &temp_sign,1);
  30.         binpos+=1;
  31.         f.seekg(binpos);
  32.         f.read((char*) &temp_xabs,2);
  33.         if (temp_sign==255) {temp_x = temp_xabs*-1;}
  34.         if (temp_sign==0)  {temp_x = temp_xabs;}
  35.         //END OF X POS
  37.         // YPOS
  38.         binpos+=2;
  39.         f.seekg(binpos);
  40.         f.read((char*) &temp_sign,1);
  41.         binpos+=1;
  42.         f.seekg(binpos);
  43.         f.read((char*) &temp_yabs,2);
  44.         if (temp_sign==255) {temp_y = temp_yabs*-1;}
  45.         if (temp_sign==0)  {temp_y = temp_yabs;}
  46.         //END OF Y POS
  48.         // Frame index !
  49.         binpos+=2;
  50.         f.seekg(binpos);
  51.         f.read((char*) &temp_frame_index,2);
  52.         // End of frame index
  54.         //xscale !
  55.         binpos+=2;
  56.         f.seekg(binpos);
  57.         f.read((char*) &temp_sign,1);
  58.         if (temp_sign==255) {temp_xscale = -1;}
  59.         if (temp_sign==0)  {temp_xscale =  1;}
  60.         // end of xscale
  62.         //yscale !
  63.         binpos+=1;
  64.         f.seekg(binpos);
  65.         f.read((char*) &temp_sign,1);
  66.         if (temp_sign==255) {temp_yscale = -1;}
  67.         if (temp_sign==0)  {temp_yscale =  1;}
  68.         // end of yscale
  70.         //layer !
  71.         binpos+=1;
  72.         f.seekg(binpos);
  73.         f.read((char*) &temp_layer,1);
  74.         // end of layer
  76.         // FILL THE ARRAY !
  77.         set_value(i,temp_x,temp_y,temp_frame_index,temp_xscale,temp_yscale,temp_layer);
  79.         binpos+=1;
  80.         f.seekg(binpos);
  81.         i++;
  82.         }
  83.     f.close();


 
on dirait qu'il se décale progressivement, en gros au lieu de lire 05 00 (les bytes sont inversé, donc en dec ça fait 5), il me lit 00 05 (1280 en dec)
 
D'où viens mon problème ? :(
 
Merci d'votre aide et @++ :D


---------------
Perhaps you don't deserve to breathe
Reply

Marsh Posté le 27-11-2010 à 22:31:31   

Reply

Marsh Posté le 27-11-2010 à 23:31:16    

char[2] ne fait pas forcément un short, ça dépend des architectures or c'est que tu sous-entends avec le read(&xxx,2). ça peut venir de là...

 

Perso, l'utilisation massive de seekg ne me plait gère (ainsi que les binpos+=xxx).

 

Pourquoi ne fais-tu pas plutôt l'utilisation de >>, c'est fait pour non ?
Par exemple (j'ai pas testé)

Code :
  1. //Read of temp_x
  2. f >> temp_sign >> temp_xabs;
  3. temp_x = (temp_sign==0)?(temp_xabs):(-temp_xabs);
  4. //Read of temp_y
  5. f >> temp_sign >> temp_yabs;
  6. temp_y = (temp_sign==0)?(temp_yabs):(-temp_yabs);
  7. //Read of temp_frame index
  8. f >> temp_frame_index;
  9. //Read of xscale
  10. f >> temp_sign;
  11. temp_xscale = (temp_sign==0)?(1):(-1);
  12. //Read of yscale
  13. f >> temp_sign;
  14. temp_yscale = (temp_sign==0)?(1):(-1);
  15. //Read of layer
  16. f >> temp_layer;
  17. //Set values and move forward in the table
  18. set_value(i,temp_x,temp_y,temp_frame_index,temp_xscale,temp_yscale,temp_layer);
  19. ++i;
 

Message cité 1 fois
Message édité par dreameddeath le 27-11-2010 à 23:42:59
Reply

Marsh Posté le 28-11-2010 à 11:34:30    

dreameddeath a écrit :

char[2] ne fait pas forcément un short, ça dépend des architectures or c'est que tu sous-entends avec le read(&xxx,2). ça peut venir de là...
 
Perso, l'utilisation massive de seekg ne me plait gère (ainsi que les binpos+=xxx).
 
Pourquoi ne fais-tu pas plutôt l'utilisation de >>, c'est fait pour non ?
Par exemple (j'ai pas testé)

Code :
  1. //Read of temp_x
  2. f >> temp_sign >> temp_xabs;
  3. temp_x = (temp_sign==0)?(temp_xabs):(-temp_xabs);
  4. //Read of temp_y
  5. f >> temp_sign >> temp_yabs;
  6. temp_y = (temp_sign==0)?(temp_yabs):(-temp_yabs);
  7. //Read of temp_frame index
  8. f >> temp_frame_index;
  9. //Read of xscale
  10. f >> temp_sign;
  11. temp_xscale = (temp_sign==0)?(1):(-1);
  12. //Read of yscale
  13. f >> temp_sign;
  14. temp_yscale = (temp_sign==0)?(1):(-1);
  15. //Read of layer
  16. f >> temp_layer;
  17. //Set values and move forward in the table
  18. set_value(i,temp_x,temp_y,temp_frame_index,temp_xscale,temp_yscale,temp_layer);
  19. ++i;


 


Le code a pas marché :(
 
la structure de mon fichier est comme ça : signx(1B) 255 pour -1 et 0 pour 1 ; posx(2B) unsigned short ;  signy(1B) 255 pour -1 et 0 pour 1 ; posy(2B) unsigned short ; frame_index(2B) unsigned short ; xscale(1B) 255 pour -1 et 0 pour 1 ; yscale(1B) 255 pour -1 et 0 pour 1 ; layer (1B) unsigned char
 
Comment j'peux adapter ton code ? j'comprends pas trop :D  


---------------
Perhaps you don't deserve to breathe
Reply

Marsh Posté le 28-11-2010 à 16:18:27    

Ok, bon, c'était mon fichier qui était bidon :D

 

Donc maintenant ça marche niquel, et maintenant faut que je fasse passer une adresse, je m'explique :

 

ma DLL charge les données d'un fichier qu'elle stocke dans son tableau (local), j'ai besoin de pouvoir me servir de plein de tableaux, donc je suis sensé faire passer son pointeur pour ensuite récupérer les données dans le bon tableau, c'est ça ? comment je peux retrouver un tableau via une adresse en double ? j'ai lu comme quoi fallait utiliser &lepointeur, mais je sais pas trop..

 

halp !

 

edit : le problème vient de la :

 
Code :
  1. double get_x(double marray,double n)
  2. {
  3.     return(double) (*marray)[n].x;
  4. }


marray est sensé être l'adresse du tableau en double, faut que je fasse quoi pour qu'il la reprenne bien ?


Message édité par Terminapor le 28-11-2010 à 16:55:40

---------------
Perhaps you don't deserve to breathe
Reply

Marsh Posté le 30-11-2010 à 11:56:51    

sauf que marray est un double, et pas un **double.

Reply

Marsh Posté le 02-12-2010 à 22:20:04    

ça marche pas non plus :/
 
jsuis sensé faire quoi ?


---------------
Perhaps you don't deserve to breathe
Reply

Marsh Posté le 02-12-2010 à 22:43:21    

double get_x(double* marray,double n)
 
genre un pointeur quoi ...

Message cité 1 fois
Reply

Marsh Posté le 04-12-2010 à 21:03:36    

Joel F a écrit :

double get_x(double* marray,double n)
 
genre un pointeur quoi ...


A marche pas non plus :/
 
ça peut venir d'un mauvais export d'array ?
 

Code :
  1. double open_animation(char* file)
  3. {
  4.     vector <animdata> *array;
  5.     array = new vector <animdata>;
  6.     array->resize(framecount);
  7.     // tout mon bordel d'array
  8.     return (int) array;
  9. }


---------------
Perhaps you don't deserve to breathe
Reply

Marsh Posté le 05-12-2010 à 11:00:24    

mais pourquoi cast tu ton pointeur en int alors que tu renvoies un double ?
GOTO un livre de C++ là :E

Message cité 1 fois
Reply

Marsh Posté le 05-12-2010 à 11:08:44    

Joel F a écrit :

mais pourquoi cast tu ton pointeur en int alors que tu renvoies un double ?
GOTO un livre de C++ là :E


pasqu'après y'a une autre fonction qui le reprend et qui l'exporte en double, sinon j'ai une erreur "pointer value used where a floating point value where expected" :/


---------------
Perhaps you don't deserve to breathe
Reply

Marsh Posté le 05-12-2010 à 11:08:44   

Reply

Marsh Posté le 05-12-2010 à 14:41:24    

non mais attends ... tu sais ce qu'est un pointeur au moins ? TU es conscient que tu écris un truc qui n'a aucun sens ?

Message cité 1 fois
Reply

Marsh Posté le 05-12-2010 à 14:56:13    

Joel F a écrit :

non mais attends ... tu sais ce qu'est un pointeur au moins ? TU es conscient que tu écris un truc qui n'a aucun sens ?


Bah c'est l'adresse où y'a mon tableau, non ?


---------------
Perhaps you don't deserve to breathe
Reply

Marsh Posté le 05-12-2010 à 15:39:22    

ca te choque pas de caster un pointeur de vector en double ?
Au lieu de caster comme un gros porc pour masquer tes erreurs, apprends à lire et réfléchi sur l'effet désiré de ta méthode.

Message cité 1 fois
Message édité par Joel F le 05-12-2010 à 15:39:51
Reply

Marsh Posté le 05-12-2010 à 15:53:06    

Joel F a écrit :

ca te choque pas de caster un pointeur de vector en double ?
Au lieu de caster comme un gros porc pour masquer tes erreurs, apprends à lire et réfléchi sur l'effet désiré de ta méthode.


Oui mais je peux pas faire autrement, je dois récupérer cette valeur en double.


---------------
Perhaps you don't deserve to breathe
Reply

Marsh Posté le 05-12-2010 à 16:45:15    

non mais arretes ... réfléchis 10s au lieu d'écrire n'imp.

Message cité 1 fois
Reply

Marsh Posté le 05-12-2010 à 17:26:43    

Joel F a écrit :

non mais arretes ... réfléchis 10s au lieu d'écrire n'imp.


Ben, c'est la première fois que je touche au c++, j'suis déjà content d'avoir pu écrire tout ce bordel :sweat:
 
Mais là j'sais vraiment pas comment m'y prendre, j'ai vu des infos sur les '*' et '&' mais j'arrive pas à le faire marcher.. :/


---------------
Perhaps you don't deserve to breathe
Reply

Marsh Posté le 05-12-2010 à 18:22:41    

lis un bouquin. Tu joues pas au bingo, tu programmes ...

Reply

Marsh Posté le 11-12-2010 à 17:10:23    

Baah, j'm'en sors pas :sweat:
 
j'ai abandonné l'idée de faire passer des adresses en double, c'est trop crasseux :/
Du coups j'ai tenté de stocker mes tableaux dans un array global, voilà mon code :  
 

Code :
  1. vector <animstruct> *garray;
  2.    unsigned int line;
  4. void array_load (unsigned int n, unsigned int array, unsigned int framecount)
  5. {
  6.     (*garray)[n].anim = array;
  7.     (*garray)[n].framecount = framecount;
  8. }
  10. void set_value(unsigned int array,unsigned int n,int x,int y, unsigned short frame_index,char xscale,char yscale,unsigned char layer)
  11. {
  12.      (*array)[n].x = x;
  13.      (*array)[n].y = y;
  14.      (*array)[n].frame_index = frame_index;
  15.      (*array)[n].xscale = xscale;
  16.      (*array)[n].yscale = yscale;   
  17.      (*array)[n].layer = layer;
  18. }
  20. export double open_animation(char* file)
  21. {
  22. // initialisation des variables
  23.     vector <animdata> *array;
  24.     array = new vector <animdata>;
  25.     garray = new vector <animstruct>;
  26.     array->resize(framecount);
  28.       for (i=0;i<framecount;i++)
  29.     {
  30.      // import des infos à partir du fichier
  31.         set_value(array,i,temp_x,temp_y,temp_frame_index,temp_xscale,temp_yscale,temp_layer); // j'remplis mon tableau
  32.     }
  33.         line++;
  34.         garray->resize(line);
  35.         array_load(line,array,framecount);
  36.         return (line); // vu qu'il faut que je puisse retrouver mon tableau, je renvois la ligne à laquel il est sensé être stocké
  37. }


 
Mais voilà, j'ai cette erreur : "invalid conversion from `std::vector<animdata, std::allocator<animdata> >*' to `unsigned int' " dans la fontion set_value(array,etc) et aussi "invalid type argument of `unary*' sur tout les lignes (*array)[n] d'où vient l'erreur ?


---------------
Perhaps you don't deserve to breathe
Reply

Marsh Posté le 12-12-2010 à 19:30:18    

même caca que la dernière fois, tu code sans savoir.
Une adresse n'est pas un entier ni un double. Réfléchi à l'adaptation des tes interfaces de fonctions plutot que de faire n'imp ...
 
Vraiment, relit un bouqin de C.

Reply

Marsh Posté le 19-12-2010 à 19:41:07    

Aah, ben j'ai finalement compris :D
 
Maintenant, j'ai un autre ptit problème :  
 
Mes vecteur sont stocké dans un vecteur global (garray), chacun ont un numéro de ligne, par exemple (*garray)[0].anim (c'est une structure avec un vector<animdata> *anim et un unsigned int framecount) ça sera le premier vecteur.
 
dans ma fonction double get_x(double id,double line) je sais pas comment récupérer le vecteur via un truc du style (*garray)[id].anim, et le membre nommé x de sa structure; en gros je voudrais fabriquer un vecteur appelé 'tarray' qui récupère tout les informations du vecteur n°0 stocké dans garray par exemple
 
 
Merci d'vot aide !


---------------
Perhaps you don't deserve to breathe
Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed