wie ktos dlaczego ten program niedziala?

Question

#include <iostream>
#include <cstdlib>
#include <windows.h>
#include <algorithm>
#include <string>
#include <stdio.h>

using namespace std;

string napis;
char wybor;
int dlugosc = napis.length();

int main()
{
    cout << "witaj!" << endl;
    Sleep(300);
    cout << "podaj twoj text: ";
    getline(cin, napis);
    Sleep (300);
    system("cls");
    cout << "      Menu" << endl;
    cout << "1 = odwracanie tekstu"<< endl;
    cout << "2 = zmiana wielkosci liter"<< endl;
    cout << "3 = koniec programu "<< endl;
    cin >> wybor;
    Sleep(100);
    system("cls");

    switch(wybor)
    {
    case '1':
    {


    for (int i=dlugosc-1; i>=0; i--)
    {
    cout << napis[i];
    }
    }
    break;


    case '2': transform(napis.begin(),napis.end(),napis.begin(),::toupper);
    cout << napis << endl;
    break;

    case '3': exit(0);
    break;
    }
    getchar();getchar();






    return 0;
}

 

Comments

Powinieneś napisać co konkretnie nie działa, żeby nie marnowac  naszego czasu

---

Zgadzam się z tobą, że powinien napisać co konkretnie nie działa albo dać jakieś wskazówki w jaki sposób powtórzyć ten błąd u siebie.

Jednakże wystarczyło tylko kod uruchomić i sprawdzić pierwszą opcję na pierwszym lepszy tekście, żeby zobaczyć co jest nie tak.

---

Dzięki, nie wpadłbym na to

Answer 1

Jeżeli chodzi Ci o opcję pierwszą, to nie działa z prostej przyczyny. Na początku inicjalizujesz zmienną dlugosc jako napis.lenght(), tylko, że w tym momencie napis ma długość zero więc zmienna dlugosc w tym momencie też ma taką samą wartość. To, że zainicjalizowałeś tą zmienną w taki sposób nie oznacza, że przez cały program będzie przyjmowała wartość długości napisu. To co należy zrobić:

1) Na początku najlepiej ustaw sobie po prostu int dlugosc

2) Pod getline ustaw jej wartość.

getline(cin, napis);
dlugosc = napis.length();

  

 

Answer 2

Ustawiasz zmienną dlugosc zanim ustawiasz napis, czyli długość zawsze jest równa zero.