Podmiana jpg na gif i w drugą stronę


(Kernelus) #1

Witam,

na szybko zrobiłem podmianę jpg na gif i działa ok, ale jak już leci gif to po kliknięciu w niego chciałbym znów ustawić go na jpg ale nie działa.

Tutaj kod:

[/code]

[code]function changeImage() { if(document.getElementById("karton").src="images/karton.jpg") { document.getElementById("karton").src="images/karton.gif"; } else { document.getElementById("karton").src="images/karton.jpg"; } }


(ra-v) #2

Popraw

if(document.getElementById("karton").src=="images/karton.jpg")

(Kernelus) #3

Kurcze, zapomniałem o "==" zamiast "=". Nienawidzę jak robię literówki. :stuck_out_tongue: Dzięki za poprawę. :smiley:

-- Dodane 19.11.2013 (Wt) 8:20 --

Hmmm nie wiem czemu teraz nie chcę w ogóle działa. O.o


(jacko1998) #4
function changeImage() {

   var karton = document.getElementById("karton").src;

   if(karton == "images/karton.jpg")

   {

    karton = ("images/karton.gif");

   }

   else

   {

   karton = ("images/karton.jpg");

   }

}

-- Dodane 19.11.2013 (Wt) 14:17 --

Ew. źle wywołujesz tą funkcję w HTML


(Kernelus) #5

Też nie chce działa :frowning:

Wywołuję ją w ten sposób:

[/code]

(floyd) #6

Co do Twojego skryptu: Wadą, a czasmi zaletą Javascript jest, że nie pokazuje blędów czy niepoprawnych zapisów. Tobie np. wydawało się, że jedna zamiana zostala wykonana poprawnie, ale tylko dlatego, że

"if(document.getElementById("karton").src=="images/karton.jpg")" zostało całkowicie pominięte i wykonana została następna instrukcja zamieniając jpg na gif. Możesz, to latwo sprawdzić wpisując np. if(document.getElementById("karton").src=="ble ble ble") i też dokonana zostanie jednokrotna zamiana.

To działa napewno:

<html>
<head>
<script type="text/javascript">
var i=0;
function changeImage()
 {
	if(i==0)
	{
        document.getElementById("karton").src="images/karton.gif";
         i=1;
	}
	else
	{
	document.getElementById("karton").src="images/karton.jpg";
         i=0;
	}
 }
</script>  
</head>
<body>
<img src="images/karton.jpg" id="karton" onclick="changeImage()"/>
</body>
</html>