2016-05-21 18 views
0

Ich versuche, eine grundlegende Galerie mit JQuery zu erstellen.JQuery Gallery Next-Taste funktioniert nicht richtig?

Die Grundidee ist, dass alle Bilddateien x.png heißen (wobei x eine Zahl ist), und das Programm fügt der aktuellen Zahl eine Zahl hinzu, die x + 1.png und so weiter erzeugt.

Der Code ich habe, ist:

function gal2(){ 
    var amount = $(".imagelist > img").length; 
    var next = $("#display").attr('src').replace('.png', ''); 
    if ($("#display").attr('src').replace('.png', '') >= amount) { 
     $("#display").attr('src', next+".png"); 
     next++; 
    } else { 
     $("#display").attr('src', next+".png"); 
     next++; 
    }; 
} 

gal2 auf einen Knopf drücken <input type="button" onclick="gal2()"> genannt wird,

.imagelist ist ein div die Bilder enthält,

#display ist das Hauptbild,

gezeigt wird Example Website

Das Problem ist, dass nichts passiert Stifte, außer wenn eine ausgewählt ist, wird sie jedes Mal auf die ursprüngliche zurückgesetzt.

P. S: Es ist für ein Jahr 9 Sekundärschulprojekt

+0

Und das Problem ist ...? – Endless

+0

Sorry, habe den wichtigsten Teil vergessen. –

Antwort

1

Sie einige Änderungen an Ihrem Skript gemacht und es jetzt gut funktioniert: https://jsfiddle.net/IA7medd/qwmt7Lep/2/

function gal2(){ 
    var amount = $(".imagelist > img").length; 
    var current = parseInt($("#display").attr('src').replace('.png', '')); 
    var next = current + 1; 
    if (current < amount) { 
     $("#display").attr('src', next+".png"); 
    } else { 
     $("#display").attr('src', "1.png"); 
    }; 
} 
+0

Betrag ist schon ein int – Endless

+0

Ja es ist schon ein Int, ich werde es bearbeiten. aber irgendwie wird es kein Problem machen. –

+0

@ Ahmed-Salama, so dass anstelle einer Basisvariable das aktuelle Bild angezeigt wird? –

Verwandte Themen