Odwracanie wyrazu z użyciem własnej funkcji

Question

Chciałem zrobić program który odwraca wyraz to znaczy wypisuje go od tyłu z użyciem funkcji żeby je lepiej zrozumieć (właśnie jestem na etapie  9 odc czyli funkcje). Program po wpisaniu jakiegoś wyrazu wyświetla mi odwrócony wyraz i ten który wpisałem na początku np wpisze "wyraz" to wyświetli "zarywwyraz". Nie mogę znaleźć błędu..

using namespace std;

string odwroc (string x, int y);

int main ()
{
   string wyraz;
   int dlugosc;

            cout << "Podaj wyraz do odwrocenia: ";
            cin >> wyraz;

            dlugosc = wyraz.length();
            cout << odwroc(wyraz,dlugosc);

 return 0;
}

string odwroc (string x, int y)
{
    for (int i=y-1; i>=0; i--)
    {
        cout << x[i];
    }
    return x;
}

Answer 1

Wszystko działa dobrze:

1. Wywołujesz funkcję odwroc. Wypisuje ona wyraz od tyłu na ekran (warto dodać że nie przechodzi do nowej linii),

2. Funkcja zwraca wyraz, który otrzymała, czyli ten nieodwrócony,

3. W funkcji main ponownie wypisujesz wyraz na ekran (został on zwrócony przez funkcję), powstaje właśnie coś takiego.

Pytanie brzmi: co chcesz osiągnąć?

Comments

#EDIT Jeżeli nie chcesz drugi raz wypisywać wyrazu to usuń return w funkcji i zmień jej typ zwracany na void.

---

Chciałem żeby mi wypisywała tylko odwrocony wyraz, zrobiłem tak jak kolega napisał wyżej czyli wykasowałem return x i teraz wyskakuje komunikat  że porgam przestał działać

Answer 2

Wykasuj return x; w funkcji

Comments

O teraz działa jak należy tylko wyskakuje komunikat że program przestał działać :/

---

Zmień linijkę

cout << odwroc(wyraz,dlugosc);

na

odwroc(wyraz,dlugosc);

 

 

---

ale tamto return x; zostaw

Answer 3

Skompilowałem sobie Twój kod i mi działa. Wygląda na dobry. Jedynie nie masz na samej górze "#include <iostream>".

Comments

Chyba, że nie chcesz, żeby wypisywał Ci ten wyraz początkowy ? Wtedy funkcje zmieniasz na voida i wyrzucasz "return x;" w mainie natomiast nie wypisujesz niczego coutem tylko wywołujesz funkcję o tak : "odwroc(wyraz,dlugosc);"

---

Tak właśnie o to mi chodziło, dzięki!

Answer 4

Gdy masz funkcję odwroc, to już ona wyświetla odwrocony tekst używając cout, tak więc przy uzyciu funkcji nie musisz uzywać cout, bo ona już nic nie będzie zwracać bez return :D

chodzi mi konkretnie o ten moment:

  cout << odwroc(wyraz,dlugosc);