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!
javascript chrome-debugging
add a comment |
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!
javascript chrome-debugging
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
add a comment |
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!
javascript chrome-debugging
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
javascript chrome-debugging
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
add a comment |
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
add a comment |
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
Did you try it?
– Navy Flame
Nov 20 at 0:14
add a comment |
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 ==
add a comment |
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
});
}
});
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
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
Did you try it?
– Navy Flame
Nov 20 at 0:14
add a comment |
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
Did you try it?
– Navy Flame
Nov 20 at 0:14
add a comment |
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
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
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
add a comment |
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
add a comment |
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 ==
add a comment |
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 ==
add a comment |
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 ==
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 ==
answered Nov 20 at 0:09
Dinesh Pandiyan
2,088823
2,088823
add a comment |
add a comment |
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.
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
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
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
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
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