一亩三分地论坛

 找回密码
 获取更多干货,去instant注册!

扫码关注一亩三分地公众号
查看: 822|回复: 2
收起左侧

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

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

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

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

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的地址,所以这种情况正确。
回复 支持 反对

使用道具 举报

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

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

使用道具 举报

本版积分规则

请点这里访问我们的新网站:一亩三分地Instant.

Instant搜索更强大,不扣积分,内容组织的更好更整洁!目前仍在beta版本,努力完善中!反馈请点这里

关闭

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

手机版|小黑屋|一亩三分地论坛声明 ( 沪ICP备11015994号 )

custom counter

GMT+8, 2016-12-7 02:38

Powered by Discuz! X3

© 2001-2013 Comsenz Inc. Design By HUXTeam

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