c++ alfabet morse'a

Question

Cześć,

czy ktoś mógłby mi proszę pomóc poprawić ten kod ,w jak najłatwiejszy  sposób, tak aby działał poprawnie ?

powinien to być program tłumaczący słowa  języku polskim na alfabet morse'a ( dodatkową funkcją mogłoby być odwrotne tłumaczenie.

Z góry dziękuję :))

 

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

main()
{
    string tekst;
    string wynik = "s";
    char alfabet [26] = {'A', 'B', 'C', 'D', 'E', 'F', 'G', 'H', 'I', 'J', 'K', 'L', 'M', 'N', 'O', 'P', 'Q', 'R', 'S', 'T', 'U', 'V', 'W', 'X', 'Y', 'Z'};
    string alfabet_morsea[26] ={".-", "-...", "-.-.", "-..", ".", "..-.", "--.", "....", "..", ".---", "-.-", ".-..", "--", "-.", "---", ".--.", "--.-", ".-.", "...", "-", "..-", "...-", ".--", "-..-", "-.--", "--.." };




    cout << "podaj slowo ktore chcesz przetlumczyæ " << endl;
    cin >> tekst;

    for (int x=0; x<=tekst.length(); x++)
    {
        for (int y=0; y<=26; y++)
        {
            if ( tekst.at(x) == alfabet [y] )
            {
                wynik.push_back( alfabet [y]);
                wynik.push_back( '/');
            }
        }

    }

    cout << endl << wynik << endl << endl;




}

 

Answer 1

• Linia 18: nie <= tylko <, to samo w linii 20.
• Linia 22: to at jest tam zbędne, bo x, jeśli poprawisz warunek, nigdy nie przekroczy zakresu.
• Linia 24: popraw na wynik.append( alfabet_morsea [y]);

Answer 2

Używasz funkcji push_back dla stringa,a powinieneś zrobić to dla vectora. Nie możesz również zapisać alfabet_morse w ten sposób, przydałaby sie tablica dwuwymiarowa ,następna kwestia to jeśli ktoś wprowadzi dane z klawiatury małymi literami  program nie zadziała poprawnie.

Comments

Żeś pojechał ;) std::string posiada metodę push_back, więc może jej użyć (choć może nie tu). Tablica alfabet_morsea jest jak najbardziej ok.