Discussion :

[mimish]

Bonjour,

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

Main

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[AF_2.8]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[mimish]

Merci pour votre réponse ,

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 ?

[stranger]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[screetch]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :-/

[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