0

我正在尝试将卡片的图像拖放到 javascript 的画布上，但即使 mouseup 事件在 main() 内，它似乎也不起作用。选择卡片后，它会跟随鼠标，但当我松开鼠标时它似乎没有松开。我也知道由于不清除屏幕而重复图像。

function main(){
var ctx = cvs.getContext("2d");
var img = new Image();
img.src = "allcards.png";
var imgX = 75;
var imgY = 75;
draw();

function draw(){
    ctx.drawImage(img,0,0,97,129,imgX,imgY,100,100);    
}

cvs.addEventListener("mouseup", function(ev){
        greaterX = false;
        lessX = false;
        greaterY = false;
        lessY = false;
    }
);

cvs.addEventListener("mousedown", function(ev){
        if(ev.clientX <= (imgX + 97)){
            var greaterX = true;
        }
        if(ev.clientY <= (imgY + 129)){
            var greaterY = true;
        }
        if(ev.clientX >= imgX){
            var lessX = true;
        }
        if(ev.clientY >= imgY){
            var lessY = true;
        }
        if(greaterX == true)
            if(greaterY == true)
                if(lessX == true)
                    if(lessY == true)
                        cvs.addEventListener("mousemove", function(ev){
                            var offsetX = (ev.clientX - imgX);
                            var offsetY = (ev.clientY - imgY);
                            imgX = imgX + offsetX;
                            imgY = imgY + offsetY;
                            draw();
                        });
    });
};
4

1 回答 1

3

greaterX,lessX等都var在你的函数内部定义mousedown，这意味着它们的范围仅限于mousedown函数。

因此，尝试将它们设置回函数false内部是没有用的mouseup。您需要在函数的主要部分声明变量：

function main() {
    var greaterX, lessX, greaterY, lessY;
    var ctx = cvs.getContext("2D");
    //etc...

现在，简单地将greaterX,lessX等设置回 false 是不够的，因为mousemove里面的事件检查器mousedown仍然处于活动状态。当您应用事件侦听器时，它会一直留在那里，直到您将其移除。

因此，下一步是将事件函数分离为它自己的函数（我使用“mouseMoveHandler”作为名称）并使用inside ofmousemove删除事件侦听器。.removeEventListener(type, listener, useCapture)mouseup

mousemove功能：

function mouseMoveHandler(ev) {
    offsetX = (ev.clientX - imgX);
    offsetY = (ev.clientY - imgY);
    imgX = imgX + offsetX;
    imgY = imgY + offsetY;
    draw();
}

功能（mousedown重要部分）：

  if (greaterX === true) { //you need the brackets for multi-line if statements
        if (greaterY === true) {
            if (lessX === true) {
                if (lessY === true) {
                    cvs.addEventListener("mousemove",mouseMoveHandler,false);
                }
            }
        }
    }

最后，mouseup函数：

cvs.addEventListener("mouseup", function(ev) {
    greaterX = false;
    lessX = false;
    greaterY = false;
    lessY = false;
    cvs.removeEventListener('mousemove',mouseMoveHandler,false);
});

这是一个带有解决方案的jsFiddle，但不使用您的图像。

于 2012-08-28T03:25:43.093 回答