在有向图中查找循环，返回状态

Question

我编写了一个有效的解决方案，以在有向图中找到一个循环，但是我想知道在从递归堆栈中删除一个键后包含returnfalse语句的原因

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()

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

多谢各位

Answer 1

如果你问原因，显而易见的答案是可读性

如果没有显式的return False，您将隐式地返回None，这与片段if self._dfs(neighbour, visited, recstack):的工作原理相同。但我仍然不清楚其意图，并认为您忘记了返回它（或者想告诉读者案例中没有函数状态）

引用documentation：

None is frequently used to represent the absence of a value, as when default arguments are not passed to a function. Assignments to None are illegal and raise a SyntaxError.

您的值并不缺少，它是为您的函数完美定义的。除了遵循良好的实践之外，它不会改变代码中的任何内容

顺便说一句，类的名称通常是大写和CamelCase，而函数使用snake_大小写