Du har fortfarande inte gjort en del av de ändringar jag skrev om: - Du har name="largeImg1" i stället för id="largeImg1", så document.getElementById("largeImg" + imgNr) blir null, och skriptet avbryts med fel (du borde sett detta som ett fel i Firebug). - Du har en del variabler som inte deklareras med 'var', så de blir globala (pictNr, randNumber, score, points). (Det orsakar inga fel på sidan, men det är väldigt dåligt.) - Fortfarande syntaxfel som <a href="#" return false;> och language="javascript". (I och för sig inte heller något som orsakar synliga effekter, men du borde rätta sådana fel så fort du ser dem, så att koden är i ett så bra tillstånd som möjligt när du skriver vidare på den.) (- Jag såg nu också att du inte använder argumentet till randomInteger alls, varför?)
Sedan, för att komma till faktiska, relevanta, fel: - Först är ju
//void threePict( int pictNr, int imgNr, int randNumber ) //if( (points == points2) || (points == points3) || (points2 == points3) ) points + points2 + points3;
ett uppenbart syntaxfel. Funktioner deklareras med "function foo(bar)", inte "void foo(int bar)" som i, säg, C. points+points2+points3 hamnar också i globalt scope och körs bara när sidan laddas. Men jag antar att detta är WIP, så *shrug*.
- document.dice.result = + points; är väldigt fel. Först är document.dice odefinierat, med 'dice' som ett div menar du nog document.getElementById('dice'). Dess .result är bara ett attribut på elementet, precis som, säg, style eller id. Att sätta det till något gör inget alls. Det jag tror att du har tänkt är att ha ett textfält med poängen. Din markup bör i så fall ändras från
Notera att <option> är ett fält i en <select>-box, inte en textruta eller en värdehållare på något sätt, och att +points+ inte gör någonting, överhuvudtaget, eftersom det är HTML, inte JavaScript, och att det även om det vore JavaScript bara skulle evaluera point en gång, vid sidladdning. För att den ska uppdateras dynamiskt behöver du ändra den från skriptet, varje gång. Egentligen borde man nog skriva om det som
Score: <input type="text" id="result">
För att man ska kunna komma åt fältet på modernt vis med document.getElementById.
- Till sist, din poänguträkning är flummig. När du gör
score = showRandomPict(1) + showRandomPict(2);
anropar du showRandomPict igen två gånger, så att bilderna ändras ytterligare, du vill antagligen kunna räkna ut poäng även från showRandomPict när bara en tärning ändras, och din logik följer inte vad du beskrivit.
De förslag jag ger resulterar i ungefär den här koden:
När du använder document.getElementById måste du ange namnen på elementen som id="", inte name="", jag antar att det är detta du har missat. (Om inte bör du nog posta hela koden.)
Och med den uppgiften är detta nog den enklaste lösningen: - Ha en global array (om du inte lärt dig det än kan du använda tre separata variabler också, eller så kan du bara kolla upp det för det är inte så svårt) som innehåller värdena på tärningsslagen. - Låt showRandomPict ändra rätt värde i arrayen (typ, dice[imgNr-1] = pictNr;). - I slutet på showRandomPict lägger du lite kod som beräknar poängen utefter vad arrayen innehåller, och visar den i ett span.
När jag skriver: document.getElementById("largeImg" + imgNr) får jag att det "is null" i firebug. Vad har jag gjort för fel då?
<i>Lägg till kod i funktionen i steg 2( i mitt fall threePict), så att tärningarnas värden jämförs och en poängsumma beräknas enligt följande: om två tärningar är lika, får man summan av dessa två tärningar; om alla tre tärningarna är lika, får man dubbla summan av tärningarna; i övrigt är poängen 0. Den framräknade poängen visas i den span-tagg eller textfält som du la till i HTML-koden.</i>
Detta står i uppgiften, vilka värden ska jag ha med (i vad jag gissar I-satserna)?
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.
