2d-rpg / card-playroom-client

カードゲームシミュレーター
https://play.google.com/store/apps/details?id=net.banatech.card_playroom_client
0 stars 0 forks source link

ルーム一覧の取得とルーム作成およびルーム入室をgraphQLからwebsocketでできるように変更 #21

Closed YasuakiYoshii closed 3 years ago

YasuakiYoshii commented 3 years ago

概要

ルーム一覧の取得とルーム作成およびルーム入室をgraphQLからwebsocketでできるように変更.

動作確認

  1. サーバを起動する。参照
  2. 端末を2つ用意し、トップ画面からRoomlistタブを押す。(この段階でなぜかリストが表示されない
  3. Homeタブを押してもう一度Roomlistタブを押すと.ルームリストが表示される。
  4. 1つ目の端末でルームリストにあるルームを1つタップすると青い四角が出る。
  5. 2つ目の端末で1つ目と同じルームに入ると、端末間で青い四角オブジェクトの位置が同期する。
  6. 2つ目の端末でフローティングボタンから新しくルームを作る(自動的にルームに参加する)と、1つ目の端末とは位置が同期しない。

Updated

alt text

関連するサーバーサイドの変更

サーバー側でルームリストをjsonで返すように変更.(https://github.com/2d-rpg/card-playroom-server/pull/16)

bana118 commented 3 years ago
bana118 commented 3 years ago
bana118 commented 3 years ago

気になった点として

YasuakiYoshii commented 3 years ago

気になった点として

* ルーム名が重複したら既存のルームに参加扱いになってる?

* 四角を同時に動かそうとするとなんかすごい動きになるので同期の仕方は工夫が必要そう

* ルームリスト更新ボタンはフローティングボタンに欲しいかも
  ただこれらはこのPRマージ後でいいと思う

1つめはそう.ルームをidではなく,ルーム名(string)で判別しているため. 2つ目は確認しますた.やばいなぁ...競合しちゃってるので何かしらの相互排他処理が必要かも 3つ目はtab押した時は更新しないようにするってこと??

bana118 commented 3 years ago

気になった点として

* ルーム名が重複したら既存のルームに参加扱いになってる?

* 四角を同時に動かそうとするとなんかすごい動きになるので同期の仕方は工夫が必要そう

* ルームリスト更新ボタンはフローティングボタンに欲しいかも
  ただこれらはこのPRマージ後でいいと思う

1つめはそう.ルームをidではなく,ルーム名(string)で判別しているため. 2つ目は確認しますた.やばいなぁ...競合しちゃってるので何かしらの相互排他処理が必要かも 3つ目はtab押した時は更新しないようにするってこと??

ルーム作成時に重複したら,そのルームに参加っていうのはやばそう.いっそルーム名は重複しないようにサーバー側で作成してもいいかもしれない. ルームリスト更新にタブ切り替えしてもらうっていうのはUI的によくないのでタブ押したときも更新ボタンを押したときも更新する,とした方がよさそう

bana118 commented 3 years ago
YasuakiYoshii commented 3 years ago

サーバ側のルームの実装にid持たせるように変更した方がいいかもね プレイヤー人数とか取得できるとなお良いけど,並列処理バージョンでそれができるのか,,,