Javascript Craps Game Simulation Unhandled Error?

Question

I was testing out javascript games and functions and on this particular example it keeps getting stuck after clicking on the play button, after I inspected the elements of the source code it showed me that error is in the two functions pasted below, however I dont know why are these exceptions given , any help would be greatly appreciated

    function rollDice()
        {
            dicerolling.play();

            die1Value = NaN;
            die2Value = NaN;
            showDice();

            die1Value = Math.floor(1 + Math.random() * 6);
            die2Value = Math.floor(1 + Math.random() * 6);
            return die1Value + die2Value;
        }

        function showDice()
        {
            setImage( rollDie1Img, die1Value );
            setImage( rollDie2Img, die2Value );
        }

        function setImage( dieImg, dieValue )
        {
            if ( isFinite ( dieValue ) )
                dieImg.src = "images//" + dieValue + ".png";
            else
                dieImg.src = "images//blank.png";
        }

the complete code is pasted here if needed :

http://codepaste.net/o29v92

the error given by chrome :

chrome error

dsgriffin · Accepted Answer · 2014-04-26 16:28:29Z

1

You're using the same ID twice in your elements:

<img id="rollDie1" src = "images/blank.png" alt = "Die 1 of Roll Value">
<img id="rollDie1" src = "images/blank.png" alt = "Die 1 of Roll Value">

Judging from your code, you should just use rollDie2 for the second one:

<img id="rollDie1" src="images/blank.png" alt="Die 1 of Roll Value">
<img id="rollDie2" src="images/blank.png" alt="Die 2 of Roll Value">

jsFiddle here

answered Apr 26, 2014 at 16:28

dsgriffin

68.9k17 gold badges141 silver badges139 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

Rage91 Over a year ago

oh wow I spent so much time but only at the script section , should pay more attention on the body part , thank you so much for pointing it out, but I must admit these things sure do make me confused sometimes

dsgriffin Over a year ago

@Rage91 No worries, I used console.log() to track what was dieImg was and noticed that it was working for the first time but not the second, that's why I went to the HTML

Collectives™ on Stack Overflow

Javascript Craps Game Simulation Unhandled Error?

1 Answer 1

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related