correction et conseil sur un petit exercice sur les iterateurs

Voilà, comme d’hab je progresse à mon rythme en suivant le cours de C++ de zeste de savoir (actuellement au chapitre sur les conteneurs Parti I)… Bref dans ce petit exercice, j’avais pour but de remplacer les espaces à l’intérieur d’une chaîne de caractères par un caractère quelconque saisie. J’aimerais avoir des retours sur ce code afin de m’améliorer encore plus, et dire si possible, si je peux ajouter certaines choses pour m’entraîner (ou bien des sites dans lesquels je puisse trouver des exos <avec correction> pour débutant). Pour info, j’ai emprunté la petite chaîne de caractères de ce tutoriel, n’y voyez pas noir . Merci en avance…

Voici le code :

#include <iostream>
#include <string>
#include <algorithm>

int main ()
{
    std::string p { "Exemple illustrant le tutoriel C++ de zeste de savoir." };
    
    char espace { ' ' };
    std::cout << "Entre un caractere : ";
    char caractere { '_' };
    std::cin >> caractere;
    

    std::string::iterator it_debut_sequence { p.begin() };
    std::string::iterator it_fin_sequence { p.end() };

    for (std::string::iterator it { p.begin() }; it != it_fin_sequence; ++it)
    {
        std::string::iterator it_find { std::find(it_debut_sequence, it_fin_sequence, espace) };
        
        if (* it_find == espace)
        {
            * it_find = caractere;
            it_debut_sequence = it_find;
        }

        else
        {
            break;
        }
    }

    std::cout << p << std::endl;

    return 0;
}

Je ne sais pas si je peux sécuriser l’entré utilisateur dans ce cas de figure… Même si je rentre tout et n’importe quoi, un tronque s’effectue automatiquement sans que j’interviens, du coup ça m’arrange un peu.

Salut je viens de débuter aussi neanmoins j’ai déjà fini de lire ce cours. Je me demandais juste si l’on aurait pas fait simple comme ceci:

#include <iostream> #include <string> #include <algorithm> #include <ctype>

int main () { std::string p { "Exemple illustrant le tutoriel C++ de zeste de savoir." };

std::cout << "Entre un caractere : ";
char caractere { '_' };
secured_input(caractere) //En admettant qu'on l'ait implementé

std::for_each(std::begin(p),std::end(p),[caractere](auto &element)->{
if(isspace(element))
   element=caractere;
});

std::cout<<p<<std::endl;
return 0;

}

Je l’ai pas testé ….

Le plus simple, c’est la fonction replace comme suggère olybri.

Sinon, vos programme n’ont pas un comportement identique, car tu utilises la fonction (std::)isspace qui identifie les caractères qui ont une sémentique de séparation (d’après la doc ' ', '\f', '\r', '\n', '\t' et '\v') alors qu’amalure ne remplace que les caractères ' '. Voir la spec (la consigne) pour savoir ce qu’il faut vraiment faire.

Aussi, remarque que tu peux utiliser le for-range pour parcourir les conteneurs en entier, ça t’évites de faire des lambdas

et la syntaxe de la lambda c’est [capture](arguments)->type de retour{corps de fonction}

Oui je vois , merci. J’ai omis le type de retour.

Bonjour j’ai décidé de revoir certains acquis car j’avais vu que certaines choses que je croyais digérer ne les sont pas ou les sont moins que je les pensais. Donc je prendrai un peu de temps avant de poster un autre message ici (Merci bcp pour ces retours)

Bref je n’étais pas au courant de l’existence de la fonction std::replace()

J’ai appris que std::find() renvoie un itérateur sur le premier élément correspondant à la recherche. Du coup je voulais qu’elle renvoie un itérateur sur le deuxième élément correspondant a la recherche et ainsi de suite, et obtenir le tout par l’intermédiaire d’une boucle.

Et c’est à partir de là qu’il m’est venu l’idée de remplacer la valeur de chaque itérateur que cette fonction renvoie par un underscore, puis de laisser le choix à l’utilisateur. Pour être concis je voulais m’entraîner. Alors qu’avec la fonction std::replace() je ne m’exerce point comme je le fais… En tout cas ça m’a bcp aidé de savoir qu’elle existe. Je prends vos conseils et je l’appliquerai… bref je reposterai bientôt

Aucun soucis, n’hésite surtout pas à poster, c’est pas grave de faire ces erreurs, c’est plus grave de ne pas les voir, et ça on peut t’y aider bien plus efficacement que ce que tu peux t’en rendre compte seul.

On peut pas tout savoir, essaie de repérer les problématiques très commune et prends l’habitude de rechercher ce qui existe déjà pour y répondre.
Prends aussi le temps de parcourir la doc du standard dans une vue d’ensemble, ça te permettra d’avoir une idée de ce qu’il y a à disposition pour y revenir dessus en détail quand tu en auras besoin.

Et c’est bien ce que tu fais, c’est juste que le choix de ta boucle n’est pas adapté donc ça la rend compliquée à relire (et entraine quelques petites allocations supplémentaires mais ça limite on s’en fous).

Oui c’est pour ça qu’elle n’apparaît même pas dans mon post et qu’elle arrive qu’en petit à la fin de celui d’Olybri en mention "pour info". On a bien compris que le but est de t’entrainer sur les structures de contrôle.

au plaisir