onclick only works once

[Steve]

This simple script works first time but if you click a second time on
the button nothing happens. I expected that the roll function would be
executed again but it doesn't. Could someone explain why.

http://members.chello.at/stephen.jou...hotspots1.html

javascript is at
http://members.chello.at/stephen.jou...e/hotspots1.js

Thanks in advance

[Conrad Lender]

On 2008-11-20 22:00, Steve wrote:

This simple script works first time but if you click a second time on
the button nothing happens. I expected that the roll function would be
executed again but it doesn't. Could someone explain why.

function roll() {
^^^^
for(var i=0; i<10; i++) {
var roll = Math.floor(Math.random()*6+1);
^^^^

Oops :-)
- Conrad

[Conrad Lender]

On 2008-11-20 22:10, Conrad Lender wrote:

Oops :-)

Err, scratch that. Sorry.

for(var i=0; i<10; i++) {
var roll = Math.floor(Math.random()*6+1);
rolls.push(roll);
^^^^^^^^^^
}
for(var i=0; i<10; i++) {
var dice="die" + i;
document.getElementById(dice).innerHTML=rolls[i];
^^^^^

You're pushing new values to the rolls array, but always read the same
10 values from the start.
- Conrad

[Gregor Kofler]

Steve meinte:

This simple script works first time but if you click a second time on
the button nothing happens. I expected that the roll function would be
executed again but it doesn't. Could someone explain why.

http://members.chello.at/stephen.jou...hotspots1.html

javascript is at
http://members.chello.at/stephen.jou...e/hotspots1.js

Thanks in advance

You are constantly pushing values onto the rolls array, and feeding the
output always with the same first few entries. A quick run with a
debugger (e.g. Firebug) would have revealed the problem.

A

rolls = [];

at the beginning of the roll function should suffice. (The rest of the
code is ..er... could be better.)

Gregor

[Steve]

On Nov 20, 10:17*pm, Conrad Lender <[e-mail]:

On 2008-11-20 22:10, Conrad Lender wrote:

>
You're pushing new values to the rolls array, but always read the same
10 values from the start.

Thanks Conrad.

[rf]

"Steve" <st***********@chello.atwrote in message
news:24***************************@news.chello.at. ..

This simple script works first time but if you click a second time on the
button nothing happens. I expected that the roll function would be
executed again but it doesn't. Could someone explain why.

It is being executed again. You are just putting exactly the same numbers
back there.