Why is my return value undefined (JavaScript) [duplicate]

Question

I have an array called questionSets full of objects. The createSet function should create new or create a copy of an existing questionSets object. The function getQuestionsFromSet is used if createSet is used to make a copy. For some reason when I call getQuestionsFromSet() from inside createSet() I always get a returned value of 'undefined'. I can't figure out why because when I do a console.log() of the value to be returned by getQuestionsFromSet() I see exactly what I want.

I have these two functions.

function createSet(name, copiedName) {
    var questions = [];
    if (copiedName) {
        questions = getQuestionsFromSet(copiedName);
    }
    console.log(questions); // undefined. WHY??
    questionSets.push({
        label: name,
        value: questions
    });
}; // end createSet()

function getQuestionsFromSet(setName) {
    $.each(questionSets, function (index, obj) {
        if (obj.label == setName) {
            console.log(obj.value); // array with some objects as values, which is what I expect.
            return obj.value;
        }
    });
}; // end getQuestionsFromSet()

console.log is showing values?

Awais Qarni
– Awais Qarni

2013-06-24 13:24:51 +00:00
Commented Jun 24, 2013 at 13:24 — Awais Qarni
– Awais Qarni, Commented Jun 24, 2013 at 13:24
@AnkitJaiswal There is no Ajax call there!

epascarello
– epascarello

2013-06-24 13:38:21 +00:00
Commented Jun 24, 2013 at 13:38 — epascarello
– epascarello, Commented Jun 24, 2013 at 13:38

techfoobar · Accepted Answer · 2013-06-24 13:23:47Z

8

Because getQuestionsFromSet() does not return anything and so is implicitly undefined.

What you need is probably something like:

function getQuestionsFromSet(setName) {
    var matched = []; // the array to store matched questions..
    $.each(questionSets, function (index, obj) {
        if (obj.label == setName) {
            console.log(obj.value); // array with some objects as values, which is what I expect.
            matched.push(obj.value); // push the matched ones
        }
    });
    return matched; // **return** it
}

answered Jun 24, 2013 at 13:23

techfoobar

66.8k14 gold badges117 silver badges138 bronze badges

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

Comments

haim770 · Accepted Answer · 2013-06-24 13:24:53Z

2

return obj.value; is nested within the inner $.each(function{}), and getQuestionsFromSet is indeed not returning anything.

answered Jun 24, 2013 at 13:24

haim770

49.2k7 gold badges113 silver badges139 bronze badges

Collectives™ on Stack Overflow

Why is my return value undefined (JavaScript) [duplicate]

2 Answers 2

Comments

Comments

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Linked

Related