Discussion :

[zenux]

Bonjour,

Voici un code qui ne veut pas compiler :

Code :

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

1
2
for (int j=dirs_size()-2, vector<string>::iterator i = dirs.begin() ; j!=0 ; ++i, --j) //ligne 30
	reverse_dirs[j] = *i;

==>
30: erreur: expected initializer avant « i »
30: erreur: « i » n'a pas été déclaré dans cet horizon

Mais si je met "int j=dirs_size()-2" en dehors de la boucle, je n'ait plus d'erreur...

Pourquoi mon code ne veut-il pas compiler ?

Merci d'avance...

[Laurent Gomila]

Parce que tu ne peux pas déclarer deux variables de types différents dans la même instruction (ie. séparés par l'opérateur virgule). Donc il faut bien en déclarer une en dehors de la boucle.

[obliveon]

De plus ce n'est pas très lisible, pas très sympathique pour celui qui devra se charger de la maintenance du code (même si c'est pour soi même).

[niglo]

Code :

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

1
2
for (int j=dirs_size()-2, vector<string>::iterator i = dirs.begin() ; j!=0 ; ++i, --j) //ligne 30
	reverse_dirs[j] = *i;

devient donc :

Code :

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

1
2
3
4
5
int j=dirs_size()-2;
for (vector<string>::iterator i = dirs.begin() ; j!=0 ; ++i, --j)
{
	reverse_dirs[j] = *i;
}

et pense à

[Luc Hermitte]

Petites pistes d'"améliorations":
- std::reverse, std::reverse_copy
- std::copy + reverse_iterator