<code>var nickNames = {}; handleMessageBroadcasting = function(socket, nickNames){ socket.on('message', function(message){ socket.broadcast.to(message.room).emit('message', { text: nickNames[socket.id] + ': ' + message.text }); }); }; handleMessageBroadcasting(socket, nickNames); </code>
上面handleMessageBroadcasting函数内部用到了nickNames,我是将nickNames当参数传入再使用呢?还是直接当成全局变量来使用好呢?为什么?
<code>var nickNames = {}; handleMessageBroadcasting = function(socket, nickNames){ socket.on('message', function(message){ socket.broadcast.to(message.room).emit('message', { text: nickNames[socket.id] + ': ' + message.text }); }); }; handleMessageBroadcasting(socket, nickNames); </code>
上面handleMessageBroadcasting函数内部用到了nickNames,我是将nickNames当参数传入再使用呢?还是直接当成全局变量来使用好呢?为什么?
传参,可以降低代码耦合度
编程里最禁忌的就是全局变量了吧,全局变量越少越好,就算项目规模比较小,也不妨养成这个好习惯
建议是传参,全局变量可能会污染全局环境
能不用全局变量尽量不用
楼上几位说的很对。还是传参比较好,特别说明一下我对经历,因为当时写对js
代码比较多定义了全局的倒置我其他地方用参数的时候,怎么设置都不行,就是受到全局的影响了。
都是在建议传参,但是有没有发现你这段代码传参或全局,都是一样的结果,都会有且只有两个 nickNames
和 socket
全局变量。(只是针对当前代码)
所以我反而觉得,这里不是全局变量个数的问题,而成了是否需要将功能函数进行良好封装的问题。