2016-05-02 15 views
0

n番目の行までのパスカルの三角形を生成する再帰関数を実行しようとしています.nはユーザー入力です。Pascalの三角関数のPython再帰関数

def printPascal(l,n): 
    while n != 1: 
     temp = [None]*(len(l)+1) 
     temp[0] = 1 
     temp[len(l)] = 1 
     for i in range(1,len(temp)-1): 
      temp[i] = l[i] + l[i-1] 
      l = temp 
      print(temp) 
     n = n-1 
     printPascal(l,n) 


n = int(input("Enter a value for n:")) 
l = [1,1] 
printPascal(l,n) 

そして、これは、それが私を与えるエラーです:これは、これまでに私のコードです

Traceback (most recent call last): 
    File "C:\Users\User\Desktop\test.py", line 16, in <module> 
    printPascal(l,n) 
    File "C:\Users\User\Desktop\test.py", line 11, in printPascal 
    printPascal(l,n) 
    File "C:\Users\User\Desktop\test.py", line 7, in printPascal 
    temp[i] = l[i] + l[i-1] 
TypeError: unsupported operand type(s) for +: 'NoneType' and 'int' 

事は、私はちょっと問題を理解して、無駄にそれを追跡しようとしています。私は何とかtemp[i] = l[i] + l[i-1]コードでl[i]またはl[i -1]のいずれかが"None"であることを知っています。なぜそれがわかりません。

私のこの小さな苦境にお時間をいただきありがとうございます。

答えて

1

コードにインデントエラーが少しありました。

def printPascal(l,n): 
    while n != 1: 
     temp = [0]*(len(l)+1) 
     temp[0] = 1 
     temp[len(l)] = 1 
     for i in range(1,len(temp)-1): 
      temp[i] = l[i] + l[i-1] 
     l = temp 
     print(temp) 
     n = n-1 
     printPascal(l,n) 
+0

。 0で初期化するだけで問題が隠されます。 – hoyland

+0

はい、あなたはそれについて正しいです。しかし、それは私が実際にエラーを実現するために有効でした:) – Keiwan

1

"for"ループ内にコードが多すぎます。 tempにすべての値を入力する前に、の値をtempの値に置き換えます。

ループには最初のステートメントのみを含める必要があります。

+0

ありがとう、私の問題を修正した。ループには他の行も含まれていることはわかりませんでした。 –

1

の代わりにあなたのコードを固定し、私はあなたに最適化されたアプローチを紹介します:

def pascal_triangle(n, triangle=[[1]]): 
    if n > len(triangle): 
     last_row = triangle[-1] 
     next_row = [a+b for (a, b) in zip([0] + last_row, last_row + [0])] 
     return pascal_triangle(n, triangle + [next_row]) 
    return triangle 

あなたはそれを実行するにはどうすればよい:

n = int(input("Pascal's triangle size: ")) 
print(*pascal_triangle(n), sep="\n") 

例の出力を入力9ため:

[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] 

か、よりきれいな印刷コマンドを使用してください:

print(*[" ".join(map(str, line)).center(40) for line in pascal_triangle(9)], sep="\n") 

それは次のようになります。私はきちんと読んでいる場合には、犯人のインデントエラーです

    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    

See this code running on ideone.com