I prefer this
var distance = metersToNauticalMiles(
utils.getDistance(
utils.toPoint(coordinates.get(0)),
utils.toPoint(coordinates.get(1))
)
);
Instead of this
var distance = metersToNauticalMiles(
utils.getDistance(
utils.toPoint(coordinates.get(0)),
utils.toPoint(coordinates.get(1))));
I'll do this if I need to reuse the intermediate values
var point1 = utils.toPoint(coordinates.get(0)),
point2 = utils.toPoint(coordinates.get(1)),
meters = utils.getDistance(point1, point2),
distance = metersToNauticalMiles(meters);
This is ok
var point1 = utils.toPoint(coordinates.get(0));
var point2 = utils.toPoint(coordinates.get(1));
var meters = utils.getDistance(point1, point2);
var distance = metersToNauticalMiles(meters);
I don't like this at all
var distance = metersToNauticalMiles(utils.getDistance(utils.toPoint(coordinates.get(0)), utils.toPoint(coordinates.get(1))));
Even worse
var distance = metersToNauticalMiles(
utils.getDistance.apply(utils,
[0, 1].map(coordinates.get.bind(coordinates))
.map(utils.toPoint.bind(utils))))
Are there other styles I'm missing?
http://benalman.com/news/2012/05/multiple-var-statements-javascript/