how to execute several functions with same arguments js

Question

I have an object and 3 methods, now my code to execute it looks like this:

var cmpAmount = 50,
    addLessAmount = 6,
    addMoreAmount = 14;
priceChanger.changeSmallPrice(cmpAmount, addLessAmount, addMoreAmount);
priceChanger.changeBigPrice(cmpAmount, addLessAmount, addMoreAmount);
priceChanger.changeTooltipPrice(cmpAmount, addLessAmount, addMoreAmount);

so, if arguments are the same how to execute it three with sending them the parameters. I mean how to change my code to make it less.

May be there is something wrong in your code, I guess not following the DRY principle. — Minko Gechev
– Minko Gechev, Commented Feb 2, 2014 at 19:57
How about creating one function in your object that executes the other three, just pass the arguments array along. — adeneo
– adeneo, Commented Feb 2, 2014 at 19:59

hacky · Accepted Answer · 2014-02-02 19:59:03Z

1

I think it would be best to make a method "changeAllPrices(cmpAmount, addLessAmount, addMoreAmount)"

answered Feb 2, 2014 at 19:59

hacky

262 bronze badges

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

Comments

Goran.it · Accepted Answer · 2014-02-02 19:57:07Z

0

Maybe using a object to store all properties :

var args = {cmpAmount:50,addLessAmount: 6, addMoreAmount: 14};

priceChanger.changeSmallPrice(args);
priceChanger.changeBigPrice(args);
priceChanger.changeTooltipPrice(args);

answered Feb 2, 2014 at 19:57

Goran.it

6,3772 gold badges25 silver badges26 bronze badges

Comments

megawac · Accepted Answer · 2014-02-02 20:02:53Z

0

You can use a simple helper function, such as the one below, to call each function with an array of arguments. I believe some javascript libraries include a similar composition function called composeArgs

function execute(fns, args) {
   for(var i = 0; i < fns.length; i++) {
       fns[i].apply(this, args);
   }
}

Then call execute

var cmpAmount = 50,
    addLessAmount = 6,
    addMoreAmount = 14;

execute([priceChanger.changeSmallPrice, priceChanger.changeBigPrice, priceChanger.changeTooltipPrice], 
        [cmpAmount, addLessAmount, addMoreAmount]);

answered Feb 2, 2014 at 20:02

megawac

11.4k6 gold badges43 silver badges61 bronze badges

Collectives™ on Stack Overflow

how to execute several functions with same arguments js

3 Answers 3

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related