San Jose各个房价 <1.5m 区域买房总结

一亩三分地论坛

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

[Leetcode] Leetcode: Clone Graph , bug 求答疑!!!

[复制链接] |试试Instant~ |关注本帖
pan_y 发表于 2015-3-24 05:23:53 | 显示全部楼层 |阅读模式

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

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

x
下面这个是有Bug的,红色标出:
/**
* Definition for undirected graph.
* struct UndirectedGraphNode {
*     int label;
*     vector<UndirectedGraphNode *> neighbors;
*     UndirectedGraphNode(int x) : label(x) {};
* };
*/
class Solution {
private:
    unordered_map<int, UndirectedGraphNode*> mNodeMap;
public:
    UndirectedGraphNode *cloneGraph(UndirectedGraphNode *node) {
      if(!node) return node;

      unordered_map<int,UndirectedGraphNode*>::iterator it = mNodeMap.find(node->label);
      if(it==mNodeMap.end()){
          UndirectedGraphNode * newNode = new UndirectedGraphNode(node->label);
          mNodeMap[node->label] = newNode;
          for (int i = 0;i<node->neighbors.size();i++)
            newNode->neighbors.push_back(cloneGraph(node->neighbors));
          return newNode;
      }else{
          return node;
      }
    }
};

test case 和出错提示如下:
Input:
{0,0,0}
Output:
Node with label 0 was not copied but a reference to the original one.
Expected:
{0,0,0}



稍微一改成这样就好了,不知道怎么回事,

/**
* Definition for undirected graph.
* struct UndirectedGraphNode {
*     int label;
*     vector<UndirectedGraphNode *> neighbors;
*     UndirectedGraphNode(int x) : label(x) {};
* };
*/
class Solution {
private:
    unordered_map<int, UndirectedGraphNode*> mNodeMap;
public:
    UndirectedGraphNode *cloneGraph(UndirectedGraphNode *node) {
      if(!node) return node;

      unordered_map<int,UndirectedGraphNode*>::iterator it = mNodeMap.find(node->label);
      if(it==mNodeMap.end()){
          UndirectedGraphNode * newNode = new UndirectedGraphNode(node->label);
          mNodeMap[node->label] = newNode;
          for (int i = 0;i<node->neighbors.size();i++)
            newNode->neighbors.push_back(cloneGraph(node->neighbors));
          return newNode;
      }else{
          return it->second;
      }
    }
};
求答疑!!!!!

stellari 发表于 2015-3-24 11:18:41 | 显示全部楼层
其实错误信息已经基本上解释了原因。cloneGraph中的递归调用cloneGraph(node->neighbors)里传入的是“原node的neighbor地址”,然后你在这次递归调用中如果if匹配不成功,会执行return node,换句话说,这次递归调用返回的就是传入的“原node的neighbor地址”,而题中要求的是“副本node的neighbor地址”。自然不正确。

改过以后的it->second是从mNodeMap中找出来的,而mNodeMap中只存放副本node的地址,所以这种情况正确。
回复 支持 反对

使用道具 举报

全球28万学生4.7分推荐
 楼主| pan_y 发表于 2015-3-26 04:06:35 | 显示全部楼层
stellari 发表于 2015-3-24 11:18
其实错误信息已经基本上解释了原因。cloneGraph中的递归调用cloneGraph(node->neighbors)里传入的是“原nod ...

3Q!!!!!!!!!!!
回复 支持 反对

使用道具 举报

本版积分规则

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

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

关闭

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

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

custom counter

GMT+8, 2018-5-26 10:19

Powered by Discuz! X3

© 2001-2013 Comsenz Inc. Design By HUXTeam

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