Polskie znaki - własna funkcja

Question

Witam wszystkich.

Mam problem z napisaną  funkcją ,która jak napomknąłem w temacie dodaje polskie znaki w konsoli

 

Kod:

#include <iostream>

using namespace std;

string pl(string napis)
{
    int i;
    int dlugosc=napis.length();

    	for(i=0;i<dlugosc;i++)
    {

    int pozycja=napis.find('ą');
    if (pozycja!=string::npos)
        {
            for (i=0;i<=dlugosc;i++)
            {

                if (napis[i]=='ą')
                {
                    napis.replace(i,1,"\245");

                }
            }
        }
    }

    for(i=0;i<dlugosc;i++)
    {

    int pozycja=napis.find('ć');
    if (pozycja!=string::npos)
        {
            for (i=0;i<=dlugosc;i++)
            {

                if (napis[i]=='ć')
                {
                    napis.replace(i,1,"\206");

                }
            }
        }
    }

    for(i=0;i<dlugosc;i++)
    {

    int pozycja=napis.find('ę');
    if (pozycja!=string::npos)
        {
            for (i=0;i<=dlugosc;i++)
            {

                if (napis[i]=='ę')
                {
                    napis.replace(i,1,"\251");

                }
            }
        }
    }

    for(i=0;i<dlugosc;i++)
    {

    int pozycja=napis.find('ł');
    if (pozycja!=string::npos)
        {
            for (i=0;i<=dlugosc;i++)
            {

                if (napis[i]=='ł')
                {
                    napis.replace(i,1,"\210");

                }
            }
        }
    }

    for(i=0;i<dlugosc;i++)
    {

    int pozycja=napis.find('ń');
    if (pozycja!=string::npos)
        {
            for (i=0;i<=dlugosc;i++)
            {

                if (napis[i]=='ń')
                {
                    napis.replace(i,1,"\344");

                }
            }
        }
    }

    for(i=0;i<dlugosc;i++)
    {

    int pozycja=napis.find('ó');
    if (pozycja!=string::npos)
        {
            for (i=0;i<=dlugosc;i++)
            {

                if (napis[i]=='ó')
                {
                    napis.replace(i,1,"\242");

                }
            }
        }
    }

    for(i=0;i<dlugosc;i++)
    {

    int pozycja=napis.find('ś');
    if (pozycja!=string::npos)
        {
            for (i=0;i<=dlugosc;i++)
            {

                if (napis[i]=='ś')
                {
                    napis.replace(i,1,"\230");

                }
            }
        }
    }

    for(i=0;i<dlugosc;i++)
    {

    int pozycja=napis.find('ż');
    if (pozycja!=string::npos)
        {
            for (i=0;i<=dlugosc;i++)
            {

                if (napis[i]=='ż')
                {
                    napis.replace(i,1,"\276");

                }
            }
        }
    }

    for(i=0;i<dlugosc;i++)
    {

    int pozycja=napis.find('ź');
    if (pozycja!=string::npos)
        {
            for (i=0;i<=dlugosc;i++)
            {

                if (napis[i]=='ź')
                {
                    napis.replace(i,1,"\253");

                }
            }
        }
    }

    for(i=0;i<dlugosc;i++)
    {

    int pozycja=napis.find('Ą');
    if (pozycja!=string::npos)
        {
            for (i=0;i<=dlugosc;i++)
            {

                if (napis[i]=='Ą')
                {
                    napis.replace(i,1,"\244");

                }
            }
        }
    }

    for(i=0;i<dlugosc;i++)
    {

    int pozycja=napis.find('Ć');
    if (pozycja!=string::npos)
        {
            for (i=0;i<=dlugosc;i++)
            {

                if (napis[i]=='Ć')
                {
                    napis.replace(i,1,"\217");

                }
            }
        }
    }

    for(i=0;i<dlugosc;i++)
    {

    int pozycja=napis.find('Ę');
    if (pozycja!=string::npos)
        {
            for (i=0;i<=dlugosc;i++)
            {

                if (napis[i]=='Ę')
                {
                    napis.replace(i,1,"\250");

                }
            }
        }
    }

    for(i=0;i<dlugosc;i++)
    {

    int pozycja=napis.find('Ł');
    if (pozycja!=string::npos)
        {
            for (i=0;i<=dlugosc;i++)
            {

                if (napis[i]=='Ł')
                {
                    napis.replace(i,1,"\235");

                }
            }
        }
    }

    for(i=0;i<dlugosc;i++)
    {

    int pozycja=napis.find('Ń');
    if (pozycja!=string::npos)
        {
            for (i=0;i<=dlugosc;i++)
            {

                if (napis[i]=='Ń')
                {
                    napis.replace(i,1,"\343");

                }
            }
        }
    }

    for(i=0;i<dlugosc;i++)
    {

    int pozycja=napis.find('Ó');
    if (pozycja!=string::npos)
        {
            for (i=0;i<=dlugosc;i++)
            {

                if (napis[i]=='Ó')
                {
                    napis.replace(i,1,"\340");

                }
            }
        }
    }

    for(i=0;i<dlugosc;i++)
    {

    int pozycja=napis.find('Ś');
    if (pozycja!=string::npos)
        {
            for (i=0;i<=dlugosc;i++)
            {

                if (napis[i]=='Ś')
                {
                    napis.replace(i,1,"\227");

                }
            }
        }
    }

    for(i=0;i<dlugosc;i++)
    {

    int pozycja=napis.find('Ż');
    if (pozycja!=string::npos)
        {
            for (i=0;i<=dlugosc;i++)
            {

                if (napis[i]=='Ż')
                {
                    napis.replace(i,1,"\275");

                }
            }
        }
    }

    for(i=0;i<dlugosc;i++)
    {

    int pozycja=napis.find('Ź');
    if (pozycja!=string::npos)
        {
            for (i=0;i<=dlugosc;i++)
            {

                if (napis[i]=='Ź')
                {
                    napis.replace(i,1,"\215");

                }
            }
        }
    }
    return napis;
}



string abc="ąćęłńóśżźĄĆĘŁŃÓŚŻŹ";

int main()
{
    cout << pl(abc) << endl;
    return 0;
}

 

Problem głównie dotyczy litery "Ć" oraz "ą"  , poniewarz "Ć" czyta jako "Ź" ,a "ą" jako "Ą".Co dziwne problemu niema gdy zostawie tylko  te linie:

#include <iostream>

using namespace std;

string pl(string napis)
{
    int i;
    int dlugosc=napis.length();

    	for(i=0;i<dlugosc;i++)
    {

    int pozycja=napis.find('ą');
    if (pozycja!=string::npos)
        {
            for (i=0;i<=dlugosc;i++)
            {

                if (napis[i]=='ą')
                {
                    napis.replace(i,1,"\245");

                }
            }
        }
    }

    for(i=0;i<dlugosc;i++)
    {

    int pozycja=napis.find('Ć');
    if (pozycja!=string::npos)
        {
            for (i=0;i<=dlugosc;i++)
            {

                if (napis[i]=='Ć')
                {
                    napis.replace(i,1,"\217");

                }
            }
        }
    }

}

string abc="ąćęłńóśżźĄĆĘŁŃÓŚŻŹ";

int main()
{
    cout << pl(abc) << endl;
    return 0;
}

Pomużcie prosze.

Answer 1

Zbyt przekombinowane jak dla mnie. Ja kiedyś napisałem taką funkcję i posługuję się nią do dzisiaj:

#include <iostream>
#include <string>

using namespace std;
string pl( string x )    //Funkcja konwertujaca znaki
{
	for( int i = 0; i < x.length(); i++ )
	{
		switch( x[ i ] )
		{
		case 'Ą': x[ i ] = (char)164; break;
		case 'Ź': x[ i ] = (char)171; break;
		case 'Ć': x[ i ] = (char)143; break;
		case 'Ę': x[ i ] = (char)168; break;
		case 'Ł': x[ i ] = (char)157; break;
		case 'Ń': x[ i ] = (char)227; break;
		case 'Ó': x[ i ] = (char)224; break;
		case 'Ś': x[ i ] = (char)151; break;
		case 'Ż': x[ i ] = (char)189; break;
		case 'ą': x[ i ] = (char)165; break;
		case 'ć': x[ i ] = (char)134; break;
		case 'ę': x[ i ] = (char)169; break;
		case 'ł': x[ i ] = (char)136; break;
		case 'ń': x[ i ] = (char)228; break;
		case 'ó': x[ i ] = (char)162; break;
		case 'ś': x[ i ] = (char)152; break;
		case 'ź': x[ i ] = (char)171; break;
		case 'ż': x[ i ] = (char)190; break;
		}
	}
	return x;
}

int main()
{
	cout << pl( "Zażółć gęślą jaźń" );
	
	return 0;
}

Jak piszesz programy pod Windowsem, to najłatwiej byłoby użyć po prostu funkcji
setlocale( "PL", LC_ALL ) i pisać normalnie z polskimi znakami. W taki sposób:

#include <iostream>
#include <Windows.h>

using namespace std;

int main()
{
	setlocale( "PL", LC_ALL );
	
	cout << "Zażółć gęślą jaźń";
	return 0;
}

Niestety nie w każdym środowisku taka sztuczka działa :-)

Answer 2

Mało komu będzie się to chciało sprawdzić, ze względu na strasznie długi kod. Parę uwag mam: 1. Ponoć lepsza jest preinkrementacja od postinkrementacji. 2. Na pewno potrzebujesz aż tyle pętli for, tyle ifow? Jeżeli ja bym miał coś takiego zrobić, to sprobowalbym umieścić wszystkie polskie znaki w tabeli, a potem pętla sprawdzająca czy wyraz[i] znajduje się w tabeli, jeśli tak, to w zależności od tego, gdzie, tak bym podmienil. Tu chyba nawet switcha można by było użyć. 3. Funkcje też nie powinny być zbyt długie - ok 10 linii 4. Pomużcie, poniewarz, strasznie rażą w oczy

Comments

Dzięki za rady