Erreur d'une fonction simple qui retourn un Vector

Version imprimable

Bonjour,

J'ai fait un programme hyper simple pour voir comment une fonction retourne un vecteur ,
voici l'exemple :

Main

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
#include <string.h>
#include <stdio.h>
#include <stdlib.h>
 
#include <vector>
#include "Essaye.h"
using namespace std;
 
 
int main()
{
	Essaye Etud;
 
	Etud.MaFonction("Jean","Jack");
 
    return 0;
}

Classe "Essaye.h"

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
#include <vector>
#include <string.h>
#include <iostream>
using namespace std;
 
 
#ifndef ESSAYE_H_
#define ESSAYE_H_
 
class Essaye {
 
public: struct etudiant {string nom;string prenom;};
 
public:
	Essaye();
	vector<etudiant> MaFonction(string,string);
 
 
private :
string monNom;
string monPrenom;
 
};
#endif /* ESSAYE_H_ */

et le fichier "Essaye.cpp"

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
#include <string.h>
#include <stdio.h>
#include <stdlib.h>
 
#include <vector>
#include "Essaye.h"
using namespace std;
 
Essaye::Essaye()
{
	monNom="nom1";
	monPrenom="prenom1";
}
 
vector<etudiant> Essaye::MaFontion(string nom,string prenom)
{
	etudiant etud;
	vector<etudiant> vect;
 
	etud.nom = nom;
	etud.prenom = prenom;
 
	vect.push_back(etud);
 
	return vect;
}

Voici l'erreur qui m'affiche :

Code:

1
2
3
4
5
Essaye.cpp:18: error: ‘etudiant’ was not declared in this scope
../Essaye.cpp:18: error: template argument 1 is invalid
../Essaye.cpp:18: error: template argument 2 is invalid
../Essaye.cpp:18: error: no ‘int Essaye::MaFontion(std::string, std::string)’ member function declared in class ‘Essaye’
make: *** [Essaye.o] Error 1

Pourquoi cette erreur s'il vous plait ?

Merci

En plus de ton erreur de compilation, ton programme ne peux pas marcher. En effet, tu déclares une variable locale dans ta fonction qui doit retourner un vector. Cette variable sera détruite à la sortie de ta fonction et donc tu ne peux rien récupérer.
Tu dois créer un pointeur sur un vector.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 vector<etudiant> Essaye::MaFontion(string nom,string prenom) { etudiant etud; vector<etudiant> *vect = new vector<etudiant>(); etud.nom = nom; etud.prenom = prenom; vect->push_back(etud); return *vect; }
Dans ton fichier .cpp il est inutile de remettre les includes de string et vector car ils sont déjà dans le .h de même le using namespace std est déjà dans le .h.
Tu dois te méfier des déclarations double de using namespace std car elles ce cumulent ce qui peux donner des choses assez étranges.

Concernant ton problème de compilation, je dirais que tu déclares cette structure uniquement dans la classe il faudrait qu'elle soit définie en dehors de la classe pour qu'elle soit connue du compilateur.

18/03/2011, 10h42
mimish

Merci pour votre réponse ,

Citation:

Envoyé par andnicam

Concernant ton problème de compilation, je dirais que tu déclares cette structure uniquement dans la classe il faudrait qu'elle soit définie en dehors de la classe pour qu'elle soit connue du compilateur.

Je dois la définir ou a part dans la classe ?
18/03/2011, 14h09
stranger
Citation:

Envoyé par mimish

Merci pour votre réponse ,

Je dois la définir ou a part dans la classe ?

Tu définis ta structure à l’extérieur de la classe.
Code:

1 2 3 4 5 6 7 8 struct etudiant{ [...] }; class Essaye { [...] };
comme mentionné par andnicam , c'est la source de ton erreur de compil.
18/03/2011, 14h29
screetch

tu n'es pas obligé de le sortir de la classe, tu peux plutot faire:

Code:

vector<Essaye::etudiant> Essaye::MaFontion(string nom,string prenom)

l'idée du pointeur est aussi très mauvaise, ton premier code marchait tres bien. La méthode va retourner une copie de ta variable locale avant de détruire la variable locale. La méthode de andnicam crée juste un leak mémoire :-/
18/03/2011, 15h35
stranger

C'est pas faux pour la fuite mémoire et aussi pour l'accès à ta structure, ça t'évite de modifier le code avec la dernière solution :)