Skip to content

Added game history api.#319

Open
gtivansan wants to merge 2 commits intogame-historyfrom
game-history-api
Open

Added game history api.#319
gtivansan wants to merge 2 commits intogame-historyfrom
game-history-api

Conversation

@gtivansan
Copy link
Copy Markdown
Member

No description provided.

Copy link
Copy Markdown
Member

@lounres lounres left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Меня немного смущает, что из-за того, что история передаётся только при входе в комнату, всё может навернуться: из-за плохого подключения и т.п. часть "истории" может быть не услышана одним из игроков.

А так всё ОК.

P.S. Можно ещё конечно передавать временну́ю информацию (как долго угадывал; она собирается также для Sombrero), но её уже не сможет нормально замерять клиент и, видимо, показывать такое мы пока не хотим.

@gtivansan
Copy link
Copy Markdown
Member Author

Не очень понял. Кажется сокеты гарантируют, что все сообщения приходят в правильном порядке. А если какое-то не дошло, то значит произошёл disconnect. В таком случае при повторном подключении вся информация передастся.

@gtivansan gtivansan added API Specifying the API between front and back teams documentation Improvements or additions to documentation labels Jan 29, 2021
@lounres
Copy link
Copy Markdown
Member

lounres commented Jan 29, 2021

Не очень понял. Кажется сокеты гарантируют, что все сообщения приходят в правильном порядке. А если какое-то не дошло, то значит произошёл disconnect. В таком случае при повторном подключении вся информация передастся.

ОК, хорошо. Видимо, я зря волнуюсь.

@alexdany657
Copy link
Copy Markdown
Member

ОК, хорошо. Видимо, я зря волнуюсь.

Ну, кажется, что не совсем зря. Если соединение идет по WebSockets, то да, порядок гарантируется. Однако в редких случаях (не более 3% по статистике socket.io) соединение может устанавливаться другими методами, не гарантирующие этого.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

API Specifying the API between front and back teams documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants