回答編集履歴

2

誤字修正

2020/05/18 11:04

投稿

sakura_hana
sakura_hana

スコア11427

test CHANGED
@@ -36,7 +36,7 @@
36
36
 
37
37
  上記のように「localScaleで左右表示を切り替えている」と分かれば、変数を挟まずに
38
38
 
39
- `if (transform.localScale == 1) {}`
39
+ `if (transform.localScale.x == 1) {}`
40
40
 
41
41
  と直で調べてしまうことも出来ます。(これは右向きの時に条件を満たす)(transformの参照は適宜正しく取ってください)
42
42
 

1

質問文編集を受けて追記

2020/05/18 11:04

投稿

sakura_hana
sakura_hana

スコア11427

test CHANGED
@@ -45,3 +45,127 @@
45
45
  これは「"右向き"というのはどういうことか?」を考えて、それぞれのコードが意味するところが分かっていれば導けます。
46
46
 
47
47
  最初は難しいかもしれませんが理解すると色々出来るので頑張ってみてください。
48
+
49
+
50
+
51
+ ---
52
+
53
+
54
+
55
+ (質問文編集を受けて追記)
56
+
57
+
58
+
59
+ 質問文には元々方向反転の記述がありますよね。
60
+
61
+ 何故消してしまったのでしょうか。そのまま利用すればいいです。
62
+
63
+
64
+
65
+ また、これは動かしてみれば分かりますが、boolは値型の変数なので「代入したタイミングの値をコピー」します。
66
+
67
+ つまりStartメソッド内に`isLeft = player.GetComponent<Player>().isLeft;`がありますが、
68
+
69
+ これだと「StartしたタイミングでのPlayerクラスのisLeft」しか反映されません。
70
+
71
+ 必要なタイミング(今回は発射ボタンを押したタイミング)で代入しましょう。
72
+
73
+
74
+
75
+ 更にこれはオマケですが、負荷についても考慮しましょう。
76
+
77
+ Updateはそのままだと毎フレーム実行されます。向き情報(offsetの用意)が必要なのは弾が発射された時だけなので、`if (Input.GetButtonDown("Fire1"))`の中に入れると余計な実行が減らせます。
78
+
79
+ また、出来るだけGetComponentの回数は減らした方が負荷が減るので、(今回はそう大差無いでしょうが)そういう風に書く癖を付けておきましょう。
80
+
81
+
82
+
83
+ という訳でPlayerBulletクラスを以下のように書くと上手くいくと思います。
84
+
85
+
86
+
87
+ ```C#
88
+
89
+ public class PlayerBullet : MonoBehaviour
90
+
91
+ {
92
+
93
+ // InspectorでPrefab化したBulletを指定する
94
+
95
+ [SerializeField]
96
+
97
+ public GameObject bullet;
98
+
99
+ public Player player; //編集:Playerクラスをそのまま受け取ります
100
+
101
+
102
+
103
+ //private bool isLeft; //編集:1つのメソッドの中でしか使わないので不要です
104
+
105
+
106
+
107
+ // ユーザーがどちらを向いているか
108
+
109
+ void Start()
110
+
111
+ {
112
+
113
+ //isLeft = player.GetComponent<Player>().isLeft; //ここも不要
114
+
115
+ }
116
+
117
+
118
+
119
+ void Update()
120
+
121
+ {
122
+
123
+ // 弾オブジェクトを生成して飛ばす関数を呼び出す
124
+
125
+ ShotAction();
126
+
127
+ }
128
+
129
+
130
+
131
+ void ShotAction()
132
+
133
+ {
134
+
135
+ //offsetの用意はボタンを押した時の処理内に移動します
136
+
137
+
138
+
139
+ if (Input.GetButtonDown("Fire1"))
140
+
141
+ {
142
+
143
+ Vector3 offset = new Vector3(1.0f, 0.0f, 0.0f); //移動してきました
144
+
145
+ bool isLeft = player.isLeft; //ここで受け取り・Playerクラスをそのまま受け取っているのでGetComponentは要りません
146
+
147
+ if (isLeft)
148
+
149
+ {
150
+
151
+ offset.x = -offset.x; //元コードと同じです
152
+
153
+ }
154
+
155
+
156
+
157
+ // Bulletコンポーネントがアタッチしている前提
158
+
159
+ var bulletObj = Instantiate(bullet, transform.position + offset, transform.rotation);
160
+
161
+ bulletObj.GetComponent<Bullet>().SetDirection(isLeft);
162
+
163
+ }
164
+
165
+ }
166
+
167
+ }
168
+
169
+
170
+
171
+ ```