node.js set property from string property name

Question

I was trying to create a function able to set a certain value of an object, having the "path" of the property:

reflectionSet = function(obj, propString, value) {
    var current = obj;
    var splitted = propString.split('.');
    splitted.forEach(function(k) {
        current = current[k];
    })
    current = value;
}
var test = {
    a: {
        s: 'asd',
        g: 'asasdasdd'
    }
};

reflectionSet(test, 'a.g', "otherValue");

and it should become:

{
    a: {
        s: 'asd',
        g: 'otherValue'
    }
}

Unfortunately this doesn't work at all.. Thanks

That is because JavaScript is "pass-by-value". Which means that when you collect the variable g you wont be changing the value in the Object. Just the value you collected from the Object. Here is a more informative explanation — magnudae
– magnudae, Commented Oct 20, 2014 at 14:05

thefourtheye · Accepted Answer · 2014-10-20 14:17:40Z

1

You can use split the properties based on . and then using Array.prototype.reduce, get to the inner most part of the Object and update it like this

function reflectionSet(obj, propString, value) {
    return propString.split(".").reduce(function(result, part, index, array) {
        if (index === array.length - 1) {
            result[part] = value;
            return obj;
        }
        return result[part];
    }, obj);
}

var test = {
    a: {
        s: 'asd',
        g: 'asasdasdd'
    }
};

console.log(reflectionSet(test, 'a.g', "otherValue"));

Output

{
    a: {
        s: 'asd',
        g: 'otherValue'
    }
}

edited Oct 20, 2014 at 14:17

answered Oct 20, 2014 at 14:06

thefourtheye

241k53 gold badges466 silver badges505 bronze badges

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

2 Comments

M4rk Over a year ago

currentItam is not set

thefourtheye Over a year ago

@rodi Oops, there was a typo. I fixed it now. Please check.

techfoobar · Accepted Answer · 2014-10-20 14:10:56Z

1

This corrected version of your function should do it.

reflectionSet = function(obj, prop, value) {
    prop = prop.split('.');
    var root = obj, i;
    for(i=0; i<prop.length; i++) {
        if(typeof root[prop[i]] == 'undefined') root[prop[i]] = {};
        if(i === prop.length - 1) root[prop[i]] = value;
        root = root[prop[i]];
    }
    return obj;
};

Now:

var test = { a: { s: 'asd', g: 'asasdasdd' } };
reflectionSet(test, 'a.g', "otherValue");

Will return { a: { s: 'asd', g: 'otherValue' } }

answered Oct 20, 2014 at 14:10

techfoobar

66.8k14 gold badges117 silver badges138 bronze badges

Collectives™ on Stack Overflow

node.js set property from string property name

2 Answers 2

2 Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related