"TypeError：リストインデックスはリストではなく整数またはスライスでなければなりません" // 2つのリストパラメータを持つ関数の場合

Question

私がしようとしているのは、同じサイズの2つのリストを取り、新しいリストを返す関数を作ることです製品と一緒に。

def mult_list(x=[],y=[]): 
    z = [] 
    for i in (x,y): 
     if len(x) == len(y): 
      z.append(x[i]*y[i]) 
     else: 
      print("length of list not equal") 
    return x,y 

をしかし、私はのような何かしようとすると、私はエラーを取得しています：コードをテストするためにmult_list([8,1,4,8],[2,5,5,8])を例x = [1,2,3]とy = [4,5,6]は、これは私がこれまで持っているものであるリスト[4,10,18]

を返します。それが何らかの理由で二回print文を繰り返し以来

そして、私はmult_list([8,1,8],[2,5,5,8])

のように不均等な大きさのリストをテストするとき、私は推測している私の出力に

>>> mult_list([8,1,8],[2,5,5,8]) 
length of list not equal 
length of list not equal 
([8, 1, 8], [2, 5, 5, 8]) 

を取得するには、間違っています..

Answer 1

コードにはいくつか問題があります。まず、for i in (x, y)はあまり意味をなさない。これにより、iが最初にxに割り当てられ、次にyに割り当てられます。 iをインデックスとすると、for i in range(len(x))が必要です。これは、長さが等しいことを前提としています。ループの各繰り返しではなく、最初にチェックする必要があります（そのため、printが繰り返されています）。最後に、z（x, yではなく）を返す必要があります。また、デフォルトのパラメータx=[], y=[]はあまり意味がありませんIMHO。

def mult_list(x, y): 
    z = [] 
    if len(x) == len(y): 
     for i in range(len(x)): 
      z.append(x[i]*y[i]) 
     return z 
    else: 
     print("length of list not equal") 

これはxとyから対応する値のペアを取得するためにzip(x, y)を使用して、しかし、改善することができる - それはあなたがfor i in (x, y)と考えていたものはおそらくです。

def mult_list(x, y): 
    z = [] 
    if len(x) == len(y): 
     for xi, yi in zip(x, y): 
      z.append(xi * yi) 
     return z 
    else: 
     print("length of list not equal") 

最後に、我々は簡単なリスト内包している複数行forループを置き換えることができます。

def mult_list(x, y): 
    if len(x) == len(y): 
     return [xi * yi for xi, yi in zip(x, y)] 
    else: 
     print("length of list not equal")