Can Javascript and/or jQuery return objects this way?

Question

I'm pulling a blank as to why this code isn't working as expected (jsFiddle here):

data = [
{
    "id": 1,
    "value": 4.56
}, 
{
    "id": 2,
    "value": 7.89
}];


function FindMe(searchID) 
{
    $.each(data, function (i, v) 
    { 
        // i=index, v=value (which is an object)
        if (v.id === searchID) 
        {
            console.log("Found: ");
            console.log(v);
            return v; // pass the desired object back to caller
        }
    });
}

console.clear();
var test = FindMe(2); // causes the console to show the correct object
console.log("Returned: ");
console.log(test); // shows "undefined" instead of a returned object

The function clearly does its job to find the correct array element (the console shows this as "Found"), but the return isn't happening. What's going wrong here?

Mārtiņš Briedis · Accepted Answer · 2014-04-13 14:55:58Z

4

That is because the looping function returns the found item, not the function FindMe.

Find me returns nothing.

function FindMe(searchID) {
 var result;    
    $.each(data, function (i, v) { // i=index, v=value (which is an object)
        if (v.id === searchID) {
            console.log("Found: ");
            console.log(v);
            result = v; // pass the desired object back to caller
            return false;
        }
    });
  return result;
}

answered Apr 13, 2014 at 14:55

Mārtiņš Briedis

17.9k5 gold badges59 silver badges78 bronze badges

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

Comments

Assaf Lavie · Accepted Answer · 2014-04-13 14:55:46Z

2

Your FindMe function has no return statement. You're just calling $.each...

answered Apr 13, 2014 at 14:55

Assaf Lavie

76.7k37 gold badges152 silver badges207 bronze badges

Comments

Joseph · Accepted Answer · 2014-04-13 14:56:46Z

0

jQuery.each just iterates, it does not collect. What you need is something similar to the native Array.prototype.filter like this one here:

function FindMe(searchID) {
    return data.filter(function(datum){
        return (datum.id === searchID);
    });
}

Most browsers should have Array.prototype.filter by now, except <= IE8.

answered Apr 13, 2014 at 14:56

Joseph

120k31 gold badges186 silver badges238 bronze badges

Comments

cookie monster · Accepted Answer · 2014-04-13 15:09:35Z

0

you can use:

data = [{
"id": 1,
    "value": 4.56
}, {
    "id": 2,
        "value": 7.89
}];


function FindMe(searchID) {
     var searchObj=null;
    $.each(data, function (i, v) { // i=index, v=value (which is an object)
        if (v.id === searchID) {
            console.log("Found: ");
            console.log(v);
           searchObj = v; // pass the desired object back to   caller
            return false;
        }
    });
    return searchObj;
}

console.clear();
var test = FindMe(2); 
object
console.log("Returned: ");
console.log(test);
object

edited Apr 13, 2014 at 15:09

cookie monster

11k4 gold badges33 silver badges45 bronze badges

answered Apr 13, 2014 at 15:05

Maneesh Singh

5752 silver badges13 bronze badges

Collectives™ on Stack Overflow

Can Javascript and/or jQuery return objects this way?

4 Answers 4

Comments

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related