Есть счётчик клиентов socket.io на сервере, клиенты прибавляются но при отключении не убавляются, что делать?

Question

[historydev]

Клиент:

var host = 'http://localhost:3000/';
var socket = io(host);
var hp = 10000;
      		
document.querySelector('.send').addEventListener('click', ev => {
  ev.preventDefault();
      			
  let text = document.querySelector('.input').value;     		
  hp = hp-100;
 		if(hp <= 0) hp = 0;
     socket.emit('message',hp);
      		
});	

socket.emit('forceDisconnect');

socket.on('new message', function(data) {
  hp = data;
  let li = document.createElement('li');
  li.textContent = hp;
      			
  document.querySelector('body').appendChild(li);
});
      	       	 
socket.on('disconnect', function(data) {
  alert('Сервер заполнен'); 
 // window.location.replace(host + 'serverfull');
});

window.onbeforeunload = function() { 
	return "Данные не сохранены. Точно перейти?";
	socket.disconnect(true);
	socket.emit('forceDisconnect');
};

Сервер:

var express = require('express'); // Get the module 
var app = express(); // Create express by calling the prototype in var express
var http = require('http').Server(app);
var connections=0,max_connections=2,port = 3000;

let io = require('socket.io')(http);

app.use(express.static(__dirname + '/client'));

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

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

io.on('connection', function(socket) {
	console.log('connected new client');
			
			connections+=1;
			
			if(connections > max_connections) {
				connections-=1;
				socket.on('forceDisconnect', function(data){
					socket.disconnect(true);
					console.log('Server full!');
				});
			}
			
			console.log(connections);
			
			socket.on('message', function(data) {
   			io.emit('new message', data);
   	});
   	
   	
   	
});
console.log(connections);

Comments

[Сергей Сергей]

надо добавить убавление при дисконекте
socket.on('close', () => connections-=1 );
а в другом месте "connections-=1" убрать.

[historydev]

Сергей Сергей, на сервер?

[historydev]

Сергей Сергей, нет, в другом он нужен для правильного числа подключений, там выходит +1 к кол-ву подключений, поэтому я при условии убираю 1

[historydev]

Сергей Сергей, не работает

[Сергей Сергей]

какое событие на закрытие клиента? туда надо вписать эти строки.
И убавление будет происходить для всех закрываемых, даже для forceDisconnect.

[historydev]

Сергей Сергей, на сервер добавил, вот

[Сергей Сергей]

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

[historydev]

Сергей Сергей, forceDisconnect, при вызове клиент закрывается

[Александр]

Сергей Сергей, HistoryART, событие disconnect.

[Сергей Сергей]

сложно всё.
Переполнение проверять сразу, без увеличения счетчика.
В случае переполнения на сокет вешается forceDisconnect как сейчас, но счетчик убавлять не надо.
В случае незаполненности увеличивается счетчик и на сокет вешается disconnect ( Александр, ) с убавителем.

socket.on( 'disconnect', () => { connections -= 1; console.log( 'client disconnected' ) }

странным я считаю то, что клиент запрашивает разрыв соединения, зачем ждать какого-то события (forceDisconnect), сразу закрывать с сообщением о переполнении.

[historydev]

Сергей Сергей, не работает

Answer 1

[historydev]

Вот так выглядит решение, forceDisconnect - отключает клиента, socket.on disconnect, ведёт счётчик, спасибо за помощь.

connections+=1;
			
			if(connections > max_connections) {
				connections-=1;
				socket.on('forceDisconnect', () => {
						socket.disconnect(true);
						console.log('Server full!');
				});
			}
			
			socket.on( 'disconnect', () => { 
				connections -= 1; 
				console.log( 'client disconnected' ) ;
				
			});