nodejs 中关于阻塞的疑惑

2016-07-21 16:04:49 +08:00
 Sirormy

上传使用了 formidable, 有些处理用到了 listener, 那么问题来了, 监听是异步的, 如何中断返回结果呢. 看下代码

var received = 0;
form.on('progress', function(bytesReceived, bytesExpected) {
    received += bytesReceived;
    if (received > 1024*1024) {
    	res.json({err: "图片过大"})
    	return;
    }
});

...

form.parse(req, function(err, fields, files) {
    var file = files.icon;
    res.json({url: file.path.replace('public', '')})
});

这个写法是错误的, 会报 Can't set headers after they are sent. 应该如何改进呢

3067 次点击
所在节点    Node.js
6 条回复
broadliyn
2016-07-21 16:31:55 +08:00
检查 http body 的大小不是应该先去检查 http 请求头的 content-length 大小么?
Arrowing
2016-07-21 16:34:23 +08:00
不太明白你为什么这样写
因为执行了 2 次 res.json
可以加个标识变量

```
var isSended = false;
var received = 0;
form.on('progress', function(bytesReceived, bytesExpected) {
received += bytesReceived;
if (!isSended && received > 1024*1024) {
res.json({err: "图片过大"})
return;
}
});
form.parse(req, function(err, fields, files) {
var file = files.icon;
!isSended && res.json({url: file.path.replace('public', '')})
});
```
Arrowing
2016-07-21 16:35:08 +08:00
忘记加上更改了 0.0
res.json 后面加上 isSended = true;
Sirormy
2016-07-21 16:59:38 +08:00
@broadliyn 对 Http 请求没搞明白, 先这么写,再慢慢改,这个不是主要的问题
Sirormy
2016-07-21 17:01:36 +08:00
@Arrowing 你那个写法不行,没法中断程序的进行,后边是执行写文件的操作,我那样做是先判断文件大小,要阻止后边的代码
magicdawn
2016-07-21 17:09:33 +08:00

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/293970

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX