Skip to content

Commit d3ac4c5

Browse files
authored
Merge pull request #9 from volf52/master
Combinations without using factorials-Python
2 parents 10e186c + 0bbecb2 commit d3ac4c5

3 files changed

Lines changed: 41 additions & 0 deletions

File tree

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,21 @@
1+
MIT License
2+
3+
Copyright (c) 2017 Muhammad Arslan <rslnkrmt2552@gmail.com>
4+
5+
Permission is hereby granted, free of charge, to any person obtaining a copy
6+
of this software and associated documentation files (the "Software"), to deal
7+
in the Software without restriction, including without limitation the rights
8+
to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
9+
copies of the Software, and to permit persons to whom the Software is
10+
furnished to do so, subject to the following conditions:
11+
12+
The above copyright notice and this permission notice shall be included in all
13+
copies or substantial portions of the Software.
14+
15+
THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
16+
IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
17+
FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
18+
AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
19+
LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
20+
OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
21+
SOFTWARE.
Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
Created on 2017-09-02
2+
3+
Author : Muhammad Arslan <rslnkrmt@gmail.com>
4+
5+
Licence : MIT
6+
7+
Finding the number of combinations (nCr) without use of factorials
8+
------------------------------------------------------------------
9+
10+
Using factorials recursively can greatly reduce performance and use a lot of memory especially when finding nCr of large sets. This aims to correct that to some extent.
11+
Enjoy and provide me with suggestions if any.
Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,9 @@
1+
import operator as op
2+
3+
4+
def ncr(n, r):
5+
r = min(r, n-r)
6+
if x == 0: return 1
7+
num = reduce(op.mul, xrange(n, n-1, -1))
8+
denom = reduce(op.mul, xrange(1, r+1))
9+
return num // denom

0 commit comments

Comments
 (0)