BFS 实现中的 Haskell 空间泄漏

Question

连续几天我一直在努力解决 Haskell 空间泄漏(自然是堆栈溢出类型)的问题。这令人沮丧，因为我试图直接从 CLR 模仿 BFS 算法，这不是自然递归的。注意:我已经启用了 BangPatterns 并且我在每个可以去的地方前面放了一个 bang，试图解决这个问题，但没有任何效果。我之前曾与空间泄漏作斗争，我不愿意放弃并在这个问题上寻求帮助，但在这一点上我被卡住了。我喜欢在 Haskell 中编码，并且我非常了解函数式编程的 Zen，但是调试空间泄漏就像在满是图钉的地板上滚来滚去一样有趣。

也就是说，我的麻烦似乎是典型的“蓄能器”类型的空间泄漏。堆栈显然是围绕以下代码中对 bfs' 的调用构建的。非常感谢任何空间泄漏的 Protips。

import qualified Data.Map as M
import qualified Data.IntSet as IS
import qualified Data.Sequence as S
import qualified Data.List as DL

data BfsColor = White | Gray | Black deriving Show
data Node =
Node {
  neighbors :: !IS.IntSet,
  color     :: !BfsColor,
  depth     :: !Int
   }

type NodeID = Int
type NodeQueue = S.Seq NodeID
type Graph = M.Map NodeID Node

bfs :: Graph -> NodeID -> Graph
bfs graph start_node =
  bfs' (S.singleton start_node) graph

bfs' :: NodeQueue -> Graph -> Graph
bfs' !queue !graph
  | S.null queue = graph
  | otherwise =
  let (u,q1) = pop_left queue
      Node children _ n = graph M.! u
      (g2,q2) = IS.fold (enqueue_child_at_depth $ n+1) (graph,q1) children
      g3 = set_color u Black g2
  in bfs' q2 g3

enqueue_child_at_depth :: Int -> NodeID -> (Graph, NodeQueue)
                                        -> (Graph, NodeQueue)
enqueue_child_at_depth depth child (graph,!queue)  =
  case get_color child graph of
    White     -> (set_color child Gray $ set_depth child depth graph,
                   queue S.|> child)
    otherwise -> (graph,queue)

pop_left :: NodeQueue -> (NodeID, NodeQueue)
pop_left queue =
  let (a,b) = S.splitAt 1 queue
  in (a `S.index` 0, b)

set_color :: NodeID -> BfsColor -> Graph -> Graph
set_color node_id c graph =
  M.adjust (\node -> node{color=c}) node_id graph

get_color :: NodeID -> Graph -> BfsColor
get_color node_id graph = color $ graph M.! node_id

set_depth :: NodeID -> Int -> Graph -> Graph
set_depth node_id d graph =
  M.adjust (\node -> node{depth=d}) node_id graph

Answer 1

这看起来更容易理解。 (不过，您仍然可以将代码缩小 1/2。)

现在，空间泄漏的性质变得显而易见。也就是说，永远不会评估的一件事是深度。会堆积成大表情1+1+... .您可以删除所有的爆炸模式并在以下位置添加一个

enqueue_child_at_depth !depth child (graph,queue)

摆脱空间泄漏。

(进一步的代码提示:您可以将 IS.IntSet 替换为一个简单的列表。队列最好按照以下方式进行解构和重构

go depth qs graph = case viewl qs of
    EmptyL  -> graph
    q :< qs ->
        let
            qs' = (qs ><) . Seq.fromList
                . filter (\q -> isWhite q graph)
                . neighbors q $ graph
        in ...

)