菜鸟又来了：关于字典使用for循环后出现的问题，求大神解答...

norrysun

其实是第37讲课后动动手最后一题遇到的一点小问题，我就把出问题的代码摘出来吧，如下：

1. def move(position):
   
2.     import random
   
3.     step=1  
   
4.     direction=random.randint(0,1)#用0代表x轴，用1代表y轴
   
5.     minus=random.randrange(-1,2,2)#用-1代表负方向，用1代表正方向
   
6.     position[direction] += minus*step#一次移动
   
7.     if position[direction]>10:#超过边界反方向运动
   
8.         position[direction] = 10-(position[direction]-10)
   
9.     elif position[direction]<(-10):
   
10.         position[direction] = -10+((-10)-position[direction])
    
11.     return position
    
12. 
    
13. #设置初始值
    
14. times=10
    
15. f={}
    
16. f0=f.fromkeys(range(10),[0,0])
    
17. list1=list(f0.keys())
    
18. 
    
19. for i in list(f0.keys()):#10个鱼分别移动一次
    
20.     temp=move(f0[i])
    
21.     f0[i]=temp
    
22.     print(i,f0[i])
    
23. print(f0)
    

复制代码

结果就会发现，f0的所有元素都是相同的，而且都是最后一次迭代后的值，我需要的每个元素都是随机的结果呀！！！！。。。实在看不懂为什么，求解答！！

zltzlt

1. def move(position):
   
2.     position = position[:]
   
3.     import random
   
4.     step = 1
   
5.     direction = random.randint(0, 1)  # 用0代表x轴，用1代表y轴
   
6.     minus = random.randrange(-1, 2, 2)  # 用-1代表负方向，用1代表正方向
   
7.     position[direction] += minus * step  # 一次移动
   
8.     if position[direction] > 10:  # 超过边界反方向运动
   
9.         position[direction] = 10 - (position[direction] - 10)
   
10.     elif position[direction] < (-10):
    
11.         position[direction] = -10 + (-10 - position[direction])
    
12.     return position
    
13. 
    
14. 
    
15. # 设置初始值
    
16. times = 10
    
17. f = {}
    
18. f0 = f.fromkeys(range(10), [0, 0])
    
19. list1 = list(f0.keys())
    
20. 
    
21. for i in list(f0.keys())[:]:  # 10个鱼分别移动一次
    
22.     temp = move(f0[i])
    
23.     f0[i] = temp
    
24.     print(i, f0[i])
    
25. print(f0)

复制代码

冬雪雪冬

把f0=f.fromkeys(range(10),[0,0])改为
f0={i: [0,0] for i in range(10)}
试试

norrysun

冬雪雪冬 发表于 2020-3-7 10:54
把f0=f.fromkeys(range(10),[0,0])改为
f0={i: [0,0] for i in range(10)}
试试

谢谢，感觉又get到新技能

norrysun

zltzlt 发表于 2020-3-7 10:51

谢谢，看到这一讲的参考答案之后，感觉迭代中使用列表的确要非常小心。

十月故里

1. def move(p):
   
2.     import random
   
3.     direction=random.randint(0,1)#用0代表x轴，用1代表y轴
   
4.     minus=random.randrange(-1,2,2)#用-1代表负方向，用1代表正方向
   
5.     p[direction] += minus#一次移动
   
6.     return p
   
7. 
   
8. #设置初始值
   
9. f={}
   
10. #f={i: [0,0] for i in range(3)}
    
11. f=f.fromkeys('abc', [0, 0])
    
12. f1=dict(a=[1,1],b=[2,2],c=[3,3])
    
13. for i in (f.keys()):#10个鱼分别移动一次
    
14.     f[i]=move(f1[i])
    
15.     print(f)

复制代码

我把你的代码简化了一下，然后这样改了之后就正常了，我试了几次觉得原因是出在了你22行move（f0[i]）这里的，其次是f0是通过fromkeys这个工厂函数生成的，然后我是这么个理解不知道对不对哈
你循环的时候修改f0[i]的时候，由于它是绑定的是fromkeys这个工厂函数的，于是代码在执行move这个函数的时候，先去fromkeys这里查询这个值，然后再跑去执行move，然后带着move后的值返回fromkeys这里，按工厂函数的方法给各个key赋值一次，所以其实你每次循环的值，都是对每个key的值重新赋了个一样的值
所以只要避开这两点就好了，两种方法
第一种
就是版主 冬雪雪冬 说的把你创建f0的方法改成别的
第二种
给move传入的参数改成别的，不是你的f0的其他东西都可以

ps：好像py读取东西都是往上读取，好比如你的def move是写在最上面一样，然后按着这个思路来理解的，不一定对，不过我试过zltzlt版主的方法，好像是行不通的

norrysun

十月故里 发表于 2020-3-7 21:46
我把你的代码简化了一下，然后这样改了之后就正常了，我试了几次觉得原因是出在了你22行move（f0）这里的 ...

谢谢你认真的解释，你提到我代码里的f0绑定的是fromkeys这个工厂函数的时候，让我想起小甲鱼说的“直接赋值相当于是在同一个位置新加了一个标签”，突然就觉得有一点点能理解了~