这个递归能写得更高效吗？

Question

我有一个层次结构，它是一棵树，如果父赞助子节点，则节点就是子节点。看来我可以用这段代码遍历树

def get_team(self, person, team):
    firstline = User.query(User.sponsor == person.key).fetch(99999999)
    if firstline:
        for person in firstline:
            team.append(person)
            newdownline = self.downline(person, team)        
    return team

使用上面的方法，我可以得到一个用户的组织

downline=user.get_team(user, [])

但是有没有更有效的方法，因为我必须为单个请求多次执行此操作，并且大量递归可能效率低下？或者代码会很好，因为它可以正确地遍历树？在我的第一个版本中，我使用了三个变量，我发现我可以将代码重新排列为只有两个变量，而不是这样：

def downline(self, person, team, teamlist):
    firstline = User.query(User.sponsor == person.key).fetch(99999999)
    if firstline:
        for person in firstline:
            teamlist.append(person)
            newdownline = self.downline(person, team, teamlist)        
            team.append(newdownline)
    return teamlist 

我发现 teamlist 变量并不是真正需要的，所以我删除了它。我这样做的方式是首先使用一个变量太多：

people = user.downline(user, [], [])

Answer 1

是的，有一种更有效的方法可以做到这一点，具体取决于您的计算权衡。

您目前正在对树进行深度优先遍历，这是一种非常好的方法。您可以通过缓存结果来提高速度，但会消耗一些 RAM，因此：

if person.id in downline_cache:
      team.append(downline_cache[person.id])
else:
      downline_cache[person.id] = results
      team.append(results)

如果树相当小，您可以预先缓存整个内容，每个线程一次。这比您正在做的事情需要更多的 RAM，但比每次您关心结果时进行深度优先遍历要快得多。很大程度上取决于您的使用模式和您存储的数据量。

如果您使用缓存，则必须确保有某种方法来处理基础数据的更改，或者使用超时和“最终正确”的类型保证，或者跟踪必须何时以及如何擦除缓存，或者缓存的元素。