Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: accept player connection #4

Open
AsPulse opened this issue Aug 14, 2024 · 3 comments
Open

feat: accept player connection #4

AsPulse opened this issue Aug 14, 2024 · 3 comments

Comments

@AsPulse
Copy link
Contributor

AsPulse commented Aug 14, 2024

  • thiserror, tracingクレートを導入してください。これ以降の関数はinstrument化しましょう.
  • InstanceId(u32), PlayerId(u32) を表す構造体をNewTypeパターンで作ってください.
  • Instanceを管理するグリーンスレッドと、メッセージパッシングでコミュニケーションを整えてください.
  • パケットの種類と、IDの対応を作ってください.
  • Codecを実装してください.( PacketId: u64, length: u64, Payload: [u8; length] が構成)
  • PacketのRequestJoinInstance, AcceptJoinInstance を実装してください.
@HMasataka
Copy link
Contributor

HMasataka commented Sep 22, 2024

一回のPRで終わらせようとすると大きくなりすぎそうなので分割して作業します。
TODOリスト代わりに以下利用

  • thiserror, tracingクレートを導入してください。これ以降の関数はinstrument化しましょう.
  • InstanceId(u32), PlayerId(u32) を表す構造体をNewTypeパターンで作ってください.
  • Instanceを管理するグリーンスレッドと、メッセージパッシングでコミュニケーションを整えてください.
  • パケットの種類と、IDの対応を作ってください.
  • Codecを実装してください.( PacketId: u64, length: u64, Payload: [u8; length] が構成)
  • PacketのRequestJoinInstance, AcceptJoinInstance を実装してください.

@AsPulse
Copy link
Contributor Author

AsPulse commented Sep 22, 2024

ここらへんのメッセージパッシングもservifyの領域かも……

@HMasataka
Copy link
Contributor

それはめっちゃ思ってました
ひとまずservifyが最低要件満せるくらいはそっち優先しますか

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

No branches or pull requests

2 participants