Polskie znaki dla wyrazów znajdujących się w zmiennej string.

Question

Witam. Piszę z problemem dotyczącym polskich znaków w konsoli. Mianowicie wczytuję z pliku tekstowego do zmiennej string napis z polskimi znakami. Wyświetlając go w konsoli pokazują się krzaki. Chciałem załatwić to za pomocą funkcji tr(nazwa_zmiennej) jednak to nie działa. Pojawia się error: cannot convert 'std::string {aka std::basic_string<char>}' to 'char*' for argument '1' to 'char* tr(char*)'|. Prosił bym o pomoc w wymyśleniu jakiegoś sposobu.

PROGRAM:

int main()
{
    string kk;
    fstream plik;
    plik.open("dokument.txt", ios::in);
    string linia;
    getline(plik, linia);
    kk=linia;
    cout <<linia<< endl;
    return 0;
}

FUNKCJA TR:

char *tr(char *str)
  {
   static char buff[256];
   char cp[]="\245\206\251\210\344\242\230\276\253\244\217\250\235\343\340\227\275\215ąćęłńóśżźĄĆĘŁŃÓŚŻŹ";
   if(strlen(str)>=sizeof(buff)) return str;
   char *bf=buff;
   while(*str)
     {
      char *pos=strchr(cp+18,*str);
      *(bf++)=pos?*(pos-18):*str;
      ++str;
     }
   *bf=0;
   return buff;
  }

 

Answer 1

Zrób tak:

cout<<strdup(tr(linia.c_str()))<<'\n';

 

Comments

Znowu error: invalid conversion from 'const char*' to 'char*' [-fpermissive]|

---

sry odwrotnie napisałem :)

cout<<tr(strdup(linia.c_str()))<<endl;

 

---

Dzięki wielkie! Działa. 

---

A po co to strdup tam? Użycie tej funkcji w taki sposób powoduje wyciek pamięci, a to błąd.

std::string tr(const char *str)
{
	const char * cp = "\245\206\251\210\344\242\230\276\253\244\217\250\235\343\340\227\275\215ąćęłńóśżźĄĆĘŁŃÓŚŻŹ";
	
	std::string buff;

	while(*str)
	{
		char *pos = strchr(cp + 18, *str);
		buff.push_back( pos ? *(pos - 18) : *str );
		++str;
	}

	return buff;
}

// użycie
cout << tr( linia.c_str() ) << '\n';

 

Można dodać jeszcze taką wersję dla std::stringa:

std::string& tr(std::string &str)   // nie tworzy kopii, działa na stringu źródłowym
{
	const char * cp = "\245\206\251\210\344\242\230\276\253\244\217\250\235\343\340\227\275\215ąćęłńóśżźĄĆĘŁŃÓŚŻŹ";
	
	std::string::iterator i1 = str.begin();
	std::string::iterator i2 = str.end();

	while(i1 != i2)
	{
		char *pos = strchr(cp + 18, *i1);
		*i1 = pos ? *(pos - 18) : *i1;
		++i1;
	}
	
	return str;
}

// użycie
cout << tr(linia) << '\n';

cout << tr("śćźżół") << '\n'; // <-- poprzednia wersja

 

---

strdup() bo chciał to dla funkcji char* tr(char*)

a Ty zmieniłeś funkcję tr() ;)

Ale przyznaję rację Twój pomysł jest lepszy

---

A dlaczego nie std::transform(...) ?

---

@JKluseczka, tak, ale wtedy wystarczyłoby chamskie rzutowanie, albo po prostu dodanie const do parametru.

@mokrowski, chciałem użyć std::transform, ale pomyślałem, że dla jasności przekazu lepiej będzie przerobić funkcję tak, żeby OP wiedział, co tam się dzieje.