r�f�rence � un vector

Bonjour,

j'ai lu quelques postes � ce sujet mais je n'y ai pas trouv� les r�ponses que j'attendais.
mon probl�me est que j'ai fait un programme douloureusement long :)
alors jme suis dit qu'il y avait surement un probl�me, d'ou mon interet pour la question de recopie des vectors.

dans les differents postes que j'ai lu, �a parlait surtout de mettre une reference � un vector comme variable d'entr�e d'une fonction.

moi au contraire j'ai plusieurs fonctions qui renvoient des vectors, et d'autres fonctions qui utilisents des valurs de ces vectors.

exemple :

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 27 28 29 30 31 32 33 34 35 36 37 38

vector<double> C1() { string line; ifstream myfile ("test2.txt"); vector<double> coord;   if (myfile.is_open()) {   int i=0;   while ( getline (myfile,line)) {   stringstream ss(line);   double nombre1; double nombre2; double nombre3; char sep;     while(ss >> nombre1 >> sep >> nombre2 >> sep >> nombre3) { coord.push_back(nombre1); }   ++i;   }   myfile.close(); }   else cout << "Unable to open file"; return coord; }

---

-> fonction qui cree un vector

dans mes autres fonctions, j'utilise c1()[i] pour appeler i+1 �me valeur de mon vector.

vaut-il mieux utiliser une reference? et si oui , comment faire ? ma fonction doit elle renvoyer une reference plutot que le vector, ou dois-je par la suite appeler une reference plutot que le vector?

merci pour vos reponses!

tu peux faire comme �a:

Code:

---

1 2 3 4 5 6 7 8 9 10 11

void C1( vector< double > & coord ) { // remplit ton vector }   int main() { vector< double > coord; C1( coord ); }

---

Sinon pour ton ifstream, tu n'as pas � appeler close(), ce serait fait automatiquement � la fin du scope.

Avec cette fonction, utiliser une r�f�rence ferait, si tu as de la chance, planter ton application. Pourquoi ? Car tu utilises une variable "morte" : renseigne toi sur la dur�e de vie des variables et sur les blocs d'instructions.

Ce qu'il faudrait faire pour optimiser ton application c'est de cr�er une seule fois ce vecteur ( et �ventuellement le recr�er si le fichier �tait modifi� ) . C�d :

Code:

---

1 2 3

const vector<double>& vc1(C1()); // ou vector<double> vc1(C1()); // s'il doit être modifié par la suite.

---

Dans d'autre situation, si tu veux �viter la recopie inutile de vector tu peux faire qqch comme :

Code:

---

1 2 3 4 5 6 7 8

void f(std::vector<int>& ret) { // modification de ret. // pas de return. } ... void g(void) { std::vector<int> vi; f(vi); }

---

PS : tu devrais utiliser les exceptions dans C1 � mon avis ( quand le fichier ne peut �tre lu ) .

effectivement mon vector n'est plus modifi� par la suite donc je peux utiliser const.
mais j'ai pas compris comment je devais l'�crire? dans le prototype de ma fonction?

Non, dans le type de l'objet qui r�cup�re le vecteur. Comme ma premi�re ligne de code ci-dessus.

j'ai fait ce que Nikko34 sugg�rait. c'est d�sormais tr�s rapide (2s au lieu de 50s :) ).
mais je me demande si on ne paut pas l'am�liorer encore car j'ai deux fonctions au debut desquelles je rappelle les vectors.
en gros estce que je ne pourrai pas les creer une seule fois, en vector globaux?

Disons que tout est possible, mais que g�n�ralement les variables globales montre un probl�me de conception.

Si tu passes � chaque fois ton vecteur par r�f�rence � tes fonctions, je ne pense pas que l'utilisation de variable globale am�liore les perfs. Ce qui te p�nalisait avant �tait la recopie. En passant par r�f�rence, tu n'as plus de recopie. Je pense que tu devrais bencher ton code pour voir ce qui prend du temps et te concentrer l�-dessus.