KeyError: 0 python 与 python 字典

Question

So I'm trying to re-create a spaceInvaders game with python. Here is where my aliens are stored:

aliens = dict()

and each time we increase the game level then a new alien is added with it's id as a key and his positions and health(each alien has 2 health and dies after two shots which deal one damage each) as his value, so the aliens dictionary can look like that:

aliens = {
0: [(100, 50), 2],
1: [(50, 200), 1],
etc...
}

now each time an alien dies I set it's health to -1 and set his positions to (-100, -100) to put him out of the view, now the problem occurs when I call a cleanAlien() function which iterates through the alien array and pops every alien that has a health equal to -1 or an out of bounds position, here is that function:

def cleanAliens():
    global aliens
    for i in range(len(aliens)):
        if(aliens[i][0] == (-100, -100) or aliens[i][1] < 0):
            aliens.pop(i);
            i -= i

However when I try to run that I get an error:

if(aliens[i][0] == (-100, -100) or aliens[i][1] < 0):
KeyError: 0

Any ideas?

Thanks in advance.

Answer 1

I'm not sure for the reason of this specific error, but it seems like the structure of aliens is not as expected.

However, you can iterate over the keys of the dictionary instead of over the size of it like so:

for i in list(aliens.keys()):
    if aliens[i][0] == (-100, -100) or aliens[i][1] < 0:
        aliens.pop(i)

And that way you don't get into this key trouble.

The list(aliens.keys()) returns a copy of the keys in the dictionary so you iterate over it, and after you remove one from the dictionary, you just continue to the next one.

(btw, the i -= i line is meaningless because i will be set next iteration according to its iterator, so you can't actually change its value)

Answer 2

Instead of:

for i in range(len(aliens)):
    if(aliens[i][0] == (-100, -100) or aliens[i][1] < 0):
        aliens.pop(i);
        i -= i

you should have used a comprehension:

aliens = { k:v for k,v in aliens.items() if v[0] != (-100,100) and v[1] >= 0 }

---

UPDATE: testing after receiving the comment about "the same error":

>>> aliens = {
... 0: [(100, 50), 2],
... 1: [(50, 200), 1],
... }
>>> aliens = { k:v for k,v in aliens.items() if v[0] != (-100,100) and v[1] >= 0 }
>>> 

There's no error.

Answer 3

EDIT: Fixed my error by creating a list which is appended each key to be removed which is then passed to this function:

def Aliens():
    global aliens
    clean = list()
    for i in aliens.keys():
        if(aliens[i][0] == (-100, -100) or aliens[i][1] < 0):
             clean.append(i)
    cleanAliens(clean)

and

def cleanAliens(cleanList):
    global aliens
    for i in range(len(cleanList)):
            aliens.pop(cleanList[i]);