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

N人プレイ対応 #6

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

amatsuda
Copy link
Member

4人プレイ限定じゃなくて、接続されているgamepadの数だけプレイヤーが出現するようにしました。
せっかくgamepadオブジェクトを取ってるんだったらそのままPlayerのコンストラクタに渡せばいいじゃん、とかツッコめるところは色々あるところかと思いますが、ひとまずやりたいことは実現できたのでここまででぷるります。

@yancya
Copy link
Contributor

yancya commented Mar 16, 2017

これだと、ゲーム起動後に gamepad を接続した場合に reload しないといけなくなりますね
現状は単にゲームパッド未接続のプレイヤーは背景がグレーになって、接続したら適宜ホワイトになります

後から接続については gamepadconnected の hook を使ってなんとかなりそうではありますが、プレイヤーの表示順を index 順にするのがめんどくさそう

@yancya
Copy link
Contributor

yancya commented Mar 16, 2017

とは言え、index は内部的なものなので ”Player 1" とか "Player 2" という表記をやめさえすれば問題なさそうですね

@yancya
Copy link
Contributor

yancya commented Mar 16, 2017

gamepadconnected hook 、思った通りに発火してくれない......

@yancya
Copy link
Contributor

yancya commented Mar 16, 2017

現状のままでも、4人揃わなくてもゲームはできるので、特に別途少人数対応をしなくてもいいんじゃないかなと思い始めました

@amatsuda
Copy link
Member Author

なるほど。起動後に乱入のケースは考えていなかったし、思ったよりめんどくさそうなのも理解しました。じゃあひとまずこの件は放置で良いですー。

@yancya
Copy link
Contributor

yancya commented Mar 16, 2017

はいー

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

Successfully merging this pull request may close these issues.

2 participants