Sprawdznie string.charAt() daje zawsze null

pytanie zadane 13 grudnia 2018 w JavaScript przez gnu_ewm Gaduła (3,340 p.)
zamknięte 31 maja 2020 przez gnu_ewm

Witam. Tworzę program w JavaScript który na podać ilość zdań, liter, słów oraz całą długość.

var letters;
var words;
var phrases;
var text;
var textLength;
var lastToken = "";

function process()
{
  text = document.getElementById("inputText").value;
  textLength = text.length;
  for(i = 0; i <= text.length; i++)
  {
    if(text.charAt(i) == " " && lastToken != " ") words++;
    else if(text.charAt(i) == "." || text.charAt(i) == "!" || text.charAt(i) == "?" && lastToken != "." && lastToken != "!" && lastToken != "?") phrases++;
    else letters++;

    lastToken = text.charAt(i);
  }

  document.getElementById("processResoult").innerHTML = "Litery: " + letters + " Wyrazy: " + words + " Zdania: " + phrases + " Długość: " + textLength;
}

<!DOCTYPE html>
<html lang="pl">
  <head>
    <meta charset="utf-8">
    <title>Słowa</title>
    <link rel="stylesheet" href="style.css">
    <link href="https://fonts.googleapis.com/css?family=Ubuntu:500&amp;subset=latin-ext" rel="stylesheet">
  </head>
  <body>
    <div id="inputs">
      <input id="inputText" type="text" placeholder="Tekst">
      <input id="processButton" type="button" value="Process" onclick="process()">
    </div>
    <div id="processResoult"></div>

    <script src="scripts.js"></script>
  </body>
</html>

Tylko sprawdzanie ilości liter daje poprawny wynik, inne dają wartość null lub undifined. Przeglądarka to Firefox, a system to Ubuntu.

komentarz zamknięcia: Zimny pot mnie zalewa kiedy czytam moje stare pytania.

1 odpowiedź

odpowiedź 13 grudnia 2018 przez Kamil Naja Nałogowiec (27,410 p.)

Moim zdaniem błąd polega na tym, że robisz if -else zamiast kilku osobnych ifów. Wykonuje się pierwszy warunek a reszta jest omijana. Możesz to łatwo sprawdzić w debuggerze. Nie podoba mi się omijanie nawiasów {} przy ifach, dodatkowo zmniejsza to czytelność.

Kod funkcji jest bardzo zawikłany, rozbij go na kilka osobnych funkcji i je wywołuj.

Np sprawdzanie ilości słów

function returnWordsCount(data) {
    return data
        .split(' ')
        .map(item => item[0])
        .length;
}

1	komentarz 13 grudnia 2018 przez gnu_ewm Gaduła (3,340 p.) Problem został rozwiązany przez przypisanie wartości 0 zmiennych words i phrases przy ich definiowaniu. Dziękuję za pomoc i cenne wskazówki.