message d'erreur

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::vector<vecteur>vect(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<class vecteur,class std::allocator<class vecteur> >' to 'class vecteur *'
 comment je peux résoudre ce problème?
Merci pour toutes proposition

La fonction veut une valeur de retour de type vecteur et toi, tu lui retourne un std::vector.
Un peu de cohérence...


Buno
----------------------------------------
L'urgent est fait, l'impossible est en cours. Pour les miracles, prévoir un délai...

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!!!

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();
 
};

Euh... je réponds avant DeathCrash:

fais plutôt: void vecteur:: readvecteur(vecteur *vect,int *nbrev,char *fichier)


Buno
----------------------------------------
L'urgent est fait, l'impossible est en cours. Pour les miracles, prévoir un délai...

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évoir un délai...

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

Et si je met ca est ce que ca pose un problè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)
 

Euh... je viens de réfléchir: si ta fonction readvecteur fais partie de ta classe vecteur, tu n'es pas obligé de retourner un vecteur!!! :)
Donc, je te propose ceci (attention, je n'ai pas testé!!!!!!):

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évoir un délai...

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émoire ou est ce que je peux mettre ca pour ne pas avoir de problème??