奇怪的 Nodejs 内存泄漏答案

【问题标题】：Strange Nodejs memory leak奇怪的 Nodejs 内存泄漏
【发布时间】：2020-12-19 00:54:32
【问题描述】：

嘿，我在 heroku 中托管的 nodejs 服务器上遇到了奇怪的内存泄漏。我试图找到泄漏两天，在我一个一个删除我的功能后，我发现这个导致了问题，但我仍然不确定问题到底出在哪里。我可以清楚地看到 heroku 指标存在内存泄漏。有人知道为什么这个函数会产生内存泄漏。谢谢

completeTicTacToeGame: function (game, winnerIndex, reasonForFinish, userId) {
    if (userId) {
      User.findOne({
        _id: userId
      }, function (err, user) {

        if (err || user == null) {
        }
        else {
          userLivesController.removeUserHeartWithoutResponeAndSaving(user, 'ticTacToeHearts', function (user, canPlay) {
            if (canPlay) {
              dateFormatterController.checkIfDateIsToday(user.lastCompitedTicTacToeGame, function (isToday, isPrevious) {
                var earnCredits = 0
                if (winnerIndex == 0) { earnCredits = 1 }
                if (winnerIndex == 1) { earnCredits = 4 }

                user.credits = user.credits + earnCredits
                user.lifetimeCredits = user.lifetimeCredits + earnCredits


                //increase ad today counter
                user.lastCompitedTicTacToeGame = (new Date()).getTime().toString()

                userBadgesController.checkIfUserNeedToWinBadgeForCredits(user, function (user) {

                  user.save(function (err, user) {
                    if (err) {
                      errorHandlingController.generalSendErrorWithMessage(req, res, 'serverError')
                    }
                    else {
                      var canPlayMoreGames = user.ticTacToeHearts > 0
                      if (earnCredits > 0) {
                        notificationController.sendNotificationToUserForNewTicTacToeGame(user, earnCredits)
                        rankingController.updateRankings(user, earnCredits, 'ticTacToe')
                      }

                      var canWatchAdToDoubleCredits = false
                      if (user.gamesWithoutAd >= ConfigParams.numberOfGamesForAd() && earnCredits > 0) {
                        canWatchAdToDoubleCredits = true
                      }
                      var timeLeftToNextHeart = (((ConfigParams.minutesForHeart() * 60000)) + parseInt(user.lastGivenHeart)) - ((new Date()).getTime())

                      var haveMaxHearts = user.mathGameHearts == ConfigParams.maxHearts() && user.memoryHearts == ConfigParams.maxHearts() && user.ticTacToeHearts == ConfigParams.maxHearts()
                      var canWatchVideoForHeart = true && haveMaxHearts == false
                      if (user.lastWatchedVideoForHeart) {
                        var canWatchVideoForHeart = (((ConfigParams.minutesForHeartWatchedAd() * 60000)) + parseInt(user.lastWatchedVideoForHeart)) - ((new Date()).getTime()) < 0 && haveMaxHearts == false
                      }

                      module.exports.sendMessageToSocketForComlitedGame(game, user._id.toString(), user.credits, earnCredits, canPlayMoreGames, canWatchAdToDoubleCredits, reasonForFinish, user.ticTacToeHearts, ConfigParams.maxHearts, null, user.gamesWithoutAd >= ConfigParams.numberOfGamesForAd() - 1, timeLeftToNextHeart, canWatchVideoForHeart)
                    }
                  })
                })
              })
            }
            else {
              var timeLeftToNextHeart = (((ConfigParams.minutesForHeart() * 60000)) + parseInt(user.lastGivenHeart)) - ((new Date()).getTime())

              var haveMaxHearts = user.mathGameHearts == ConfigParams.maxHearts() && user.memoryHearts == ConfigParams.maxHearts() && user.ticTacToeHearts == ConfigParams.maxHearts()
              var canWatchVideoForHeart = true && haveMaxHearts == false
              if (user.lastWatchedVideoForHeart) {
                var canWatchVideoForHeart = (((ConfigParams.minutesForHeartWatchedAd() * 60000)) + parseInt(user.lastWatchedVideoForHeart)) - ((new Date()).getTime()) < 0 && haveMaxHearts == false
              }

              module.exports.sendMessageToSocketForComlitedGame(game, user._id.toString(), user.credits, 0, false, false, reasonForFinish, user.ticTacToeHearts, ConfigParams.maxHearts, null, user.gamesWithoutAd >= ConfigParams.numberOfGamesForAd() - 1, timeLeftToNextHeart, canWatchVideoForHeart)
            }
          })
        }
      })
    }
  },

【问题讨论】：

你有什么证据让你认为你有内存泄漏？
我附上了一张来自 heroku 指标平台的图片。你可以看到服务器使用的内存是如何逐分钟增加的
垃圾收集循环发生。这种内存使用对我来说看起来很正常。
可以看到内存增加超过6小时这是不正常的。之后我们达到最大内存并重新启动服务器，这就是内存下降的原因。
你提供的这个函数有很多外部调用者，可能其中一个或几个是泄漏的结果。正如@evgenifotia 所说，替换每个调用以查看是否泄漏内存。其他解决方案是封装并尽可能多地转换为功能代码，从而尽可能减少意外泄漏。

标签： javascript node.js express memory-leaks

【解决方案1】：

就像其他人在这里评论一样，通过查看代码很难找到明显的原因。

您应该查看的一件事可能是在您的应用中调用此函数的频率。您致电User.findOne。是否会因连接池大小限制而以某种方式受到限制，即排队或阻塞请求。

如果函数调用多于可用资源/连接您将开始看到内存使用量逐渐增加。

不是说这是原因，但绝对是我要开始寻找的地方。

【讨论】：

【解决方案2】：

这里看起来可能有问题的是“游戏”对象和创建的用户对象。因此，取决于他们去哪里（两者都被传递到sendMessageToSocketForComlitedGame），他们可能在排队或以其他方式保留。我在这里担心的是闭包泄漏，可能是一些数组或队列使它们保持活动状态，或者可能是一个计时器循环。您是否尝试过将“sendMessageToSocketForComlitedGame”函数设为 noop 以查看内存问题是否消失？

另一个想法 - 当用户会话或游戏正在进行时，这可能是内存状态数据吗？是否有其他一些长期存在的数据结构在事件时被删除？因为在这里它看起来不像是永久泄漏，而是您将数据保留了一段时间。

由于您已经为此苦恼了几天，我相信您已经阅读了所有类型的关于 javascript 泄漏的文章，但 I found this one to be a nice one。我会查看这个函数的上方和下方，看看是否有任何东西保持对实例化闭包或对象的引用。

祝你好运！ -达林

【讨论】：

【解决方案3】：

我不是在优化，也不是指出导致内存泄漏的代码的任何循环漏洞，但这里有一些技巧可以帮助您避免此类泄漏

全局变量： 由于 JavaScript 中的全局变量由根节点（window 或全局 this）引用，因此它们在应用程序的整个生命周期中都不会被垃圾回收，并且只要应用程序运行就会占用内存。这适用于全局变量及其所有子变量引用的任何对象。从根目录引用大量对象可能会导致内存泄漏。

多次引用： 当多个对象引用同一个对象时，如果其中一个引用悬空，可能会导致内存泄漏。

关闭： JavaScript 闭包有一个很酷的特性，就是记住它周围的上下文。当闭包持有对堆中大对象的引用时，只要闭包在使用中，它就会将对象保留在内存中。这意味着您很容易陷入保存此类引用的闭包可能被不当使用而导致内存泄漏的情况。

计时器和事件： 使用 setTimeout、setInterval、Observers 和事件侦听器时，如果在回调中保留大量对象引用而没有适当处理，则会导致内存泄漏。

【讨论】：