How come my code cannot calculate count correctly in jquery?

Question

I am trying to limit keyword input.

Whenever a user enter a keyword, it will append 'span' to that keyword, e.g.

like the html code below.

I can limit words, but the count is not accurate but when it reach 5, it stop entering.

If I refresh page, I can enter one more again, then stop me again too.

The weird things, it doesn't work if I use id for var count=$('#result_tag').text(). I have to use class.

I had tried to change /\S+/g to /\s+/g, not working, I have to keep S capital.

JQuery:

$( "#tag" ).on( "keydown", function( event ) {

        var count=$('.result_tag').text();
        var words=count.match(/\S+/g).length;

        if(words >= 8 ){

            alert('No more than eight words');  
        }       

        //save tag after press enter key   
        else if ( event.which==188 || event.which == 13 ) {

        event.preventDefault();
        //codes will do the action

HTML:

<span id="result_tag" class="result_tag">keyword that user enter</span>
<span id="result_tag" class="result_tag">keyword that user enter</span>
<span id="result_tag" class="result_tag">keyword that user enter</span>

You cannot have same ID's.

roshan
– roshan

2015-02-20 05:15:52 +00:00
Commented Feb 20, 2015 at 5:15 — roshan
– roshan, Commented Feb 20, 2015 at 5:15

rjdown · Accepted Answer · 2015-02-20 05:19:53Z

4

ids should be unique, that's why "it doesn't work if I use id".

As for your other problem, your method of counting seems a little convoluted. Sounds like all you need to do is count the spans themselves:

var words = $('.result_tag').length;

answered Feb 20, 2015 at 5:19

rjdown

9,2073 gold badges35 silver badges48 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Hearaman · Accepted Answer · 2015-02-20 05:33:17Z

1

Use length property to get the count of keywords.

    $("#tag").on("keydown", function(event) 
    {
         var count = $('.result_tag').length;
         if (count >= 8) 
         {
               alert('No more than eight words');
         }

         ...........
         ...........

edited Feb 20, 2015 at 5:33

answered Feb 20, 2015 at 5:25

Hearaman

8,78813 gold badges45 silver badges58 bronze badges

1 Comment

rjdown Over a year ago

FYI: The .size() method is deprecated as of jQuery 1.8. Use the .length property instead.

Collectives™ on Stack Overflow

How come my code cannot calculate count correctly in jquery?

2 Answers 2

Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related