setTimeout virker ikke

Plejer den ikke at være inde under window


jeg plejer ikke at kunne få min kode til at køre ved mindre jeg skriver:

window.setTimeout

Plejer den ikke at være inde under window


jeg plejer ikke at kunne få min kode til at køre ved mindre jeg skriver:

Kode 

window.setTimeout

window er at global object alle objekter arver fra, så window. er faktisk ligegyldigt, vil nok sige at du skal droppe din eval sætning og i stedet skrive noget ala:

 window.setTimeout(function()
 {
      marioFall(x);
 },acc);

Ikke testet...

Ok jeg har fiflet lidt
Jeg har slået marioJump og marioFall sammen til een funktion, et hop med både op og ned hvor mario er tilbage til det samme x som han startede ved når funktionen stopper:

var g_intMarioJumpTime;
var g_numMarioVelocity;
var GRAVITY = 10.0; //pixels per second per second
var g_numMarioX; //number describing mario x
var g_intActualMarioX; //integer describing actual marioX
function  marioJump(){
	//initialize jump
	marioAction = 'jump';
	g_intMarioJumpTime = new DateTime().getTime();
	g_numMarioVelocity = -10.0; //initial velocity 10 pixels pr second up
	g_numMarioX = 0.0;
	g_intActualMarioX = 0;
	setTimeout(marioJumpInterrupt,1);
}

function marioJumpInterrupt(){
	var intCurrentTime = new DateTime().getTime();
	var intDeltaTime = intCurrentTime - g_intMarioJumpTime;
	//update velocity
	g_numMarioVelocity += intDeltaTime/1000.0 * GRAVITY;
	var dX = intDeltaTime/1000.0 * g_numMarioVelocity;
	g_numMarioX += dX; //non rounded x position of Mario
	var dXActual = Math.Round(g_numMarioX) - g_intActualMarioX; //rounded move of mario;
	if(dXActual>0){
		if (g_numMarioVelocity > 0) {
			marioAction = 'stand'; //we are falling			
		}
		g_intActualMarioX += dXActual;
		if(g_intActualMarioX < 0){
			dXActual += g_intActualMarioX; //return to x=0
			changeMarioTop(dxActual);
			return; //jump done
		}
		changeMarioTop(dxActual);		
	
	}
        g_intMarioJumpTime = intCurrentTime;
	setTimeout(marioJumpInterrupt,1);	}

Så vidt jeg kunne se satte du marioAction = 'stand' når han nåede toppen.

Koden er ikke testet overhovedet men måske kan du få den til at virke?

Du kan skrue på GRAVITY og den initiellle fart for at få hoppet hhv. hurtigere og højere

Men du brugte alligevel selv det globale objekt der

Men du brugte alligevel selv det globale objekt der

Ja, det er bare en good practice... Men ikke nødvendigt..

Jakob: Jeg kan ikke helt forstå hvordan din kode skulle kunne virke. Men under alle omstendigheder så ved jeg at det ikke er min eval der er noget galt med, for den brugte jeg også i mit andet kode eksempel hvor den virkede fint.

Simon, hvad siger Firefox debugger til din kode? Og jeg mener hele det der system. Hvis det er en fejl må den da næsten brokke sig.

Simon, hvad siger Firefox debugger til din kode? Og jeg mener hele det der system. Hvis det er en fejl må den da næsten brokke sig.

Nope. Hverken Explorer, Firefox eller Chrome melder nogen fejl når jeg kører siden, men det virker ikke.

Det er et Mario-linende spil jeg er ved at lave. Hele min JavaScript kode er her vis det har nogen interesse:

//##########LEVEL FUNKTIONER
//##########LEVEL FUNKTIONER
//##########LEVEL FUNKTIONER

//Funktionen der tegner banen på skærmen
function drawLevel(level) {
	eval('drawLevel' + level + '();');
}

//Funktion der generere CSS'en og positionere samtlige 20 linjer
//Jo længere banen skal være jo større er ROW
function drawLines(rows) {

	//Følgende WHILE køres 20 gange og X er lig med det antal gange den er blevet kørt
	var x = 1;
	while (x <= 20) //Funktionen skal køres 20 gange da der er tyver rækkker i banen
		{
		var name = "line" + x; //Name er navnet på DIV'en. Der er en div for værd række i banen
		document.write('<div id="' + name + '">'); //Skriver DIV NAME ud i HTML
		//Skriver inholdet af DIV NAME ud på skærmen
		var y = 1;
		while (y <= rows) // DIVen skal indeholder ROWS antal kolonner
			{
			document.write('<span id="' + x + '-' + y + '" class="square"><img src="sky.png"></span>');
			y ++;
		}
		
	document.write('</div>'); //Indholdet af diven er lavet og  derefter lukkes den
	
	//DIV'en er nu lavet og lukket, og vi positinere den derfor nu med DOM
	document.getElementById(name).style.position = 'absolute';
	document.getElementById(name).style.top = (x - 1) * 20 + 'px';
	document.getElementById(name).style.left = '0px';
	document.getElementById(name).style.width = rows * 20 + 'px';
	
	x ++; //X forøges og møllen køre igen hvis X stadig er mindre end 20
	}
}

function drawHorizontalLine(row,startColumn,endColumn,fill) //Denne funktion tegner en vandret linje fra venstre mod højre. row er hvilken række linjen skal være i. starColumn og endColumn er hendholdvis hvilken kollone linjen skal starte i, og hvilken den skal slutte i. fill representere navnet på den png fil der skal fyldes ud i den pågældende linje.
	{
	while (startColumn <= endColumn)
		{ //Funktionen køre så længe at starColumn ikke er større end endColumn, for hvis den er det er linjen tegnet færdig.
		var square = row + '-' + startColumn; //square representere det felt der skal ændres
		document.getElementById(square).innerHTML = '<img src="' + fill + '.png">'; //Indholdet af square ID'et bliver = fill.
		startColumn ++; //Start column bliver forøget med en sådan at den næste square i linjen bliver udfyldt ved næste loop
	}
}

function drawLevel1() {
	drawLines(50); //Kalder funktionen drawlines der laver en bane med 50 linjer
	drawHorizontalLine(17,1,50,'gro');
	drawHorizontalLine(18,1,50,'gro');
	drawHorizontalLine(17,7,50,'sky');
	createMario(101,51); //Laver Mario
	
}


//##########MARIO FUNKTIONER
//##########MARIO FUNKTIONER
//##########MARIO FUNKTIONER

var marioAction = 'stand'; //Har værdien 'stand' hvis Mario ikke falder pt. og værdigen 'fall' hvis han er ved at falde, 'jump' hvis han er i spring
var key39 = 'up'; //RIGHT
var key37 = 'up'; //LEFT
var key38 = 'up'; //LEFT

//Denne funktion laver Mario, og placerer ham et sted på skærmen baseret på X og Y
function createMario(x,y)
	{
	document.write('<div id="mario"><img src="mario.png"></div>');
	var mario = document.getElementById("mario");
	mario.style.left = x + 'px';
	mario.style.top = y + 'px';
	mario.style.zIndex = '2';
	marioFall(1);
}
//Funktionen der får mario til at falde af
function marioFall(x) //Funktionen tjækker om Mario er på jorden, og hvis han ikke er det kaldes funktonen der får ham til at falde
	{
	var acc;
	is = isMarioOnGround(); //Hvis mario er på jorden bliver is=yes og hvis ikke bliver is=no
	if (is != 'yes' && marioAction == 'stand') //Hvis mario ikke er på jorden og han ikke allerede er ved at falde
		{
		changeMarioTop(1);
		marioAction = 'fall';
		acc = x / 2;
		x++;
		eval('setTimeout("marioFall(' + x + ')",acc)');
	} else if (is == 'yes' && marioAction == 'fall')
		{
		marioAction = 'stand';
		key38 = 'up';
	}
}

//Funktionen der tjekker om Mario står på jorden, den giver 'yes' hvis han er på jorden og 'no' hvis at han ikke er på jorden
function isMarioOnGround()
	{
	var y = getMarioTop();
	var row = y / 20 + 3; //Dividere Y med 20 for at finde hans placering i forhold til felterne, der alle er 20x20
	if (Math.ceil(row) != Math.floor(row)) //Hvis ROW ikke er et heltal rundes det op, og derved findes den række bunden af mario befinder sig i.
		{
		row = Math.ceil(row);
	}
	var x = getMarioLeft();
	x = x / 20;
	if (Math.ceil(x) == Math.floor(x)) // Hvis X er et heltal rundes det ikke af
		{
		var x1 = x+1;
		if (document.getElementById(row + '-' + x).innerHTML == '<img src="sky.png">' && document.getElementById(row + '-' + x1).innerHTML == '<img src="sky.png">')
			{
			return 'no';
		} else {
			return 'yes';
		}
	} else {
		x = Math.ceil(x);
		var x1 = x+1;
		var x2 = x+2;
		if (document.getElementById(row + '-' + x).innerHTML == '<img src="sky.png">' && document.getElementById(row + '-' + x1).innerHTML == '<img src="sky.png">' && document.getElementById(row + '-' + x2).innerHTML == '<img src="sky.png">')
			{
			return 'no';
		} else {

			return 'yes';
		}
	}
}

function moveMario(e) //Denne funktion aktiveres når der trykkes på en knap
	{
	var key=e.keyCode;
	if (key == '39' && key39 == 'up') //Hvis der er blevet trykket "højre"
		{
		key39 = 'down';
		moveLeft = setInterval('changeMarioLeft(1)',9);
	} else if (key == '37' && key37 == 'up') // Hvis der er blevet trykket venstre
		{
		key37 = 'down';
		moveRight = setInterval('changeMarioLeft(-1)',9);
	} else if (key == '38' && key38 == 'up') // Hvis der er blevet trykket op
		{
		key38 = 'down';
		marioJump(1);
	}
}

function stopMoveMario(e) //Denne funktion aktiveres når der trykkes på en knap
	{
	var key=e.keyCode;
	if (key == '39') //Hvis "højre" blev løftet
		{
		key39 = 'up';
		clearInterval(moveLeft);
	} else if (key == '37') // Hvis venstre blev løftet
		{
		key37 = 'up';
		clearInterval(moveRight);
	}
}

function marioJump(x) //Funktionen der gør at Mario kan hoppe
	{
	marioAction = 'jump';
	var acc;
	if (x <= 40) {
		changeMarioTop(-2);
		acc = x * 2;
		x++;
		eval('setTimeout("marioJump(' + x + ')",acc)');
	} else {
		marioAction = 'stand';
		marioFall(1);
	}
}

function getMarioTop() //Denne funktion returnere top positions værdien af Mario som en int
	{
	mario = document.getElementById("mario")
	var y = mario.style.top;
	y = y.substring(0,y.length-2);
	y = parseInt(y);
	return y;
}
function changeMarioTop(x) //Denne funktion ændre top positions værdien af mario med X
	{
	mario = document.getElementById("mario")
	var y = mario.style.top;
	y = y.substring(0,y.length-2);
	y = parseInt(y);
	mario.style.top = y + x + 'px';
}
function getMarioLeft() //Denne funktion returnere left position værdien af Mario som en int
	{
	mario = document.getElementById("mario")
	var x = mario.style.left;
	x = x.substring(0,x.length-2);
	x = parseInt(x);
	return x;
}
function changeMarioLeft(y) //Denne funktion ændre left positions værdien af mario med X
	{
	mario = document.getElementById("mario")
	var x = mario.style.left;
	x = x.substring(0,x.length-2);
	x = parseInt(x);
	mario.style.left = x + y + 'px';
	if (marioAction != 'jump')
		{
		marioFall(1);
	}
}

Det er aldrig en god idé at bruge eval, den er langsom og giver nemt problemer, som regel kan man klare sig uden.
Din kode:

eval('setTimeout("marioFall(' + x + ')",acc)');

Er temmeligt mystisk og man skal holde tungen meget lige i munden for at regne ud hvad den gør.
x indsætter du værdien af i eval strengen mens acc lader du blive som en variabel, hvis x er 1:

setTimeout("marioFall(1)", acc)

Det er som regel også en dårlig ide at bruge strenge til setTimeout, dat det svarer til en eval, det er som regel bedre at bruge funktion

du kan lave ovenstående på denne måde i stedet:

setTimeout(function(count){
 return function(){
  marioFall(count)
 }
}(x), acc)

humlen er at setTimeout tager en funktion uden parametre. For at få det, laver du en funktion det tager en parameter (count) og returnere en ny funktion (uden parametre) der kalder marioFall med count parametren, denne funktion kalder du så med x. (Med (x) til sidst)
Det svarer lidt til at du får en funktion der har x's aktuelle værdi "inde i maven" så den ikke ændre sig selvom x ændrer sig.

Men spørgsmålet er om det overhovedet er det du vil?

Kan du ikke forklare hvad din setTimeout skal gøre?

Det er aldrig en god idé at bruge eval, den er langsom og giver nemt problemer, som regel kan man klare sig uden.
Din kode:

Kode 

eval('setTimeout("marioFall(' + x + ')",acc)');

Er temmeligt mystisk og man skal holde tungen meget lige i munden for at regne ud hvad den gør.
x indsætter du værdien af i eval strengen mens acc lader du blive som en variabel, hvis x er 1:

Kode 

setTimeout("marioFall(1)", acc)

Det er som regel også en dårlig ide at bruge strenge til setTimeout, dat det svarer til en eval, det er som regel bedre at bruge funktion

du kan lave ovenstående på denne måde i stedet:

Kode 

setTimeout(function(count){
 return function(){
  marioFall(count)
 }
}(x), acc)

humlen er at setTimeout tager en funktion uden parametre. For at få det, laver du en funktion det tager en parameter (count) og returnere en ny funktion (uden parametre) der kalder marioFall med count parametren, denne funktion kalder du så med x. (Med (x) til sidst)
Det svarer lidt til at du får en funktion der har x's aktuelle værdi "inde i maven" så den ikke ændre sig selvom x ændrer sig.

Men spørgsmålet er om det overhovedet er det du vil?

Kan du ikke forklare hvad din setTimeout skal gøre?

Jeg har godt hørt at det skulle være en dårlig idé at bruge eval(), desværre ved jeg ikke helt hvad alternativet skulle være, og jeg forstår ikke helt den kode du skrev.

Men jeg kan godt forklarer hvad setTimeout skal gøre. Funktionen marioFall(x) rykker figuren i mit spil (kaldet mario af mangel på bedre) 1px ned hver gang den bliver kaldet, hvis figuren vel og mærke ikke står på jorden. Pointen er at når funktionen kaldes første gang, altså når figuren begynder at falde, bliver den kaldet med 1, så X er lig med 1. Derefter bliver X forøget med 1 hver gang marioFall() kaldes, det gør at variablen acc (en forkortelse af acceleration) hele tiden bliver mindre, da den er lige med 2 / x. Det medføre at tidsrummet mellem hver gang marioFall() kaldes, hele tiden bliver mindre, og derfor falder figuren hurtigere og hurtigere. Meningen er altså at hastigheden figuren falder med, accelerer under faldet.