Функция выполняется несколько раз, почему ( Node JS + Socket IO )?

Question

[Роман Королёв]

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

Нужно сделать так, чтобы она отправлялась 1 раз в 3 секунды. Может проблема в setTimeout? ( Функция usersCount() )

Пример кода:

io.sockets.on('connection', function(socket){
    console.log('connect');
   setTimeout(usersCount, 3000);
   
});

/* Доступ к статистике */ 

function usersCount() {
		connection.query('SELECT COUNT(*) FROM `users`', function(err, rows) {
		users = rows['0']['COUNT(*)'];
		});  
		
		https_server.getConnections(function (err, count) {
		online = count;
		});
		
		connection.query('SELECT COUNT(*) FROM `drops`', function(err, rows) {
		drops_count = rows['0']['COUNT(*)'];
		});


			var json = {
			
			users: users,
			online: online,
			drops: drops_count
			
			};

        socket.emit('stats',json); // отправим в сокеты

   setTimeout(usersCount, 5*3000);
}

Answer 1

[Роман Королёв]

Разобрался, убрал
io.sockets.on('connection', function(socket){

Answer 2

[Кирилл Кудрявцев]

С помощью SetTimeout вы не ограничивает кол-во выполнения в секунду, а просто откладываете вызов на 3 секунды. И если было 2 коннекта подряд, то через 3 секунды usersCount вызовится 2 раза.

Comments

[Роман Королёв]

а как тогда правильно написать можете подсказать ?

Answer 3

[Владимир Скибин]

Вам необходимо выполнять функцию отправки только при условии наличия подключенных пользователей и отправлять по всем подключениям (далее их можно разделить по комнатам/группам и по каждой комнате отправлять нужные данные).

setIterval(usersCount, 5 * 3000);

function usersCount() {
  io.sockets.clients((err, clients) => {
    if (err) throw err;
    if (clients.length) {
     // do some sql
     io.sockets.emit('stats', json);
    }
  });
}