Python：クラスのすべての要素を集計する方法

Question

class Neuron: 
    def __init__(self, inbound_neurons=[], label=''): 
     self.label = label 
     self.inbound_neurons = inbound_neurons 
     self.outbound_neurons = [] 
     self.value = None 

     for n in self.inbound_neurons: 
      n.outbound_neurons.append(self) 

    def forward(self): 
     raise NotImplemented 

class Input(Neuron): 
    def __init__(self): 
     Neuron.__init__(self) 

    def forward(self, value=None): 
     if value is not None: 
      self.value = value 

class Add(Neuron): 
    def __init__(self, *inputs): 
     Neuron.__init__(self, inputs) 

    def forward(self): 
     for n in self.inputs: 
      self.value = self.value + n 

Add（）はNeuronクラスのサブクラスです。入力配列のすべての要素を追加するにはいくつかの問題があります。

Answer 1

class Add(Neuron): 

    def __init__(self, *inputs): 
     Neuron.__init__(self, inputs) 

    def forward(self): 
     self.value = 0 
     for n in self.inbound_neurons: 
      self.value = self.value + n.value 

     return(self.value) 

クラスの追加で「前進」関数は、合計するループを持っていますinbound_neuronsのすべての要素。

Answer 2

Firtオフのコード行は次のようになります。

for n in self.inbound_neurons: 
     self.outbound_neurons.append(self) 

self.inputsは、あなたのクラスで定義されていませんでした。入力をループするためには、あなたが持っている必要があるだろう：入力が2つの項目、別のリストと、多分それの文字列を持つリストになるよう

def __init__(self, *inputs): 
     self.inputs = inputs 
     Neuron.__init__(self, inputs) 

をしかし、それが見えます。これらは一緒に結ばれません。代わりに、self.inbound_neuronsの合計を合計したいと思うように見えます。

Answer 3

あなたの質問には関係ありませんが、非常に重要です：関数/メソッドのデフォルトのために、リストとして変更可能なデータ型を使用すべきではありません。

あなたのコードは次のように更新する必要があります：あなたはこの方法を行う必要がありますなぜ

class Neuron: 
    def __init__(self, inbound_neurons=None, label=''): 
     self.inbound_neurons = inbound_neurons or [] 
     # ... 

がここで説明されています"Least Astonishment" and the Mutable Default Argument