In JavaScript Numbers Can Bite, I talked about a problem that can crop up in JavaScript when you’re doing calculations with integers.

At the end of the post I hinted at a better way to write the combination function. Let’s go ahead and implement it. I’ll throw a permutation function in as well.

Let’s look at combinations. The formula is:

n!/(k!*(n-k)!)

That’s for n items, taken k at a time. In my mom’s family, there were seven children. How many combinations of them are there if you take 4 at a time?

7!/(4!*3!)

Now, if we write out the factorials, we immediately see a simplification (not in notation, but in floating point operations).

(7*6*5*4*3*2*1)/(4*3*2*1*3*2*1) = (7*6*5)/(3*2*1)

What we need here is a function that multiplies a range of numbers. Then we have:

productRange(5,7)/productRange(1,3)

or more generally,

productRange(k+1,n)/productRange(1,n-k)

function productRange(a,b) {
var product=a,i=a;
while (i++<b) {
product*=i;
}
return product;
}
function combinations(n,k) {
if (n==k) {
return 1;
} else {
k=Math.max(k,n-k);
return productRange(k+1,n)/productRange(1,n-k);
}
}

So that’s combinations, where the order of the items doesn’t matter. What about permutations, where the order does matter? The formula is:

n!/(n-k)!

That one is simple…

function permutations(n,k) {
return productRange(k+1,n);
}

### Like this:

Like Loading...

*Related*

## TNO said,

November 8, 2008 at 5:46 pm

Math.max can be slow, it’s more efficient to use the inline if statement:

k=(k < n-k) ? n-k : k;

## Nosredna said,

November 8, 2008 at 6:04 pm

Really? Good to know. I would have thought that a built-in library function would be quick. Maybe it’s doing a lot of type checking?

It probably depends on the browser.

## TNO said,

November 8, 2008 at 6:26 pm

Operators are always faster than method calls since there is no lookup involved

## Nosredna said,

November 8, 2008 at 7:26 pm

Note that that line could be done away with.

## Paul Hanlon said,

November 9, 2008 at 6:25 am

Hi Nosredna,

Very elegant. I love algorithms that reduce complexity, and in so doing, run faster.

Nice site too. Learned a pile!

Paul

## ken larkin said,

July 9, 2009 at 10:07 am

This formula is incorrect, if you have 7 in your family you have only one combination of seven. This formula returns 8 for combinations(7,7)

## Nosredna said,

April 13, 2010 at 8:53 pm

ken, obviously I suck. I’ll take a look and see what I did wrong. I hope you’ve read my disclaimer:

https://dreaminginjavascript.wordpress.com/about/

## Nosredna said,

April 14, 2010 at 1:30 am

Oh. I bet it’s because zero is a special case. 0! is defined as 1.

## Nosredna said,

April 14, 2010 at 1:58 am

OK. I just fixed combinations, hopefully. I expect you’ll tell me next that permutations has a similar problem?

## James Heo said,

May 23, 2011 at 7:37 am

var n=parseInt(prompt(“n!, input interger”,”100″));

if(isNaN(n)==true){alert(‘Please, input interger’);}

var big_integer=Math.pow(2,52.5);

var R_TXT=[];var cnt=0; var factorial_past=0;

function F(m,n) {

var factorial=m;var multiply=1;

var multi_past;

while(factorialbig_integer){R_TXT[cnt++]=(factorial-1)+”!/”+factorial_past+”! = “+(factorial_past+1)+” ~ “+(factorial-1)+” = “+multi_past;

factorial_past=(factorial-1);return F(factorial,n);}

factorial++;}

return multiply;}

var N=F(1,n);

R_TXT[cnt++]=n+”!/”+factorial_past+”! = “+(factorial_past+1)+” ~ “+n+” = “+N;

document.write(n+’ Factorial ‘+(R_TXT.length-1)+” Numbers”+(R_TXT).join(”));

## James Heo said,

May 23, 2011 at 7:43 am

Oh, No…

“” where is this? Do not erase~!

## James Heo said,

May 23, 2011 at 7:45 am

~than big sign, ~than small sign ..

do not erase~!

## Davis said,

January 20, 2012 at 3:04 pm

Unfortunately this formula is still not correct when there is a zero involved.. Using combinations(2,0) it returns 3. It should return 1.

eg. n=2, k=0

then in the function k is set to max(0,2-0) = 2

then calling productRange(2+1,2) = 3 / productRange(1,0) = 1

= 3