Socket.io教程创建了额外的消息

时间:2016-05-12 15:25:34

标签: node.js sockets socket.io

我正在关注Socket.IO tutorial,但我遇到的问题是页面上显示的消息数量呈指数级增长,导致聊天客户端无效。

一些粗略的搜索告诉我它涉及事件处理程序,但我没有找到关于如何在这种情况下使用它们的确定性。 我需要使用这些事件处理程序的内容和位置,以及为什么?

我的index.js:

var app = require('express')();
var http = require('http').Server(app);
var io = require('socket.io')(http);

app.get('/', function(req, res){
  res.sendFile(__dirname + '/index.html');
});

io.on('connection', function(socket){
  // console.log('a user connected');
  // socket.on('disconnect', function(){
  //   console.log('user disconnected');
  // });
  socket.on('chat message', function(msg){
    //console.log('message: ' + msg);
    io.emit('chat message', msg);
  });
});

http.listen(8080, function(){
  console.log('listening on *:8080');
});

我的HTML:

<!doctype html>
<html>
  <head>
    <title>Socket.IO chat</title>
    <style>
      * { margin: 0; padding: 0; box-sizing: border-box; }
      body { font: 13px Helvetica, Arial; }
      form { background: #000; padding: 3px; position: fixed; bottom: 0; width: 100%; }
      form input { border: 0; padding: 10px; width: 90%; margin-right: .5%; }
      form button { width: 9%; background: rgb(130, 224, 255); border: none; padding: 10px; }
      #messages { list-style-type: none; margin: 0; padding: 0; }
      #messages li { padding: 5px 10px; }
      #messages li:nth-child(odd) { background: #eee; }
    </style>
  </head>
  <script src="/socket.io/socket.io.js"></script>
  <script src="http://code.jquery.com/jquery-1.11.1.js"></script>
  <script>

   function doDid(){
      var socket = io();
      $('form').submit(function(){
        socket.emit('chat message', $('#m').val());
        $('#m').val('');
        return false;
      });
      socket.on('chat message', function(msg){
        $('#messages').append($('<li>').text(msg));
      });
   };
  </script>
  <body>
    <ul id="messages"></ul>
    <form action="">
      <input id="m" autocomplete="off" /><button onclick="doDid()">Send</button>
    </form>
  </body>
</html>

2 个答案:

答案 0 :(得分:1)

var socket = io();

此行创建与socket.io的连接。每次调用它都会创建另一个连接。尝试只调用一次而不是每次发送。

为了澄清,io()功能是工厂而不是访问者

修改

看起来socket.io客户端实际上会创建它创建的缓存套接字并且不会创建多个连接。

但是,我也注意到你在该函数中绑定事件,但是每次单击都会调用它,所以每次都要重新绑定。仅在启动时调用您的函数一次。

答案 1 :(得分:1)

问题是,每次按下按钮都会订阅“聊天消息”事件。

您应该只运行一次此代码:

 var socket = io();

  socket.on('chat message', function(msg){
    $('#messages').append($('<li>').text(msg));
  });

所以你应该改变你的代码:

<script>
  var socket = io();

  socket.on('chat message', function(msg){
    $('#messages').append($('<li>').text(msg));
  });

   function doDid(){
      $('form').submit(function(){
        socket.emit('chat message', $('#m').val());
        $('#m').val('');
        return false;
     });  
   };
  </script>