使用for循环将onclick方法附加到多个画布

时间:2014-07-28 20:29:10

标签: javascript html onclick

我需要在a中的多个画布上添加一个onclick函数。我使用for循环遍历所有画布,然后使用attachEvent添加onclick事件。

我已经尝试使用eventlistener或直接添加onclick事件但似乎没有任何效果请告诉我,如果你能看到我的错误......

这是我的JavaScript代码:

        //here's the loop where the canvas get their width, height and onclick 
        //event.
    function canvasSize() {
        var canvas=document.getElementsByTagName('canvas');
        var ctx;
        for (var i=0;i<canvas.length;i++) {
            canvas[i].width="50";
            canvas[i].height="50";
            ctx=canvas[i].getContext('2d');
            ctx.fillRect(0,0,50,50);
            ctx.fillStyle="blue";
            canvas[i].attachEvent("click", onClick(this,"voitures"));
        }
    };


         //onclick() function
    function onClick(elem,theme) {
        var images=preloadImages(theme);
        var canvas=getElementsByTagName("canvas");
        for (var i=0;i<canvas.length;i++) {
            if (canvas[i]==elem) {
                elem.drawImage(images[0],0,0)
            }
        }
    };

        //function where I preload every images
    function preloadImages(theme) {
        var tab=[];
        var imageObj= new Image();
        if (theme=="cars") {
            tab[0]="0.jpg"
            tab[1]="1.jpg"
            tab[2]="2.jpg"
            tab[3]="3.jpg"
            tab[4]="4.jpg"
            tab[5]="5.jpg"
            tab[6]="6.jpg"
            tab[7]="7.jpg"
            tab[8]="8.jpg"
            tab[9]="9.jpg"
            tab[10]="10.jpg"
            tab[11]="11.jpg"
            tab[12]="12.jpg"
        //...
        }

        for (var i=0;i<32;i++) {
            imageObj.src=tab[i];
            //imageObj.onload=alert("img "+i+" is loaded");
        }

        return tab;
    }
}

在添加onclick函数之前代码崩溃。希望有人能帮助我找到我的错误。

1 个答案:

答案 0 :(得分:0)

一:attachEvent仅在IE中可用。

二:不要命名函数onClick这是一个保留global event handler

因此,为了修复您的for循环,我会将您的最后一行更改为:

for (var i=0;i<canvas.length;i++) {
    var self = this;
    if(window.attachEvent){
        canvas[i].attachEvent("click", function(){
            newNameForOnClickFunction(self,  "voitures");
        }, false);
    }else{
         canvas[i].addEventListener("click", function(){
            newNameForOnClickFunction(self,  "voitures");
         }, false);
    }
}

我打赌你想要将canvis传递给你的下一个功能而不是对你的canvasSize功能的尊重。如果是这种情况,那么您应该通过canvas[i]而不是this(或上面示例中的self