cout nie wyświetla zawartości stirng.

Question

Witam, 

mam za zadanie odwrócić wyraz z jednej zmiennej string i zapisać już odwrócony do drugiej. 
Dlaczego program nic nie pokazuje? 

#include <iostream>

using namespace std;

int main()
{
  string liczby, pomoc1, pomoc2;
  int dl, j=0;



        cin>>liczby;
        dl=liczby.length();


          pomoc1=liczby;


        for(int i=(dl-1); i>=0; i--)
        {
         pomoc2[j]=liczby[i];
         j++;
        }
            cout<<pomoc2<<endl;


    return 0;
}

 

Comments

Gwoli ścisłości, możesz użyć std::reverse:

reverse(liczby.begin(), liczby.end());
cout << liczby;

 

---

@amtrax, dziękuje panowie za pomoc. 

Jesteście wielcy <3 

Temat zamykam. 
Pozdrowienia 

Answer 1

Zauważ, że

pomoc2[j] = liczby[i];

Przypisuje jeden znak z "liczby" do jednego elementu "pomoc2", tylko problem jest taki, że "pomoc2" jest pusta i elementy, do których się odwołujesz nie istnieją. Żeby to rozwiązać lepiej byłoby abyś od końca dodawał znaki za pomocą operator +=. Jeśli nie zrozumiałeś o co chodzi z opisu to spójrz na ten przerobiony kod (przy okazji ci go upiększyłem):

std::string liczby, pomoc;

std::cin >> liczby;
const int dl = liczby.length();

for (int i = dl - 1; i >= 0; --i)
     pomoc += liczby[i];

std::cout << pomoc << "\n";

PS. Rozwiązanie jakie proponuje @Velta jest lepsze niż użycie pętli.   

Answer 2

Witam

Mi wyrzuca segmentation fault - cały problem leży tutaj:

	for (int i = (dl - 1); i >= 0; i--){
		pomoc2[j] = liczby[i];
		j++;
	}

Zmienna pomoc2 jest pusta, a zmieniamy jej j-ty element.

Zmień na:

	for (int i = (dl - 1); i >= 0; i--){
		pomoc2 += liczby[i];
		j++;
	}

Operator += dodaje na koniec stringa liczby[i].

Dodam, że nie przechodziło mi bez dodania biblioteki string - nie szło "dać" do cout string'a.

Answer 3

jako ze string pomoc2 nie jest zadeklarowany na długość [0] i jak probujesz przypisać wartość do pomoc2[1] wychodzisz poza zakres tablicy, możesz zadeklarować pomoc2=liczby;

wtedy będziesz miał odpowiednia długość tablicy taki kod:

#include <iostream>

using namespace std;

int main()
{
	string liczby, pomoc1, pomoc2;
	int dl, j = 0;

	cin >> liczby;
	dl = liczby.length();

	pomoc1 = liczby;
	pomoc2 = liczby; // <---- tytaj deklaracja

	for (int i = (dl - 1); i >= 0; i--)
	{
		pomoc2[j] = liczby[i];
		j++;
	}

	cout << pomoc2 << endl;
	return 0;
}

Ps. zmienna długosc1 jest niepotrzeba w kodzie bo i tak w petli masz liczby[i];

Mozesz tez rozwiazać to w taki sposob, do zmiennej dlugosc2 dodac poszczegulne znaki z stringa liczby, lepjej to zobaczysz w kodzie:

#include <iostream>

using namespace std;

int main()
{
	string liczby, pomoc2;
	int dl;

	cin >> liczby;
	dl = liczby.length();

	for (int i = (dl - 1); i >= 0; i--)
	{
		pomoc2 += liczby[i]; //<---- znak += jest kluczowy
	}
	cout << pomoc2 << endl;
	return 0;
}