2017-03-03 80 views
-1

代碼說明:Car對象會根據鼠標位置定期更新其位置和方向。我是requestAnimationFrame函數,可以更平滑地渲染幀。我的程序有什麼問題 - 短程序

問題: 當我在繪圖函數中時,汽車對象的x和y座標是NAN。 汽車不顯示在畫布上。

代碼:

<!DOCTYPE html> 
<html> 
<head> 
<style type="text/css"> 
body { 
    overflow:hidden; 
} 
</style> 

</head> 
<body> 

<canvas id="myCanvas"> 
</canvas> 

<script> 

// requestAnimationFrame initialization with cross-browser compatibility 
(function(){ 
    var requestAnimationFrame = window.requestAnimationFrame || 
           window.mozRequestAnimationFrame || 
           window.webkitRequestAnimationFrame || 
           window.msRequestAnimationFrame; 
    window.requestAnimationFrame = requestAnimationFrame; 
})(); 



// canvas and context objects 
var myCanvas = document.getElementById("myCanvas"); 
myCanvas.width = window.innerWidth; 
myCanvas.height = window.innerHeight; 
var ctx = myCanvas.getContext("2d"); 
myCanvas.addEventListener('mousemove', updateMousePos, false); 


//mouse position coordinates 
var mousex; 
var mousey; 

//constructor for a Car object with methods to update position, orientation, and draw the car 
function Car(x, y, orientation, id, type) { 
    //x and y are the cooredinates of the center of a car object 
    this.x = x; 
    this.y = y; 
    this.type = type; 
    this.speed = 5; //default speed = 5 
    this.isAlive = 1; 
    this.stillExists = 1; 
    this.id = id; 
    this.orientation = orientation; 
    this.img = new Image(); 
    this.img.source = 'car1.png'; 
    this.img.width = 200; 
    this.img.height = 100; 

    //this method computes a new positin and orientation of our car. 
    this.updatePosAndOrien = function(){ 

     //caclcuate car orientation using mousex and mousey and x y position of our car using atan2 
     var targetX = mousex - this.x; 
     var targetY = mousey - this.y; 
     var trgtOrien = Math.atan2(targetY, targetX); 
     this.orientation = trgtOrien; 

     //calculate new positon of car using speed and current location of car 
     var dx = mousex -this.x; 
     var dy = mousey - this.y; 
     //distance between mouse and car 
     var distance = Math.sqrt(dx*dx + dy*dy); 

     //Now we compute xspeed and yspeed of car - which are displacements along x and y axis 
     var factor = distance/this.speed; 
     var xspeed = dx/factor; 
     var yspeed = dy/factor; 

     //set new positon of car 
     this.x = this.x + xspeed; 
     this.y = this.y+ yspeed; 
    }; 

    //draw method that draws the car on canvas 
    this.draw = function() { 
     this.img.x = this.x; 
     this.img.y = this.y; 
     this.img.orientation = this.orientation; 

     this.img.onload = function() { 

      ctx.save();//save context 
      //translate context origin to center of image 
      ctx.translate(Math.round(this.x), Math.round(this.y)); 
      ctx.rotate(this.orientation); //rotate context 
      ctx.drawImage(img, -(this.width/2), -(this.height/2), 
       this.width, this.height);//draw img 
      ctx.restore(); //restore context 
     } 
    }; 
} 

/*this function update mouse position upon mouse movement*/ 
function updateMousePos(evt) { 
    var rect = myCanvas.getBoundingClientRect(); 
    mousex = evt.clientX - rect.left; 
    mousey = evt.clientY - rect.top; 
    //log mouse position 
    console.log("mouse postion: "+mousex+", "+mousey); 
} 

//defining car and starting the rendering 
var ourCar = new Car(300, 400, 2, 111, 1); 
console.log("car: "+ourCar.x+", "+ourCar.y); 


/*This function draws everything using requestFrameAnimation(). */ 
function drawIt() { 
    // Use the identity matrix while clearing the canvas 
    ctx.setTransform(1, 0, 0, 1, 0, 0); 
    ctx.clearRect(0, 0, myCanvas.width, myCanvas.height); 

    //update orientation of player 
    ourCar.updatePosAndOrien(); 
    console.log("car in drawIt: "+ourCar.x+", "+ourCar.y); //prints NAN for both 

    //Draw car 
    console.log("drawing the car"); 
    ourCar.draw(); 

    requestAnimationFrame(drawIt); 
} 

//start rendering 
requestAnimationFrame(drawIt); 


</script> 
</body> 
</html> 
+0

在'ctx.drawImage(IMG, - (this.width/2), - (this.height/2),this.width,this.height);'你使用未定義的'img'變量而不是'this.img'。 – Xufox

+0

瘋狂的猜測:'mousex'和'mousey'是'undefined'這將導致'NaN's遍佈'updatePosAndOrien()' – Thomas

回答

1

有幾件事情會在這裏的...

首先,requestAnimationFrame(drawIt)運行在頁面加載。如果鼠標還沒有移動,那麼mousex和mousey是NaN。此外,drawIt會繼續調用它自己,所以如果你看看控制檯,它只會重複NaN。我不確定你的目標是什麼,但是你可能想要檢查ourCar.x和ourCar.y的有效值,如果沒有任何設置,請返回。

至於圖片,首先像xufox說的,你使用的是img而不是this.img。但是,由於此時您處於函數內部,因此您不想使用它,因爲它指的是實際的img標記本身。您應該在該功能之外設置一個變量並將其設置爲與此相同。

其次,爲了讓img.onload觸發,你需要設置圖像的src屬性。

JS小提琴和下面的代碼與評論和控制檯消息來說明,以及爲了使它工作的更新,您可能需要重構您的需求。

https://jsfiddle.net/2dd4rv9u/

// canvas and context objects 
var myCanvas = document.getElementById("myCanvas"); 
myCanvas.width = window.innerWidth; 
myCanvas.height = window.innerHeight; 
var ctx = myCanvas.getContext("2d"); 
myCanvas.addEventListener('mousemove', updateMousePos, false); 

//mouse position coordinates 
var mousex; 
var mousey; 

//constructor for a Car object with methods to update position, orientation, and draw the car 
function Car(x, y, orientation, id, type) { 
    //x and y are the cooredinates of the center of a car object 
    this.x = x; 
    this.y = y; 
    this.type = type; 
    this.speed = 5; //default speed = 5 
    this.isAlive = 1; 
    this.stillExists = 1; 
    this.id = id; 
    this.orientation = orientation; 
    this.img = new Image(); 
    this.img.source = 'http://www.iconsdb.com/icons/preview/black/car-xxl.png'; 
    this.img.width = 200; 
    this.img.height = 100; 

    //this method computes a new positin and orientation of our car. 
    this.updatePosAndOrien = function(){ 

     //caclcuate car orientation using mousex and mousey and x y position of our car using atan2 
     var targetX = mousex - this.x; 
     var targetY = mousey - this.y; 
     var trgtOrien = Math.atan2(targetY, targetX); 
     this.orientation = trgtOrien; 

     //calculate new positon of car using speed and current location of car 
     var dx = mousex -this.x; 
     var dy = mousey - this.y; 
     //distance between mouse and car 
     var distance = Math.sqrt(dx*dx + dy*dy); 

     //Now we compute xspeed and yspeed of car - which are displacements along x and y axis 
     var factor = distance/this.speed; 
     var xspeed = dx/factor; 
     var yspeed = dy/factor; 

     //set new positon of car 
     this.x = this.x + xspeed; 
     this.y = this.y+ yspeed; 
    }; 

    //draw method that draws the car on canvas 
    this.draw = function() { 
     this.img.x = this.x; 
     this.img.y = this.y; 
     this.img.orientation = this.orientation; 
     var self = this; 

     this.img.onload = function() { 
      console.log('DRAWING'); 
      console.log(this); 
      console.log(this.img); 
      console.log(self.img.x); 

      ctx.save();//save context 
      //translate context origin to center of image 
      ctx.translate(Math.round(self.x), Math.round(self.y)); 
      ctx.rotate(self.orientation); //rotate context 

      // I set these to 0 because I didn't check the math and wanted to show 
      // that the car would draw 
      ctx.drawImage(self.img, 0, 0);//draw img 
      ctx.restore(); //restore context 
     } 

     this.img.src = this.img.source; 
    }; 
} 

/*this function update mouse position upon mouse movement*/ 
function updateMousePos(evt) { 
    var rect = myCanvas.getBoundingClientRect(); 
    mousex = evt.clientX - rect.left; 
    mousey = evt.clientY - rect.top; 
    //log mouse position 
    console.log("mouse postion: "+mousex+", "+mousey); 
} 

//defining car and starting the rendering 
var ourCar = new Car(300, 400, 2, 111, 1); 
console.log("car: "+ourCar.x+", "+ourCar.y); 


/*This function draws everything using requestFrameAnimation(). */ 
function drawIt() { 
    if (isNaN(ourCar.x)) return; 

    // Use the identity matrix while clearing the canvas 
    ctx.setTransform(1, 0, 0, 1, 0, 0); 
    ctx.clearRect(0, 0, myCanvas.width, myCanvas.height); 

    //update orientation of player 
    ourCar.updatePosAndOrien(); 
    console.log("car in drawIt: "+ourCar.x+", "+ourCar.y); //prints NAN for both 

    //Draw car 
    console.log("drawing the car"); 
    ourCar.draw(); 

    requestAnimationFrame(drawIt); 
} 

//start rendering 
requestAnimationFrame(drawIt); 
+0

感謝您的幫助。 –