5
\$\begingroup\$

I have written a program that takes a number n and prints Pascal's triangle having n number of rows.

Here is my code -

n = int(input("Enter number of rows: "))
a = []

for i in range(n):
    a.append([])
    a[i].append(1)

    for j in range(1, i):
        a[i].append(a[i - 1][j - 1] + a[i - 1][j])
    if (n != 0):
        a[i].append(1)

for i in range(n):
    print("   " * (n - i), end = " ", sep = " ")

    for j in range(0, i + 1):
        print('{0:5}'.format(a[i][j]), end = " ", sep = " ")
    print()

Here are some example outputs -

Enter number of rows: 3
              1 
           1     1 
        1     2     1 

Enter number of rows: 6
                       1 
                    1     1 
                 1     2     1 
              1     3     3     1 
           1     4     6     4     1 
        1     5    10    10     5     1 

Enter number of rows: 10
                                   1 
                                1     1 
                             1     2     1 
                          1     3     3     1 
                       1     4     6     4     1 
                    1     5    10    10     5     1 
                 1     6    15    20    15     6     1 
              1     7    21    35    35    21     7     1 
           1     8    28    56    70    56    28     8     1 
        1     9    36    84   126   126    84    36     9     1 

Therefore, I would like to know whether I could make this program shorter and more efficient.

Any help would be highly appreciated.

\$\endgroup\$
2
  • 1
    \$\begingroup\$ You say "more efficient" but it doesn't look like you care with all those prints. \$\endgroup\$
    – Peilonrayz
    Jun 7, 2019 at 14:55
  • 1
    \$\begingroup\$ As a general rule in Python, you want to avoid calling append as much as possible. It's usually easy to do so. \$\endgroup\$
    – jpmc26
    Jun 8, 2019 at 14:10

4 Answers 4

11
\$\begingroup\$

One issue with your approach is your memory usage is \$O(n^2)\$. You are building up the entire Pascal's Triangle, and subsequently printing it. If you simply want to print Pascal's Triangle, you don't need to keep the entire triangle in memory. You can simply generate and print it row by row. Since you generate the next row from only the previous row, the memory requirement is simply two rows, which is \$O(n)\$.

>>> row = [1]
>>> for _ in range(5):
...    row = [1] + [x + y for x, y in zip(row[:-1], row[1:])] + [1]
...    print(row)
... 
[1, 1]
[1, 2, 1]
[1, 3, 3, 1]
[1, 4, 6, 4, 1]
[1, 5, 10, 10, 5, 1]
>>> 

Demystifying [1] + [x + y for x, y in zip(row[:-1], row[1:])] + [1]:

row[:-1] is all the values in row, except the last one. For example, if row is [1, 3, 3, 1], then row[:-1] is [1, 3, 3]. Similarly, row[1:] is all the values of row except the first, so [3, 3, 1]. zip() takes the first value of each of the lists, and returns those as the tuple (1, 3), which get assigned to x and y respectively, which get added together to get 4. Then, zip() emits the next pair of values, (3, 3) which results in 6, and then the final pair (3, 1) results in 4, producing the list [4, 6, 4]. The [1] + ... + [1] adds the bookends, which results in the next row of the triangle: [1, 4, 6, 4, 1].

Instead of needing to keep track of the last row, and feed that back to generate the next row, we can package that up into a generator function, which can prime the loop and do the feedback itself:

>>> def pascals_triangle(n):
...     row = [1]
...     yield row
...     for _ in range(n):
...         row = [1] + [x + y for x, y in zip(row[:-1], row[1:])] + [1]
...         yield row
... 
>>> for row in pascals_triangle(5):
...     print(row)
... 
[1]
[1, 1]
[1, 2, 1]
[1, 3, 3, 1]
[1, 4, 6, 4, 1]
[1, 5, 10, 10, 5, 1]
>>> 

Format the resulting rows as desired.

>>> for i, row in enumerate(pascals_triangle(5)):
...     print(("   "*(5-i) + "{:6}"*(i+1)).format(*row))
... 
                    1
                 1     1
              1     2     1
           1     3     3     1
        1     4     6     4     1
     1     5    10    10     5     1
>>> 
\$\endgroup\$
6
  • 1
    \$\begingroup\$ If you want to get really picky about performance, your one-liner print probably has to be rethought a little. Maybe use writes to sys.stdout to avoid the implicit buffer flush with print; don't do string concatenation; maybe other micro-optimizations associated with the format string. \$\endgroup\$
    – Reinderien
    Jun 8, 2019 at 3:09
  • 1
    \$\begingroup\$ @Reinderien The performance was the generation of the triangle rows. My print is horrendously inefficient 2-line statement I used for demonstration purposes; I said “as desired” ... mine was just a (bad/cool) example. \$\endgroup\$
    – AJNeufeld
    Jun 8, 2019 at 3:12
  • \$\begingroup\$ It's a fair point. Also, if you want to get technical, you've generated one too many rows given n=5. \$\endgroup\$
    – Reinderien
    Jun 8, 2019 at 3:19
  • \$\begingroup\$ @Reinderien Yup! I can’t from zero. \$\endgroup\$
    – AJNeufeld
    Jun 8, 2019 at 3:19
  • \$\begingroup\$ Uhh. Five things counting from zero is still five things, not six things. \$\endgroup\$
    – Reinderien
    Jun 8, 2019 at 3:41
5
\$\begingroup\$

Functions & code organisation

At the moment, your code is basically split into 3 parts:

  • getting the input from the user
  • generating a triangle
  • printing a triangle

Things would be easier to understand/test/reuse if we could split them into 3 logical parts. Functions could be a good way to do things here.

Also, it could be a good occasion to use the if __name__ == "__main__": trick to split function definitions from the code using it and performing Input/Output.

def generate_pascal(n):
    ...
    return a

def print_pascal(a):
    ...

if __name__ == "__main__":
    n = int(input("Enter number of rows: "))
    p = generate_pascal(n)
    print_pascal(p)

Simplifying print_pascal

You can define end and sep as parameters to print_pascal.

Loop like a native: I highly recommend Ned Batchelder's excellent talk called "Loop like a native". You usually do not need to write loops based on the length of the list you are working on, you can just iterate over it.

In our case, this leads to the following code:

def print_pascal(a, end = " ", sep = " "):
    n = len(a)
    for i, line in enumerate(a):
        print("   " * (n - i), end = end, sep = sep)
        for e in line:
            print('{0:5}'.format(e), end = sep, sep = sep)
        print()

Note: This hilight an issue with the structure provided to the printing procedure as the first line contains 2 numbers.


Simplifying generate_pascal

In generate_pascal, you add a list to a there refer to that list using a[i].

It would be clearer and more efficient to just define a new list that you fill and then eventually add to a.

We get:

def generate_pascal(n):
    a = []
    for i in range(n):
        line = []
        line.append(1)
        for j in range(1, i):
            line.append(a[i - 1][j - 1] + a[i - 1][j])
        if (n != 0):
            line.append(1)
        a.append(line)
    return a

Fixing a bug in generate_pascal

The issue hilighted previously can probably be fixed by replacing if (n != 0) with if (i != 0) which is more usually written without the superfluous parenthesis: if i != 0 or even if i.

More generally, this if could be applied to most of the logic in the loop:

def generate_pascal(n):
    a = []
    for i in range(n):
        line = []
        line.append(1)
        if i:
            for j in range(1, i):
                line.append(a[i - 1][j - 1] + a[i - 1][j])
            line.append(1)
        a.append(line)
    return a

Now, we can take this chance to access a[i - 1] via a variable with a proper name for instance prev_line. Also, The conventional way to access the latest element from an array is to use the -1 index. We could write:

        prev_line = a[-1]
        for j in range(1, i):
            line.append(prev_line[j - 1] + prev_line[j])

This this stage, the full code looks like:

def generate_pascal(n):
    p = []
    for i in range(n):
        line = [1]
        if i:
            prev_line = p[-1]
            for j in range(1, i):
                line.append(prev_line[j - 1] + prev_line[j])
            line.append(1)
        p.append(line)
    return p

def print_pascal(p, end = " ", sep = " "):
    n = len(p)
    for i, line in enumerate(p):
        print("   " * (n - i), end = end, sep = sep)
        for e in line:
            print('{0:5}'.format(e), end = sep, sep = sep)
        print()

if __name__ == "__main__":
    n = int(input("Enter number of rows: "))
    p = generate_pascal(n)
    print_pascal(p)


\$\endgroup\$
5
  • \$\begingroup\$ Do we review code organization and simplification if performance is the goal? \$\endgroup\$ Jun 7, 2019 at 15:41
  • 1
    \$\begingroup\$ @ThomasWeller you can review whatever you like in the question no matter what the tags are. \$\endgroup\$
    – Peilonrayz
    Jun 7, 2019 at 15:50
  • \$\begingroup\$ @ThomasWeller I've edited my answer to add details that may have a positive performance impact. \$\endgroup\$
    – SylvainD
    Jun 7, 2019 at 16:24
  • \$\begingroup\$ @Josay - To be continued - Is there still more? \$\endgroup\$
    – Justin
    Jun 16, 2019 at 3:06
  • \$\begingroup\$ @Justin nothing more than what the others have said. I've edited my answer accordingly \$\endgroup\$
    – SylvainD
    Jun 16, 2019 at 6:53
4
\$\begingroup\$

There's an even better way to generate the Pascal's triangle that only requires O(1) memory. It can even calculate a certain row directly without requiring previous rows to be computed.

It works like this:

  • the first number in each row is always 1.
  • consider the fraction (n / 1) where n in the row index (1-based)
  • the 2nd number is obtained by multiplying the first number by this fraction
  • decrement the numerator of the fraction by 1 and increment the denominator by 1
  • obtain the 3rd number by multiplying the fraction with the 2nd number
  • and so on, for (n + 1) numbers

Worked Example:

suppose n = 4

  • 1st number = 1
  • 2nd number = 1 * (4 / 1) = 4
  • 3rd number = 4 * (3 / 2) = 6
  • 4th number = 6 * (2 / 3) = 4
  • 5th number = 4 * (1 / 4) = 1

which is exactly the 4th row of triangle

Code:

def print_row(n):
    numerator, denominator = n, 1
    cur = 1
    for _ in range(n + 1):
        print(cur, end='\t')
        cur = (cur * numerator) // denominator
        numerator -= 1
        denominator += 1
    print()

for row in range(5):
    print_row(row)
\$\endgroup\$
1
  • \$\begingroup\$ While this might be a great SO answer for Code Review it would be better if you made some observations about the posters code as well. \$\endgroup\$
    – pacmaninbw
    Jun 8, 2019 at 13:24
4
\$\begingroup\$

The following is a profiling run of all of the methods I've seen above, with some improvements to the I/O.

from io import StringIO
from timeit import timeit
from sys import stdout, stderr
import numpy as np


def justin(n):
    a = []

    for i in range(n):
        a.append([])
        a[i].append(1)

        for j in range(1, i):
            a[i].append(a[i - 1][j - 1] + a[i - 1][j])
        if n != 0:
            a[i].append(1)

    for i in range(n):
        print("   " * (n - i), end=" ", sep=" ")

        for j in range(0, i + 1):
            print('{0:5}'.format(a[i][j]), end=" ", sep=" ")
        print()


def josay(n):
    def generate_pascal():
        p = []
        for i in range(n):
            line = [1]
            if i:
                prev_line = p[-1]
                for j in range(1, i):
                    line.append(prev_line[j - 1] + prev_line[j])
                line.append(1)
            p.append(line)
        return p

    def print_pascal(p, end=" ", sep=" "):
        for i, line in enumerate(p):
            print("   " * (n - i), end=end, sep=sep)
            for e in line:
                print('{0:5}'.format(e), end=sep, sep=sep)
            print()

    print_pascal(generate_pascal())


def neufeld_pascals_triangle(n):
    row = [1]
    yield row
    for _ in range(n - 1):
        row = [1] + [x + y for x, y in zip(row[:-1], row[1:])] + [1]
        yield row


def neufeld(n):
    for i, r in enumerate(neufeld_pascals_triangle(n)):
        print(("   "*(n-i) + "{:6}"*(i+1)).format(*r))


def neufeld_stdout(n):
    for i, r in enumerate(neufeld_pascals_triangle(n)):
        stdout.write(('   '*(n-i-1) + '{:<6}'*(i+1) + '\n').format(*r))


def neufeld_noenum(n):
    for r in neufeld_pascals_triangle(n):
        i = len(r)
        stdout.write(('   '*(n-i) + '{:<6}'*i + '\n').format(*r))


def neufeld_fmt(n):
    fmt = '   '*n
    for r in neufeld_pascals_triangle(n):
        fmt = fmt[3:] + '{:<6}'
        stdout.write((fmt + '\n').format(*r))


def neufeld_onewrite(n):
    fmt = '   '*n
    msg = ''
    for r in neufeld_pascals_triangle(n):
        fmt = fmt[3:] + '{:<6}'
        msg += fmt.format(*r) + '\n'
    stdout.write(msg)


def neufeld_strio(n):
    fmt = '   '*n
    msg = StringIO()
    for r in neufeld_pascals_triangle(n):
        fmt = fmt[3:] + '{:<6}'
        msg.write(fmt.format(*r) + '\n')
    stdout.write(msg.getvalue())


def tri_rishav(n):
    for y in range(n):
        numerator, denominator = y, 1
        cur = 1
        row = [None] * (y + 1)
        for x in range(y + 1):
            row[x] = cur
            cur = (cur * numerator) // denominator
            numerator -= 1
            denominator += 1
        yield row
def rishav_strio(n):
    fmt = '   '*n
    msg = StringIO()
    for r in tri_rishav(n):
        fmt = fmt[3:] + '{:<6}'
        msg.write(fmt.format(*r) + '\n')
    stdout.write(msg.getvalue())


def tri_numpy(n):
    row = []
    for _ in range(n):
        new_row = np.ones(1 + len(row), np.int32)
        new_row[1:-1] = row[1:] + row[:-1]
        row = new_row
        yield row
def numpy_strio(n):
    fmt = '   '*n
    msg = StringIO()
    for r in tri_numpy(n):
        fmt = fmt[3:] + '{:<6}'
        msg.write(fmt.format(*r) + '\n')
    stdout.write(msg.getvalue())


def tri_onearray(n):
    row = np.ones(n, np.int32)
    x = n - 1
    for y in range(n):
        yield row[x: x+y+1]
        row[x: x+y] += row[x+1: x+y+1]
        x -= 1
def onearray_strio(n):
    fmt = '   '*n
    msg = StringIO()
    for r in tri_onearray(n):
        fmt = fmt[3:] + '{:<6}'
        msg.write(fmt.format(*r) + '\n')
    stdout.write(msg.getvalue())


methods = ((n, globals()[n]) for n in (
    'justin', 'josay', 'neufeld',
    'neufeld_stdout', 'neufeld_noenum', 'neufeld_fmt', 'neufeld_onewrite',
    'neufeld_strio', 'rishav_strio',
    'numpy_strio', 'onearray_strio'))


def profile():
    # IMPORTANT - run redirecting stdout to /dev/null !
    first = None
    for name, method in methods:
        method = globals()[name]
        n = 50

        def run():
            method(n)
        reps = 200
        stdout.flush()
        dur = timeit(run, number=reps) / reps
        if first is None:
            first = dur
            rel = 0
        else:
            rel = first/dur - 1
        print(f'n={n} method={name:16} {1e3*dur:5.3f}ms speedup={rel:7.2%}',
              file=stderr, flush=True)


def print_test():
    for name, method in methods:
        method(6)
        print(name)


profile()
# print_test()

This yields the following results:

n=50 method=justin           7.276ms speedup=  0.00%
n=50 method=josay            6.956ms speedup=  4.60%
n=50 method=neufeld          2.110ms speedup=244.85%
n=50 method=neufeld_stdout   1.947ms speedup=273.78%
n=50 method=neufeld_noenum   1.951ms speedup=273.02%
n=50 method=neufeld_fmt      1.925ms speedup=277.90%
n=50 method=neufeld_onewrite 1.577ms speedup=361.45%
n=50 method=neufeld_strio    1.555ms speedup=367.87%
n=50 method=rishav_strio     1.714ms speedup=324.50%
n=50 method=numpy_strio      2.687ms speedup=170.80%
n=50 method=onearray_strio   2.529ms speedup=187.66%

Of note:

  • It was quite interesting to me that neither Rishav's computed method nor Numpy vectorization are able to beat Neufeld's simple, array-based method
  • Building up the printed output in memory before writing it to stdout is much faster than writing to stdout on each iteration, especially if you avoid print
\$\endgroup\$
1
  • 1
    \$\begingroup\$ You should really compare displaying separately from generating. Also you've made your tests really biased against Josay. wtf is their code in closures, but no-one elses is? Making all the functions return the same output (yes they don't) and build a list in the same way I get this graph. \$\endgroup\$
    – Peilonrayz
    Jun 8, 2019 at 19:20

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Not the answer you're looking for? Browse other questions tagged or ask your own question.