Problem w kodzie, test na palindrom.

Question

Witam, niedawno zacząłem się uczyć z pomocą genialnych tutoriali pana Mirosława zelenta. Ostatnia sam wiele kombinowałem, a po odcinku o łańcuchach znaków, spróbowałem stworzyć program sprawdzający, czy dane słowo jest palindromem. Coś jednak poszło nie tak i mam problemy z jego uruchomieniem. Bardzo prosiłbym o wskazówki dot. tego co zmienić.

KOD:

 

#include <iostream>
#include <cstdlib>
using namespace std;

int main()
{
    string slowo;
    int dlugosc;

    cout << "Podaj swoje slowo, ktore zostanie sprawdzone testem na palindrom: ";
    cin >> slowo;

    dlugosc=slowo.length();
    int a=0;
    int b=dlugosc-1;

    if
        (
         for(a, b; (a==b)||(a<b); a++, b--)
        {
            slowo[a]==slowo[b];
        }
        )
        cout << "Slowo jest palindromem. " << endl;
    else
        cout << "Slowo nie jest palindromem. :(" << endl;

    return 0;
}

 

Answer 1

Coś jednak poszło nie tak i mam problemy z jego uruchomieniem.

A co dokładnie poszło nie tak ? Jeśli pojawił się komunikat o błędzie to podaj go w całości. Jeśli program daje niepoprawne dane to podaj co wprowadzasz na wejściu, co otrzymujesz na wyjściu a co chciałbyś aby poprawnie dawało na wyjściu.

Na forum jest specjalny przycisk do wstawiania kodu przez co jest kolorowany i bardziej czytelny. Jak możesz popraw Swój kod.

---

Rzuciłem okiem na kod i ta konstrukcja gdzie `for` jest wewnątrz `if` wygląda bardzo nietypowo - pierwszy raz widzę coś takiego. To na pewno nie robi tego co tobie się wydaje, że powinno robić.

Comments

Linie 19: 1. expeceted primary-expression before "for" 2. expected ')' before 'for' 3. exptected ';' before ')' token.

Linia 23: Ad.1 ---\\--- before ')'  oraz to samo co 19.3

---

`if` oczekuje, że wewnątrz () będzie warunek, ktory zwraca `true/false` (lub coś podobnego do `true/false`). Instrukcja `for` zaś nie zwraca żadnej wartości - nie do tego służy.

Jak już to należało by zrobić:

// domyślnie zakładam, ze to palindrom
bool palindrom = true;

// sprawdzam kolejne litery
for( ; a =< b; a++, b--)
{
    if( slowo[a] != slowo[b]) {
        // a jednak nie palindrom
        palindrom = false;

        // nie ma co sprawdzać reszte liter
        break;
    }
}

if( palindrom == true ) {
	cout << "TRUE" << endl;
} else {
	cout << "FALSE" << endl;
}

 

Answer 2

Spróbuj coś takiego:

#include <iostream>
#include <cstdlib>
using namespace std;
 
int main()
{
    string slowo;
    int dlugosc;
 
    cout << "Podaj swoje slowo, ktore zostanie sprawdzone testem na palindrom: ";
    cin >> slowo;
 
    dlugosc=slowo.length();
    int a=0;
    int b=dlugosc-1;
    bool czy_jest_palindrom = true;

    for(a, b; (a==b)||(a<b); a++, b--)
    {
            if(slowo[a] != slowo[b])
            {
                 czy_jest_palindrom = false;
                 break;
            }
    }
 
    if (czy_jest_palindrom)
        cout << "Slowo jest palindromem. " << endl;
    else
        cout << "Slowo nie jest palindromem. :(" << endl;
 
    return 0;
}

 

Answer 3

Czemu w warunku przy ifie dałeś pętlę (linia 19)? Takie coś jest w C++ nie możliwe!

Answer 4

Nie wiem jak żeś rozkminił tego fora w ifie :D ale to nie ma prawa działać. Nie łatwiej po prostu:

for(short i = 0; i < slowo.length()/2; i++)
        if( slowo[i] != slowo[slowo.length() - i - 1]) //nie jest palindromem;

A jeśli koniecznie chcesz to zrobić na takiej zasadzie jak ty to zrobiłeś, to poczytaj o wyrażeniach lambda. Wtedy jesteś w stanie zrobić coś takiego.

Btw. - to co robisz nie do końca ma sens - łatwiej jest sprawdzić czy NIE jest palindromem. Sprawdzasz czy literki się od siebie różnią, jeśli chociaż jedna para się różni to nie jest i koniec.

Comments

Bardzo dziękuję za pomoc i wskazówki dot. zmian kodu :)