Javascriptでif文が機能しない

Question

前提・実現したいこと

Zキーを押すと自機から弾が発射されるようにしたい。

発生している問題

Zキーを押すと自機から弾が発射されるようにするためにキーが入力されたら弾が発射されるようにしようとしましたが、Zキーを押しても反応がありませんでした。

該当のソースコード

var app_height = 600;
var app_width = 420;
tm.main(function () {
	app = tm.display.CanvasApp("#app");	//appという名前のcanvasに描画する
	app.fps = 60;						//fpsを60にする。
	app.resize(app_width, app_height);	//解像度をapp_width,app_widthにし、ウィンドウいっぱいにする。
	app.replaceScene(MainScene());
	app.run();
});

tm.define("MainScene", {
	superClass: "tm.app.Scene",
	init: function () {
		this.superInit();
		

		{var gradation = tm.graphics.LinearGradient(0, 0, 500, 500)		//背景(グラデーション)
		.addColorStopList(Array.range(0, 20).map(function(i) {
			return {
				offset: i / 20,
				color: "hsl({0}, 80%, 80%)".format(360 * i / 20),
			};
		}))
		.toStyle();
		tm.display.RectangleShape(app_width*2, app_height*2, {
			fillStyle: gradation
		}).setPosition(200, 200).addChildTo(this);}


		this.fromJSON({							//出現キャラ
			children: {
				enemy: {
					type: "tm.display.CircleShape",
					init: [50, 50],
					x: app_width * 0.5, y: app_height * 0.1	//敵の初期位置
				},
				poyoenemy:{
					type: "tm.display.HeartShape",
					init: [20,20],
					x:app_width , y:app_height *0.3			//poyo敵
				},
				player: {
					type: "Player",
					x: app_width * 0.5, y: app_height * 0.9	//プレイヤーの初期位置
				}
			}
		});


		var params = {
			target: this.player,	//ターゲットをプレイヤーに設定
			createNewBullet: function (runner,attr) {	//弾を生成
				var bullet = tm.bulletml.Bullet(runner).addChildTo(this);
				bullet.boundingType = "circle";
				bullet.radius = 2;
				bullet.onenterframe = function () {

					if (this.x < 0 || app_width < this.x || this.y < 0 || app_height < this.y) {
						this.remove();
					}
					if (this.isHitElement(Player.SINGLETON)) {
						this.remove();
					}
				};
			}.bind(this)

		};

		var pparams = {
			target: this.enemy,
			createNewBullet: function(runner) {
				var bullet = tm.bulletml.Bullets(runner).addChildTo(this);
				bullet.boundingType = "circle";
				bullet.radius = 2;
				bullet.onenterframe = function (){
					if (this.x < 0 || app_width < this.x || this.y < 0 || app_height < this.y) {
						this.remove();
					}
				};
			}.bind(this)
		};

		tm.define("MyBullet",{
			superClass:"tm.bulletml.Bullets",
			init: function(runner,attr){
				this.superInit(runner);
				this.color = attr.color;
				this.removeChildren();
				//colorプロパティに応じて初期化
				switch(attr,color){
					case "blue":
					tm.display.Splite("bullet",20,20)
							.setFrameIndex(0)
							.addChildTo(this);
						break;
					case "red":
						tm.display.Splite("bullet",20,20)
							.setFrameIndex(1)
							.setScale(1.5,0.5)
							.addChildTo(this);
						break;
				}

				this.beforeX = this.x;
				this.beforeY = this.y;
			},
			onenterframe:function(){
				if(this.color ==="blue"){
					this.rotation +=10;
				}else if(this.color ==="red"){
					this.rotation = Math.atan2(this.y -this.beforeY,this.x - this.beforeX)*Math.RAD_TO_DEG;
				}

				this.beforeX = this.x;
				this.beforeY = this.y;
			}
		})

		bulletml.dsl();
		var DANMAKU_SAKURETU = new bulletml.Root({
			top: action([
				repeat(100, [		//100回fireとwaitを繰り返す。
					fire(bullet(actionRef("explosion"))),
					//fire(方向,弾)
					//方向1.aim(自機狙い)2.absolute(絶対方向)3.sequence(直前に打った弾と相対方向)単位は°(度)
					wait(200)//wait(x) xフレーム待機。自然数のみ

				])
			]),
			explosion: action([
				wait(100),
				changeSpeed(speed(0), 20),
				wait(20),
				repeat(8, [
					fire(direction("$rand *360"), bullet({color:"red"}))
				]),
				vanish()
			])
		});
		var DANMAKU_RENSHA = new bulletml.Root({
			top: action([
				repeat(999, [
					fire(speed(2.1), bullet({color:"blue"})),
					wait("$rand *20")
				])
			]),
		});
		var DANMAKU_KAITEN = new bulletml.Root({
			top: action([
				repeat(990, [
					fire(direction(23, "sequence"), bullet()),
					wait(1)
				])
			])
		});
		var DANMAKU_KOTEI = new bulletml.Root({
			top: action([
				repeat(999, [
					fire(direction(180, "absolute"), speed(2.1), bullet()),
					wait("$rand * 20"),
				]),
			]),
		});
		var DANMAKU_NWAY = new bulletml.Root({
			top: action([
				repeat(999, [
					fire(direction(-30), speed(1.5), bullet()),
					repeat(4, [
						fire(direction(15, "sequence"), speed(0, "sequence"), bullet()),
					]),
					wait(120),
				]),
			]),
		});
		var DANMAKU_ALL = new bulletml.Root({
			top: action([
				repeat(999, [
					fire(speed(1.5), bullet()),
					repeat(35, [
						fire(direction(10, "sequence"), speed(0, "sequence"), bullet()),
					]),
					wait(120),
				]),
			]),
		});
		var DANMAKU_WHIP = new bulletml.Root({
			top: action([
				repeat(999, [
					fire(speed(1.5), bullet()),
					repeat(10, [
						wait(5),
						fire(direction(0, "sequence"), speed(0.5, "sequence"), bullet()),
					]),
					wait(120),
				]),
			]),
		});
		var DANMAKU_WINDER = new bulletml.Root({
			top: action([
				repeat(10000, [
					bindVar("d", "Math.sin($loop.index * 0.03) * 20"),
					fire(direction("179 + $d", "absolute"), speed(5.0), bullet(), offsetX(-40)),
					fire(direction("181 + $d", "absolute"), speed(5.0), bullet(), offsetX(40)),
					wait(5),
				]),
			]),
		});

		var DANMAKU0 = new bulletml.Root({
			top: action([
				repeat(999, [
					fire(speed(2.1), bullet({ color: "red" })),
					wait("$rand * 20"),
					fire(direction(180, "absolute"), speed(2.1), bullet({ color: "blue" })),
					wait("$rand * 20"),
				]),
			]),
		});

		var PLAYER_BULLET = new bulletml.Root({
			top: action([
				repeat(10000,[
				fire(direction(360,"absolute"),speed(20,0),bullet()),
				wait(5),
			]),
			]),
		});


		{
			{//////////////問題の場所はここです///////////////////////////
			var kb =app.keyboard;
			if(kb.getKey("Z")){
			this.player.startDanmaku(PLAYER_BULLET,pparams);
			}///////////////////////////////////////////////////////////
		this.enemy.startDanmaku(DANMAKU0, params);
		}

		}
		//	上の定義を敵が打ち始める。
	}
});

tm.define("Player", {
	superClass: "tm.display.TriangleShape",

	init: function () {
		this.superInit(25, 25);			//プレイヤーのサイズ
		this.boundingType = "circle";	//当たり判定の形
		this.radius = 1.5;				//当たり判定のサイズ
		Player.SINGLETON = this;
	},

	update: function (app) {			//キー入力監視
		var X =4;
		var kb = app.keyboard;
		if(kb.getKey("shift")){			//shiftで低速移動
			X -= 3;
		}
		if (this.x < 1) this.x =1;
		if (app_width-2 < this.x)this.x=app_width-1;
		if (this.y < 1 ) this.y =1;
		if (app_height-1 < this.y) this.y=app_height-1;

		this.position.add(kb.getKeyDirection().mul(X));	
	}
});
Player.SINGLETON = null;

試したこと

Zキーを押すと弾が発射されるのではなく、Zキーが押されていなかったら弾が発射されるようにすると、Zキーを押されていないときも押されているときも弾が発射され続けました。

補足情報（FW/ツールのバージョンなど）

tmlib.js
tmlib.bulletml.js
bulletml.js

Comments

[y_waiwai]

どこのif文のはなしなんでしょうか

[nikuman1251]

tm.define("MainScene"の一番下です

[Udomomo]

if文がずっとelseになるというのは具体的に何が起こっているのでしょうか。

[nikuman1251]

Zキーを押しても反応がありませんでした。試しにelseに処理を書いてみたところ、その処理がずっと行われました。

[mather]

kb.getKey("tab") とあるのに「Zキーを押しても反応がありませんでした」とは…？

[nikuman1251]

すみません。ソースコードがまちがえていました。正しくは、kb.getKey("Z")です。

[mather]

実際に動かないときのソースコードを提示しましょう。「Zキーを押しても反応がありませんでした。」などの情報は質問に追記してください。ifが動かない、といきなり決めつけるのではなく「どんな処理をしようとして、どんな操作を行ったが、〇〇という期待した結果にならず、〇〇という結果になってしまう」と具体的に書いてください。

[think49]

tmの定義コードが見あたりませんが、質問文に貼って頂いたコードのみで動作しますか。

[nikuman1251]

tmlib.js、tmlib.bulletml.js、bulletml.jsを使用しているため、このコードだけでは動作しません。

Answer 1

どこかのコードをまるっとコピペした感じですかね。
「tm」が指すライブラリは存じませんが、kb.getKeyを書く場所がそもそもおかしいと感じます。
今書かれている場所は、「init」のブロックですので、初期化処理なのでしょう。
そこでコード入力を受け付けようとしても、まだ起動していない状態なのでキーは反応しないと思います。
(なので、yambejpさんの提案で書かれたconsole.logはfalseになった)

ですから、key.getKeyを書くとしたら、コード下部にある「Player」の

update: function (app) {            //キー入力監視

の部分じゃないでしょうか。
Shiftキーを押した際の動作を書かれているようですので、それを真似たらいいと思います。

正直、この問題一つですむものではないと思いますので、まずは自分でコードを理解される学習の期間を置かれたほうが良いかと思います。

Answer 2

勉強不足により、まだまだ基礎的な知識が足りていないため、勉強をしてから、再度挑戦しようと思います。

Answer 3

if(kb.getKey("Z")){

の前行で

console.log(kb.getKey("Z"));

などでチェックすることですかね
あとは「Z」を「z」にして再度チェックしてみるとか

Comments

[nikuman1251]

console.log(kb.getKey("Z"));
でチェックしてみると、1度だけfalseが出力されました。その後はいくらZを押してもコンソールに出力されることはありませんでした。

[yambejp]

では、単にgetKeyの使い方が違うのでしょう。