Что не так с кодом?

Question

[Quintis_Mikeliy]

Доброго времени суток , не могу обнулить счетчик когда textarea пустая , вот код :
https://codepen.io/Mikeliy/pen/EqWMGZ

$('#js-myText').on('keyup',function(){

	var count = $('#js-myText').val().match(/\w{1,}/g);

   $('#js-wordCount').text(count.length);
 




if (count.length < 10) {
$('#js-wordCount').removeClass("js-text-color");
  $('#js-myText').removeAttr("maxlength");
} else  if (count.length >= 10) {    
$('#js-myText').attr("maxlength", "0"); 
$('#js-wordCount').addClass("js-text-color"); 
} else {
     $('#js-wordCount').text(0);
   alert(win);
}


});

<p><textarea id="js-myText" onpaste="return false" ></textarea></p>
		<p class="count-number"> Word count: <span id="js-wordCount">0</span>

Answer 1

[lloydbanks]

Потому что у вас метод matchничего не находит при пустой строке.
Замените

var count = $('#js-myText').val().match(/\w{1,}/g);

на:

var count = $('#js-myText').val().match(/\w{1,}/g) || 0;

Comments

[Quintis_Mikeliy]

Большое человеческое СПАСИБО) целый день сидел на кодом )

[Quintis_Mikeliy]

А можете объяснить как код работает , ведь когда нету символов count.length меньше 10 но срабатывает последний else

[Quintis_Mikeliy]

Это потому что с пустой строки нельзя высчитать count.length и код исполняет последний else ?

[lloydbanks]

Сорри за долгий ответ, я не так часто сюда захожу.

А можете объяснить как код работает , ведь когда нету символов count.length меньше 10 но срабатывает последний else

Упс, ошибочка вышла. Я забыл, что matchвозвращает массив. Вместо || 0 используйте || [].
Последний else срабатывает из-за того, что у нуля свойство length - это undefined, а оно не может быть < 10, поэтому мы не попадали в первое условие. Скрин для наглядности (выделил желтым):

[Quintis]

Спасибо !)