那些年,在nodejs上踩过的坑(一)
发布于 3年前 作者 jifeng 5956 次浏览

自己写nodejs也有一段时间,踩过很多坑(而且大部分是自己给自己埋),也见过很多别人踩过的坑,原因其实也很简单,要么是对这个知识点理解不够深入,要么就是编码的习惯不好。这段响应朋春大牛的号召,打算陆陆续续整理下这些坑,算是给自己一个备忘,同时也希望能对大家有所帮助。

1. callback

事件回调是nodejs非常常见的一个应用场景,那大家先来看看以下这段代码是否存在什么问题?

get(params, function(err, data) {
  if (err) {
   callback(err);
 }
 //对data进行操作
 var row = data[0];
});

看出来了吧。对,就是err存在时,callback之后,接下来的代码还是要执行的。而这时,data值是什么,我们往往是没办法控制的。如果data这时返回的是undefined,那么就悲剧了,程序肯定报错。当然解决方法很简单,就是在callback之前加个return即可:

get(params, function(err, data) {
  if (err) {
    return callback(err);
  }
 //对data进行操作
 var row = data[0];
});

这个知识点并不是很难,但往往是初学者特别容易犯的错,甚至已经写了很久代码的同学也会偶尔犯这种低级错误。

再来一个更隐蔽的:

db.get(key, function(err, data) {
  if (err) {
    return callback(err);
  }
  try {
    callback(null, JSON.parse(data.toString()))
  } catch(e) {
    callback(e);
  }
});

看似没有任何问题吧。嘿嘿,揭晓答案, 对,被回调两次。callback(null, Error)一次,callback(e),具体比如:

function asyncfun(data, callback) {
  try {
    callback(null, JSON.parse(data.toString()));
  } catch (e) {
    callback(e);
  }
}
var json = {'a': 'b'};
var jsonstr = JSON.stringify(json);
var d = new Buffer(jsonstr);

asyncfun(d, function(err, data) {
  console.log(err);
  throw new Error('new Error');
});

运行结果:

null
[Error: new Error]

这在一个大项目绝对是坑爹了,排错都需要很久。

ps:之前的描述存在问题,谢谢苏千的指正。

2. buffer

还是老规矩,先看代码:

var data = "";  
res.on('data', function (chunk) {  
  data += chunk;  
})  
.on("end", function () {  
});

这段代码在chunk都是ascii码数据或者数据量比较少时是没有问题,但如果你的数据是大量中文的话,恭喜你,中枪了,会出现乱码。其原因是两个chunk(Buffer对象)的拼接并不正常,相当于进行了buffer.toString() + buffer.toString()。如果buffer不是完整的,则toString出来后的string是存在问题的(比如一个中文字被截断)。具体可以参见朴灵写得这篇文章:http://cnodejs.org/topic/4faf65852e8fb5bc65113403

3. 深度嵌套

很多刚开始写nodejs代码的人,由于思路还停留在同步的思维,所以或多或少写过这样的代码:

func1(err, function(err1, data1) {
  func2(err1, function(err2, data2) {
    func3(err3, function(err3, data3) {
      func4(err4, function(err4, data4) {
        .......
      })
    })
  })
})

先别说这样的代码是否易于维护,光样子就够难看,代码都“斜”了。不怕大家耻笑,我自己刚开始就写时也写过这样“坑爹”的代码,为此自己当时还写了篇《如何让nodejs同步操作》讲如何解决这个问题。不过从源头上,大家还是要尽量避免采用这种同步的方法,因为nodejs得优势就在于异步,硬生生要做成同步绝对吃力不讨好

14 回复

还好,一路踩过来了

  try {
    callback(null, JSON.parse(data.toString()))
  } catch(e) {
    callback(e);
  }

这个应该是callback 本身抛异常了,导致callback被执行两次。。。

最后一个“深度嵌套”的说法不对,你完全没法让JS同步执行的啊,你那篇文章里的“同步执行”只是用一些流程控制的办法或是类库而已。

还有,用Jscex啊Jscex。

踩过。。。

try … catch 那个坑哥也干过…

是也乎,是也乎,形式同步,本质异步,表让程序员的头脑生转异步,就是正道,

嗯,之前的说法的确是有问题,我想说的是以下这种情况:

function asyncfun(data, callback) {
  try {
    callback(null, JSON.parse(data.toString()));
  } catch (e) {
    callback(e);
  }
}
var json = {'a': 'b'};
var jsonstr = JSON.stringify(json);
var d = new Buffer(jsonstr);

asyncfun(d, function(err, data) {
  console.log(err);
  throw new Error('new Error');
})

执行结果:

null
[Error: new Error]

@ZoomQuiet 哥们这话说得好,也是我现在想表达的意思,也谢谢老赵的指正。

哈哈哈哈

好文一枚,顶

好 学习了 那个.on()是什么意思?拼接吗?

回到顶部