Python Scoping/Static Misunderstanding

Question

次のコードブロック1が出力2の代わりに出力1になるのはなぜですか？

コードブロック1：

class FruitContainer: 
     def __init__(self,arr=[]): 
      self.array = arr 
     def addTo(self,something): 
      self.array.append(something) 
     def __str__(self): 
      ret = "[" 
      for item in self.array: 
       ret = "%s%s," % (ret,item) 
      return "%s]" % ret 

arrayOfFruit = ['apple', 'banana', 'pear'] 
arrayOfFruitContainers = [] 

while len(arrayOfFruit) > 0: 
    tempFruit = arrayOfFruit.pop(0) 
    tempB = FruitContainer() 
    tempB.addTo(tempFruit) 
    arrayOfFruitContainers.append(tempB) 

for container in arrayOfFruitContainers: 
    print container 

**Output 1 (actual):** 
[apple,banana,pear,] 
[apple,banana,pear,] 
[apple,banana,pear,] 

**Output 2 (desired):** 
[apple,] 
[banana,] 
[pear,] 

このコードの目的は、配列を反復し、親オブジェクトの各ラップすることです。これは私の実際のコードを減らしたもので、すべてのリンゴをリンゴの袋などに加えています。私の推測は、何らかの理由で、同じオブジェクトを使用しているか、フルーツコンテナが静的配列を使用しているかのように動作していることです。私はこれをどのように修正するのか分かりません。

Answer 1

あなたのコードは、クラスを初期化するデフォルトの引数があります。解決策は以下の通り__init__の機能を変更することです。デフォルトの引数の値はコンパイル時に1回評価されるため、すべてのインスタンスは同じリストで初期化されます。そのようにそれを変更します。

def __init__(self, arr=None): 
    if arr is None: 
     self.array = [] 
    else: 
     self.array = arr 

私はここに、より完全にこれを議論：なしに渡すよりもHow to define a class in Python

Answer 2

メソッドのデフォルト引数に変更可能な値（[]など）を使用しないでください。この値は1回計算され、すべての呼び出しに使用されます。既定値として空のリストを使用すると、前の関数呼び出しによって値が変更されても、引数なしでメソッドが呼び出されるたびに同じリストが使用されます。

代わりにこれを行います：ネッドが言うように

def __init__(self,arr=None): 
    self.array = arr or [] 

Answer 3

、問題はあなたがデフォルトの引数としてリストを使用しています。詳細はhereです。

 def __init__(self,arr=None): 
      if arr is not None: 
       self.array = arr 
      else: 
       self.array = [] 

Answer 4

よりよい解決策 - この特定のインスタンスでは、むしろ一般的にはよりを - にARRのパラメータを処理することです__init__の項目の列挙セットが内部ストレージに使用するとFruitContainerはなく、配列を事前に初期化するよう：これはあなたのコンテナを初期化するために、あなたは他の列挙タイプに渡すことができるようになります

class FruitContainer: 
    def __init__(self, arr=()): 
    self.array = list(arr) 
    ... 

、より高度なPythonのユーザーが行うことができることを期待なる：それはまた別の潜在的なエイリアシングバグ打開ます

myFruit = ('apple', 'pear') # Pass a tuple 
myFruitContainer = FruitContainer(myFruit) 
myOtherFruit = file('fruitFile', 'r') # Pass a file 
myOtherFruitContainer = FruitContainer(myOtherFruit) 

：このページのすべての他の実装では

myFruit = ['apple', 'pear'] 
myFruitContainer1 = FruitContainer(myFruit) 
myFruitContainer2 = FruitContainer(myFruit) 
myFruitContainer1.addTo('banana') 
'banana' in str(myFruitContainer2) 

を、これはTrueを返します、ので、誤ってコンテナの内部ストレージに別名を付けたことがあります。

注：このアプローチは正しい答えではない常にです：「ないなし場合は、」他の例では、より良いではありません。私はオブジェクトのセット、または変更可能なコンテナを渡していますか？オブジェクトを渡しているクラス/関数が、私が与えたストレージを変更する場合、それは（a）驚くか、（b）望ましいでしょうか？この場合、私はそれが（a）であると主張します。したがって、リスト（...）呼び出しが最善の解決策です。 （b）の場合、「もしそうでなければ」は正しいアプローチであろう。