當了一年的 Facebook Rotational Software Engineer 心得分享

一亩三分地论坛

 找回密码
 Sign Up 注册获取更多干货
E轮2.5亿美元融资
K12教育独角兽一起作业
北京-诚聘人工智能/教育/大数据岗
坐标湾区
Games Startup
招聘游戏开发工程师
游戏初创公司招聘工程师、UIUX Designer和游戏策划
码农求职神器Triplebyte:
不用海投
内推多家公司面试
把贵司招聘信息放这里
查看: 1088|回复: 6
收起左侧

[算法题] 做Combination Sum II 遇到的一个python问题

[复制链接] |试试Instant~ |关注本帖
woshidxfdxf 发表于 2014-10-22 11:23:36 | 显示全部楼层 |阅读模式

注册一亩三分地论坛,查看更多干货!

您需要 登录 才可以下载或查看,没有帐号?Sign Up 注册获取更多干货

x
以前用c++写的,现在用python重写的时候遇到很tricky的问题,我已经绕晕了,求python大神解释一下:
刚开始写的,在dfs()里面打印结果都是对的,但是传出来总是空的,比如题目给的例子跑出来就是[ [ ][ ][ ][ ] ]
  • class Solution:
  •     # @param candidates, a list of integers
  •     # @param target, integer
  •     # @return a list of lists of integers
  •     def combinationSum2(self, candidates, target):
  •         Solution.res = []
  •         candidates.sort()
  •         self.dfs(candidates,0,0,target,[])
  •         return Solution.res
  •     def dfs(self,candidates,index,s,target,solu):
  •         if s>target: return
  •         if s==target:
  •             return Solution.res.append(solu)
  •         while index < len(candidates):
  •             solu.append(candidates[index])
  •             self.dfs(candidates,index+1,s+candidates[index],target,solu)
  •             solu.pop()
  •             while index<len(candidates)-1 and candidates[index]==candidates[index+1]:
  •                 index+=1
  •             index+=1
  •         return





后来改成这样就可以了:
  •             solu.append(candidates[index])
  •             self.dfs(candidates,index+1,s+candidates[index],target,solu)
  •             solu.pop()

改成
  •             self.dfs(candidates,index+1,s+candidates[index],target,solu+[candidates[index]])



自己想了很久,越想越晕乎了,求python大神解释一下到底是什么原理?? 谢了啊
 楼主| woshidxfdxf 发表于 2014-10-23 05:53:37 | 显示全部楼层
求大神解惑啊
回复 支持 反对

使用道具 举报

全球28万学生4.7分推荐
头像被屏蔽
kinslayer 发表于 2014-10-27 14:27:53 | 显示全部楼层
提示: 作者被禁止或删除 内容自动屏蔽
回复 支持 反对

使用道具 举报

manifold 发表于 2014-10-28 13:32:21 | 显示全部楼层
楼上的解释正确啊 list是mutable object, 作为参数传递给函数时, 传递的reference。 所以, pop的时候, 把已经append到res里的solu也pop了。

把line 16改为 return Solution.res.append(list(solu)) 即可。 为solu创建一个新的copy, 把copy push到res中去。
回复 支持 反对

使用道具 举报

 楼主| woshidxfdxf 发表于 2014-10-29 04:56:42 | 显示全部楼层
kinslayer 发表于 2014-10-27 14:27
因为你Solution.res.append(solu),此时Solution.res已经和solu相关了(pass by reference);你之后再solu.p ...

我又翻书看了看 确实是这样
之前一直没弄明白list1.append(list2)是shallow copy的
非常感谢!
回复 支持 反对

使用道具 举报

 楼主| woshidxfdxf 发表于 2014-10-29 04:58:07 | 显示全部楼层
manifold 发表于 2014-10-28 13:32
楼上的解释正确啊 list是mutable object, 作为参数传递给函数时, 传递的reference。 所以, pop的时候,  ...

非常感谢!
return Solution.res.append(list(solu)) 就是append的时候新建list复制solu,理解了
回复 支持 反对

使用道具 举报

Frankhappens 发表于 2014-10-29 09:29:09 | 显示全部楼层
传参和引用的问题么?
回复 支持 反对

使用道具 举报

本版积分规则

提醒:发帖可以选择内容隐藏,部分板块支持匿名发帖。请认真读完以下全部说明:

■隐藏内容方法: [hide=200]你想要隐藏的内容比如面经[/hide]
■意思是:用户积分低于200则看不到被隐藏的内容
■可以自行设置积分值,不建议太高(200以上太多人看不到),也不建议太低(那就没必要隐藏了)
■建议只隐藏关键内容,比如具体的面试题目、涉及隐私的信息,大部分内容没必要隐藏。
■微信/QQ/电子邮件等,为防止将来被骚扰甚至人肉,以论坛私信方式发给对方最安全。
■匿名发帖的板块和方法:http://www.1point3acres.com/bbs/thread-405991-1-1.html

关闭

一亩三分地推荐上一条 /5 下一条

手机版|小黑屋|一亩三分地论坛声明

custom counter

GMT+8, 2018-5-20 20:40

Powered by Discuz! X3

© 2001-2013 Comsenz Inc. Design By HUXTeam

快速回复 返回顶部 返回列表