Question

我编写了一个WORKING解决方案以在DIRECTED图中查找循环，但是我想知道从递归堆栈中删除键后包含return False语句的原因。


class vertex:

  def __init__(self, key):

    self.neighbours = {}
    self.key = key

  def add(self, key, edge):
    self.neighbours[key] = edge

class graph:

  def __init__(self):

    self.root = {}
    self.nbnodes = 0

  def addnode(self, key1, key2, edge=0):
    if key1 not in self.root:
      self.root[key1] = vertex(key1)
      self.nbnodes += 1
    if key2 not in self.root:
      self.root[key2] = vertex(key2)
      self.nbnodes += 1

    self.root[key1].add(key2, edge)

  def dfs(self):

    visited = set()
    recstack = []

    for key in self.root:
      if key not in visited:
        if self._dfs(key, visited, recstack):
          print('cycle')
        else:
          print('No cycle')


  def _dfs(self, key, visited, recstack):

    visited.add(key)
    recstack.append(key)
    for neighbour in self.root[key].neighbours:
      if neighbour not in visited:
        if self._dfs(neighbour, visited, recstack):
          return True
      elif neighbour in recstack:
        return True
    recstack.remove(key)  
    #return False



if __name__ == '__main__':
  a = graph()
  a.addnode(0,1)
  a.addnode(0,2)
  a.addnode(1,3)
  a.addnode(1,4)
  a.addnode(2,4)
  a.addnode(4,5)
  a.dfs()

上面是我的最终解决方案代码，我注释了return False语句，我想知道为什么要包含它。

谢谢

Answer 1

如果您询问原因，那么显而易见的答案就是可读性。

没有显式return False，您将隐式返回None，因此，片段if self._dfs(neighbour, visited, recstack):的工作方式与注释的return False相同。仍然我不清楚这个意图，并且会假设您忘记返回它了（或者想指出读者在这种情况下没有功能状态）。

引用documentation：

没有人经常使用表示缺少值，例如默认参数不存在时传递给函数。分配给“无”是非法的，并会导致语法错误。

您的值不存在，它是为您的函数完美定义的。除了遵循良好做法之外，它不会更改您的代码中的任何内容。

顺便说一句。类的名称通常使用大写字母和CamelCase命名，而函数使用snake_case。

在有向图中查找循环，返回语句

1 个答案: