JavaScript Złe Działanie Sprawdzenie czy wyraz to palindron

pytanie zadane 31 marca 2017 w JavaScript przez Kitsugaya Nowicjusz (200 p.)
zamknięte 31 marca 2017 przez Kitsugaya

Witam! Otóż mam kod(podam niżej) i jeśli wpiszę w niego wyraz który jest faktycznie palindronem wypisuje się że to palindron ale jeśli wpiszę normalne słowo które nim nie jest dzieje się to samo ktoś ma pomysł?

<html>
<head>
<script type="text/javascript">

function palindrom(){
    var tekst = document.getElementById('tekst').value;
    var dlugosc=tekst.length;
    var napis = "";

    for(var i = 0; i < dlugosc; i++){
        if(tekst.charAt(i) != tekst.charAt(dlugosc -i -1)){napis = "Wyraz nie jest palindromem!"}
        else {napis = "Wyraz jest palindromem"}
    }
    document.getElementById('pole').innerHTML = napis;
}

</script>
</head>
<body>
Podaj wyraz: <input type="text" id="tekst">

<input type="button" onClick="palindrom()" value="Sprawdz!">
<div id="pole"></div>
</body>

komentarz zamknięcia: Rozwiązane

1 odpowiedź

odpowiedź 31 marca 2017 przez niezalogowany
wybrane 31 marca 2017 przez Kitsugaya

Najlepsza

W momencie w którym stwierdzisz, że wyraz nie jest palindromem, musisz wyjść z pętli for

komentarz 31 marca 2017 przez Kitsugaya Nowicjusz (200 p.)

Dzięki wielkie! :)

komentarz 31 marca 2017 przez niezalogowany

Tak z ciekawości, jak z niej wyszedłeś?

komentarz 31 marca 2017 przez Kitsugaya Nowicjusz (200 p.)

for(var i = 0; i < dlugosc; i++)

        if(tekst.charAt(i) != tekst.charAt(dlugosc -i -1)){napis = "Wyraz nie jest palindromem!"}
        else {napis = "Wyraz jest palindromem"}
    document.getElementById('pole').innerHTML = napis;

Nie wiem, czy do końca prawidłowo, ale działa tak jak powinno.

komentarz 31 marca 2017 przez niezalogowany

według mnie, nie działa - sprawdź na przykład "abaa"

komentarz 31 marca 2017 przez Kitsugaya Nowicjusz (200 p.)

To znaczy tak myślę, że tyle do szkoły co mogę mieć na poprawie powinno wystarczyć. A można to jakoś zabezpieczyć?

komentarz 31 marca 2017 przez niezalogowany

Zostaw poprzednią wersję z nawiasami.

Jeśli wyraz nie jest palindromem, ustaw zmienną i na długość, dzięki temu, pętla for nie wykona się już więcej razy

(przy kolejnych iteracjach, zmienna napis ponownie zostałaby nadpisana - możliwe, że błędnym wynikiem)

komentarz 31 marca 2017 przez niezalogowany

Miej na uwadze, że to najprostsze rozwiązanie problemu przy obecnym kodzie. Najlepiej byłoby zrobić osobną funkcję, do sprawdzania, czy string, rzeczywiście jest palindromem. W momencie wykrycia, że nie jest - po prostu return false