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: <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8" />
<title>DICE</title>
<link rel="stylesheet" type="text/css" href="Dice1.css" />
<script type="text/javascript" language="javascript">
function showPict(randNumber , imgNr){
if (randNumber == 1 ) imgUrl = "dice/1.gif";
else if (randNumber == 2 ) imgUrl = "dice/2.gif";
else if (randNumber == 3 ) imgUrl = "dice/3.gif";
else if (randNumber == 4 ) imgUrl = "dice/4.gif";
else if (randNumber == 5 ) imgUrl = "dice/5.gif";
else if (randNumber == 6 ) imgUrl = "dice/6.gif";
document["largeImg" + imgNr].src = imgUrl;
}
function randomInteger(randNumber) {
var randNumber = Math.random();
var randNumber = Math.floor(6*randNumber)+1;
return randNumber;
}
function showRandomPict(imgNr) {
var pictNr = randomInteger(6);
showPict(pictNr , imgNr);
}
</script>
</head>
<body>
Roll The Dice! </p>
<div id= largePict>
<a href="#" return false;>
<img name="largeImg1" src="dice/1.gif" alt="Ett" onclick="showRandomPict(1)" />
</a>
<a href="#" return false;>
<img name="largeImg2" src="dice/1.gif" alt="Ett" onclick="showRandomPict(2)" />
</a>
<a href="#" return false;>
<img name="largeImg3" src="dice/1.gif" alt="Ett" onclick="showRandomPict(3)" />
</a>
</div>
<div id="dice">
<a href="#" onclick = "showRandomPict(1); showRandomPict(2); showRandomPict(3); return false">Roll all Dices</a>
</p>
<form name="result">
</form>
</p>
</div>
</body>
</html> |
| 
