Javascript filter was unfair











up vote
-3
down vote

favorite












I'm trying to get existCount from an array, which has id in selected array.



But something went wrong, I had an item with id = 5493 but existCount.length 0



My JS code:



var existCount = $scope.selectedScript.filter(function (item) {
return item.id === script.script_id;
});
console.log('existCount.length ', existCount.length);
console.log('$scope.selectedScript ', $scope.selectedScript);
console.log('script.script_id ', script.script_id);


Chrome Console view:



https://i.stack.imgur.com/4UVWw.png



// Sorry I forgot the first output line, but this line's at the top of $scope.selectedScript and it was existCount.length = 0



Where my fault?



How can I fix it?



Thanks!










share|improve this question
























  • Please don't post images of code. Edit your question to format your code as blocks using the { } icon or Ctrl+K.
    – Patrick Roberts
    Nov 20 at 0:02










  • @PatrickRoberts Ok, I will edit now, thank for your help.
    – Anh Tuan
    Nov 20 at 0:04















up vote
-3
down vote

favorite












I'm trying to get existCount from an array, which has id in selected array.



But something went wrong, I had an item with id = 5493 but existCount.length 0



My JS code:



var existCount = $scope.selectedScript.filter(function (item) {
return item.id === script.script_id;
});
console.log('existCount.length ', existCount.length);
console.log('$scope.selectedScript ', $scope.selectedScript);
console.log('script.script_id ', script.script_id);


Chrome Console view:



https://i.stack.imgur.com/4UVWw.png



// Sorry I forgot the first output line, but this line's at the top of $scope.selectedScript and it was existCount.length = 0



Where my fault?



How can I fix it?



Thanks!










share|improve this question
























  • Please don't post images of code. Edit your question to format your code as blocks using the { } icon or Ctrl+K.
    – Patrick Roberts
    Nov 20 at 0:02










  • @PatrickRoberts Ok, I will edit now, thank for your help.
    – Anh Tuan
    Nov 20 at 0:04













up vote
-3
down vote

favorite









up vote
-3
down vote

favorite











I'm trying to get existCount from an array, which has id in selected array.



But something went wrong, I had an item with id = 5493 but existCount.length 0



My JS code:



var existCount = $scope.selectedScript.filter(function (item) {
return item.id === script.script_id;
});
console.log('existCount.length ', existCount.length);
console.log('$scope.selectedScript ', $scope.selectedScript);
console.log('script.script_id ', script.script_id);


Chrome Console view:



https://i.stack.imgur.com/4UVWw.png



// Sorry I forgot the first output line, but this line's at the top of $scope.selectedScript and it was existCount.length = 0



Where my fault?



How can I fix it?



Thanks!










share|improve this question















I'm trying to get existCount from an array, which has id in selected array.



But something went wrong, I had an item with id = 5493 but existCount.length 0



My JS code:



var existCount = $scope.selectedScript.filter(function (item) {
return item.id === script.script_id;
});
console.log('existCount.length ', existCount.length);
console.log('$scope.selectedScript ', $scope.selectedScript);
console.log('script.script_id ', script.script_id);


Chrome Console view:



https://i.stack.imgur.com/4UVWw.png



// Sorry I forgot the first output line, but this line's at the top of $scope.selectedScript and it was existCount.length = 0



Where my fault?



How can I fix it?



Thanks!







javascript chrome-debugging






share|improve this question















share|improve this question













share|improve this question




share|improve this question








edited Nov 20 at 0:35

























asked Nov 19 at 23:58









Anh Tuan

42




42












  • Please don't post images of code. Edit your question to format your code as blocks using the { } icon or Ctrl+K.
    – Patrick Roberts
    Nov 20 at 0:02










  • @PatrickRoberts Ok, I will edit now, thank for your help.
    – Anh Tuan
    Nov 20 at 0:04


















  • Please don't post images of code. Edit your question to format your code as blocks using the { } icon or Ctrl+K.
    – Patrick Roberts
    Nov 20 at 0:02










  • @PatrickRoberts Ok, I will edit now, thank for your help.
    – Anh Tuan
    Nov 20 at 0:04
















Please don't post images of code. Edit your question to format your code as blocks using the { } icon or Ctrl+K.
– Patrick Roberts
Nov 20 at 0:02




Please don't post images of code. Edit your question to format your code as blocks using the { } icon or Ctrl+K.
– Patrick Roberts
Nov 20 at 0:02












@PatrickRoberts Ok, I will edit now, thank for your help.
– Anh Tuan
Nov 20 at 0:04




@PatrickRoberts Ok, I will edit now, thank for your help.
– Anh Tuan
Nov 20 at 0:04












2 Answers
2






active

oldest

votes

















up vote
0
down vote



accepted










Change return item.id === script.script_id; to return item.id == script.script_id;



In your case: item.id was a number, script.script_id was a string. You can see it in chrome debug by color, black for string, blue for number.



=== is the hard way to compare in JS.



You can see at https://stackoverflow.com/a/359509/8572205



So === return false and no item added into existCount






share|improve this answer























  • Did you try it?
    – Navy Flame
    Nov 20 at 0:14


















up vote
0
down vote













if you use triple-equals ===, make sure both the values in comparison are of same type. I suspect script.script_id is a string.



Try changing === to ==






share|improve this answer





















    Your Answer






    StackExchange.ifUsing("editor", function () {
    StackExchange.using("externalEditor", function () {
    StackExchange.using("snippets", function () {
    StackExchange.snippets.init();
    });
    });
    }, "code-snippets");

    StackExchange.ready(function() {
    var channelOptions = {
    tags: "".split(" "),
    id: "1"
    };
    initTagRenderer("".split(" "), "".split(" "), channelOptions);

    StackExchange.using("externalEditor", function() {
    // Have to fire editor after snippets, if snippets enabled
    if (StackExchange.settings.snippets.snippetsEnabled) {
    StackExchange.using("snippets", function() {
    createEditor();
    });
    }
    else {
    createEditor();
    }
    });

    function createEditor() {
    StackExchange.prepareEditor({
    heartbeatType: 'answer',
    convertImagesToLinks: true,
    noModals: true,
    showLowRepImageUploadWarning: true,
    reputationToPostImages: 10,
    bindNavPrevention: true,
    postfix: "",
    imageUploader: {
    brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
    contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
    allowUrls: true
    },
    onDemand: true,
    discardSelector: ".discard-answer"
    ,immediatelyShowMarkdownHelp:true
    });


    }
    });














    draft saved

    draft discarded


















    StackExchange.ready(
    function () {
    StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53384348%2fjavascript-filter-was-unfair%23new-answer', 'question_page');
    }
    );

    Post as a guest















    Required, but never shown

























    2 Answers
    2






    active

    oldest

    votes








    2 Answers
    2






    active

    oldest

    votes









    active

    oldest

    votes






    active

    oldest

    votes








    up vote
    0
    down vote



    accepted










    Change return item.id === script.script_id; to return item.id == script.script_id;



    In your case: item.id was a number, script.script_id was a string. You can see it in chrome debug by color, black for string, blue for number.



    === is the hard way to compare in JS.



    You can see at https://stackoverflow.com/a/359509/8572205



    So === return false and no item added into existCount






    share|improve this answer























    • Did you try it?
      – Navy Flame
      Nov 20 at 0:14















    up vote
    0
    down vote



    accepted










    Change return item.id === script.script_id; to return item.id == script.script_id;



    In your case: item.id was a number, script.script_id was a string. You can see it in chrome debug by color, black for string, blue for number.



    === is the hard way to compare in JS.



    You can see at https://stackoverflow.com/a/359509/8572205



    So === return false and no item added into existCount






    share|improve this answer























    • Did you try it?
      – Navy Flame
      Nov 20 at 0:14













    up vote
    0
    down vote



    accepted







    up vote
    0
    down vote



    accepted






    Change return item.id === script.script_id; to return item.id == script.script_id;



    In your case: item.id was a number, script.script_id was a string. You can see it in chrome debug by color, black for string, blue for number.



    === is the hard way to compare in JS.



    You can see at https://stackoverflow.com/a/359509/8572205



    So === return false and no item added into existCount






    share|improve this answer














    Change return item.id === script.script_id; to return item.id == script.script_id;



    In your case: item.id was a number, script.script_id was a string. You can see it in chrome debug by color, black for string, blue for number.



    === is the hard way to compare in JS.



    You can see at https://stackoverflow.com/a/359509/8572205



    So === return false and no item added into existCount







    share|improve this answer














    share|improve this answer



    share|improve this answer








    edited Nov 20 at 0:18

























    answered Nov 20 at 0:10









    Navy Flame

    167112




    167112












    • Did you try it?
      – Navy Flame
      Nov 20 at 0:14


















    • Did you try it?
      – Navy Flame
      Nov 20 at 0:14
















    Did you try it?
    – Navy Flame
    Nov 20 at 0:14




    Did you try it?
    – Navy Flame
    Nov 20 at 0:14












    up vote
    0
    down vote













    if you use triple-equals ===, make sure both the values in comparison are of same type. I suspect script.script_id is a string.



    Try changing === to ==






    share|improve this answer

























      up vote
      0
      down vote













      if you use triple-equals ===, make sure both the values in comparison are of same type. I suspect script.script_id is a string.



      Try changing === to ==






      share|improve this answer























        up vote
        0
        down vote










        up vote
        0
        down vote









        if you use triple-equals ===, make sure both the values in comparison are of same type. I suspect script.script_id is a string.



        Try changing === to ==






        share|improve this answer












        if you use triple-equals ===, make sure both the values in comparison are of same type. I suspect script.script_id is a string.



        Try changing === to ==







        share|improve this answer












        share|improve this answer



        share|improve this answer










        answered Nov 20 at 0:09









        Dinesh Pandiyan

        2,088823




        2,088823






























            draft saved

            draft discarded




















































            Thanks for contributing an answer to Stack Overflow!


            • Please be sure to answer the question. Provide details and share your research!

            But avoid



            • Asking for help, clarification, or responding to other answers.

            • Making statements based on opinion; back them up with references or personal experience.


            To learn more, see our tips on writing great answers.





            Some of your past answers have not been well-received, and you're in danger of being blocked from answering.


            Please pay close attention to the following guidance:


            • Please be sure to answer the question. Provide details and share your research!

            But avoid



            • Asking for help, clarification, or responding to other answers.

            • Making statements based on opinion; back them up with references or personal experience.


            To learn more, see our tips on writing great answers.




            draft saved


            draft discarded














            StackExchange.ready(
            function () {
            StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53384348%2fjavascript-filter-was-unfair%23new-answer', 'question_page');
            }
            );

            Post as a guest















            Required, but never shown





















































            Required, but never shown














            Required, but never shown












            Required, but never shown







            Required, but never shown

































            Required, but never shown














            Required, but never shown












            Required, but never shown







            Required, but never shown







            Popular posts from this blog

            Costa Masnaga

            Fotorealismo

            Sidney Franklin