2012-01-01 23 views
2

単純な描画アプリケーションがあります。問題は座標(redraw関数)です:マウスでなければなりませんが、2倍のマウスに近いです。コードの問題は何ですか?キャンバスに描画するときのオフセット

<html> 
<head> 
    <style type="text/css" media="screen"> 
body{ 
    background-color: green; 
} 

#workspace{ 
    width: 700px; 
    height:420px; 
    margin: 40px auto 20px auto; 
    border: black dashed 1px; 
} 

#canvas{ 
    background: white; 
    width: 100%; 
    height: 100%; 
} 
    </style> 
<script type="text/javascript" src="jquery-1.7.1.js"></script> 
<script type="text/javascript"> 
$(document).ready(
    function() { 
     var context = document.getElementById('canvas').getContext("2d"); 
     var clickX = new Array(); 
     var clickY = new Array(); 
     var clickDrag = new Array(); 
     var paint; 

     $('#canvas').mousedown(function(e){ 
      var mouseX = e.pageX - this.offsetLeft; 
      var mouseY = e.pageY - this.offsetTop; 
      paint = true; 
      addClick(mouseX,mouseY, false); 
      redraw(); 
     }); 

     $('#canvas').mousemove(function(e){ 
      if(paint){ 
       addClick(e.pageX - this.offsetLeft, e.pageY - this.offsetTop, true); 
       redraw(); 
      } 
     }); 

     $('#canvas').mouseup(function(e){ 
      paint = false; 
     }); 

     $('#canvas').mouseleave(function(e){ 
      paint = false; 
     }); 

     function addClick(x, y, dragging) 
     { 
      clickX.push(x); 
      clickY.push(y); 
      clickDrag.push(dragging); 
     } 

     function redraw(){ 
      canvas.width = canvas.width; // Clears the canvas 

      context.strokeStyle = "#df4b26"; 
      context.lineJoin = "round"; 
      context.lineWidth = 5; 

      for(var i=1; i < clickX.length; i++) 
      { 
       context.beginPath(); 
       if(clickDrag[i] && i){ 
        context.moveTo(clickX[i-1], clickY[i-1]); 
       }else{ 
        context.moveTo(clickX[i], clickY[i]); 
       } 
       context.lineTo(clickX[i], clickY[i]); 
       context.closePath(); 
       context.stroke(); 
      } 

      console.log(clickX[clickX.length-1]+" "+clickY[clickX.length-1]); 
     } 
    }); 
</script> 
</head> 
<body> 
<div id="workspace"> 
<canvas id="canvas"/> 
</div> 
</body> 
</html> 

答えて

11

CSSでキャンバスの幅/高さを設定しないでください。解像度を上げるのではなく、キャンバスを伸ばします。これは、座標は正しいものの、視覚的にどこかで終わることを意味します。

たとえば、100のx座標は、視覚的にx座標が200(またはそれ以外の場合は、少なくとも伸びているので100より大きい)になります。

代わりに、使用:

<canvas id="canvas" width="700" height="420" /> 

とCSSでwidth: 100%を削除します。

http://jsfiddle.net/euXJC/1/

関連する問題