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.