Niechciane znaki w łańcuchu

Question

Ostatnio postanowiłem napisać sobie prosty program zamieniający liczby podane przez użytkownika na złote. Ustawiłem wszystko tak jak chciałem program działa w większości przypadków gdy podaje liczbę 23 wyświetla 23 zł itp. Gdy podam jakąś literę wtedy program wraca na początek, ale gdy podam coś w stylu "2s2" program wyświetla to jako "2s2 zł". Chciałbym aby jedynym akceptowanym znakiem przez program oprócz cyfr była kropka.

#include <iostream>

using namespace std;

void pieniadze(string lancuch); //prototyp funkcji

int main()
{
    bool dalej;
    dalej = true;
    do
    {
        cin.clear();
        cin.sync();

        string napis;
        cout << "Podaj liczbe ktora mam zamienic na PLN ";
        getline(cin,napis);

        int dlugosc;
        dlugosc = napis.length();

        for (int i=0; i<dlugosc; i++)
        {
            if (napis[i]<45||napis[i]==46||napis[i]==47||napis[i]>57)
                dalej = false;
            else
                dalej = true;
        }

        if (dalej==true)
            pieniadze(napis);


    }
    while(dalej==false);

    return 0;
}
void pieniadze(string lancuch)
{
    char kropka = '.';
    size_t szukaj = lancuch.find(kropka);

    if (szukaj!=string::npos)
    {
        string zlote = lancuch.substr(0,szukaj);
        lancuch.erase(0,szukaj+1);

        int dlugosc_lancucha = lancuch.length();
        if (dlugosc_lancucha>2)
            lancuch.erase(2,dlugosc_lancucha-2);
        else if (dlugosc_lancucha==1)
            lancuch.insert(0,"0");
        cout << zlote << " zl " << lancuch << " gr";
    }
    else
        cout << lancuch << " zl";
}

 

Answer 1

• Aby kod dobrze działał powinieneś dołączyć nagłówek <string> w którym znajduje się klasa basic_string<> oraz jej konkretyzacje takie jak string,wstring,u16string,u32string, oraz metody do konwersji pomiędzy typami łańcuchowymi a numerycznymi i odwrotnie, nie zapominając o klasie cech typowych char_traits<>.
• Zamiast kropki lepiej użyć obiektu locale, który sprawi, że można użyć innych separatorów dziesiętnych zależnie od języka, kultury, kraju.
• Warto użyć algorytmu find_if z argumentami str.cbegin(),str.cend() oraz funkcją lub lambdą sprawdzającą czy to liczba lub separator dziesiętny.
• Standard języka definiuje również inne sposoby, pamiętaj by nie używać nic powodującego "niezdefiniowane zachowanie" (zdefiniowane przez implementacje, nie przez standard języka).