Блог

Учитывая подобный код из node.js новичок вроде меня:

 require('http').createServer(function( req, resp ) {
    var pathname = require('url').parse( req.url ).pathname;

    if( req.method === "POST" ) {
        var postData = "";
        req.addListener( "data", function( postDataChunk ) {
            postData  = postDataChunk;
        });
        req.addListener( "end", function() {
            // do my thing with postData
            // and end the response
        });
    }
}).listen(8888);
  

… каждый раз, когда поступает POST запрос, набор анонимных функций, отправляемых слушателям, воссоздается заново.

Это кажется (почти) необходимым, поскольку node не отправляет второй аргумент накопителя "data" слушателю или аргумент конечного результата "end" слушателю. Таким образом, анонимным функциям необходимо ссылаться на postData переменную, чтобы построить результат.

Я предпочитаю, чтобы именованные функции для прослушивателей создавались вне обработчика запросов сервера, вот так:

 function data_listener( postDataChunk ) {
    this.some_unique_property.accumulator  = postDataChunk;
}
function end_listener() {
    // do my thing with postData
    // and end the response
}

require('http').createServer(function( req, resp ) {
    var pathname = require('url').parse( req.url ).pathname;

    if( req.method === "POST" ) {
        req.some_unique_property = {
            response: resp,
            pathname: pathname,
            accumulator: "",
        };
        req.addListener( "data", data_listener );
        req.addListener( "end", end_listener );
    }
}).listen(8888);
  

Поскольку this в функциях прослушивателя находится request объект, я позаимствовал request объект для накопления данных post, чтобы я мог создавать свои функции один раз и повторно использовать их.

Является ли это приемлемым / безопасным подходом?

Я предпочитаю последний подход по сравнению со многими анонимными функциями, потому что это упрощает организацию кода. Если код для обработки данных станет большим, первый может стать гигантской функцией с анонимными функциями внутри, и его гораздо сложнее поддерживать.

С помощью последнего я могу легко разбить код на отдельные модули, чтобы все было организовано. Это также позволяет мне легко документировать отдельные обработчики, если я использую что-то вроде JSDoc для документации.

Единственная рекомендация, которую я могу дать в отношении последнего, — это некоторый комментарий, указывающий, что this должно представлять, чтобы кому-то другому, читающему код, не пришлось тратить слишком много времени на обдумывание этого.

В этом, вероятно, нет ничего плохого, но звучит ли это как хорошая идея?

Анонимные функции дешевы. Любой текущий движок JS может создавать миллионы таких объектов в секунду, что на несколько порядков превышает достижимую частоту запросов.

А) Я не вижу причин не использовать подобные функции, я думаю, это делает вещи чище, в стеке вызовов уже будет другой уровень, будь то анонимный или выполненный подобным образом, поэтому сделайте код программиста читаемым.

Б) как это не то, для чего используется уровень маршрутизации? Я не вижу разницы. Конечно, я предпочитаю инвертировать мой req-метод и мой path, но они эффективно работают одинаково.

Мне это кажется преждевременной оптимизацией. Способ, которым я обычно обрабатываю такого рода вещи, выглядит следующим образом:

 require('http').createServer(function( req, res ) {
  if( req.method === "POST" ) {
    var buf = [];
    req.on('data', [].push.bind(buf)).on('end', function(){
      // do my thing with buf
    });
  }
}).listen(8888);
  

И, кстати, во втором примере вам следует исключить require('url') .

Если вы действительно беспокоитесь о производительности, вы могли бы отредактировать саму библиотеку http, чтобы при необходимости настроить унаследованные объекты запроса.

По-видимому, http-библиотека уже предоставляет вам возможность переопределить ваш класс request, и это должно выглядеть примерно так:

 function MyNewRequest( options, cb ){
    http.ClientRequest.call(self, options, cb); //Call the original constructor

    this.on('data', this.customData ); //Hook the event, and now we are the object
}
util.inherits(MyNewRequest, http.ClientRequest); //Inherit the original object

MyNewRequest.prototype.customData = function(data){

}

var oldRequest = http.request;
http.request = function( options, cb ){ //Override the original ClientRequest provider
    if( options.path == "/upload" ) //If it is our special path, return our new object 
        return new MyNewRequest( options, cb );

    return oldRequest( options, cb );
}