Discussion :

[Nikolai]

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 :

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
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!

[nikko34]

tu peux faire comme �a:

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

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.

[hiura]

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 :

S�lectionner tout - Visualiser dans une fen�tre � part

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 :

S�lectionner tout - Visualiser dans une fen�tre � part

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 ) .

[Nikolai]

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?

[hiura]

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

[Nikolai]

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?

[hiura]

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

[3DArchi]

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.