-
Notifications
You must be signed in to change notification settings - Fork 1
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
扉の開閉 #41
扉の開閉 #41
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
WIPになっていますがしばらくコミットがないのでとりあえず今のところまでのコードを読みました.
いくつか修正を指示したので対応してください
else if (!isOpen) | ||
{ | ||
StartCoroutine("Close"); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ドアの開閉命令はコルーチンではなく,メソッド(Open,Close)を作ってその中でやってください.
(isOpenの値が変化したらドアの開閉を行うメソッドを呼ぶ)
コルーチンを使う場合はOpen,Closeの中で開始するようにしてください
else if (!isOpen) | ||
{ | ||
StartCoroutine("Close"); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
14行目からここまでの処理はUpdateの中でやってください.( isOpenの値の監視は今回は一時的にUpdate内で行う)
基本的にInteract文の中ではisOpenの値だけを切り替えるようにしてください
…jp/SSP into _iieiieiie/door-open-close
遅すぎてすいませんでした |
個人的にはitweenを導入してギミック周りの細かい移動はitweenを使うのが良いと思うのですが、らうじぃさん的やiieiieくんはどう思いますか? |
すいません使ったことがないのでわかりません… |
「unity itween」で検索したらすぐ出てくるんですけど、iTweenは簡単なアニメーションをスクリプトで定義して操作できるようにするアセットです。 iTween.MoveTo( gameObject, iTween.Hash( "x", 1f ) ); コード量的にも効率的にもこちらの方がかなり良いのでできればiTweenを使ってもらえるとうれしいです。 |
返信遅れてすみません |
お待たせしました |
importしたiTweenもpushお願いします |
importしたアセットはAssetsフォルダに入れるようにしてください。 DoorController.csもSSPプロジェクトの直下においてありますがAssetsフォルダの中に移動してください。 それと編集したシーンやプレハブもpushするようにしてください。 |
本当にすみません |
###対応issue
#25
確認項目