Нет события beforeunload — есть какое-то объяснение?

Question

[Abcdefgk]

Столкнулся с такой странностью.
Простая на клиенте вещь, вот такие две строчки:

window.addEventListener('beforeunload', function() {
  sendMessage(room);
});

Содержание функции простое, можно было так написать - socket.emit('message', room)
Решил опробовать WebRTC, а мне обязательно нужно на сервере считать количество клиентов в "комнатах", для чего есть объектик, в котором имя комнаты - ключ, и его значение - массив со случайными id присоединившихся к ней. Предполагается, что если клиент уходит (закрывает окно), то и нужно отправить имя комнаты, из которой он ушел, чтобы отнять элемент из соответствующего массива.

Я всё это дело сделал дома, и на localhost это всё нормально работает. Решил попробовать, так сказать, "в эфире" - выпросил 14 дней бесплатного VPS, залил туда всё, запустил... ... - а нет этого события, месадж не отправляется, сервер ничего не получает при закрытии окна. Количество присоединившихся к комнатам в массивы плюсуется, а при уходе - не уменьшается, естественно.

Повторю: на localhost всё работает. Я уже и в пяти вариациях это писал, и во всех на локалхосте работает, а ровно то же самое в энторнетах - нет события, нет сообщения, сервер ничего не получает.
Есть этому хоть какое-то объяснение?

Comments

[Илья Шатохин]

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

[Abcdefgk]

Илья Шатохин, объясните это лаборатории Google -->

[Илья Шатохин]

Abcdefgk, ткнул по ссылке, увидел гайд для новичков по webRTC. Не понятно причем тут beforeunload.

[Abcdefgk]

Илья Шатохин, Я понял, что вы - круче Гугла. Вы им тогда и объясните, в чём тут ошибка, как её исправить, как правильно пользоваться событиями.
А то, я почему-то не вижу здесь ответа на вопрос, а вижу только пальцы, распущенные веером.
Я-то придумал решение, и у меня в результате всё работает.
Там, по ссылке, есть весь клиентский код, и в нём использовано событие именно в таком виде. А в конце есть ссылка на комюнити в Гитхабе, куда вы можете написать - и научить их, как делать правильно.
А меня оставьте в покое.

[Илья Шатохин]

Abcdefgk, снова не понял вашей демагогии. Где там ошибка то? И пальцы веером пока пускаете только вы, впрочем, как и хамите каждому, кто вам отвечает.

[Abcdefgk]

Илья Шатохин, "Вы бы хоть почитали что такое beforeunload" - это, по вашему, не хамство?
Вот вам фрагмент огиринально гугловского кода:

function maybeStart() {
  console.log('>>>>>>> maybeStart() ', isStarted, localStream, isChannelReady);
  if (!isStarted && typeof localStream !== 'undefined' && isChannelReady) {
    console.log('>>>>>> creating peer connection');
    createPeerConnection();
    pc.addStream(localStream);
    isStarted = true;
    console.log('isInitiator', isInitiator);
    if (isInitiator) {
      doCall();
    }
  }
}

window.onbeforeunload = function() {
    sendMessage('bye');
};

/////////////////////////////////////////////////////////

function createPeerConnection() {
  try {
    pc = new RTCPeerConnection(null);
    pc.onicecandidate = handleIceCandidate;
    pc.onaddstream = handleRemoteStreamAdded;
    pc.onremovestream = handleRemoteStreamRemoved;
    console.log('Created RTCPeerConnnection');
  } catch (e) {
    console.log('Failed to create PeerConnection, exception: ' + e.message);
    alert('Cannot create RTCPeerConnection object.');
    return;
  }
}

15 и 16 строчку видите?
Можете объяснить им ошибку.

Answer 1

[ой всё]

https://stackoverflow.com/questions/23399293/nodej...

Comments

[Abcdefgk]

Спасибо, но это не годится - я думал про это. Событие disconnect генерится, и оно исправно приходит на сервер - я проверял, но с ним же ничего не приходит, кроме объекта с одним ключом и одним значением - что процесс прибился, а мне надо имя комнаты передать непременно для дальнейших с ним манипуляций.

[ой всё]

чессгря, первый раз такое слышу, чтобы бэкэнд ломал голову над закрытием страницы

обычно делается сессия и ей устанавливается лимит времени

https://stackoverflow.com/questions/5292506/is-the...

[Abcdefgk]

AnneSmith, чессгря, я, глядя со стороны, вообще не вижу тут повода, над которым можно было бы ломать голову. Но есть проблема - оно не работает, и я не понимаю, почему оно, вообще, может не работать.

[ой всё]

Abcdefgk, так по ссылке написано почему :)
вы не читали?

[Abcdefgk]

AnneSmith, чего там написано, что событие beforeunload - совершенно бесполезно и никто не знает, зачем оно существует? Интересная трактовка браузерных событий.
Ну да ладно, я всё переделал - чтоб работало.
Если интересно - можете прочитать ответ меня себе.

[ой всё]

Abcdefgk, а так вы английского не знаете
извините, я почему-то думала, что для программистов английский - стандарт

[Abcdefgk]

AnneSmith, Это вы русский не знаете.

[ой всё]

Abcdefgk, ну вы ниже подтвердили, что изобрели велосипед, на который вам дали ссылку :)

[Abcdefgk]

AnneSmith, Не велосипед, а конкретный способ применения. А вы не любите изобретать? Небось, всю жизнь проводите за чтением stackoverflow.

[ой всё]

Abcdefgk, нет, не всю жизнь, сделала исключение для вас лично :)

Answer 2

[Кирюшка Цисарь]

window.onbeforeunload = function() {
  return sendMessage(room);
};

?

Comments

[Abcdefgk]

Во-первых, я это пробовал (хотя, это я уже по настоятельной рекомендации Мозилла-девелопер написал addEventListener), и всё равно - "дома" всё работает, там - не работает.
А return тут смысла не имеет, я же говорю - можно так написать (что я тоже пробовал):

window.onbeforeunload = function() {
  socket.emit('message', room);
};

Чего тут возвращать?

Answer 3

[Abcdefgk]

Короче, если кому интересно, то я всё сделал иначе, и оно работает, вроде.
Я сам себе придумал проблему, когда для подсчёта клиентов стал записывать в массивы комнат случайные типа-айдишники.
Тогда как каждое socket-соединение имеет собственный уникальный id - их-то, уникальные id, и надо в эти массивы складывать. И тогда я могу обойтись без имени комнаты, из которой клиент ушёл.
При событии socket.on('disconnect', ... (а оно неизбежно, как смерть и налоги) мы легко можем зафиксировать, какой сокет отключился, - по его этому самому id. Остаётся только перебрать весь объект-хранилище и удалить этот конкретный элемент. Ну вот так:

socket.on('disconnect', function() {
	for(var key in clientsArray) {
		for(var i=0; i<clientsArray[key].length; i++) {
			if(clientsArray[key][i] == socket.id) {
				clientsArray[key].splice(i, 1);
			}
		}
	}
});

Вуаля.
Правда, с событием beforeunload непонятка так и осталась.

Comments

[ой всё]

то есть ровно, что было написано в stackoverflow :)

[Abcdefgk]

AnneSmith, Что было написано в stackoverflow? что существует событие disconnect? Но это написано и в документации socke.io - тоже мне, новость.

[ой всё]

Abcdefgk, а что вам мешало сразу воспользоваться этой неновостью в таком случае? :))
научитесь признавать свои очевидные косяки, а не хамить тем, кто старался вам помочь

[Abcdefgk]

AnneSmith, Вообще-то, я вас поблагодарил в первом же каменте. Вы очень нервная. Расслабтесь, это же просто хобби.

[ой всё]

Abcdefgk, и тут же нахамили :))
у вас хобби, а у меня - благотворительность

[Abcdefgk]

AnneSmith, Ну извините. Не обидайтесь сильно-то. Я старый, больной человек, меня девушки не любят.

[Илья Шатохин]

Поиск в массиве, да через for-in? При онлайне в пару тысяч такой код будет насиловать CPU по полной. Проще использовать Map или объект. В общем, этот код из серии "вредных советов" или как делать НЕ НАДО.