message d'erreur

Question

Salut tous le monde, j'ai le fonction suivante [cpp] vecteur* vecteur:: readvecteur(int *nbrev,char *fichier) { int i=0,cp=0,j=0; float value,val1,val2; ifstream Al; std::vectorvect(50); Al.open("fichier.txt"); while (i <(100*d) ) // *2 ici car on va lire x et y { Al >> value; val1=value; Al >> value; val2=value; vect[cp].x=val1; vect[cp].y=val2; vect[cp].id=compt++; cp++; i=i+d; } (*nbrev)=cp; Al.close(); return vect; }/cpp avec la class vecteur: [cpp] class vecteur { public : float x,y; int id; float reald; vecteur(); vecteur(const vecteur &p); vecteur(float a,float b,float c, float d); virtual ~vecteur(); }; /cpp le message d'erreur: error C2440: 'return' : cannot convert from 'class std::vector >' to 'class vecteur *' comment je peux résoudre ce problème? Merci pour toutes proposition

BunoCS · Answer

La fonction veut une valeur de retour de type vecteur et toi, tu lui retourne un std::vector.
Un peu de coh&#233;rence...

Buno
----------------------------------------
L'urgent est fait, l'impossible est en cours. Pour les miracles, pr&#233;voir un d&#233;lai...

DeAtHCrAsH · Answer

Tu dois declarer vect en tant que pointeur de vecteur.
Sinon a part ca je te déconseille vivement de continuer dans cette vois.

Une fonctiion qui renvoie un pointeur, doit renvoyer un pointeur sur une variable globale (dans ton cas elle doit etre globale a ta classe).

Toi tu renvoies le pointeur d'une variable locale a ta fonction, tout en sachant qu'a la fin de ta fonction ton pointeur sera detruit.
Tu n'es donc plus dans la certitude de renvoyer un pointeur valide.

NE JAMAIS CREER DES FONCTIONS QUI RENVOI DES POINTEURS PROPRE A LA FONCTION!!!

imanedaoudi · Answer

ci j'ai bien compris ta remarque je faire comme ca:
vecteur* vecteur:: readvecteur(int *nbrev,char *fichier)
{
 int i=0,cp=0,j=0;
    float value,val1,val2; 
 ifstream Al;
 vecteur *vect;
vect=new vecteur[500];
 Al.open("fichier.txt");
 while (i <(100*d) )  // 
 {
   Al >> value;
   val1=value;
  Al >> value;
  val2=value;
  vect[cp].x=val1;
  vect[cp].y=val2;
  vect[cp].id=compt++;
  cp++;
  i=i+d;
 }
(*nbrev)=cp;
data=vect;
Al.close();
return vect; 
}
avec la class vecteur:

class vecteur  
{
  public :


  float x,y;
  int id;
  float reald;
  vecteur *data;
  
vecteur();
vecteur(const vecteur &p);
vecteur(float a,float b,float c, float d);
 virtual ~vecteur();
 
};

BunoCS · Answer

Euh... je r&#233;ponds avant DeathCrash:


fais plut&#244;t: void vecteur:: readvecteur(vecteur *vect,int *nbrev,char *fichier)

Buno
----------------------------------------
L'urgent est fait, l'impossible est en cours. Pour les miracles, pr&#233;voir un d&#233;lai...

BunoCS · Answer

Du coup, tu peux enlever:
- vecteur *vect;
- return vect;

Ah oui! Il vaut mieux faire le new en dehors de la fonction...
Buno
----------------------------------------
L'urgent est fait, l'impossible est en cours. Pour les miracles, pr&#233;voir un d&#233;lai...

imanedaoudi · Answer

merci Buno, .. juste pour rendre les choses un peu clair, l'allocation de la m&#233;moire 
vect=new vecteur[500]; dois se faire &#224; l'ext&#233;rieur de la fonction non??

imanedaoudi · Answer

Et si je met ca est ce que ca pose un probl&#232;me!!!
En gardant bien sur la meme classe vecteur
vecteur::vecteur(vecteur a,vecteur b,char *fichier)
{   
 int nb;
 data=new vecteur[500];
 readvecteur(data,&nb,fichier);
 id=nb;
}
et en tenant compte bien sur de ta proposition Buno:
void vecteur:: readvecteur(vecteur *vect,int *nbrev,char *fichier)

BunoCS · Answer

Euh... je viens de r&#233;fl&#233;chir: si ta fonction readvecteur fais partie de ta classe vecteur, tu n'es pas oblig&#233; de retourner un vecteur!!! :)
Donc, je te propose ceci (attention, je n'ai pas test&#233;!!!!!!):

int main(int argc, char ** argv)
{
  vecteur vect;
  int nb=500;

  vect.readvecteur(&nb,"fichier.txt");
  ...
  return 0;
}

void vecteur::readvecteur(int * nb, char * fichier)
{
 int i=0,cp=0,j=0;
 float value,val1,val2; 
 ifstream Al;
 Al.open("fichier.txt");
 while (i <(100*d) )  // 
 {
   Al >> val1;
  Al >> val2;
  data[cp].x=val1;
  data[cp].y=val2;
  data[cp].id=compt++;
  cp++;
  i=i+d;
 }
(*nbrev)=cp;
Al.close();
}

typedef struct
{
  float x;
  float y;
  int id;
} truc;



class vecteur  
{
  public :


  float reald;
  truc *data;
  
vecteur();
vecteur(const vecteur &p);
vecteur(float a,float b,float c, float d);
 virtual ~vecteur();
 
};

Buno
----------------------------------------
L'urgent est fait, l'impossible est en cours. Pour les miracles, pr&#233;voir un d&#233;lai...

imanedaoudi · Answer

je comprend, mais je ne vois pas ou tu alloue de la memoire pour *data??
je pense que pour realiser ca:
data[cp].x=val1;
  data[cp].y=val2;
  data[cp].id=compt++;
  cp++;

il faut reserver de l'espace m&#233;moire ou est ce que je peux mettre ca pour ne pas avoir de probl&#232;me??

BunoCS · Answer

Effectivement, mais j'ai suppos&#233; que tu l'avais fait dans le constructeur... ce n'est pas le cas????

Buno
----------------------------------------
L'urgent est fait, l'impossible est en cours. Pour les miracles, pr&#233;voir un d&#233;lai...

BunoCS · Answer

[probl&#232;me: l'editbox ne prend pas mon texte en entier]
Je disais:
constructeur
vecteur::vecteur(int taille)
{
  data = new truc[taille];
}

destructeur:
vecteur::~vecteur()
{
  delete data;
}
Buno
----------------------------------------
L'urgent est fait, l'impossible est en cours. Pour les miracles, pr&#233;voir un d&#233;lai...

imanedaoudi · Answer

voila c'est ca ma question!! j'ai pas mis ca dans le constructeur, par contre si je le met dans la fonction  read vecteur cela ne posera pas de probl&#232;mes non??

BunoCS · Answer

Du coup, dans le constructeur, tu initialise data &#224; NULL, tu fais l'allocation dans readvecteur et tu n'oublie pas le delete dans le destructeur... &#231;a devrait &#234;tre correct...

Buno
----------------------------------------
L'urgent est fait, l'impossible est en cours. Pour les miracles, pr&#233;voir un d&#233;lai...

imanedaoudi · Answer

jevais eclaircir un peu plus le probl&#233;me, en faite je connais pas la taille exacte de data c'est en lisant les donn&#233;es du fichier que je peux r&#233;cuperer l taille exacte de mon vecteur data (cp) dans la fonction donc je reserve un espace memoire tres grand(500) puis je calcul la taille exacte de mon vecteur (la fct read vecteur me renvoi la taille de data et le vecteur data).

imanedaoudi · Answer

ta proposition marche tr&#233;s bien buno merci.

BunoCS · Answer

Si ma soluce te convient, merci de cliquer sur "r&#233;ponse accept&#233;e" pour clore le sujet
Bonne prog'
Buno
----------------------------------------
L'urgent est fait, l'impossible est en cours. Pour les miracles, pr&#233;voir un d&#233;lai...

imanedaoudi · Answer

ce n'est pas encore fini!!
ci ca te derange pas je pose une autre question??

BunoCS · Answer

Vas-y, je suis pay&#233; pour...euh non...enfin, je suis l&#224; pour &#231;a :)

Buno
----------------------------------------
L'urgent est fait, l'impossible est en cours. Pour les miracles, pr&#233;voir un d&#233;lai...

imanedaoudi · Answer

dans la meme fonction readfile,  normalement le connais pas exactement la taille de mes donn&#233;es &#224; lire j'ai mis :
Al.open("fichier.txt");
 while (i <(100*d) )  // 
 {
   Al >> value;
   val1=value;
  Al >> value;
  val2=value;
  vect[cp].x=val1;
  vect[cp].y=val2;
  vect[cp].id=compt++;
  cp++;
  i=i+d;
 }

la taille (100*d) rien que pour lire un tres grand nombres de valeurs possibles,comment faire alors pour lire tous les donnees du fichier??

BunoCS · Answer

Je ne connais pas par coeur la classe ifstream mais tu dois avoir une m&#233;thode qui te renseignes sur la fin du stream, genre is_empty... et tu boucle tant que cette condition n'est pas v&#233;rifi&#233;e (c'est-&#224;-dire, tant qu'il y a quelquechose dans ton stream)

Buno
----------------------------------------
L'urgent est fait, l'impossible est en cours. Pour les miracles, pr&#233;voir un d&#233;lai...

Message d'erreur

32 réponses

Votre réponse

Discussions similaires