chaîne de caractères.find() ne retourne pas les -1

Le code ci-dessous est très simple. Que je sache, si string::find() n'a pas trouvé matches elle retourne -1. Mais pour certaines raisons, le code ci-dessous ne fonctionne pas. Chaque fois que j'exécute ce code j'obtiens boucle sans fin. Merci pour l'aide!

#include <string>
#include <iostream>
using namespace std;

int main()
{
    string text;
    text = "asdasd ijk asdasd";
    string toReplace = "ijk";
    cout<<text<<endl;
    int counter = 0;
    while ( text.find(toReplace) != -1)
        counter++;

    cout<<counter<<endl;

    system("pause");
}
  • Il me semble certain temps passé à lire la documentation de std::basic_string::find() serait susceptible de faire des merveilles dans l'apprentissage du fonctionnement de quelque chose.
  • Mise à JOUR: j'ai essayé d'utiliser tout ( le texte.trouver(toReplace) != string::obnl ) mais j'ai toujours boucle sans fin 🙁
  • Avez-vous essayez de faire quelque chose dans ce tout-corps en plus d'incrémenter un compteur de sens? C'est là la première case, le temps à lui seul n'est pas pour le faire aucune faveur. Peut-être changer d'endroit où vous commencez à la recherche après chaque touche réussie.
InformationsquelleAutor eoLithic | 2013-04-01
  1. 10

    Côté de l'autre les réponses qui sont tout à fait correct, je voulais juste ajouter que votre boucle while aurait produit une boucle sans fin de toute façon. Par exemple:

    while(text.find(toReplace) != std::string::npos)
    counter++;

    sera une boucle sans fin, car il va continuer à essayer de trouver le toReplace chaîne dans text et il sera toujours le trouver (c'est parce que trouver commence à partir du début de la chaîne à chaque fois). Ce n'est probablement pas ce que vous souhaitiez.

    • Merci!!!! C'est pourquoi je m'en boucle à chaque fois!
    • Cheers! Content d'avoir pu aider
    • ha ha, c'est ce nouveau paradigme ou modèle de conception?
    InformationsquelleAutor Memento Mori
  2. 8

    std::string::find retourne std::string::npos si la recherche de sous-chaîne n'est pas trouvée, pas -1. La valeur exacte de npos la mise en œuvre est définie, afin de l'utiliser npos, comme dans

    while ( text.find(toReplace) != std::string::npos)

    Venez pour penser à elle, find ne pouvait pas retourner -1 même si elle le voulait, parce que le type de retour de la trouver est de std::size_t qui est un unsigned type.

    En outre, les trouver va toujours de recherche pour le première occurrence de la sous-chaîne, peu importe combien de fois vous l'appelez. Si vous souhaitez parcourir toutes les occurrences que vous devez utiliser la surcharge de find qui prend un deuxième paramètre de la position à partir de laquelle commencer la recherche.

    • npos est -1 (converti en size_type). -1 est sans doute moins élégant, mais pas incorrect.
    • "est -1" et "est -1 converti à size_t" sont des choses différentes 🙂
    • Vrai, mais ce n'est certainement pas de mise en œuvre définies, et en comparant les size_type résultat de find avec -1 donnera toujours le même résultat que la comparaison avec npos.
    • Intéressant à savoir, je ne connaissais pas la valeur des obnl qui a été spécifié par le standard
    InformationsquelleAutor Armen Tsirunyan
  3. 5

    Celui qui vous a dit ceci ou partout où vous le lisez, il a menti à vous.

    Si std::string::find échoue, elle renvoie std::string::npos, ce qui n'est pas -1.

    Vous devez vérifier la documentation au sujet de telles choses, quand vous n'êtes pas sûr.

    Donc, votre while sera quelque chose comme:

    while ( std::string::npos != text.find(toReplace) )

    Ce qui concerne votre commentaire:

    Mise à JOUR: j'ai essayé d'utiliser tout ( le texte.trouver(toReplace) != string::obnl ) mais j'ai toujours en boucle 🙁 – user2167403 10 secondes il y a

    Vous devriez vraiment apprendre à lire l' la documentation. Utiliser une variable pour stocker le dernier résultat de std::string::find (différent de std::string::npos) et l'utilisation std::string::finddeuxième paramètre - pos ( en passant la valeur last_match_position + 1).

    L'omission de la deuxième paramètre, std::string::find commence toujours à partir du début de la chaîne, ce qui entraîne la boucle sans fin.

    InformationsquelleAutor Kiril Kirov
  4. 0

    Dans l'extrait de code que vous avez fourni text variable contient la sous-chaîne "ijk" qui est stocké dans le toReplace variable. Aussi longtemps que dans tout le cycle ni text ou toReplace variables sont modifiées, trouver la méthode retournera toujours pas une valeur de -1 qui est la condition de tout le cycle de continuer.

    Comme déjà dessus décrite dans d'autres commentaires, vous devriez vérifier pas de -1, mais pour std::string::npos.

    InformationsquelleAutor Pavel Krasnopevtsev
  5. 0

    Il ne permet de lire la page de manuel (string::obnl est la réponse).

    Voir http://www.cplusplus.com/reference/string/string/find/

    InformationsquelleAutor Ed Heal