thisの参照位置を変更したい

Question

###前提・実現したいこと
javascriptで画像を配置し、アニメーションを行えるような関数を制作しています。複数回インスタンス生成したいので、「this.変数名」を使って様々な値を設定したいのですが、参照している場所が思い通りにいきません。
実現したいことは、最初に座標の値を保持する変数を０で初期化したのち、描画の関数の引数で座標を指定し、それを上の変数で記録です。

###該当のソースコード

function Character(URL) {
	this.x = 0;
	this.y = 0;
	this.width = 0;
	this.height = 0;
	//画像配置
	var pict = new Image();
	pict.src = URL;

　　//描画（引数は、canvas名,x座標,y座標,幅,高さ）
	this.draw = function (ctx,　x, y, width, height) {
　　　　　//Charactor.xの部分を参照したい
		this.x = x;
		this.y = y;
		this.width = width;
		this.height = height;

　　　　　//URLが入っていたら画像をキャンバスに書き込む
		if (pict.src != "") {
			pict.onload = function () {
				console.log("" + this.x + "" + this.y + "");
				ctx.save();
				ctx.drawImage(pict, this.x, this.y,);
				ctx.restore();
			}
			
		}
	}
}

Answer 1

イベントハンドラ内でthisを参照しているので, Characterオブジェクトだと思っていたものが実は変数pictが参照しているImageオブジェクトになっています. ですから, xとyの値が取れないのです.

function Character(URL) {
    this.x = 0;
    this.y = 0;
    this.width = 0;
    this.height = 0;
    //画像配置
    var pict = new Image();
    pict.src = URL;

　　//描画（引数は、canvas名,x座標,y座標,幅,高さ）
    this.draw = function (ctx,　x, y, width, height) {
　　　　　//Charactor.xの部分を参照したい
        this.x = x;
        this.y = y;
        this.width = width;
        this.height = height;

　　　　　//URLが入っていたら画像をキャンバスに書き込む
        if (pict.src != "") {
            pict.onload = function () {
                console.log("" + this.x + "" + this.y + "");//ここのthisはpict！
                ctx.save();
                ctx.drawImage(pict, this.x, this.y,);
                ctx.restore();
            }

        }
    }
}

また, コードの内容を見るにloadイベントやsave/restoreメソッドを利用する必要は無いものと思われます.

function Character(URL) {
    this.x = 0;
    this.y = 0;
    this.width = 0;
    this.height = 0;
    //画像配置
    var pict = new Image();
    pict.src = URL;

　　//描画（引数は、canvas名,x座標,y座標,幅,高さ）
    this.draw = function (ctx,　x, y, width, height) {
　　　　　//Charactor.xの部分を参照したい
        this.x = x;
        this.y = y;
        this.width = width;
        this.height = height;

　　　　　//画像の読み込みが完了していたらpictをcanvasに描く
        if(pict.complete){
            ctx.drawImage(pict, this.x, this.y);
        }
    }
}

「画像の読み込みが完了していたら」はcompleteプロパティで判ります. なお画像の読み込み失敗(画像が見つからない等)時にもcompleteがtrueとなるので, できれば, errorイベントを使って画像が正しく読み込めなかった場合の代替処理についても考慮したほうが良いでしょう.

Comments

[meron-pan]

コードを修正したところ無事、動かすことができました。ありがとうございます。

Answer 2

こういうことですよね？
普通に指定通りにうごきませんか？

function func_a(){
  this.x=0;
  this.func_b=function(x){
    this.x=x;
  }
}
var myObj=new func_a();
myObj.func_b(1);
console.log(myObj.x);

Comments

[meron-pan]

画像が配置されている場所を２１行目のconsole.log("" + this.x + "" + this.y + "");で確認しているのですが、結果は００でさらに画像も左上から一ミリも動いていないので、指定通りには理論上動くはずなんですけど、なんかダメです。

[yambejp]

drawメソッドの中でpict.onload を処理していますが
drawは実行とpictのonloadに整合性はあるのでしょうか？

[meron-pan]

座標をURLと一緒に引数を指定したくなかったのと、onloadは「画像の読み込みができていたら」という認識で使っています。そのため、画像の読み込みの後に描画をしたいのでそこに格納しています。

Answer 3

javascriptでは「this」の使用箇所によって意味が変わるので、このように修正すれば行けるかと。

  var self = this;
  
  //描画（引数は、canvas名,x座標,y座標,幅,高さ）
  this.draw = function (ctx,　x, y, width, height) {
  //Charactor.xの部分を参照したい
    self.x = x;
    self.y = y;
    self.width = width;
    self.height = height;

    //URLが入っていたら画像をキャンバスに書き込む
    if (pict.src != "") {
      pict.onload = function () {
        console.log("" + self.x + "" + self.y + "");
        ctx.save();
        ctx.drawImage(pict, self.x, self.y,);
        ctx.restore();
      }

    }
}

Comments

[meron-pan]

その場合、drawメソッドで指定した引数を上で用意した変数に入れることは可能なのでしょうか？

[defghi1977]

通常(グローバル)変数selfはスクリプトの実行時コンテキスト(例えばwindow)を参照するので, 別の名称を使ったほうが良いと思います. (スコープで分離されていれば別ですが)