Question

我正在尝试在链接列表中找到一个圆圈，并在圆圈的开头返回该节点。例如，如果列表是A - >; B - ＆gt; C - ＆gt; D - ＆gt; C - 我们将返回C

这是我的代码：

ListNode<T> * findCircle()
{
  map<ListNode<T>*, bool> addresses;

  ListNode<T>* node = head;

  if(addresses.find(node)->second)
    cout << " wtf " << endl;

  while(node)
  { 
    if((addresses.find(node))->second)
    {
      return node;
    } 
    else
      addresses.insert(pair<ListNode<T>*,bool>(node, 1));

    node = node->next;
  }

  return NULL;

}

我有几个问题：

1）这是解决问题的正确方法

2）我是否使用最有效的方法在表格中查找/插入键和值

3）为什么不工作？当我在地图中检查头部时，在我插入头部之前，它仍然执行if语句并打印“wtf”。我的算法是将节点作为具有真值的键插入，如果在地图中找不到，否则如果该键已经在地图中则返回该节点。

我尝试用std :: set做这个但是它给了我麻烦，所以我切换到地图。令我感到困惑的是，以下代码使用完全相同的方法（使用查找表删除链表中的重复项的代码）。

  void removeDuplicates()
    {
      map<T, bool> listData;

      ListNode<T> *node;
      ListNode<T> *prev = node;
      for(node = head; node; node = node->next)
      {
        if(listData.find(node->data)->second)
        {
          prev->next = node->next;
          delete node;
        }
        else
        {
          listData.insert( pair<T, bool>(node->data, 1));
        }
        prev = node;
      }
    }

第二块代码是否符合预期，但第一块代码是不是只是一个侥幸？

Answer 1

您需要检查查找操作是否实际找到数据

auto iterator itr = address.find(node);
if(itr != address.end()){
    if(itr->second == true){ cout << "WTF" << endl; }
}

类似于while循环。至于你的方法，我认为它具有O（n）的最佳运行时。你所能做的就是降低常数。

Answer 2

addresses.find(node)->second

当find()失败时，

当前会产生未定义的行为，因为您正在尝试访问过去的迭代器的second字段。相反，使用

addresses.find(node) != addresses.end()

另外：找到一个循环的确切位置（通常称之为循环）很重要，或者你只想查看一个循环是否存在？如果是后者，那么一个非常聪明的算法可以使用两个指针解决线性时间和恒定空间中的问题，其中一个指针移动速度是另一个指针的两倍：）

使用查找表在链接列表中查找圆圈

2 个答案: