关于javascript的参数使用方式

<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 全局变量。（只是针对当前代码）

所以我反而觉得，这里不是全局变量个数的问题，而成了是否需要将功能函数进行良好封装的问题。