Комментарии 3
'{"event":"dataConfirm","userid":"' +
msgdata.userid +
'","dataid":"' +
А зачем вы тут JSON собираете конкатенацией строки, а не используете литерал объекта? Смотрится страшненько.
Также, почему смешиваете стили написания идентификаторов?
"dataConfirm","userid"
В одном случае используется заглавная буква, в другом нет.
// datasync.Mess массив сообщений
Вы при синхронизации пересылаете все сообщения сразу? Это проблем не вызовет из-за большого объема?
Результаты тестов скорости в интернете для node.js были немного быстрее при использовании контатенации. И не только для node.js stackoverflow.com/questions/29055518/are-es6-template-literals-faster-than-string-concatenation
Я еще в коде иногда var пишу.
Возможно сам пока не сталкивался, около 20 пользователей и комнат + несколько тысяч сообщений проблем не создают. О возможной проблеме в будущем знал, поправить несложно.
Я еще в коде иногда var пишу.
Возможно сам пока не сталкивался, около 20 пользователей и комнат + несколько тысяч сообщений проблем не создают. О возможной проблеме в будущем знал, поправить несложно.
Поправил, данные большого объема пересылаются частями также как и файлы, размер части зависит от настройки filepartsize.
В статье приведен пример подключения клиента, код сервера открыт, если вам что то не нравится вы всегда можете переписать как вам угодно.
В статье приведен пример подключения клиента, код сервера открыт, если вам что то не нравится вы всегда можете переписать как вам угодно.
Зарегистрируйтесь на Хабре, чтобы оставить комментарий
Клиент для «Сервер push сообщений»