敵の移動で質問があります。

Question

敵の移動パターンで質問があります。

void move_enemy0()
	{
		counter++;
		if(y<100){
		    x=300;
			y++;
	
		}
		if(y>=100){
			x+=1;
		}
		if(x>350){
			x--;
		}

上はEnemyのクラスで宣言している敵の動きです。
最初はまっすぐ進んできて、ある座標で停止し、そのあとインベーダーゲームのように左右に動くようにしたいのですが、上のプログラムのように打ち込むと何故かif(x>350)の条件だけを満たして停止してしまいます。
そのあと、動きを反転させるためにx--を入れたのですが、なぜこの条件だと停止するのでしょうか。

Answer 1

// x = 350

if(y>=100){
    //右に進ませる
    x+=1;// x = 351
}

// x が 350を超えたら
if(x>350) {
    //左に進ませる
    x--; // x = 350
}

追記
Zuishinさんの回答と同じこと書いてました。

Comments

[Zuishin]

貼られたコードに x+=1 とあるのがそれだと思います。

[abs123]

指摘ありがとうございます。

今気が付きましたが、Zuishinさんと同じようなこと書いてる・・・

Answer 2

x と y の初期値はきちんと設定されていますか？
仮に x = 400, y = 400 だとすると最後しか実行されません。

また初期設定がきちんとされていたとしても往復はしないでしょう。x は 351 になった後 351 と 350 の間しか動きません。

きちんと動かすには、まず x = 300, y = 0, dx = 0, dy = 1 と初期設定してください。
移動は x += dx, y += dy とします。
y >= 100 で dx = 1, dy = 0 に変更してください。また dx >= 350 で dx = -1 にしてください。

追記
dx >= 350 は x >= 350 の誤りです。

Comments

[taiyokarasu]

アドバイスをいただき、実際に実行してみたのですが、動きが反転することなく画面外に進んでいってしまいました。入力の仕方が悪いのでしょうか

x +=dx;
y +=dy;

if(y>=100){
dx=1;
dy=0;

if(dx>350){
dx =-1;
}
}

[Zuishin]

失礼しました。dx > 350 は間違いです。x > 350 としてください。

[abs123]

x < 0になった場合に dx = 1にする処理も必要ですね。

[Zuishin]

そうですね。dx, dy は移動する方向と距離を表しますから、どう扱えばいいのか理解できれば好きなように動かせると思います。

[taiyokarasu]

ｘ＞350の条件で350と351の間をいったりきたりしているようです
ブルブルと左右に揺れて、動けなくなってしまっています。

[Zuishin]

dx を x に直すのは一カ所だけですよ。二カ所直しませんでした？
あと、y >= 100 は一度成立してからずっと条件が満たされているので、これではまずいですね。
次のようにやってみてください。

if (y == 100 && dx == 0) {
dx = 1;
dy = 0;
} else if (x >= 350) {
dx = -1;
} else if (x <= 0) {
dx = 1;
}
x += dx;
y += dy;

[Zuishin]

また同じ間違いを繰り返したので修正しました。
上記試してみてください。

[taiyokarasu]

やはり、反転した動きをしてくれません。
それどころか、何故か画面左上から画面右下あたりを真っすぐ進み、画面外に消えてしまいました。

[Zuishin]

えっと、初期設定はされましたか？
また変数はグローバルですか？

[taiyokarasu]

変数がグローバルという意味がわかりませんが、一応

x=300
y=0
dx=0
dy=1

という感じで初期値は設定しています

[Zuishin]

その座標から出ませんか？

[taiyokarasu]

出てきます。もう一度やってみたら、うまくいきました。
ありがとうございます！
もし、何かエラーが起きたらよろしくお願いします！

Answer 3

１回しか実行されてないから。
ループもなにもなくただこのメソッドを実行しただけなら、初期状態で満たされる条件の分のみ実行されて終了します。

Comments

[taiyokarasu]

一つ一つのif文に分けていることがループができていない原因ということでしょうか？