0

この関数の何が問題になっていますか? スコープ エラーのようです (ただし、直接使用するのではなく、各 callable をリストに配置することで修正したと思います)。エラーは最大再帰深度に達しました (comp(inv,dbl,inc) の呼び出し時)...

注：問題は、なぜ最大深度に達しているのかではなく、なぜ再帰しているのかということです...

def comp(*funcs):
    if len(funcs) in (0,1):
        raise ValueError('need at least two functions to compose')
    # get most inner function
    composed = []
    print("appending func 1")
    composed.append(funcs[-1])
    # pop last and reverse
    funcs = funcs[:-1][::-1]
    i = 1
    for func in funcs:
        i += 1
        print("appending func %s" % i)
        composed.append(lambda *args, **kwargs: func(composed[-1](*args,**kwargs)))
    return composed[-1]

def inc(x):
    print("inc called with %s" % x)
    return x+1
def dbl(x):
    print("dbl called with %s" % x)
    return x*2
def inv(x):
    print("inv called with %s" % x)
    return x*(-1)

if __name__ == '__main__':
    comp(inv,dbl,inc)(2)

トレースバック (役立つ場合):

appending func 1
appending func 2
appending func 3
Traceback (most recent call last):
  File "comp.py", line 31, in <module>
    comp(inv,dbl,inc)(2)
  File "comp.py", line 17, in <lambda>
    composed.append(lambda *args, **kwargs: func(composed[-1](*args,**kwargs)))
  File "comp.py", line 17, in <lambda>
    composed.append(lambda *args, **kwargs: func(composed[-1](*args,**kwargs)))
  File "comp.py", line 17, in <lambda>
    composed.append(lambda *args, **kwargs: func(composed[-1](*args,**kwargs)))
  (...)
  File "comp.py", line 17, in <lambda>
    composed.append(lambda *args, **kwargs: func(composed[-1](*args,**kwargs)))
RuntimeError: maximum recursion depth exceeded while calling a Python object
4

2 に答える 2

5

composed作成したラムダ関数は、変数に対してクロージャーを構築します。

composed.append(lambda *args, **kwargs: func(composed[-1](*args,**kwargs)))

これは、ラムダ関数を作成するときではなく、呼び出すcomposed[-1]ときに評価されることを意味します。その効果は、それ自体が再帰的に何度も呼び出されることです。composed[-1]

ヘルパー関数 (独自のスコープを持つ) を使用してラムダ関数を作成することで、この問題を解決できます。

def comp2(f1, f2):
    return lambda *args, **kwargs: f1(f2(*args, **kwargs))

...
for func in funcs:
     composed.append(comp2(func, composed[-1]))
于 2011-08-01T15:00:41.350 に答える
1

そもそもなぜ多くの関数を生成するのかわかりません。あなたのコードの簡単なバージョンがあります：

def compose(*funcs):
    if len(funcs) in (0,1):
        raise ValueError('need at least two functions to compose')

    # accepting *args, **kwargs in a composed function doesn't quite work
    # because you can only pass them to the first function.
    def composed(arg):
        for func in reversed(funcs):
            arg = func(arg)
        return arg

    return composed

# what's with the lambdas? These are functions already ...
def inc(x):
    print("inc called with %s" % x)
    return x+1
def dbl(x):
    print("dbl called with %s" % x)
    return x*2
def inv(x):
    print("inv called with %s" % x)
    return -x

if __name__ == '__main__':
    f = compose(inv,dbl,inc)
    print f(2)
    print f(3)
于 2011-08-01T15:18:26.310 に答える