[已解决]第十六讲课后作业 解决sum()方法不能有字符串类型存在的缺陷

zzong2019

我的思路是不管什么类型的序列先把它转换成列表，然后通过循环体输出每一个列表中的元素，判断是不是字符类型，如果是的话就移出列表，最后返回一个新列表再求和。
但执行报错：RecursionError: maximum recursion depth exceeded while calling a Python object，请帮忙看看是哪里出问题了啊？

1. def sum(seq):
   
2.     list1=list(seq)
   
3.     for each in list1:
   
4.         if type(each)==str:
   
5.             list1.remove(each)
   
6.     return sum(list1)
   
7. mix=(1,2,5,10,17,'a','b','c')
   
8. sum(mix)

复制代码

最佳答案

月排行榜 / 总排行榜

zltzlt

2020-2-13 18:49:27

zzong2019 发表于 2020-2-13 18:48
我把函数名改过了，又变成这个报错了：TypeError: unsupported operand type(s) for +: 'int' and 'str'
...

把 for each in list1 改成 for each in list1[:] 。

1. def add(mix):
   
2.     list1 = list(mix)
   
3.     for each in list1[:]:
   
4.         if type(each) == str:
   
5.             list1.remove(each)
   
6.     return sum(list1)
   
7. 
   
8. 
   
9. mix = (1, 2, 5, 10, 17, 'a', 'b', 'c')
   
10. print(add(mix))

复制代码

跳转到最佳答案楼层

zltzlt

return sum(list1) 这一句导致了递归调用，因为你的函数名是 sum。将函数名换一个。

zzong2019

zltzlt 发表于 2020-2-13 18:29
return sum(list1) 这一句导致了递归调用，因为你的函数名是 sum。将函数名换一个。

我把函数名改过了，又变成这个报错了：TypeError: unsupported operand type(s) for +: 'int' and 'str'
然后我自己在idle上把这部分代码运行了一下，发现列表中的‘b’这个元素没有被去除，是怎么回事呢？

1. def add(mix):
   
2.     list1=list(mix)
   
3.     for each in list1:
   
4.         if type(each)==str:
   
5.             list1.remove(each)
   
6.     return sum(list1)
   
7. mix=(1,2,5,10,17,'a','b','c')
   
8. add(mix)

复制代码

zltzlt

zzong2019 发表于 2020-2-13 18:48
我把函数名改过了，又变成这个报错了：TypeError: unsupported operand type(s) for +: 'int' and 'str'
...

把 for each in list1 改成 for each in list1[:] 。

1. def add(mix):
   
2.     list1 = list(mix)
   
3.     for each in list1[:]:
   
4.         if type(each) == str:
   
5.             list1.remove(each)
   
6.     return sum(list1)
   
7. 
   
8. 
   
9. mix = (1, 2, 5, 10, 17, 'a', 'b', 'c')
   
10. print(add(mix))

复制代码

zzong2019

zltzlt 发表于 2020-2-13 18:49
把 for each in list1 改成 for each in list1[:] 。

就是在执行循环体的时候list1的值是不断在变化的，所以需要用拷贝列表来执行循环体，谢谢大神指导！

zltzlt

zzong2019 发表于 2020-2-13 18:52
就是在执行循环体的时候list1的值是不断在变化的，所以需要用拷贝列表来执行循环体，谢谢大神指导！

对。

zzong2019

zltzlt 发表于 2020-2-13 18:52
对。

还能再追问一个问题嘛？我刚才试了一下如果这个求和序列本身就是一个带数字的字符串，刚才那个代码运行是有问题的（字符串转换为列表，以字符串当中的数字形式的字符串求和永远都是等于0），我用try-expect语句好像是可以解决，您帮忙看看是不是还可以优化或者有没有什么问题呢？

1. def add(mix):
   
2.     list1=list(mix)
   
3.     for each in list1[:]:
   
4.         try:
   
5.             list1[list1.index(each)]=int(each)
   
6.         except:
   
7.             list1.remove(each)
   
8.     return sum(list1)
   
9. mix='1251017abc'
   
10. print(add(mix))

复制代码