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