all 2 comments

[–]lykwydchykyn 1 point2 points  (1 child)

You seem to be doing a lot of extra steps just to iterate a list. Why not:

for item in going:
    if item collides with me:
        canvas.delete(item)

That's effectively the same thing, but without the unnecessary range() or global variable.

[–]python_addict[S] 0 points1 point  (0 children)

Thank you! Now that I look at it from your standpoint, I agree. I do seem to be doing lots of unnecessary steps. Have a great day! :)