メソッドをプレーン関数で置き換えることはできますか？

Question

これは期待通りに機能しますが、私はこのアプローチについて何らかの形で不確実です。それは安全ですか？それはpythonicですか？

class Example: 
    def __init__(self, parameter): 
     if parameter == 0: 
      # trivial case, the result is always zero 
      self.calc = lambda x: 0.0 # <== replacing a method 
     self._parameter = parameter 

    def calc(self, x): 
     # ... long calculation of result ... 
     return result 

（Python2とのpython3の間に違いがあれば、私は唯一のpython3を使用しています。）

Answer 1

あなたは問題がparameterこれまで変化しなければならないので、私はそれは良い習慣は考えていないでしょう。 は代わりに、私はあなたがこれを行うべきだと思う：これは非常に混乱して

class Example: 
    def __init__(self, parameter): 
     self._parameter = parameter 

    def calc(self, x): 
     if not self._parameter: 
      return 0.0 
     # ... long calculation of result ... 
     return result 

Answer 2

。他の誰かがそれを読むと、何が起こっているのか理解できません。メソッドの先頭にifステートメントを挿入するだけです。それは0で初期化された後、あなたがself.parameterを変更した場合

def calc(self, x): 
    if self.parameter == 0: 
     return 0 
    # ... long calculation of result ... 
    return result 

また、あなたの関数はもう動作しません。

Answer 3

私はいくつかのコメントと回答の概要を投稿することに決めました。この要約には投票しないでください。代わりにを元の著者に+1してください。

• アプローチは、アプローチが
• 等、unpythonic望ましくない、または不要とみなされる特別__methods__
• を除いて使用する機能を決定するパラメータは一定でなければならない安全です。そうでない場合、このアプローチはまったく意味がありません。
• 私は一般的な例については、以下のコードと簡単な例のための明白なif cond: return 0.0を好む、いくつかの提案から：

---

class Example: 
    def __init__(self, parameter): 
     if parameter == 0: 
      self.calc = self._calc_trivial 
     else: 
      # ... pre-compute data if necessary ... 
      self.calc = self._calc_regular 
     self._parameter = parameter 

    def _calc_regular(self, x): 
     # ... long calculation of result ... 
     return result 

    @staticmethod 
    def _calc_trivial(x): 
     return 0.0