为什么客户端收不到Socket.IO房间广播？附代码写法优化疑问

一、Socket.IO房间广播无响应问题 #

核心错误原因 #

1. 事件重复注册问题：你把io.of('/game').on('connection')写在了GET /game的路由回调里，每有一个用户访问/game页面，就会给/game命名空间新增一个connection事件监听器，后续只要有用户连接到这个命名空间，所有注册过的监听器都会同时触发，会产生大量重复逻辑、变量污染，完全不符合Socket.IO的使用规范。
2. 广播机制认知偏差：Socket.IO中socket.to(房间名).emit()的语义是「向该房间内除了当前socket连接之外的所有用户发送事件」，所以新加入的用户自己是收不到这个事件的，只有房间里的老用户能收到。你直接用socket.emit是给当前socket自己发，所以能正常接收。
3. 变量作用域隐患：你当前的roomName是从GET路由的cookie里取的，放在connection回调里属于引用了外层作用域的变量，后续其他用户访问路由时这个变量会被覆盖，会出现发错房间的问题。

修正代码 #

app.js

// 全局只注册一次/game命名空间的连接监听，不要放在路由里
const gameNamespace = io.of('/game');
gameNamespace.on('connection', socket => {
    console.log('a user connected');
    // 客户端连接后主动发送加入房间的请求，携带房间号和当前用户信息
    socket.on('joinRoom', async ({roomName, currentUser}) => {
        await socket.join(roomName);
        // 需求1：通知房间内除自己外的老用户有新人加入
        socket.to(roomName).emit('join', currentUser);
        // 需求2：给自己也发通知确认加入成功（可选）
        socket.emit('join', currentUser);
        // 若要直接通知房间所有人包括自己，替换为下面这行即可
        // gameNamespace.to(roomName).emit('join', currentUser);
    })

    socket.on('disconnect', () => {
        console.log('user disconnected');
    });
});

// Game路由
app.get('/game', (req, res) => {
    const cookies = req.cookies;
    const currentUser = cookies['current_user'];
    const roomName = cookies['room_name'];

    if (roomName) {
        res.render('pages/game', {
            room: roomName,
            currentUser: currentUser
        });
    } else {
        res.redirect('/login');
    }
});

game.ejs

const socket = io("/game");
// 页面加载后主动发送加入房间的请求，把模板注入的参数传给服务端
socket.emit('joinRoom', {
    roomName: "<%= room %>",
    currentUser: "<%= currentUser %>"
})

socket.on('join', function(user) {
    console.log(`${user} joined`);
});

二、分支判断写法问题 #

你提到的替换写法完全可行，这种写法叫做卫语句，核心作用是减少代码嵌套层级，避免多层if-else嵌套导致的「缩进地狱」，可读性和可维护性都比原始的if-else写法更好，在你给出的场景下没有任何兼容性或逻辑问题，可以放心使用。

内容的提问来源于stack exchange，提问作者William Jones