3

全部。

在尝试解决一些编程测验: https ://www.hackerrank.com/challenges/missing-numbers 时,我遇到了空间泄漏。

主要功能是difference,实现多集差分。我发现 List ':' 和 Triples (,,) 使用 -hT 选项进行分析。但是,只有大列表是的两个参数,并且随着尾递归difference的不断进行而缩小。difference但是列表消耗的内存随着程序的运行而不断增加。

Triples 是临时数组结构,用于记录多重集每个元素的计数。但是三元组消耗的内存也在不断增加,我不知道为什么。

虽然我在 stackoverflow 中浏览过类似的“空间泄漏”问题,但我无法理解这个想法。我当然有很多东西要学。

我感谢任何评论。谢谢你。

ps) 可执行文件使用 -O2 开关编译。

$ ./difference -hT < input04.txt
Stack space overflow: current size 8388608 bytes.
$ ghc --version
The Glorious Glasgow Haskell Compilation System, version 7.6.3

.

import Data.List
import Data.Array

-- array (non-zero-count, start-offset, array_data)
array_size=101

myindex :: Int -> Int -> Int
myindex key offset
    | key >= offset = key - offset
    | otherwise     = key - offset + array_size

mylookup x (_,offset,arr) = arr ! idx
    where idx = myindex x offset

addOrReplace :: Int -> Int -> (Int, Int, Array Int (Int,Int)) -> (Int, Int, Array Int (Int,Int))
addOrReplace key value (count,offset,arr) = (count', offset, arr // [(idx,(key,value))])
    where idx = myindex key offset
          (_,prev_value) = arr ! idx
          count' = case (prev_value, value) of
                      (0,0) -> count
                      (0,_) -> count + 1
                      (_,0) -> count - 1
                      otherwise -> count

difference :: (Int,Int,Array Int (Int,Int)) -> [Int] -> [Int] -> [Int]
difference (count,offset,arr) [] []
    | count == 0 = []
    | otherwise  = [ k | x <- [0..array_size-1], let (k,v) = (arr ! x), v /= 0]
difference m (x:xs) y = difference new_m xs y
    where (_,v) = mylookup x m
          new_m = addOrReplace x (v + 1) m
difference m [] (y:ys) = difference new_m [] ys
    where (_,v) = mylookup y m
          new_m = if v == 0
            then m
            else addOrReplace y (v - 1) m

main = do
    n <- readLn :: IO Int
    pp <- getLine
    m <- readLn :: IO Int
    qq <- getLine
    let p = map (read :: String->Int) . words $ pp
        q = map (read :: String->Int) . words $ qq
        startArray = (0,head q, array (0,100) [(i,(0,0)) | i <- [0..100]] )
    putStrLn . unwords . map show . sort $ difference startArray q p

[编辑] 感谢 Carl 的建议,我对 value 和 Array 进行了排序。我附上堆图。

[原始堆分析] [ 原始堆] 1

[在seq'ing值之后v]

difference m (x:xs) y = difference new_m xs y
    where (_,v) = mylookup x m
          new_m = v `seq` addOrReplace x (v + 1) m

seq'ing v 后堆

[在seq'ing值v和之后Array]

difference m (x:xs) y = new_m `seq` difference new_m xs y
    where (_,v) = mylookup x m
          new_m = v `seq` addOrReplace x (v + 1) m

seq'ing v 和 Array 后的堆

4

1 回答 1

1

我看到这段代码存在三个主要问题。

首先(不是内存使用的原因,但绝对是性能普遍不佳的原因)对于这个用例Array来说是可怕的。当更新为 O(n) 时,O(1) 查找是无用的。

说到,存储在其中的值在循环其第一个输入Array时不是强制的。difference它们是包含指向先前版本数组中未评估查找的指针的 thunk。您可以通过多种方式确保在更新数组的同时评估值。当difference循环其第二个输入时,它实际上是通过将值与 0 进行比较来意外做到这一点的。

第三,difference在遍历它的第一个参数时甚至不强制评估正在创建的新数组。没有什么需要在循环的那部分评估旧数组。

后两个问题都需要解决以修复空间泄漏。第一个问题不会导致空间泄漏,只是比需要的开销要高得多。

于 2016-01-21T15:43:04.303 回答