JavaScript: Funktion zu ändern, Bilder mit onClick funktioniert nicht.

Habe ich 6 Bilder von der schönen Serena Williams, ich habe erforscht, wie das zu ändern Bilder jedes mal, wenn ich auf Sie klicken. Aber der code funktioniert nicht, bitte helfen Sie mir herauszufinden, ob ich habe eine Logik oder syntax Fehler:

<html>
    <head>
        <title>SerenaWilliams</title>
    </head>
    <body>
        <img src="http://1.bp.blogspot.com/-hGUyOlpoeB0/UROyeHgMXcI/AAAAAAAAAZ4/L32zLAQvQj0/s1600/Hot-Serena-Williams+_sexy+16.jpg" id="serenaGallery" onclick="changeImage" /> 
    </body>

   <script type="text/javascript">

       function changeImage() {

           var currentImage = document.getElementById("serenaGallery").src;
           var image1 = "http://1.bp.blogspot.com/-hGUyOlpoeB0/UROyeHgMXcI/AAAAAAAAAZ4/L32zLAQvQj0/s1600/Hot-Serena-Williams+_sexy+16.jpg";
           var image2 = "http://usatthebiglead.files.wordpress.com/2011/04/serena-williams-is-big-boned.jpg";
           var image3 = "http://www.strangecosmos.com/images/content/109963.jpg";
           var image4 = "http://1.bp.blogspot.com/-HiJxcIjMmFg/UWo9JtbfCEI/AAAAAAAAF1g/aUU42F3V9Ic/s1600/Serena+Williams+Hot+2013+04.jpg";
           var image5 = "http://mystarclub.com/wp-content/uploads/2012/11/Serena-Williams-is-a-Bikini-Babe.jpg";
           var image6 = "http://1.bp.blogspot.com/-vCsx4sswzeM/UA5GtbEwJ1I/AAAAAAAAACE/tMiP_p-0rB0/s1600/serena+williams+tennis+ball+in+butt.jpg";

           if (currentImage==image1){ currentImage=image2; }
           if (currentImage==image2){ currentImage=image3; }
           if (currentImage==image3){ currentImage=image4; }
           if (currentImage==image4){ currentImage=image5; }
           if (currentImage==image5){ currentImage=image6; }
           else { currentImage=image1; }

        }

    </script>

</html>
  • Ihre <script> sollte in <head> oder <body>, nicht ein unmittelbares Kind von <html>
InformationsquelleAutor Ruben | 2014-03-30
  1. 2

    Erste von allen, müssen Sie zum ändern der onclick Attribut, so wird es tatsächlich führen Sie die Funktion aus:

    onclick="changeImage()"

    Den Befehl 

    var currentImage = document.getElementById("serenaGallery").src;

    speichert nur die Quelle - Attribut des Bildes in einer Variablen; es wird nicht darauf verweisen.

    Auch, Sie ändern müssen alle if Aussagen zu else if oder die Funktion zu ändern image1 zu image2, dann image3, etc.

    Dieser funktionieren würde:

    var currentImage = document.getElementById("serenaGallery");

...

if (currentImage.src == image1){ currentImage.src=image2;}
else if (currentImage.src == image2){ currentImage.src=image3;}

...

    Schließlich ein array für das Bild-Quellen erlauben würde, Sie zu kondensieren, die if - Anweisungen in einer einzigen for Schleife. Nicht wirklich nötig für sechs Bilder, aber besser, wenn Sie weitere hinzufügen möchten.

    InformationsquelleAutor Dennis
Schreibe einen Kommentar