qu'est-ce qui ne va pas dans ce code?

Question

j'ai le code suivant

#include <iostream> 
#include <string> 
using namespace std; 
string replace(string s){ 

    for (int i=0;i<s.length();i++){ 
     if (s[i]> 'b' && s[i]<'f'){ 
      s.erase(s[i]); 

     } 

    } 
    return s; 
} 
int main(){ 

    string s; 
    cin>>s; 
    cout<<replace(s)<<endl; 


    return 0; 

} 

si j'entre géorgie il me montrer exception "abort a été appelé" pourquoi?

Answer 1

std::string::erase() prend une paire d'index, ou des itérateurs.

Regardez this link.

Ici s[i] donne un caractère, qui est par erreur converti en size_t donc, en fonction de votre chaîne, vous essayez en fait de supprimer un élément qui n'existe pas.

Une solution plus propre serait:

#include <string> 
#include <iostream> 
#include <cstdlib> 

bool should_be_removed(char c) { return (c > 'b') && (c < 'f'); } 

int main() 
{ 
    std::string s; 
    std::cin >> s; 
    s.erase(std::remove_if(s.begin(), s.end(), should_be_removed), s.end()); 

    return EXIT_SUCCESS; 
}