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

扉の開閉 #41

Closed
wants to merge 7 commits into from
Closed

扉の開閉 #41

wants to merge 7 commits into from

Conversation

iieiieiie
Copy link

###対応issue
#25

確認項目

  • DoorControllerはIInteractableを継承
  • DoorController内にbool型の変数isOpenを作成する
  • Interactメソッドが呼ばれたらisOpenの値を切り替える
  • isOpenの値が変化したらドアの開閉を行うメソッドを呼ぶ
  • isOpenの値の監視は今回は一時的にUpdate内で行う

@chromee chromee requested a review from RauziDssh September 14, 2017 03:41
Copy link
Member

@RauziDssh RauziDssh left a 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");
}
Copy link
Member

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");
}
Copy link
Member

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の値だけを切り替えるようにしてください

@iieiieiie iieiieiie changed the title [WIP]Interactメソッドから開閉メソッド 扉の開閉 Oct 1, 2017
@iieiieiie
Copy link
Author

遅すぎてすいませんでした
指摘してもらったところを修正しました

@iieiieiie iieiieiie removed the WIP label Oct 1, 2017
@chromee
Copy link
Member

chromee commented Oct 1, 2017

個人的にはitweenを導入してギミック周りの細かい移動はitweenを使うのが良いと思うのですが、らうじぃさん的やiieiieくんはどう思いますか?

@iieiieiie
Copy link
Author

すいません使ったことがないのでわかりません…

@chromee
Copy link
Member

chromee commented Oct 2, 2017

「unity itween」で検索したらすぐ出てくるんですけど、iTweenは簡単なアニメーションをスクリプトで定義して操作できるようにするアセットです。
ドアをx方向に1スライドさせるすると、下のスクリプトで完了です

iTween.MoveTo( gameObject, iTween.Hash( "x", 1f ) );

コード量的にも効率的にもこちらの方がかなり良いのでできればiTweenを使ってもらえるとうれしいです。

http://doggy.hatenablog.com/entry/2016/02/11/121327

@iieiieiie
Copy link
Author

返信遅れてすみません
itween使ってみます

@iieiieiie
Copy link
Author

お待たせしました
itweenで書いたコードです

@chromee
Copy link
Member

chromee commented Oct 22, 2017

importしたiTweenもpushお願いします

@chromee
Copy link
Member

chromee commented Oct 22, 2017

importしたアセットはAssetsフォルダに入れるようにしてください。
Assetsフォルダに入れないとUnityに読み込まれないのでiTweenを使うことができません。

DoorController.csもSSPプロジェクトの直下においてありますがAssetsフォルダの中に移動してください。
その際、Assets/SSP/Scripts/Common/DoorController.csは不要なので削除してください。

それと編集したシーンやプレハブもpushするようにしてください。
レビューする度にスクリプトが正しい挙動をしているか確認するために、いちいちレビュワーがオブジェクトを配置したりしないといけないのは大変なのでよろしくお願いします。

@iieiieiie
Copy link
Author

本当にすみません
今から用事なので明日までに何とかします

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

Successfully merging this pull request may close these issues.

3 participants