メモリリークの対処法を聞きたいです。

前提・実現したいこと

メモリリークの対処法を聞きたい。

ここに質問の内容を詳しく書いてください。
（例）PHP(CakePHP)で●●なシステムを作っています。
■■な機能を実装中に以下のエラーメッセージが発生しました。

今現在Pythonを学んでいる学生です。
質問内容は円を作成し、動かすオブジェクトを作っているのですが前のオブジェクトを削除する方法を教えてほしいです。
学習本を読み、オブジェクトを適当な変数に保存し、新しいオブジェクトを作る前にdeleteで削除するという対応を調べたのですが、どの部分にデリートを挿入するのかなどが記載されていなかったため質問いたしました。

発生している問題・エラーメッセージ

円を一つ作成して動かす（動かす処理は前の円を白にして見え無くし、消したように見せているだけです）だけなら問題ないのですが複数の円を作成し動かすとメモリリークが発生しプログラムがうまく動きません。

該当のソースコード

python
1ball=[{"x":400,"y":300,"dx":1,"dy":1,"color":"red"},
2    {"x":200,"y":100,"dx":-1,"dy":1,"color":"green"},
3    {"x":100,"y":200,"dx":1,"dy":-1,"color":"blue"}
4    ]
5
6円を動かす処理ですballに入っているリストの数だけ円を作成しています。
7def move():
8    global ball
9    for b in ball:
10        
11        canvas.create_oval(b["x"]-20,b["y"]-20,b["x"]+20,b["y"]+20,fill="white",width=0)
12        
13        b["x"]=b["x"]+b["dx"]
14        b["y"]=b["y"]+b["dy"]
15        
16        canvas.create_oval(b["x"]-20,b["y"]-20,b["x"]+20,b["y"]+20,fill=b["color"],width=0)
17
18        if b["x"]>=canvas.winfo_width():
19            b["dx"]=-1
20        if b["x"]<=0:
21            b["dx"]=1
22        if b["y"]>=canvas.winfo_height():
23            b["dy"]=-1
24        if b["y"]<=0:
25            b["dy"]=1
26        
27        
28        root.after(10,move)

Python

試したこと

学習本を読んだ。ネットで検索したがいまいち理解できなかった。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

IDLEを使っています。pythonのバージョンは3.9です。

maisumakun

2021/07/26 02:32

> プログラムがうまく動きません。具体的には、どういう状況になるのですか？

quickquip

2021/07/26 02:38

このコードは「うまく動きません」と言っているコードではなくて、うまく動いている方のコードではないでしょうか?

jbpb0

2021/07/26 03:15

pythonのコードの一番最初の行のすぐ上に ```python だけの行を追加してくださいまた、pythonのコードの一番最後の行のすぐ下に ``` だけの行を追加してください現状、コードがとても読み辛いです質問にコードを載せる際に上記をやってくれたら、他人がコードを読みやすくなり、コードの実行による現象確認もやりやすくなるので、回答されやすくなります

kaful

2021/07/26 03:20

>maisumakunさん具体的にはエラーなどは出ないのですが、メモリが圧迫してうまく描画されていない状態になります。 >quickquipさん申し訳ございません。間違えて記載していました。リストの中身を二つ以上にするとエラーというか動かなくなっていました。

行動規範の内容に同意します

回答3件

ベストアンサー

   for b in ball:
   (略)
        root.after(10,move)

ですから、リストを3つにするとroot.after(10,move)が3回よばれます。
すると10ms後にmoveが3回呼ばれるんじゃないでしょうか? （詳しくないので自信はありません）

そうだとすると、次の10ms後に9回、次の10ms後に27回、、、、となっているのではないですか?

意図しているコードは

python
1def move():
2    global ball
3    for b in ball:
4        canvas.create_oval(b["x"]-20,b["y"]-20,b["x"]+20,b["y"]+20,fill="white",width=0)
5
6        b["x"]=b["x"]+b["dx"]
7        b["y"]=b["y"]+b["dy"]
8
9        canvas.create_oval(b["x"]-20,b["y"]-20,b["x"]+20,b["y"]+20,fill=b["color"],width=0)
10
11        if b["x"]>=canvas.winfo_width():
12            b["dx"]=-1
13        if b["x"]<=0:
14            b["dx"]=1
15        if b["y"]>=canvas.winfo_height():
16            b["dy"]=-1
17        if b["y"]<=0:
18            b["dy"]=1
19
20
21    root.after(10,move)

ではないでしょうか?

(ついで)

python
1def move():
2    for b in ball:
3        canvas.create_oval(b["x"]-20,b["y"]-20,b["x"]+20,b["y"]+20,fill="white",width=0)
4
5    for b in ball:
6        b["x"]=b["x"]+b["dx"]
7        b["y"]=b["y"]+b["dy"]
8
9        canvas.create_oval(b["x"]-20,b["y"]-20,b["x"]+20,b["y"]+20,fill=b["color"],width=0)
10
11        if b["x"]>=canvas.winfo_width():
12            b["dx"]=-1
13        if b["x"]<=0:
14            b["dx"]=1
15        if b["y"]>=canvas.winfo_height():
16            b["dy"]=-1
17        if b["y"]<=0:
18            b["dy"]=1
19
20
21    root.after(10,move)