FrotendServer.js中客户端消息为什么不加上reqId呢？ about mydog HOT 7 CLOSED

pomelo的是协议里面自己加上了reqId吧，如果不统一加上reqId的话，需要每一个协议都加上标识，这个感觉挺麻烦的。 fontendServer.js的161行 this.msgHandler[cmdArr[1]][cmdArr[2]](msg, client.session, this.callBack(client, data.cmd, msg.reqId)); } else { this.doRemote(data, client.session, cmdArr); } } catch (e) { this.app.logger("msg" /* msg */, "error" /* error */, e.stack); } } /** * Callback */ callBack(client, cmd, reqId) { let self = this; return function (msg) { if (msg === undefined) { msg = {}; } msg.reqId = reqId; let buf = self.app.protoEncode(cmd, msg); client.send(buf); }; } 感觉这么加上就可以了吧？ 我刚刚看mydog，不确认这个this.callback这么改是否可行。 ***@***.*** 发件人： ahuang 发送时间： 2021-09-10 14:06 收件人： ahuangege/mydog 抄送： lyyzdearjy; Author 主题： Re: [ahuangege/mydog] FrotendServer.js中客户端消息为什么不加上reqId呢？ (#3) 如果加reqId，则前端服转发到后端服时实现需要麻烦一些。而需要reqId辨认消息的需求，可以通过服务器返回的数据里加上一些标识。综合考虑，没有加这个东西。 — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe. Triage notifications on the go with GitHub Mobile for iOS or Android.

还是觉得很奇怪，请求+回调，变成了客户端和服务器的相对两个push。 如果出现多处使用协议的地方就会很难受。 ***@***.*** 发件人： ahuang 发送时间： 2021-09-10 14:25 收件人： ahuangege/mydog 抄送： lyyzdearjy; Author 主题： Re: [ahuangege/mydog] FrotendServer.js中客户端消息为什么不加上reqId呢？ (#3) 额，基本上所有消息都不需要依赖这个reqId吧，收发消息处理就行了。极少数需要依赖的地方就通过服务器的消息加些标识。 — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe. Triage notifications on the go with GitHub Mobile for iOS or Android.

嗯，是的。消息监听和移除需要把握好。。这个reqId，我这边可能暂时不考虑加了哈。

我看了一下你的斗地主的客户端代码，基本上都是再start注册，然后destroy移除handler。 我这边可能更多的是习惯在数据层、逻辑层做通讯处理。所以频繁的调用add remove的写法会很怪。 我自己先修改frontend吧。 能否在后续的版本中将reqId加入可配？ ***@***.*** 发件人： ahuang 发送时间： 2021-09-10 14:25 收件人： ahuangege/mydog 抄送： lyyzdearjy; Author 主题： Re: [ahuangege/mydog] FrotendServer.js中客户端消息为什么不加上reqId呢？ (#3) 额，基本上所有消息都不需要依赖这个reqId吧，收发消息处理就行了。极少数需要依赖的地方就通过服务器的消息加些标识。 — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe. Triage notifications on the go with GitHub Mobile for iOS or Android.

------------------ 原始邮件 ------------------ 发件人: ***@***.***>; 发送时间: 2021年9月10日(星期五) 下午2:42 收件人: ***@***.***>; 抄送: ***@***.***>; ***@***.***>; 主题: Re: [ahuangege/mydog] FrotendServer.js中客户端消息为什么不加上reqId呢？ (#3) 我看了一下你的斗地主的客户端代码，基本上都是再start注册，然后destroy移除handler。 我这边可能更多的是习惯在数据层、逻辑层做通讯处理。所以频繁的调用add remove的写法会很怪。 我自己先修改frontend吧。 能否在后续的版本中将reqId加入可配？ ***@***.*** 发件人： ahuang 发送时间： 2021-09-10 14:25 收件人： ahuangege/mydog 抄送： lyyzdearjy; Author 主题： Re: [ahuangege/mydog] FrotendServer.js中客户端消息为什么不加上reqId呢？ (#3) 额，基本上所有消息都不需要依赖这个reqId吧，收发消息处理就行了。极少数需要依赖的地方就通过服务器的消息加些标识。 — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe. Triage notifications on the go with GitHub Mobile for iOS or Android. — You are receiving this because you commented. Reply to this email directly, view it on GitHub, or unsubscribe. Triage notifications on the go with GitHub Mobile for iOS or Android.