passage par référence

Marsh Posté le 10-06-2008 à 10:58:19

Bonjour,

j'ai un petit probleme dans mon passage par référence dans une fonctionne puisque les parametres modifiés dans la fonction ne sont pas répercutés a la sortie de celle ci.

j'ai la fonction suivante (pas tres bien programmé mais bon c'est le début) :

Code :

void createData(const string fileMID, const string fileMIF, vector < vector<string>> & myData, vector <int> &myType, vector <string> &myNameVariable,vector <string> &min,vector <string> &max)

et je l'appelle comme cela :

Code :

const string fileMID = "data/DED.MID"; // string which contains path to MIF file
const string fileMIF = "data/DED.MIF"; // string which contains path to MIF file
vector < vector<string>> myData; // vector which contains my data
vector <int> myType; // vector which contains type of the data
vector <string> myNameVariable; // vector wich contains name of the data
vector <string> min ; // min of data
vector <string> max; // max of data
createData(fileMID, fileMIF, myData, myType, myNameVariable, min,max);

dans la fonction (en mode debug) toutes les variables sont modifiés avec les bonnes valeurs.
par contre quand je sors de ma fonction toujours en mode debug mes vecteurs n'ont plus la meme taille et surtout ont des valeurs comme 0 ou -1163005939

j'utilise visual express c++ 2008 sous windows XP.

est ce que j'ai rien compris au passage par référence ou c'est juste une petite erreure quelque part ? (ou peut etre une erreure du compilo mais je pense plutot pour une erreur de ma part vu mon niveau !)

Reply

Marsh Posté le 10-06-2008 à 10:58:19

Reply

Marsh Posté le 10-06-2008 à 11:20:18

Salut,

Une petite chose m'a sauté aux yeux dans ton code :
[...]vector < vector<string>> & myData, [...]
Les deux chevrons fermants du double "vector" sont collés. Le compilo les interprète comme un opérateur de flux ">>".

Mets un espace entre les deux '>'. Ça devrait marcher un peu mieux.

Message cité 1 fois

Reply

Marsh Posté le 10-06-2008 à 11:25:02

jxano a écrit :

Salut,

Une petite chose m'a sauté aux yeux dans ton code :
[...]vector < vector<string>> & myData, [...]
Les deux chevrons fermants du double "vector" sont collés. Le compilo les interprète comme un opérateur de flux ">>".

Mets un espace entre les deux '>'. Ça devrait marcher un peu mieux.

hélas non le probleme ne vient pas de la. j'ai corrigé le code et a l'exécution j'ai toujours le meme probleme.

Reply

Marsh Posté le 10-06-2008 à 11:51:21

en fait je viens de faire un test en mettant tout en commentaire et en ayant dans mon code que les lignes suivantes :

Code :

// string library : use type string
#include <string>
using namespace std ;
// vector librairy : use type vector
#include <vector>
using std::vector;
int main(int argc, char *argv[]){
vector <string> max; // max of data
max.push_back("1" );
max.push_back("mot" );
max.push_back("mot" );
return 0;
}

et en mode debug toujours quand je fais une visualisation des donnees max reste toujours comme cela :

Code :

max [0]() std::vector<std::basic_string<char,std::char_traits<char>,std::allocator<char> >,std::allocator<std::basic_string<char,std::char_traits<char>,std::allocator<char> > > >

j'ai en fait l'empression que mon code n'est pas exécuté meme si il s'arrete bien sur la ligne avec les breakpoints em mode debug.

Reply

Marsh Posté le 10-06-2008 à 12:09:09

Si tu utilise visual vire l'optimisation du code des fois ça fait des trucs chelou en debug

Message cité 1 fois

---------------
You can't start a fire with moonlight

Reply

Marsh Posté le 10-06-2008 à 12:23:46

kyntriad a écrit :

Si tu utilise visual vire l'optimisation du code des fois ça fait des trucs chelou en debug

il était déja désactivé. donc en fait j'ai supprimé toutes mes classes et je les aient remise apres et tout fonctionne maintenant.
les mysteres de la compilation :lol:

Reply

Marsh Posté le 10-06-2008 à 12:28:27

Code :

// string library : use type string
#include <string>
// vector librairy : use type vector
#include <vector>
using namespace std ;
int main(int argc, char *argv[]){
vector <string> max; // max of data
max.push_back("1" );
max.push_back("mot" );
max.push_back("mot" );
return 0;
}

À mon avis, le namespace std::vector n'est pas très utile, mais je peux me tromper.

Comment visualise-tu les données ? Perso, j'utiliserais un itérateur pour lire le vecteur :

Code :

for (vector<string>::iterator i= max.begin(); i!=max.end(); i++)
{
cout << *i << " ";
}

Message cité 2 fois

Reply

Marsh Posté le 10-06-2008 à 12:40:48

jxano a écrit :

[cpp]

Comment visualise-tu les données ? Perso, j'utiliserais un itérateur pour lire le vecteur :

en mode debug tu peux "voir" les données locales dans une fenetre séparée avec le nom de la variables, sa valeur et son type. c'est tres pratique quand tu as beaucoup de gros tableaux par exemples

Reply

Marsh Posté le 10-06-2008 à 16:24:38

jxano a écrit :

Comment visualise-tu les données ? Perso, j'utiliserais un itérateur pour lire le vecteur :

Code :

for (vector<string>::iterator i= max.begin(); i!=max.end(); i++)
{
cout << *i << " ";
}

Mon oreillette me dit :

Code :

vector<int> V;
// ...
copy(V.begin(), V.end(), ostream_iterator<int>(cout, "\n" ));

Sinon t'as testé un passage par ref simpel dans une fonction débile ?

Reply

Marsh Posté le 29-06-2008 à 12:43:23

Bonjour,

Pardon de reprendre cette vieille discussion...

Citation :

Mon oreillette me dit :

Code :

vector<int> V;
// ...
copy(V.begin(), V.end(), ostream_iterator<int>(cout, "\n" ));

Par acquis de conscience, j'ai essayé l'algorithme "copy()" dans un de mes programmes.

Pour l'utiliser, il convient tout d'abord de spécifier les en-têtes <algorithm> et <ostream> en début de source.

Dans mon programme, j'ai remplacé le type "int" par "Entier", une classe utilisateur implantant un entier long avec un "vector<char>".

Code :

[...]
class Entier
{
private: vector<char> vcCnt;
public:
Entier () { vcCnt.push_back ('0'); }
Entier (const Entier& e) : vcCnt (e.vcCnt) {}
Entier (long l) { vcCnt.push_back ('0'+l); } // entiers < 10
void Ecrire () { for (vector<char>::reverse_iterator i= vcCnt.rbegin (); i != vcCnt.rend (); i++) cout << *i; }
Entier& operator+= (Entier & );
};
[...]
void Afficher (vector<Entier> &veR)
{
cout << "<";
//for (vector<Entier>::iterator i= veR.begin (); i != veR.end (); i++) { cout << " "; i->Ecrire (); }
copy (veR.begin(), veR.end(), ostream_iterator<Entier> (cout, " " ));
cout << ">" << endl;
}
[...]

J'ai le désagrément de constater que mon compilateur (g++ sous Kubuntu) a toutes les peines du monde à instancier cette classe ostream_iterator<Entier>.

Joel F, ta méthode est élégante, mais exigeante... en tout cas restreinte aux patrons de types prédéfinis.

EDIT : Je viens de m'apercevoir qu'il faut que je surcharge l'opérateur << pour utiliser copy(), entre autres...

Message édité par jxano le 29-06-2008 à 16:01:55

Reply

Marsh Posté le 29-06-2008 à 12:43:23

Reply

Marsh Posté le 29-06-2008 à 17:51:19

bah, c'est comme tout, tu lis la doc de la STL et tu auras la liste des concepts et interface que les types passable à copy exigent. C'est pas parce que tu n'arrive pas à la faire marcher que ca marche
Ensuite, sauf précision de ma part, les codes que je poste ne sont pas compilables (sauf ils commencent par un #include) mais représente un exemple de syntaxe. Le reste c'est dans la doc.

Enfin, operator<< est nécessaire pour ostream_iterator<> par pour copy qui lui ne demandent qu'un constructeur de copie et/ou un opérateur d'affectation.

Message édité par Joel F le 29-06-2008 à 17:52:50

Reply

passage par référence

Sujets relatifs:

Leave a Replay