Jag ser inte riktigt sammanhanget i vilket du skulle vilja använda ett returvärde av showRandomPict? Antagligen kan du bara returnera pictNr, men hur du sedan lagrar returnvärdena i variabler beror på vad du vill göra med dem.
Din kod använder för övrigt en del utdaterade/felaktiga sätt att skriva saker på, så här är bättre: <!DOCTYPE html> (<- HTML5-doctypen)
<html>
<head>
<meta charset="UTF-8"> (<- förenklad charset-deklaration i HTML5, och den standarden kräver inte heller att du avslutar tomma taggar med />)
<title>DICE</title>
<link rel="stylesheet" type="text/css" href="Dice1.css">
<script type="text/javascript"> (<- language="javascript" behövs inte längre) function showPict(randNumber, imgNr){
var imgUrl = "dice/" + randNumber + ".gif"; (<- deklarera imgUrl som 'var' för att den inte ska bli global, och du behöver inga if-satser om du ändå bara gör samma sak (lägger in slumptalet i strängen)) document.getElementById("largeImg" + imgNr).src = imgUrl; (<- document["namn"] är väldigt föråldrat; document.getElementById bör användas i stället. Notera att id's används i stället för name's på elementen)
} function randomInteger(randNumber) {
return Math.floor(6*Math.random())+1; (<- du kan ha allt på en rad (tempvariabler är onödiga ibland), och undvik förresten att deklarera variabler med 'var' två gånger)
} function showRandomPict(imgNr) {
var pictNr = randomInteger(6);
showPict(pictNr, imgNr);
return pictNr; (<- om du ska ha ett returvärde är det nog detta)
} </script>
</head>
<body> Roll The Dice! </p>
<div id="largePict">
<a href="#" onclick="return false;"> (<- du menade onclick="return false;", inte bara return false; som tolkas som två tomma HTML-attribut med namn "return" och "false;")
<img id="largeImg1" src="dice/1.gif" alt="Tärningsslag" onclick="showRandomPict(1)"> (<- mer beskrivande alt-text, eftersom den inte uppdateras senare)
</a>
<a href="#" onclick="return false;">
<img id="largeImg2" src="dice/1.gif" alt="Tärningsslag" onclick="showRandomPict(2)">
</a>
<a href="#" onclick="return false;">
<img id="largeImg3" src="dice/1.gif" alt="Tärningsslag" onclick="showRandomPict(3)">
</a>
</div>
<div id="dice">
<a href="#" onclick="showRandomPict(1); showRandomPict(2); showRandomPict(3); return false;">Roll all Dice</a> ("dice" är plural av "die", "dices" är inget ord)
</p> (<- det känns semantiskt, möjligen standardsmässigt, fel att ha <form> i , men jag är osäker)
<form name="result"> (<- ??)
</form>
</p>
</div>
</body>
</html> Simon | |
Hej!
Jag har lite problem med en uppgift jag har fått i skolan som gäller js.
Jag har kommit en bit på vägen, men nu gäller det att returnera ett värde från funktionen shovRandomPict och sedan ska jag spara de returnerade värdena i variabler. Jag har suttit och fixat med det...men kommer inte på nån lösning. Skulle verkligen uppskatta hjälp. :) Här är koden: <snip> |
| 
