1

私はpython2.7でtkinterを使用しています。私がやりたいのは、あるクラスでキャンバスを描画してから、キャンバスの周りに正方形を移動する別のクラスをクラス化することだけです。しかし、何らかの理由で私は得る

Exception in Tkinter callback
Traceback (most recent call last):
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/lib-tk/Tkinter.py", line 1410, in __call__
    return self.func(*args)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/lib-tk/Tkinter.py", line 495, in callit
    func(*args)
AttributeError: Animation instance has no __call__ method

私が見たところ、値を上書きしたり、名前が別の名前を隠したりすると、このエラーが発生します。しかし、私が乗っているかもしれないものを見ることができません。他の誰かが問題を見ることができますか?

driver.py:

from Tkinter import *
import animation
class Alien(object):
    def __init__(self):
        #Set up canvas
        self.root = Tk()
        self.canvas = Canvas(self.root, width=400, height=400)
        self.canvas.pack()
        #Vars
        self.map = [[1, 0, 0, 1, 0], [0, 1, 0, 1, 0], [0, 0, 1, 0, 0], [0, 1, 1, 0, 0], [1, 0, 0, 1, 0]]
        self.x = 0
        self.y = 0
        r = 50
        land = {}
        #Draw Init
        for i, row in enumerate(self.map):
            for j, cell in enumerate(row):
                color = "black" if cell else "green"
                land[(i, j)] = self.canvas.create_rectangle(r * i, r * j , r * (i + 1), r * (j + 1),
                                             outline=color, fill=color)
        self.creature = self.canvas.create_rectangle(r * self.x, r * self.y, r * (self.x + 1), r * (self.y + 1),
                                                     outline="red", fill="red")
        self.canvas.pack(fill=BOTH, expand=1)
        #Action
        self.root.after(0, animation.Animation(self.root, self.canvas, self.creature))
        #Clost TK
        self.root.mainloop()
a = Alien()

animation.py:

from random import randrange


    class Animation():
        def __init__(self, root, canvas, creature):
            self.x = self.y = 0
            i = randrange(1, 5)
            if i == 1:
                self.y = -1
            elif i == 2:
                self.y = 1
            elif i == 3:
                self.x = -1
            elif i == 4:
                self.x = 1
            self.canvas = canvas
            self.creature = creature
            for i in range(10):
                root.after(250, self.animate())

        def animate(self):
                #root.after(250, self.animate(canvas, creature))
                """Moves creature around canvas"""
                self.canvas.move(self.creature, self.x * 50, self.y * 50)
                #self.canvas.update() no longer needed
4

3 に答える 3

3

Animation インスタンスを after に渡し、それを呼び出そうとします。インスタンスを作成してから、その animate メソッドを渡すつもりだったのではないでしょうか?

編集:インターネットに戻ったので、少し詳しく説明します。問題は次の行にあります。

self.root.after(0, animation.Animation(self.root, self.canvas, self.creature))

これによりAnimationインスタンスが作成され、それがafterメソッドに渡されます。メソッドは、それを呼び出す方法がわからない場合に爆発します。私は次のようなことをします:

animator = animation.Animation(self.root, self.canvas, self.creature)
self.root.after(0, animator.animate) # get things going

また、他の場所で指摘されているように、アニメーション コンストラクター内のループは、少なくとも 2 つの方法で壊れています。

for i in range(10):
    root.after(250, self.animate())

10 個のコールバックをすべて現在の時間に対して設定します。つまり、1/4 秒後に一度に 10 個のコールバックをトリガーします。メソッド自体を渡すのではなく、animateメソッドを呼び出してその戻り値 ( None) をに渡すため、それらはすべて失敗します。after言い換えれば、

after(ms, function())

と同じです

value = function()
after(ms, value)

これは本当にあなたが望むものではありません。これを修正する 1 つの方法はself.root = root、コンストラクターに追加してから、次のようにすることです。

self.root.after(250, self.animate)

1/4animate秒後に別の呼び出しをトリガーします。

または、Tkinter に追跡させることもできます。このafterメソッドを使用すると、引数をコールバックに渡すことができ、それを使用してafter関数自体を渡すことができるので (!)、animateメソッドはそれを探す必要がありません。

def animate(self, after):
    ... do animation ...
    after(250, self.animate, after)

次に、root.after を 1 回呼び出して、そのメソッドを渡します。

root.after(0, animator.animate, root.after)
于 2013-07-23T11:56:01.823 に答える
2

Animationへのコールバックとしてインスタンスを渡しますself.root.after()。コールバック オブジェクトは、指定された時間後に呼び出されます。 それらを呼び出すということは、それらの__call__()メソッドが呼び出されることを意味します。あなたのAnimationクラスにはそのようなメソッドがありません。私はあなたがやりたいことを何でもするものを追加することを提案します。例えば:

class Animation():
    #
    # ... your stuff from above ...
    #
    def __call__(self):
        self.animate()

別のオプションは、呼び出されるメソッドをafter()直接指定することです。

self.root.after(0, animation.Animation(self.root, self.canvas, self.creature).animate)
于 2013-07-23T11:56:05.320 に答える
2

すでに述べた欠点に加えて、あなたも持っています

            for i in range(10):
                root.after(250, self.animate())

あなたのanimation.pyで。ここでも同じことが当てはまります:この callable を 1 回呼び出した結果 ( ) の代わりにroot.after()、 callableを渡したいとします。self.animateself.animate()

前述の()「もの」の呼び出しを行います。今すぐ呼び出す場合は、 を使用します()。そうでない場合 (この場合はそうでない場合)、()ここに入力しないでください。

于 2013-07-23T12:05:09.420 に答える