Discussion :

[tcharles]

Salut a tous,
je voudrais utiliser la template vector dans une classe pour recuperer des données et pouvoir par la suite les afficher. j'ai ecris les bouts de code suivants qui ne fonctionnent pas.

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
27
28
29
30
 
 
//  Corps de la class
#include <iostream>
#include <vector>
#include <algorithm>
#include <numeric>
#include "Klass.h"
 
using namespace std;
 
char* Cnode::GetName(){
        return m_StrName;
}
 
int Cnode::GetEdgedID(){
 
        for ( size_t i = 0, size = m_cEdgedIDs.size(); i < size; ++i ){
                cout << m_cEdgedIDs[ i ] << "  " ;
        }
        cout << endl;
}
 
void Cnode::SetName(char* Name){
        m_StrName = Name;
}
 
void SetEdgedID(int Edge){
        m_cEdgedIDs.push_back(Edge);
}

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
 
// fichier header Klass.h
#ifndef Klass_H
#define Klass_H
class Cnode {
        private:
                char* m_StrName;
                std::vector<int>  m_cEdgedIDs;  // m_cEdgedIDs
        public:
                char* GetName();                
                int GetEdgedID();
                void SetName(char*);               
                void SetEdgedID(int);              
};
#endif

Merci de m'appoter une solution a cela

[Fiquet]

j'ai ecris les bouts de code suivants qui ne fonctionnent pas.

Qu'est-ce que tu entends par ne fonctionnent pas? Pourrais-tu nous donner les messages d'erreur car sans on peut pas dire grand chose.

Sinon déjà à la volé tu as oublié d'inclure <vector> dans le header (pour la déclaration de std::vector<int> m_cEdgedIDs

[killerjeff]

il te manque juste le

Code :

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

 #include <vector>

dans Klass.h

[tcharles]

voici en fait les erreurs que j'obtient meme apres voir insere #include<vector>dans Klass.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
 
"Klasse.C", line 6: Error: Could not open include file "Klass.h".
"Klasse.C", line 10: Error: Cnode is not defined.
"Klasse.C", line 11: Error: m_StrName is not defined.
"Klasse.C", line 14: Error: Cnode is not defined.
"Klasse.C", line 16: Error: m_cEdgedIDs is not defined.
"Klasse.C", line 16: Error: Badly formed expression.
"Klasse.C", line 17: Error: m_cEdgedIDs is not defined.
"Klasse.C", line 20: Error: "GetEdgedID()" is expected to return a value.
"Klasse.C", line 22: Error: Cnode is not defined.
"Klasse.C", line 23: Error: m_StrName is not defined.
"Klasse.C", line 27: Error: m_cEdgedIDs is not defined.
"Klasse.C", line 27: Error: Badly formed expression.
12 Error(s) detected.

[Matthieu Brucher]

Alors commence par résoudre ta première ligne, il dit clairement, explicitement, qu'il n'arrive pas à ouvrir le fichier Klass.h.

[tcharles]

apres les modifications suivantes:

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
27
28
29
30
 
//  Corps de la class
#include <iostream>
#include <vector>
#include <algorithm>
#include <numeric>
#include "Klasse.h"
 
using namespace std;
 
char* Cnode::GetName(){
        return m_StrName;
}
 
int Cnode::GetEdgedID(){
 
        for ( size_t i = 0, size = m_cEdgedIDs.size(); i < size; ++i ){
                cout << m_cEdgedIDs[ i ] << "  " ;
		return m_cEdgedIDs[ i ];
        }
        cout << endl;
}
 
void Cnode::SetName(char* Name){
        m_StrName = Name;
}
 
void SetEdgedID(int Edge){
        m_cEdgedIDs.push_back(Edge);
}

et #include "Klasse.h" j'ai les erreurs suivantes:

Code :

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

1
2
3
4
 
"Klasse.C", line 28: Error: m_cEdgedIDs is not defined.
"Klasse.C", line 28: Error: Badly formed expression.
2 Error(s) detected.

[tcharles]

je me pose aussi la question s'il ne serait pas plus aise d'utiliser le vector comme variable public sans y ajouter des methodes a son utilisation.
???

[Caine]

Simple,

Code :

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

1
2
3
4
 
void Cnode::SetEdgedID(int Edge){ 
        m_cEdgedIDs.push_back(Edge); 
}

Tel que tu as déclaré SetEdgedID, ce n'est pas une fonction de classe.

[jmv]

Code :

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

1
2
3
4
5
6
7
8
9
 
int Cnode::GetEdgedID(){
 
        for ( size_t i = 0, size = m_cEdgedIDs.size(); i < size; ++i ){
              cout << m_cEdgedIDs[ i ] << "  " ;
              return m_cEdgedIDs[ i ];
        }
        cout << endl;
}

y'aurait pas un autre pb là ? c'est louche le return dans la boucle for, tu retourne la 1ere valeur du vector et c'est tout, je ne pense pas que c'est ce que tu voulais faire.

[Caine]

Au posteur initial :
Et donc la fin de l'histoire? Juste pour savoir

[Matthieu Brucher]

Code :

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

1
2
3
4
5
6
7
8
9
 
int Cnode::GetEdgedID(){
 
        for ( size_t i = 0, size = m_cEdgedIDs.size(); i < size; ++i ){
              cout << m_cEdgedIDs[ i ] << "  " ;
              return m_cEdgedIDs[ i ];
        }
        cout << endl;
}

y'aurait pas un autre pb là ? c'est louche le return dans la boucle for, tu retourne la 1ere valeur du vector et c'est tout, je ne pense pas que c'est ce que tu voulais faire.

Exact, et il a dû sortir un truc te disat que la fonction doit retourner qqch mais qu'en fait elle ne retournait rien, non ?

[tcharles]

La fin de l'histoire est la suivante: au lieux de

Code :

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

1
2
3
4
 
void SetEdgedID(int Edge){
        m_cEdgedIDs.push_back(Edge);
}

plutot:

Code :

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

1
2
3
4
 
void Cnode::SetEdgedID(int Edge){
        m_cEdgedIDs.push_back(Edge);
}

et pour

Code :

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

1
2
3
4
5
6
7
8
9
 
int Cnode::GetEdgedID(){
 
        for ( size_t i = 0, size = m_cEdgedIDs.size(); i < size; ++i ){
              cout << m_cEdgedIDs[ i ] << "  " ;
              return m_cEdgedIDs[ i ];
        }
        cout << endl;
}

simplement

Code :

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

1
2
3
4
5
6
 
int Cnode::GetEdgedID(){
 
        return m_cEdgedIDs.back();
 
}