syntaxe while avec un itérateur

Bonjour.

je m'initie au C++ et je rencontre un problème lors de la compilation d'un bout de code

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14

std::ifstream fichier_lecture {"texte.txt"}; std::string ligne {""};   while (std::getline(fichier_lecture,ligne)) { nb_ligne++; std::string::iterator it {std::find(std::begin(ligne),std::end(ligne),' ')}; while (it != std::end(ligne)) { ... } ... } }

---

compile très bien

alors que

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

std::ifstream fichier_lecture {"texte.txt"}; std::string ligne {""};   while (std::getline(fichier_lecture,ligne)) { nb_ligne++; while (std::string::iterator it {std::find(std::begin(ligne),std::end(ligne),' ')} != std::end(ligne)) { ... } ... } }

---

renvoie le message d'erreur

Code:

---

could not convert 'it' from 'std::__cxx11::basic_string<char>::iterator' to 'bool'

---

Pourquoi, dans la deuxième version, le compilateur cherche-t-il à convertir mon itérateur en booléen?
La condition du while reste bien une comparaison entre 2 itérateurs, non?

Merci de votre aide.

J'ai mis un temps à mettre le doigt dessus également :-)

Code:

---

while (std::string::iterator it {std::find(std::begin(ligne),std::end(ligne),' ')} != std::end(ligne))

---

Tu ne peux pas déclarer une nouvelle variable au sein d'une condition while comme on le fait au début d'un for, par exemple. Par contre, il reste tout-à-fait possible de construire une instance anonyme d'un type (qui formera une xvalue). Et pour ce faire, il suffit de supprimer le symbole « it » seul au sein de ton expression.

Ça compilera en l'état (ici, le corps de ta boucle est vide), par contre tu ne pourras pas faire référence à cet itérateur au sein de ta boucle et, donc, tu ne pourras pas le faire avancer non plus. Donc en l'état l'expression ne sert à rien, mais elle compile quand même.

explication limpide! :applo:
J'ai effectivement cru que l'on pouvait faire avec le while la même chose qu'avec le for.

Merci beaucoup.

Malheureusement il manque à while la zone d'initialisation qui permet de déclarer localement des nouvelles variables dans for depuis toujours en C++, et dans if et switch depuis le C++17.

En version for, il y a une façon redondante et une façon obfusquée de faire la chose
Et c'est pas que la version redondante ne soit pas obfusquée non plus dans mon implémentation: https://godbolt.org/z/ocoWTT6x1

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

#include <string_view> #include <algorithm> #include <print> // C++23   void split_redondant(std::string_view s) { for (auto it = std::find(s.begin(), s.end(), '-'), old=s.begin() ; it != s.end() ; old=it+1, it = std::find(old, s.end(), '-')) { std::string_view sub(old, it); // subview en C++26 std::println("-> {}", sub); } }   void split_obfusqué(std::string_view s) { for (auto it = s.begin(), old=s.begin() ; (it = std::find(old, s.end(), '-')) != s.end() // Tu vois l'affectation dégueulasse en plein milieu des parenthèses qui isolent le membre droit à tester? ; old=it+1) { std::string_view sub(old, it); // subview en C++26 std::println("-> {}", sub); } }

---

Ca peut paraitre surprenant, mais je préfère utiliser for pour itérer et réserver while a des conditions complexes et non itératives.

EDIT: la version obfusquée ne l'est pas assez, j'avais oublié std::exchange (une sorte de post-incrémentation avec une nouvelle valeur autre que l'ex+1). Ne faites pas ça à la maison.

Code:

---

1 2 3

for (auto it = s.begin(), old=s.begin() ; (it = std::find(std::exchange(old, it+1), s.end(), '-')) != s.end() ;)

---

Salut à tous.

Il y a surtout un problème de logique dans ton exemple.
Tu retrouveras toujours le même premier espace et ta boucle ne progressera pas.

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14

std::string::iterator it = std::begin(ligne);   while (it != std::end(ligne)) { it = std::find(it, std::end(ligne), ' ');   if (it == std::end(ligne)) break;   // Traitement de l'espace trouvé ...   ++it; // On avance après l'espace }

---