I have an application that I'm writing in Node.js which needs to make a lot of configuration and database calls in order to process user data. The issue I'm having is that after 11,800+ function calls Node will throw an error and exit the process.

我有一個應用程序,我正在Node.js編寫,需要進行大量的配置和數據庫調用,以處理用戶數據。我遇到的問題是,在11,800多個函數調用之后,Node會拋出錯誤並退出進程。

The error says: RangeError: Maximum call stack size exceeded

錯誤說明:RangeError:超出最大調用堆棧大小

I'm curious if anyone else has had this situation arise and to know how they handled this. I've already started to break up my code into a couple of extra worker files but even so each time I process a data node it needs to touch 2 databases (at most 25 calls to update various tables) and do a number of sanitization checks.

我很好奇是否有其他人已經出現這種情況,並知道他們是如何處理這個問題的。我已經開始將我的代碼分解為幾個額外的工作文件,但即便如此,每次處理數據節點時,它都需要觸摸2個數據庫(最多25次調用以更新各種表)並進行一些清理檢查。

I am totally willing to admit that I'm possibly doing something non-optimal if that is the case but would appreciate some guidance if there is a more optimal manner.

我完全願意承認,如果是這種情況,我可能會做一些非最優的事情,但如果有更優化的方式,我會很感激。

Here is an example of the code I'm running on data:

以下是我在數據上運行的代碼示例:

app.post('/initspeaker', function(req, res) {
    // if the Admin ID is not present ignore
    if(req.body.xyzid!=config.adminid) {
        res.send( {} );
        return;
    }

    var gcnt = 0, dbsize = 0, goutput = [], goutputdata = [], xyzuserdataCallers = [];

    xyz.loadbatchfile( xyz.getbatchurl("speakers", "csv"), function(data) {
        var parsed = csv.parse(data);
        console.log("lexicon", parsed[0]);

        for(var i=1;i<parsed.length;i++) {
            if(typeof parsed[i][0] != 'undefined' && parsed[i][0]!='name') {
                var xyzevent = require('./lib/model/xyz_speaker').create(parsed[i], parsed[0]);
                xyzevent.isPresenter = true;
                goutput.push(xyzevent);
            }
        }
        dbsize = goutput.length;

        xyzuserdataCallers = [new xyzuserdata(),
                                    new xyzuserdata(),
                                    new xyzuserdata(),
                                    new xyzuserdata(),
                                    new xyzuserdata(),
                                    new xyzuserdata(),
                                    new xyzuserdata(),
                                    new xyzuserdata()
                                ];
        // insert all Scheduled Items into the DB                   
        xyzuserdataCallers[0].sendSpeakerData(goutput[0]);
        for(var i=1;i<xyzuserdataCallers;i++) {
            xyzuserdataCallers[i].sendSpeakerData(8008);
        }

        //sendSpeakerData(goutput[0]);
    });

    var callback = function(data, func) {
        //console.log(data);
        if(data && data!=8008) {
            if(gcnt>=dbsize) {
                res.send("done");
            } else {
                gcnt++;
                func.sendSpeakerData(goutput[gcnt]);
            }
        } else {
            gcnt++;
            func.sendSpeakerData(goutput[gcnt]);
        }
    };

    // callback loop for fetching registrants for events from SMW
    var xyzuserdata = function() {};
    xyzuserdata.prototype.sendSpeakerData = function(data) {
        var thisfunc = this;

        if(data && data!=8008) {
            //console.log('creating user from data', gcnt, dbsize);
            var userdata = require('./lib/model/user').create(data.toObject());
            var speakerdata = userdata.toObject();
            speakerdata.uid = uuid.v1();
            speakerdata.isPresenter = true;

            couchdb.insert(speakerdata, config.couch.db.user, function($data) {
                if($data==false) {
                    // if this fails it is probably due to a UID colliding
                    console.log("*** trying user data again ***");
                    speakerdata.uid = uuid.v1();
                    arguments.callee( speakerdata );
                } else {
                    callback($data, thisfunc);
                }
            });
        } else {
            gcnt++;
            arguments.callee(goutput[gcnt]);
        }
    };

});

A couple of classes and items are defined here that need some introduction:

這里定義了幾個類和項目需要一些介紹:

• I am using Express.js + hosted CouchDB and this is responding to a POST request

我正在使用Express.js +托管的CouchDB,這是響應POST請求

• There is a CSV parser class that loads a list of events which drives pulling speaker data

有一個CSV解析器類可以加載一個驅動揚聲器數據的事件列表

• Each event can have n number of users (currently around 8K users for all events)

每個活動可以有n個用戶(目前所有活動的用戶約為8K)

• I'm using a pattern that loads all of the data/users before attempting to parse any of them

我正在使用一種模式,在嘗試解析任何數據/用戶之前加載所有數據/用戶

• Each user loaded (external data source) is converted into an object I can use and also sanitized (strip slashes and such)

每個加載的用戶(外部數據源)都會轉換為我可以使用的對象,也會被清理(條帶斜線等)

• Each user is then inserted into CouchDB

然后將每個用戶插入CouchDB

This code works in the app but after a while I get an error saying that over 11,800+ calls have been made and the app breaks. This isn't an error that contains a stack trace like one would see if it was code error, it is exiting due to the number of calls being done.

此代碼在應用程序中有效,但過了一段時間后,我收到一條錯誤消息,說已經有超過11,800多個來電並且應用程序中斷了。這不是包含堆棧跟蹤的錯誤,如果它是代碼錯誤就會看到它,由於調用次數正在退出。

Again, any assistance/commentary/direction would be appreciated.

再次,任何協助/評論/指示將不勝感激。

2 个解决方案

flow.serialForEach(goutput, xyzuserdata.sendSpeakerData, ...)

setTimeout(callback, 0, [$data, thisfunc])

 ee = eventemitter
 arr = A_very_long_array_to_process
 callback = callback_to_call_once_either_with_an_error_or_when_done

 // the worker function does everything
 processOne() {
   var 
      next = arr. shift();
   if( !arr )
      ee.emit ( 'finished' )
      return

   process( function( err, response) {
      if( err )
         callback( err, response )
      else
         ee.emit( 'done-one' )
    } );
 }    

 // here we process the final event that the worker will throw when done
 ee.on( 'finished', function() { callback( null, 'we processed the entire array!'); } );

 // here we say what to do after one thing has been processed
 ee.on( 'done-one', function() { processOne(); } );

 // here we get the ball rolling
 processOne();