为何 for 内 if 只执行一次？

This topic created in 2277 days ago, the information mentioned may be changed or developed.

例如：names[王八，王九，李大，李二，赵四]
for a in names:
if a 姓王，names.remove(a)
if a 姓李, names.remove(a)
print(names)
结果不会留下[赵四]，而是有[王九、李二、赵四]，为什么会这样？
for 循环不会把 5 个字符串遍历 5 次吗？

names

赵四

李二

王九

23 replies • 2020-02-23 19:21:27 +08:00

BrettD

Feb 22, 2020 via iPhone

是不是因为你在 for names 的循环里对 names 本体产生了修改

wwqgtxx

Feb 22, 2020 via iPhone

各个语言几乎都不允许在 foreach 语句中修改迭代集合本体，那样会使得迭代器失效
比如 c++中会导致悬空指针
python 中导致生成器意外退出

xupefei

Feb 22, 2020

楼上说的对。
实在不行你就倒着遍历。

toliho

Feb 22, 2020

@wwqgtxx 了解了

toliho

Feb 22, 2020

@xupefei 咋个倒着遍历法？ remove 改成 append 吗

crella

Feb 22, 2020

@wwqgtxx 不一定吧，ruby 的可以在循环里面删，见 https://paste.ubuntu.com/p/YHSsBXpz9c/

xiri

Feb 22, 2020

for a in names
实际执行逻辑类似
for i in range(len(names)):
a=names[i]
第一遍执行，i=0，移除“王八”，names=[王九，李大，李二，赵四]
第二遍执行，i=1，a=李大，移除，names=[王九，李二，赵四]
第三遍，i=2，a=赵四，后面就不用多说了吧

xiri

Feb 22, 2020

@xiri
注意这个 range(len(names))，就算后面 len(names)发生变化，对 i 的取值也不会有影响（可以理解成在第一次执行是生成了一个列表，后面是在这个列表中迭代，不会再计算了）

ybw

Feb 22, 2020 via Android

不要被楼上半桶水误导了
比如 c++支持迭代内删除 map、vector 等
记得事前做一个++操作，保存指向下一元素的指针就好了

wwqgtxx

Feb 22, 2020 via iPhone

@crella 这个吧取决于各个语言的实现，比如 java 的 iterator 要求先调用 next 才允许对旧的 iterator 执行 remove，否则会抛出 IllegalStateException
至于 ruby 的行为可能要看他的文档才能解释为何可以正常执行了
不过这种在 foreach 循环中删除元素的行为还是尽量避免的好，容易引起各种奇奇怪怪的问题

crella

Feb 22, 2020

@wwqgtxx 别说了，我已经懵 b 了，api doc 也没说明。

见 https://gitee.com/crella/rubycode/blob/master/drop_from_array.rb

我一会儿去 ruby china 问问

wwqgtxx

Feb 22, 2020 via iPhone

@ybw 都说了是在 foreach 中，你要是手动写 for 循环，那么一般是
if (你的判断条件) it=map.remove(it);
else it++;
当然不会有这个破问题

看别人回答麻烦看仔细，别上来就说别人半桶水

crella

Feb 22, 2020

楼主现在看我的链接，应该就明白了。我也是想了一会儿。形式可能是:

i = 0
while true
if (i < 数组.长度)
x = 数组[i]
if ....
数组.删除(x)
end
i += 1
else
break
end
end

然后数组.长度是实时更新的

crella

Feb 22, 2020 via Android

@wwqgtxx 你说的是对的。ruby 遍历数组，修改 array 的同时也影响了 array.each 的抛出的变量。我发的第一段代码只不过是刚好每隔一个数就是偶数而被删除，后面我再测试一下其他的排列，得到和楼主所得的类似的结果。

不过一般都是直接对数组进行 map 操作，楼主的操作应该也是实验性质的吧。

crella

Feb 22, 2020 via Android

puts RUBY_VERSION
names = %w(王一王二王三)
a = names.each
puts a.inspect
names.pop
puts a.inspect

结果
2.5.1
#<Enumerator: ["王一", "王二", "王三"]:each>
#<Enumerator: ["王一", "王二"]:each>

真是奇怪，我以为 enum 会保存自己的值，这里看来并不是

Feb 23, 2020 via iPhone

python 的话是不是这样可解
for name in names[:]:
....

xqin

Feb 23, 2020

当王八成立的时候, 你把王八移除了, 而此时王九变成了第一个(下标 0),

而迭代器下次遍历时, 获取的是下标 1 的, 因为 0 刚才才取过了..

所以此时取到的是李大...

依次类推...

希望你能明白..

toliho

Feb 23, 2020

哇一天没来这么多回答，多谢各位

toliho

Feb 23, 2020

@ddsfeng 听懂了

toliho

Feb 23, 2020

@ik 没太懂

toliho

Feb 23, 2020

@crella 我用了这个办法解决：
new_names =[]
for n in names:
if 姓王:
pass
elif 姓李：
pass
else:
new_names.append(n)

toliho

Feb 23, 2020

我还发现一个问题，在 for 循环里，如果用了 if if，只会执行第二个 if ；必须用 if elif elif ……或者 if elif else

Feb 23, 2020 via iPhone

@toliho #20 试一下

![20200223191636.png]( https://i.loli.net/2020/02/23/fpmgUsIvdl1BbH9.png)