✔ Problem mit JavaScript

[Kopfballstar]

Hallo,

im vorraus: Ich wäre für Hilfe sehr dankbar, denn es ist relativ dringend und wichtig das ich den Fehler finde! Vielen Dank schonmal an jeden der sich kurz Zeit nimmt.

ich habe ein dickes Problem mit einem JavaScript. Es ist so das es manchmal funktioniert, dann wieder nicht. Als Beispiel: Gestern lief es einwandfrei. Ich gehe zum Sport, und als ich wieder zu Hause bin geht es wieder nicht. Habe mit einem alert Befehl mal die Funktionen durchgeprüft: Es werden auf jeden Fall immer die richtigen Werte übergeben. Als ich das prüfte funktionierte es auch wieder. Dann habe ich den Alert Befehl rausgenommen und es lief immer noch. Dann habe ich die Seite komplett neu geöffnet und das Skript funktionierte plötzlich wieder nicht!

Es handelt sich um eine Slideshow und funktioniert folgendermaßen:

Hier wird sie eingebunden:

<td  width='138' height='137'>
     <div id='slideshow'>
	<?php include('slideshow/slideshow.php');?>
     </div>
</td>

Hier die Datei:

<img border="0" name="img" src="slideshow/1.jpg"> 
<br>
<div id='button_left'>
     <a id="right" name="next" href="javascript:void('')" onclick="nextimg()" ></a>
</div>

<div id='button_right'>
     <a id="left" name="previous" href="javascript:void('')" onclick="previmg()" ></a>
</div>

Hier das JavaScript:

var i = 1; 
images = new Array; 
images[1] = "slideshow/1.jpg" ;
images[2] = "slideshow/2.jpg" ;
images[3] = "slideshow/3.jpg" ;
images[4] = "slideshow/4.jpg" ;
images[5] = "slideshow/5.jpg" ;
images[6] = "slideshow/6.jpg" ;
images[7] = "slideshow/7.jpg" ;
images[8] = "slideshow/8.jpg" ;
images[9] = "slideshow/9.jpg" ;
images[10] = "slideshow/10.jpg" ;
images[11] = "slideshow/11.jpg" ;
images[12] = "slideshow/12.jpg" ;
images[13] = "slideshow/13.jpg" ;
images[14] = "slideshow/14.jpg" ;
images[15] = "slideshow/15.jpg" ;

function gallery(){ 
	if (i == 1){ 
	document.gallery.previous.value="";
	} 
}

function previmg(){ 
	if (i != 1) { 
		i --; 
		document.img.src = images[i];
	} 
	if (i == 1) { 
		image = images[15]; 
		document.img.src = image;
		i = 15;
	} 
} 

function nextimg(){ 
	if (i != 15) { 
		i ++;
		image = images[i];
		document.img.src = image;
	} 
	if (i == 15){ 
		image = images[1]; 
		document.img.src = image;
		i = 1;
	} 
}

 

[Sven Mintel]

Was ich so auf den ersten Blick erkenne:

images = new Array()

...die Klammern dort fehlen bei dir.

 

[con-f-use]

Die Klammern bei Array fehlen.

Außerdem: Gib dem Bild eine Id, statt eines Namens und sprech es immer über "document.getElementById('IdDesBildes')" statt "document.img" an. Außerdem ist "javascript:" in Links nicht so toll. Mach's besser so:

<a id="right" name="next" href="#" onclick="nextimg(); return false;" ></a>

Edit:
Sven war schneller.

 

[Sven Mintel]

Gib dem Bild eine Id, statt eines Namens und sprech es immer über "document.getElementById('IdDesBildes')" statt "document.img"

Was genau würde das für Vorteile bringen, mal abgesehen davon, dass es mehr Code ist und dann in evtl. noch vorhandenen Uralt-Browsern nicht mehr funktioniert?

 

[Gumbo]

Wozu überhaupt das a-Element wenn es nicht einmal ein richtigen Ziel-URL gibt? Ach ja, es soll ja so wirken, als sei es ein normaler Link.

 

[Kopfballstar]

Hallo,

ok danke. Anscheinend funktioniert es jetzt. Ich hoffe das bleibt jetzt auch so!