Python list messing up

Asked Jul 06 '18 at 05:01

Active Jul 06 '18 at 05:01

Viewed 32 times

So I have some code:

nlist = [0,0,0,0, 0,0,0,0, 0,0,0,0, 0,0,2,0]
count = 0
for i in nlist:
    if nlist[i] == 0:
        nlist.remove(nlist[i])
        count += 1
        if count == 4:
            count = 0
print(nlist)

And when ever i run it the Output is:

[0, 0, 0, 0, 0, 0, 2, 0]

Even though the output should be this:

[2]

Can someone please tell me what I am doing wrong.

asked Jul 06 '18 at 05:01

J. Doe

1,475
2
9
17

Is `i` an element or index? You're using it in both ways. – Jonathon Reinhart Jul 06 '18 at 05:04
i is an index... – J. Doe Jul 06 '18 at 05:05
@J.Doe no, it's not: `for i in nlist:` will give you the elements of `nlist`, I didn't even notice that error. But that and iterating while you mutate are the source of your problems – juanpa.arrivillaga Jul 06 '18 at 05:09
count backwards, `for i in xrange(len(nlist)-1, -1, -1):` – Alex Jul 06 '18 at 05:11
@Alex although you *can* do it that way, it is inefficient and error prone. It's almost always better just to create a new list. – juanpa.arrivillaga Jul 06 '18 at 05:15

Python list messing up

0 Answers0