2016-05-12 3 views
1

Я следую за Socket.IO tutorial, но я столкнулся с проблемой, когда количество сообщений, отображаемых на странице, увеличивается экспоненциально, что делает клиент чата неэффективным.Учебник Socket.io создает дополнительные сообщения

Некоторый беглый поиск говорит мне, что он включает обработчики событий, но я не нашел ничего определенного в том, как использовать их в этом контексте. Что и где мне нужно использовать эти обработчики событий и почему?

Моего 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> 

ответ

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> 
1
var socket = io(); 

Эта линия создает соединение с Socket.io. Каждый раз, когда вы его вызываете, вы создаете другое соединение. Попробуйте только называть это один раз, а не на каждой отправке.

Чтобы уточнить, функция io() является завод не аксессор.

Edit:

Похоже, socket.io клиент фактически делает сокеты кэша создает и не создают несколько соединений.

Однако я также заметил, что вы привязываете события к этой функции, но каждый раз нажимаете на нее каждый щелчок, поэтому вы каждый раз переплетаетесь. Вызывайте свою функцию только один раз при запуске.

+0

Полезно знать, но проблема с несколькими сообщениями по-прежнему происходит. – Thassa

+0

@Thassa вам нужно переместить 'var socket = io()' _and_ 'socket.on()' вне 'doDid()' целиком. – robertklep

+0

Собственно, просто переместите обработку событий. Вы уже привязываетесь к событию отправки, не вызывайте эту функцию при отправке. Назовите его на загрузку страницы. – Chad