Funkcja usuwająca znaki.

Question

Witam robię program który  ma usuwać znak "_" z wykorzystaniem funkcji.Niestety kiedy program urochamiam na konsoli nic się nie wyświetla.

#include<iostream>
#include<algorithm>
#include<string>
using namespace std;
string znaki(string y,string z)
{
    size_t x;
    while(x!=string::npos)
    {
        
        size_t x=y.find(z);
        y.erase(y.begin()+x);
        
    }
    
    return y;
    
}

int main()
{
    string napis="mama ma_psa_";
    string szukane="_";
    
    cout<<znaki(napis,szukane)<<endl;
   
    

    return 0;
}

 

Answer 1

Zobaczmy co się dzieje w twojej funkcji

string znaki(string y,string z)
{
    size_t x; // niezainicjowana wartość
    while(x!=string::npos) // porównanie z niezainicjowaną wartością
    {
        size_t x=y.find(z); // przesłonięcie zmiennej x (nowa zmienna o tej samej nazwie)
        y.erase(y.begin()+x);  //wywołanie erase bez sprawdzenia poprawności 
    }
     
    return y;
     
}

A teraz wersja poprawiona

string znaki(string y,string z)
{
    size_t x=y.find(z);
    while(x!=string::npos)
    {
        y.erase(y.begin()+x);
        x=y.find(z);
    }
     
    return y;
}

 

Answer 2

Chyba nie chcesz zrobić tego kasowania z linijki 12., gdy std::string::find z linijki 11. niczego nie znajdzie, nie? Wtedy x = std::string::npos, a y.begin()+x to już niezdefiniowane zachowanie (undefined behavior).