Mal wieder ein JS-Fehler^^

Vollkornbrot · 21. September 2009

Hab mal wieder bisschen gecodet, aber das ist ein fehler drin^^
Soll so ne Art Snake werden. Is nich sonderlich schön gecodet, wolltes einfach mal zum laufen bringen:
place_food funzt nicht, Firefox meint mover_con wär null.

JavaScript

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN"
     "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml" dir="ltr" xml:lang="de">
	<head>
			<meta http-equiv="Content-Type" content="text/html; charset=ISO-8859-1" />
			<meta http-equiv="Content-Script-Type" content="text/javascript" />
			<meta http-equiv="Content-Style-Type" content="text/css" />
		<style type="text/css">
			*
			{
				border:0px;
				margin:0px;
				padding:0px;
			}

			#mover
			{
				position:absolute;
				height:20px;
				width:20px;
			}

			#mover_con
			{
				height:400px;
				width:400px;
				background-image: url(background.png);
			}
		</style>

		<script type="text/javascript">
		left = true;
		top = true;
		bottom = true;
		right = true;

		document.onkeydown = key_press;

		function rand (min, max) {
			return Math.floor(Math.random() * (max - min + 1)) + min;
		}


		
		function init (){
			mover = document.getElementById("mover");
			mover.style.left = "100px";
			mover.style.top = "40px";
		}
		function key_press(events)
		{
			if(!events){
			events = window.event;
			}
				if(events.which){
					key(events.which);
				}
				else if(events.keyCode){
					key(events.keyCode);
				}

		}


		function key(code)
		{
			if(code==37 && left){
				move("left");
			}
			if(code==38 && top){
				move("top");
			}
			if(code==39 && right){
				move("right");
			}
			if(code==40 && bottom){
				move("bottom");
			}
		}

		function move(where)
		{
			mover = document.getElementById("mover");
			if(where == "left"){
				mover_left = mover.style.left;				
				dummy = mover_left.split("px");				
				left = dummy[0]-10;
				if(left>= "0"){
					mover.style.left = left+"px";
				} else {
					mover.style.left = mover.style.left;
				}
			}
			if(where == "right"){
				mover_left = mover.style.left;
				dummy = mover_left.split("px");
				dummy[0] = parseInt(dummy[0]);
				left = (dummy[0]+10);
				if(left<= "380"){
					mover.style.left = left+"px";
				} else {
					mover.style.left = mover.style.left;
				}
			}
			if(where == "top"){
				mover_top = mover.style.top;				
				dummy = mover_top.split("px");				
				top = dummy[0]-10;
				if(top>= "0"){
					mover.style.top = top+"px";
				} else {
					mover.style.top = mover.style.top;
				}
			}
			if(where == "bottom"){
				mover_top = mover.style.top;				
				dummy = mover_top.split("px");
				dummy[0] = parseInt(dummy[0]);
				top = (dummy[0]+10);
				if(top<= "380"){
					mover.style.top = top+"px";
				} else {
					mover.style.top = mover.style.top;
				}
			}
		}

		function place_food(){
			food = document.createElement("img");
			food.src = "mover.png";
			food.style.position = "absolute";
			top = rand(0, 400)+"px";
			left = rand(0, 400)+"px";
			food.style.top = top;
			food.style.left = left;
			mover_con = document.getElementById("mover_con");
			mover_con.appendChild(food);
		}

 		place_food();
		</script>


	</head>
	<body onload="init()">
		<div id="mover_con">
			<img src="mover.png" id="mover" />
		</div>







	</body>
</html>

Alles anzeigen

Achja, hier zum ausprobieren. xD

**TimWolla** · 21. September 2009

Zitat von Vollkornbrot

mover_con = document.getElementById("mover_con"); mover_con.appendChild(food);

Mach das einfach mal so:

Code

document.getElementById("mover_con").appendChild(food);

BTW Link geht nicht ;) Der lädt sich tot.

Ansonsten, veränder einfach den Ort von dem Bild was du fest eingefügt hast, ka ob das so geht wie du willst.

Vollkornbrot · 21. September 2009

Also document.getElementById("mover_con").appendChild(food); hatte ich als erstes auch, hat aber nicht funktioniert. mover_con ist weiterhin null.
Bei mir geht der Link, wahrscheinlich waren die server down...
Und ich versuch mal des bild per js einzubinden.

Fred · 22. September 2009

was ist den daran flasch oder anders gefragt , was soll funktionionieren was nicht funzt ?

weil ich seh im code keinen direkten fehler, wahrscheinlich ein denkfehler nur den kann man so nicht finden ...

**TimWolla** · 22. September 2009

Zitat von eeZy

Ansonsten, veränder einfach den Ort von dem Bild was du fest eingefügt hast, ka ob das so geht wie du willst.

Also

Code

document.getElementById("mover").top = 'xx';
document.getElementById("mover").left = 'xx';

Vollkornbrot · 22. September 2009

Fred
Das soll ja auch kein flash sein^^
Das Problem is das mover_con null is.

@eeZy
Ich check das nich.
Wo soll ich das einfügen?

**TimWolla** · 22. September 2009

Zitat von Vollkornbrot

@eeZy
Ich check das nich.
Wo soll ich das einfügen?

In der Place Food Funktion

Vollkornbrot · 22. September 2009

Zitat

document.getElementById("mover") is null

E:
Btw:
Es macht keinen Unterschied :

Zitat

document.getElementById("mover").top = 'xx';

oder

Zitat

document.getElementById("mover").style.top = 'xx';

??

**TimWolla** · 22. September 2009

Zitat von Vollkornbrot

Es mach keinen unter schied ob

Doch, du musst das mit style nehmen hätte ich gesagt:

E:/

mover_con = document.getElementById('mover_con'); -> das geht ;) Die doppelten Anführungszeichen waren wohl böse.

Chris! · 23. September 2009

was soll an doppelten anführungszeichen in javascript falsch sein? schwachsinn xD

der einzige mir ersichtliche fehler und der einzige fehler, denn die fehlerkonsole in firefox hergibt ist, dass die funktion place_foot aufgerufen wird, bevor das div mit der ID "mover_con" überhaupt geladen is.. vielleicht kannst du den aufruf von place_foot mit in init() reinquetschen oder derartiges..

Vollkornbrot · 24. September 2009

8o8o8o
Das erklärt do ziemlich jeden Javascript Fehler, den ich bisher hatte. Hab gedacht die Browser wären so klug.
Komisch, das sowas nie irgendwo erwähnt wird.

BIG THX!

**TimWolla** · 24. September 2009

Zitat von Chris!

was soll an doppelten anführungszeichen in javascript falsch sein? schwachsinn xD

Mein Fehler, hatte das über die Adressleiste so aufgerufen :/