2012-05-10 80 views
-1

我遇到了一个我试图制作的游戏的问题。这里是我的代码(非常基本):JavaScript游戏循环不起作用

var canvas = document.createElement("canvas"); 
var gameCanvas = canvas.getContext("2d"); 
canvas.width = 500; 
canvas.height = 500; 
document.body.appendChild(canvas); 

// Global variables 
var game; 
game.fps = 60; 
game._intervalId = setInterval(game.run, 1000/game.fps); 

// Game loop 
game.run = function() { 
    game.draw(); 
}; 

// Functions 
game.draw = function() { 
    // Background image 
    var bgImage = new Image(); 
    bgImage.src = "img/background.png"; 
    // Player image 
    var playerImage = new Image(); 
    playerImage.src = "img/player.png"; 
    // Draw 
    gameCanvas.drawImage(bgImage, 0, 0); 
    gameCanvas.drawImage(playerImage, 10, 10);  
} 

// Game over 
document.getElementById('end').onclick = function stop(){ 
    clearInterval(game._intervalId);  
} 

// Run 
window.onload = game.run(); 

游戏无法正常运行。我做错了什么,还是有我错过的东西?这里是一个页面的链接:http://dl.dropbox.com/u/33213779/Game/demo_me.html

谢谢。

+0

你是什么意思的“不正常运行”? – m90

+0

@ m90如果您查看链接,画布似乎无法加载,或者图像无法加载。我不知道这是游戏循环还是加载画布/图像代码的问题。 – Harry

回答

1

我可以从你写的内容中发现两个问题。

第一个是,你必须初始化你的游戏变量作为一个对象:

var game = {}; // = {} initializes a new Object 
game.fps = 60; 
.... 

的第二个问题是,你是不是预加载的图像源(而不是你加载它们各60次秒......),以便画布无法访问它们,因为它们仍在加载。

相反,你可以使用这样的机制:

var bgImage = new Image(); 
bgImage.src = "img/background.png"; 
bgImage.onload = checkPreload; 

var playerImage = new Image(); 
playerImage.src = "img/player.png"; 
playerImage.onload = checkPreload; 

var loadedPics = 0; 
function checkPreload(){ 
loadedPics++; 
if (loadedPics == 2){ 
buildGame(); //start game 
} 
} 

function buildGame(){ 
//inside this function you should be safe to refer to bgImage and playerImage 
} 

注意,这是非常过于简单化作为图像预加载是一个有很多缺陷的受灾面积。但我想你会明白的。

顺便说一句this是一个关于游戏逻辑和所有东西画布的非常好的教程。

+0

谢谢,我会阅读它! – Harry

1

在Chrome中,游戏会生成一个停止执行的错误。在var游戏之后;游戏是不确定的,那么你做你game.fps =东西,就像你在做undefined.fps =东西,和JavaScript生气。第一个补丁将做var game = {}

我不明白你为什么一次又一次地在一个循环中创建bgImage和播放器。即使这种方法有效,也会产生问题。

var canvas = document.createElement("canvas"); 
    var gameCanvas = canvas.getContext("2d"); 
    canvas.width = 500; 
    canvas.height = 500; 

     canvas.setAttribute("style","border:1px solid red"); 
    document.body.appendChild(canvas); 

    // Global variables 
    var game = {}; 
    game.fps = 60; 


    // Game loop 
    game.run = function() { 
     game.draw(); 
    }; 

    // Resource loader 
    game.loader = function() { 
     // Background image 
     game.bgImage = new Image(); 
     game.bgImage.src = "img/background.png"; 

     // Player image 
     game.playerImage = new Image(); 

     game.playerImage.src = "img/player.png"; 
    };     

    // Functions 
    game.draw = function() { 
     // Draw 
     gameCanvas.drawImage(game.bgImage, 0, 0); 
     gameCanvas.drawImage(game.playerImage, 10, 10);  
    } 
      game.loader(); 
    game._intervalId = setInterval(game.run, 1000/game.fps); 

此版本的代码似乎按照您的意图工作。

+0

等等,你建议我为游戏创建一个函数吗?请稍微清楚一点,谢谢。 – Harry

+0

@哈里,答案很清楚。 '{}'是一个新的空对象。 – kay

+0

是的,我建议创建一个initlization函数来加载资源。 – Tei