list 删除元素后，怎样减循环次数？

[omg21]

list1 = ['print','lock','china','page']
print(len(list1))
print(list1)
for i in range(0,len(list1)):
if list1[i].find('a') != -1:
list1.pop(i)

这段代码的本意是查找列表中各元素里有没有字母 a ，如果有 a 就删除这个元素，可是现在发现个问题，删除了以后 len(list1)就减少了，但循环的次数是不减的，所以到了后面就出错了。这样的情况应该怎么改？

[omg21]

list1 = ['print','lock','china','page']
for sub in list1:
能不能在这里直接删除元素？

[wannafly]

正规(高效)的做法应该是在循环中把要删除的元素往前(后)交换, 并同时记录有效元素的最后(起始)索引, 循环结束了一次性删除待删除元素.

[cheneydog]

你需要一个新的 list

[264768502]

list1 = [x for x in list1 if 'a' not in x]

[billion]

楼主的代码有一股浓浓的 Java 的味道。
Python 有更加现代化的写法。


[billion]

https://ooo.0o0.ooo/2016/10/30/5815caa3b24bd.png

图片又发不出来了吗？

[bingxx]

@billion groovy 也可以有优雅的写法。

def list1 = ['print','lock','china','page']
def list2 = list1.findAll{it?.contains("a")}
println(list2)

[omg21]

@wannafly
@cheneydog
@264768502
@billion
@bingxx

非常感谢

[ferran]

用迭代器遍历

[Mandown]

@billion 回复貌似不资词 markdown

[omg21]

@billion 没办法，以前 vb,java 用习惯了，一时半会儿改不过来。
再追加一个问题，如果 list1 = ['print','lock','china','g']，现在最后一个元素没有 a 了，我想在找到第一个有 a 的元素后不管后边的元素有没有 a 都保留下来，应该怎样做？

[liuxingou]

@omg21

VB 用习惯是一个比较坑的情景。。。

如果对内存比较敏感推荐用单向链表而不是 list 。

[luban]

上面说的对，用迭代器。
java 里面使用 foreach 可以安全的 remove 元素

[luban]

java8 的流使用 filter 过滤出需要的元素也很方便的
只会 java 路过

[forestyuan]

正确的做法是从后往前删，就没有问题了。
也就是， for(i = n-1; i >= 0; i --)，而不是 for(i = 0; i < n; i ++)

[mooncakejs]

这种操作应该用 filter 而不是循环去删

[imn1]

改变思维吧，循环过程中变更 list 长度是错误的思路

[1373569162]

@omg21
1
2 def f(n, l):
3 for x in l:
4 if n in x:
5 return l[l.index(x):]


list1 = ['print', 'lock', 'china', 'g']
list2 = f('a', list1)

[speedcell4]

如果我没理解错的话。。。

```python
list1 = ['print', 'lock', 'china', 'page']

list2 = [item for item in list1 if item.count('a') == 0]

print(list2)
# ['print', 'lock']
```

[speedcell4]

或者更直观的

list1 = ['print', 'lock', 'china', 'page']

list2 = [item for item in list1 if 'a' not in item]

print(list2)

[alittletrain]

列表推导最 pythonic

[lrh3321]

还可以使用内置 filter 函数

list1 = ['print', 'lock', 'china', 'page']

list2 = filter(lambda item: 'a' not in item,list1)

print(list2)

[billion]

@omg21 你的第二个需求

如果你的 list1 里面至少有一个元素包含字母 a, 那么可以这样写：

list2 = list1[list1.index([x for x in list1 if 'a' in x][0]) + 1 :]

如果可能所以元素都不含 a,那么建议使用 for 循环。

[iyangyuan]

倒着来