Socket.io: socket.rooms - бесполезный объект

Что ж, похоже, документация здесь не актуальна, rooms самом деле был массивом до версии 1.4, я думаю.

Примечание: я думаю, мы могли бы использовать логическое значение в качестве значения.

Думаю, это запрос функции. Сделайте socket.io снова отличным: верните массивы!

Моя цель здесь - получить массив комнат, в которых находится клиент. С массивом я предполагаю, что socket.id будет первым:

var rooms = socket.rooms.slice(1);

Используя текущую реализацию:

var roomKeys = Object.keys(socket.rooms);
var socketIdIndex = roomKeys.indexOf( socket.id );
var rooms = roomKeys.splice( socketIdIndex, 1 );

Если бы socket.id не было включено, тоже было бы нормально. Я просто не хочу, чтобы его нужно было искать и выбирать.

var rooms = Object.keys(socket.rooms);

С помощью логических значений я мог бы отфильтровать истинные значения, например (верно?):

var roomKeys = Object.keys(socket.rooms);
var rooms = roomKeys.filter(function(key){ 
  return socket.rooms[key]==true;
});

Это так же медленно, как итерация с indexOf() .

Было бы полезно включать комнаты, в которых нет клиента (например, чтобы показать комнаты чата, к которым он может присоединиться). Я просто думаю, что эта информация была бы лучше где-нибудь еще, например, в экземпляре Server :

var allRooms = io.rooms;
var myRooms = socket.rooms;

У кого-нибудь еще есть мнение по этому поводу?

Да, это кажется излишне сложным. Я действительно изо всех сил пытаюсь понять, почему идентификатор сокета пользователя представляет собой «комнату»? Или зачем нужны пары ключ: значение . Это просто должен быть список - какие бы комнаты ни присутствовали, вы в них находитесь.

['room1', 'room2', 'room3']

И, как указано выше, io.rooms, чтобы получить их все либо в виде массива строк, либо, в данном случае, в виде объекта с именами комнат в качестве ключей и массивом идентификаторов сокетов в качестве их значения.

{ room1: ['id1', 'id2', 'id3'], room2: ['id4', 'id5', 'id6'] }

Я думаю, что основная причина - это производительность поиска.
Например, когда пользователь присоединился к комнате 100, было бы быстрее использовать поиск объектов, чтобы узнать, присоединился ли он к комнате, а не повторять значение массива.

Обратитесь к моему ответу на stackoverflow

{
    "room1":"room1",
    "room2":"room2",
    ...
    "room100":"room100"
}

Чтобы проверить, присоединился ли пользователь к комнате, достаточно просто:
if(socket.rooms[roomID]) return true;

Если это тип массива, нам нужно использовать indexOf или перебирать каждый массив:
if(socket.rooms.indexOf(roomID) != -1) return true;

Но я полностью согласен, если значение ключа будет изменено на логическое:
{ room1: true, ... , room100: true }

Это может помочь сэкономить немного памяти

Я бы не прочь изменить socket.rooms так, чтобы значения были логическими. Можно ли над этим работать? Я не уверен, что это за процесс, поскольку я не видел файла CONTRIBUTING.md.

Разве Set имеет больше смысла?

if (socket.rooms.has(roomID)) return true;

Например, структура будет просто Set(4) {"<socket ID>", "room1", "room2", "room3"}

Я согласен, если временная сложность набора постоянна. Я верю в javascript.

В Socket.IO v3 Socket#rooms теперь будет Set и использовать значение, предоставленное базовым адаптером, см. Здесь .