Defining forms?

A

Andy B

Hi,
I am very new to JavaScript and my first assignment is to create a simple
game. Below is my code which works fine in IE, but not in any other browser
(Opera, Netscape, Mozilla). It has no form defined yet it uses form buttons.
Like I said, it works fine in IE but when loaded into another browser and
the Start Game! button is clicked, nothing seems to happen. There are no
error messages apperaing. Any help would be appreciated. Thanks.
--
To contact me, please use andrewbroomhead {AT} yahoo {DOT} co {DOT} uk

I have deleted some pointless HTML code from the beginning

<script type="text/JavaScript">
<!--
var GamePics = new Array(9);
var i;
var j;
var ind;
var AccClick=0;
var PlayingGame=0;
GamePics[0] = new Image();
GamePics[0].src="images/1.gif";
GamePics[1] = new Image();
GamePics[1].src="images/2.gif";
GamePics[2] = new Image();
GamePics[2].src="images/3.gif";
GamePics[3] = new Image();
GamePics[3].src="images/4.gif";
GamePics[4] = new Image();
GamePics[4].src="images/5.gif";
GamePics[5] = new Image();
GamePics[5].src="images/6.gif";
GamePics[6] = new Image();
GamePics[6].src="images/7.gif";
GamePics[7] = new Image();
GamePics[7].src="images/8.gif";
GamePics[8] = new Image();
GamePics[8].src="images/Blank.gif";
function Clicked(NumClicked)
{
if (AccClick == 0)
alert ("Please click on Start Game! to begin");
else
{
if (NumClicked == 1 && image[1].src == GamePics[8].src)
Swap(0,1);
if (NumClicked == 1 && image[3].src == GamePics[8].src)
Swap(0,3);
if (NumClicked == 2 && image[0].src == GamePics[8].src)
Swap(1,0);
if (NumClicked == 2 && image[2].src == GamePics[8].src)
Swap(1,2);
if (NumClicked == 2 && image[4].src == GamePics[8].src)
Swap(1,4);
if (NumClicked == 3 && image[1].src == GamePics[8].src)
Swap(2,1);
if (NumClicked == 3 && image[5].src == GamePics[8].src)
Swap(2,5);
if (NumClicked == 4 && image[0].src == GamePics[8].src)
Swap(3,0);
if (NumClicked == 4 && image[4].src == GamePics[8].src)
Swap(3,4);
if (NumClicked == 4 && image[6].src == GamePics[8].src)
Swap(3,6);
if (NumClicked == 5 && image[3].src == GamePics[8].src)
Swap(4,3);
if (NumClicked == 5 && image[1].src == GamePics[8].src)
Swap(4,1);
if (NumClicked == 5 && image[5].src == GamePics[8].src)
Swap(4,5);
if (NumClicked == 5 && image[7].src == GamePics[8].src)
Swap(4,7);
if (NumClicked == 6 && image[2].src == GamePics[8].src)
Swap(5,2);
if (NumClicked == 6 && image[4].src == GamePics[8].src)
Swap(5,4);
if (NumClicked == 6 && image[8].src == GamePics[8].src)
Swap(5,8);
if (NumClicked == 7 && image[7].src == GamePics[8].src)
Swap(6,7);
if (NumClicked == 7 && image[3].src == GamePics[8].src)
Swap(6,3);
if (NumClicked == 8 && image[6].src == GamePics[8].src)
Swap(7,6);
if (NumClicked == 8 && image[4].src == GamePics[8].src)
Swap(7,4);
if (NumClicked == 8 && image[8].src == GamePics[8].src)
Swap(7,8);
if (NumClicked == 9 && image[5].src == GamePics[8].src)
Swap(8,5);
if (NumClicked == 9 && image[7].src == GamePics[8].src)
Swap(8,7);
WinCheck();
}
}
function Swap(Pic1, Pic2)
{
spare=new Image();
spare.src=document.image[Pic1].src;
document.image[Pic1].src=document.image[Pic2].src;
document.image[Pic2].src=spare.src;
}
function Timer()
{
if(PlayingGame==1)
{
timeTaken.value++;
setTimeout('Timer()', 1000);
}
}
function Shuffle()
{
if(AccClick==1)
{
AccClick=0;
PlayingGame=0;
startButton.value="Start Game!";
timeTaken.value=0;
return;
}
else
startButton.value="Stop Game!";
for (j=0;j<8;j++)
{
ind=Math.round(8*Math.random());
Swap(j,ind);
}
AccClick = 1;
timeTaken.value=0;
PlayingGame=1;
Timer();
}
function WinCheck()
{
var NoCorrect=0;
for (i=0;i<8;i++)
{
if (document.image.src == GamePics.src)
NoCorrect++;
}
if (NoCorrect == 8)
{
startButton.value="Start Game!";
PlayingGame=0;
AccClick=0;
alert("Congratulations, You have completed the game! It took
"+timeTaken.value+" seconds!");
timeTaken.value=0;
}
}
//-->
</script>
<center><table class="content">
<tr>
<td class="innerContent"><table width="300" border="0" cellspacing="1"
cellpadding="1"
height="300">
<tr>
<td><A border='0' href="javascript:Clicked('1')"><img border='0'
name="image" src="images/1.gif"></A></td>
<td><A border='0' href="javascript:Clicked('2')"><img border='0'
name="image" src="images/2.gif"></A></td>
<td><A border='0' href="javascript:Clicked('3')"><img border='0'
name="image" src="images/3.gif"></A></td>
</tr>
<tr>
<td><A border='0' href="javascript:Clicked('4')"><img border='0'
name="image" src="images/4.gif"></A></td>
<td><A border='0' href="javascript:Clicked('5')"><img border='0'
name="image" src="images/5.gif"></A></td>
<td><A border='0' href="javascript:Clicked('6')"><img border='0'
name="image" src="images/6.gif"></A></td>
</tr>
<tr>
<td><A border='0' href="javascript:Clicked('7')"><img border='0'
name="image" src="images/7.gif"></A></td>
<td><A border='0' href="javascript:Clicked('8')"><img border='0'
name="image" src="images/8.gif"></A></td>
<td><A border='0' href="javascript:Clicked('9')"><img border='0'
name="image" src="images/Blank.gif"></A></td>
</tr>
</table>
<p>
<input type='button' type='button' name='startButton' value='Start
Game!' OnMouseDown='javascript:Shuffle()'>
<font color="#FFFFFF" size="3" face="Geneva, Arial, Helvetica,
sans-serif">
You have taken&nbsp;
<input class='box' type='text' name='timeTaken' readonly size='2'
value='0'>
seconds</font><br>
</p>
<p>&nbsp; </p>
<table width="100%" border="0">
<tr>
<td width="72%"><font color="#FFFFFF" size="3" face="Verdana, Arial,
Helvetica,
sans-serif">The
completed picture looks like this: </font></td>
<td width="28%"><font face="Verdana, Arial, Helvetica,
sans-serif"><img
src="Images/Complete.gif" width="200" height="200" border="2"></font></td>
</tr>
</table>
<p>&nbsp;</p></body>
</html>
 
L

Lee

Andy B said:
Hi,
I am very new to JavaScript and my first assignment is to create a simple
game. Below is my code which works fine in IE, but not in any other browser
(Opera, Netscape, Mozilla). It has no form defined yet it uses form buttons.
Like I said, it works fine in IE but when loaded into another browser and
the Start Game! button is clicked, nothing seems to happen. There are no
error messages apperaing. Any help would be appreciated. Thanks.
Click to expand...

This is USENET.
To contact you, we respond to your post in the newsgroup.

If you're not seeing error messages in Netscape, then you're not looking
in the right place.

In Netscape 7, use the pulldown menu item:

Tools > Web Development > JavaScript Console

You should see a message complaining that startButton is not defined.
Most browsers don't let you get away with referring to elements
simply by their names, without any context information.

Also note that your method of choosing random numbers doesn't really
work the way you (and possibly your instructor) think it does:
ind=Math.round(8*Math.random());
chooses 0 and 8 half as often as any of the other values.
 
M

Michael Winter

I am very new to JavaScript and my first assignment is to create a simple
game. Below is my code which works fine in IE, but not in any other
browser (Opera, Netscape, Mozilla). It has no form defined yet it uses
form buttons.
Like I said, it works fine in IE but when loaded into another browser and
the Start Game! button is clicked, nothing seems to happen. There are no
error messages apperaing. Any help would be appreciated. Thanks.
Click to expand...

I'll quote sections of your code, then comment on any issues I see. Some
of the comments will be general guidelines only, and won't actually solve
your problem. However, they should still be given some consideration.
<A border='0' href="javascript:Clicked('1')"><img border='0'
name="image" src="images/1.gif"></A>
Click to expand...

1) Don't use the JavaScript pseudo-protocol (javascript:) without a good
reason. You can do the above with either of:

<a href="" onclick="Clicked('1'); return false;"><img src="images/1.gif"
alt="short description"></a>

or:

<img src="images/1.gif" alt="short description" onclick="Clicked('1')">

Note that the former should be styled with CSS, and the latter won't work
with NN4.

2) You pass the string, '1', in the call above, but you check against a
number in the function:

function Clicked(NumClicked) {
...
if (NumClicked == 1 && image[1].src == GamePics[8].src)
Swap(0,1);

Choose one or the other.

if (NumClicked == 1 && image[1].src == GamePics[8].src)
Swap(0,1);
Click to expand...

You use 'image' above as a global variable. However, it is not - it is the
name of several HTML elements. This is where the main issue lies. You can
access the images through the document.images collection. This can be done
either by index (first image in the entire document is 0), or by name. If
you use the latter, you'll have to give each image a unique name.
<input type='button' type='button' name='startButton' value='Start
Game!' OnMouseDown='javascript:Shuffle()'>
Click to expand...

You probably want to use the onclick intrinsic event, not onmousedown.
Furthermore, prepending "javascript:" to an intrinsic event does not
specify the language in any browser other than IE. The proper way to
acheive this is to use a META element inside the HEAD section:

<script type="text/JavaScript">
<!--
Click to expand...

You don't need to enclose script content with SGML comments. The archaic
reasons for doing so are no longer valid. Also, by convention, MIME types
are written in lowercase, but there shouldn't be any difference (the type
attribute is case insensitive).

Mike

In future, please don't place anything other than your signature, below
the signature marker ("-- ").
 
A

Andy B

Michael Winter said:
I am very new to JavaScript and my first assignment is to create a simple
game. Below is my code which works fine in IE, but not in any other
browser (Opera, Netscape, Mozilla). It has no form defined yet it uses
form buttons.
Like I said, it works fine in IE but when loaded into another browser and
the Start Game! button is clicked, nothing seems to happen. There are no
error messages apperaing. Any help would be appreciated. Thanks.
Click to expand...

I'll quote sections of your code, then comment on any issues I see. Some
of the comments will be general guidelines only, and won't actually solve
your problem. However, they should still be given some consideration.
<A border='0' href="javascript:Clicked('1')"><img border='0'
name="image" src="images/1.gif"></A>
Click to expand...

1) Don't use the JavaScript pseudo-protocol (javascript:) without a good
reason. You can do the above with either of:

<a href="" onclick="Clicked('1'); return false;"><img src="images/1.gif"
alt="short description"></a>

or:

<img src="images/1.gif" alt="short description" onclick="Clicked('1')">

Note that the former should be styled with CSS, and the latter won't work
with NN4.

2) You pass the string, '1', in the call above, but you check against a
number in the function:

function Clicked(NumClicked) {
...
if (NumClicked == 1 && image[1].src == GamePics[8].src)
Swap(0,1);

Choose one or the other.

if (NumClicked == 1 && image[1].src == GamePics[8].src)
Swap(0,1);
Click to expand...

You use 'image' above as a global variable. However, it is not - it is the
name of several HTML elements. This is where the main issue lies. You can
access the images through the document.images collection. This can be done
either by index (first image in the entire document is 0), or by name. If
you use the latter, you'll have to give each image a unique name.
<input type='button' type='button' name='startButton' value='Start
Game!' OnMouseDown='javascript:Shuffle()'>
Click to expand...

You probably want to use the onclick intrinsic event, not onmousedown.
Furthermore, prepending "javascript:" to an intrinsic event does not
specify the language in any browser other than IE. The proper way to
acheive this is to use a META element inside the HEAD section:

<script type="text/JavaScript">
<!--
Click to expand...

You don't need to enclose script content with SGML comments. The archaic
reasons for doing so are no longer valid. Also, by convention, MIME types
are written in lowercase, but there shouldn't be any difference (the type
attribute is case insensitive).

Mike

In future, please don't place anything other than your signature, below
the signature marker ("-- ").
Click to expand...

Cheers guys, fixed it now and everything is ok.
Thanks very much for the help.
Andy
 
M

McKirahan

Andy B said:
Hi,
I am very new to JavaScript and my first assignment is to create a simple
game. Below is my code which works fine in IE, but not in any other browser
(Opera, Netscape, Mozilla). It has no form defined yet it uses form buttons.
Like I said, it works fine in IE but when loaded into another browser and
the Start Game! button is clicked, nothing seems to happen. There are no
error messages apperaing. Any help would be appreciated. Thanks.
Click to expand...

Try this:

<html>
<head>
<title>Andy_B.htm</title>
<script type="text/JavaScript">
<!--
var path = location.href.substr(0,location.href.lastIndexOf("/")+1)
var i;
var j;
var ind;
var AccClick=0;
var PlayingGame=0;
var GamePics = new Array(9);
GamePics[0] = new Image();
GamePics[0].src = path + "images/1.gif";
GamePics[1] = new Image();
GamePics[1].src = path + "images/2.gif";
GamePics[2] = new Image();
GamePics[2].src = path + "images/3.gif";
GamePics[3] = new Image();
GamePics[3].src = path + "images/4.gif";
GamePics[4] = new Image();
GamePics[4].src = path + "images/5.gif";
GamePics[5] = new Image();
GamePics[5].src = path + "images/6.gif";
GamePics[6] = new Image();
GamePics[6].src = path + "images/7.gif";
GamePics[7] = new Image();
GamePics[7].src = path + "images/8.gif";
GamePics[8] = new Image();
GamePics[8].src = path + "images/Blank.gif";
function Clicked(NumClicked) {
if (AccClick == 0) {
alert ("Please click on Start Game! to begin");
} else {
if (NumClicked == 1 && document.image1.src == GamePics[8].src)
Swap(0,1);
if (NumClicked == 1 && document.image3.src == GamePics[8].src)
Swap(0,3);
if (NumClicked == 2 && document.image0.src == GamePics[8].src)
Swap(1,0);
if (NumClicked == 2 && document.image2.src == GamePics[8].src)
Swap(1,2);
if (NumClicked == 2 && document.image4.src == GamePics[8].src)
Swap(1,4);
if (NumClicked == 3 && document.image1.src == GamePics[8].src)
Swap(2,1);
if (NumClicked == 3 && document.image5.src == GamePics[8].src)
Swap(2,5);
if (NumClicked == 4 && document.image0.src == GamePics[8].src)
Swap(3,0);
if (NumClicked == 4 && document.image4.src == GamePics[8].src)
Swap(3,4);
if (NumClicked == 4 && document.image6.src == GamePics[8].src)
Swap(3,6);
if (NumClicked == 5 && document.image3.src == GamePics[8].src)
Swap(4,3);
if (NumClicked == 5 && document.image1.src == GamePics[8].src)
Swap(4,1);
if (NumClicked == 5 && document.image5.src == GamePics[8].src)
Swap(4,5);
if (NumClicked == 5 && document.image7.src == GamePics[8].src)
Swap(4,7);
if (NumClicked == 6 && document.image2.src == GamePics[8].src)
Swap(5,2);
if (NumClicked == 6 && document.image4.src == GamePics[8].src)
Swap(5,4);
if (NumClicked == 6 && document.image8.src == GamePics[8].src)
Swap(5,8);
if (NumClicked == 7 && document.image7.src == GamePics[8].src)
Swap(6,7);
if (NumClicked == 7 && document.image3.src == GamePics[8].src)
Swap(6,3);
if (NumClicked == 8 && document.image6.src == GamePics[8].src)
Swap(7,6);
if (NumClicked == 8 && document.image4.src == GamePics[8].src)
Swap(7,4);
if (NumClicked == 8 && document.image8.src == GamePics[8].src)
Swap(7,8);
if (NumClicked == 9 && document.image5.src == GamePics[8].src)
Swap(8,5);
if (NumClicked == 9 && document.image7.src == GamePics[8].src)
Swap(8,7);
WinCheck();
}
}
function Swap(Pic1, Pic2) {
var imag1 = eval("document.image" + Pic1);
var imag2 = eval("document.image" + Pic2);
var spare = new Image();
spare.src = imag1.src;
imag1.src = imag2.src;
imag2.src = spare.src;
}
function Timer() {
if (PlayingGame == 1) {
document.forms[0].timeTaken.value++;
setTimeout('Timer()',1000);
}
}
function Shuffle() {
if (AccClick == 1) {
AccClick = 0;
PlayingGame = 0;
document.forms[0].startButton.value = "Start Game!";
document.forms[0].timeTaken.value = 0;
return;
} else {
AccClick = 1;
PlayingGame = 1;
document.forms[0].startButton.value = "Stop Game!";
for (j=0; j<8; j++) {
ind = Math.round(8*Math.random());
Swap(j,ind);
}
document.forms[0].timeTaken.value = 0;
Timer();
}
}
function WinCheck() {
var NoCorrect = 0;
for (i=0; i<8; i++) {
var imag0 = eval("document.image" + i);
if (imag0.src == GamePics.src) NoCorrect++;
}
if (NoCorrect == 8) {
document.forms[0].startButton.value = "Start Game!";
PlayingGame = 0;
AccClick = 0;
alert("Congratulations, You have completed the game! It
took"+document.forms[0].timeTaken.value+" seconds!");
document.forms[0].timeTaken.value = 0;
}
}
//-->
</script>
</head>
<body bgcolor="#000000">
<center>
<table class="content">
<tr>
<td class="innerContent">
<table width="300" border="0" cellspacing="1" cellpadding="1"
height="300">
<tr>
<td><A border='0' href="javascript:Clicked(1)"><img border='0'
name="image0" src="images/1.gif"></A></td>
<td><A border='0' href="javascript:Clicked(2)"><img border='0'
name="image1" src="images/2.gif"></A></td>
<td><A border='0' href="javascript:Clicked(3)"><img border='0'
name="image2" src="images/3.gif"></A></td>
</tr>
<tr>
<td><A border='0' href="javascript:Clicked(4)"><img border='0'
name="image3" src="images/4.gif"></A></td>
<td><A border='0' href="javascript:Clicked(5)"><img border='0'
name="image4" src="images/5.gif"></A></td>
<td><A border='0' href="javascript:Clicked(6)"><img border='0'
name="image5" src="images/6.gif"></A></td>
</tr>
<tr>
<td><A border='0' href="javascript:Clicked(7)"><img border='0'
name="image6" src="images/7.gif"></A></td>
<td><A border='0' href="javascript:Clicked(8)"><img border='0'
name="image7" src="images/8.gif"></A></td>
<td><A border='0' href="javascript:Clicked(9)"><img border='0'
name="image8" src="images/Blank.gif"></A></td>
</tr>
</table>
</td>
</tr>
</table>
<form>
<p>
<input type='button' type='button' name='startButton' value='Start
Game!' onclick='Shuffle()'>
<font color="#FFFFFF" size="3" face="Geneva, Arial, Helvetica,
sans-serif">You have taken &nbsp;
<input class='box' type='text' name='timeTaken' readonly size='2'
value='0'> seconds</font><br>
</p>
<p>&nbsp; </p>
<table width="100%" border="0">
<tr>
<td width="72%"><font color="#FFFFFF" size="3" face="Verdana, Arial,
Helvetica, sans-serif">The completed picture looks like this: </font></td>
<td width="28%"><font face="Verdana, Arial, Helvetica,
sans-serif"><img src="Images/Complete.gif" width="200" height="200"
border="2"></font></td>
</tr>
</table>
<p>&nbsp;</p>
</form>
</body>
</html>
 
A

Andy B

McKirahan said:
Andy B said:
Hi,
I am very new to JavaScript and my first assignment is to create a simple
game. Below is my code which works fine in IE, but not in any other browser
(Opera, Netscape, Mozilla). It has no form defined yet it uses form buttons.
Like I said, it works fine in IE but when loaded into another browser and
the Start Game! button is clicked, nothing seems to happen. There are no
error messages apperaing. Any help would be appreciated. Thanks.
Click to expand...

Try this:

<html>
<head>
<title>Andy_B.htm</title>
<script type="text/JavaScript">
<!--
var path = location.href.substr(0,location.href.lastIndexOf("/")+1)
var i;
var j;
var ind;
var AccClick=0;
var PlayingGame=0;
var GamePics = new Array(9);
GamePics[0] = new Image();
GamePics[0].src = path + "images/1.gif";
GamePics[1] = new Image();
GamePics[1].src = path + "images/2.gif";
GamePics[2] = new Image();
GamePics[2].src = path + "images/3.gif";
GamePics[3] = new Image();
GamePics[3].src = path + "images/4.gif";
GamePics[4] = new Image();
GamePics[4].src = path + "images/5.gif";
GamePics[5] = new Image();
GamePics[5].src = path + "images/6.gif";
GamePics[6] = new Image();
GamePics[6].src = path + "images/7.gif";
GamePics[7] = new Image();
GamePics[7].src = path + "images/8.gif";
GamePics[8] = new Image();
GamePics[8].src = path + "images/Blank.gif";
function Clicked(NumClicked) {
if (AccClick == 0) {
alert ("Please click on Start Game! to begin");
} else {
if (NumClicked == 1 && document.image1.src == GamePics[8].src)
Swap(0,1);
if (NumClicked == 1 && document.image3.src == GamePics[8].src)
Swap(0,3);
if (NumClicked == 2 && document.image0.src == GamePics[8].src)
Swap(1,0);
if (NumClicked == 2 && document.image2.src == GamePics[8].src)
Swap(1,2);
if (NumClicked == 2 && document.image4.src == GamePics[8].src)
Swap(1,4);
if (NumClicked == 3 && document.image1.src == GamePics[8].src)
Swap(2,1);
if (NumClicked == 3 && document.image5.src == GamePics[8].src)
Swap(2,5);
if (NumClicked == 4 && document.image0.src == GamePics[8].src)
Swap(3,0);
if (NumClicked == 4 && document.image4.src == GamePics[8].src)
Swap(3,4);
if (NumClicked == 4 && document.image6.src == GamePics[8].src)
Swap(3,6);
if (NumClicked == 5 && document.image3.src == GamePics[8].src)
Swap(4,3);
if (NumClicked == 5 && document.image1.src == GamePics[8].src)
Swap(4,1);
if (NumClicked == 5 && document.image5.src == GamePics[8].src)
Swap(4,5);
if (NumClicked == 5 && document.image7.src == GamePics[8].src)
Swap(4,7);
if (NumClicked == 6 && document.image2.src == GamePics[8].src)
Swap(5,2);
if (NumClicked == 6 && document.image4.src == GamePics[8].src)
Swap(5,4);
if (NumClicked == 6 && document.image8.src == GamePics[8].src)
Swap(5,8);
if (NumClicked == 7 && document.image7.src == GamePics[8].src)
Swap(6,7);
if (NumClicked == 7 && document.image3.src == GamePics[8].src)
Swap(6,3);
if (NumClicked == 8 && document.image6.src == GamePics[8].src)
Swap(7,6);
if (NumClicked == 8 && document.image4.src == GamePics[8].src)
Swap(7,4);
if (NumClicked == 8 && document.image8.src == GamePics[8].src)
Swap(7,8);
if (NumClicked == 9 && document.image5.src == GamePics[8].src)
Swap(8,5);
if (NumClicked == 9 && document.image7.src == GamePics[8].src)
Swap(8,7);
WinCheck();
}
}
function Swap(Pic1, Pic2) {
var imag1 = eval("document.image" + Pic1);
var imag2 = eval("document.image" + Pic2);
var spare = new Image();
spare.src = imag1.src;
imag1.src = imag2.src;
imag2.src = spare.src;
}
function Timer() {
if (PlayingGame == 1) {
document.forms[0].timeTaken.value++;
setTimeout('Timer()',1000);
}
}
function Shuffle() {
if (AccClick == 1) {
AccClick = 0;
PlayingGame = 0;
document.forms[0].startButton.value = "Start Game!";
document.forms[0].timeTaken.value = 0;
return;
} else {
AccClick = 1;
PlayingGame = 1;
document.forms[0].startButton.value = "Stop Game!";
for (j=0; j<8; j++) {
ind = Math.round(8*Math.random());
Swap(j,ind);
}
document.forms[0].timeTaken.value = 0;
Timer();
}
}
function WinCheck() {
var NoCorrect = 0;
for (i=0; i<8; i++) {
var imag0 = eval("document.image" + i);
if (imag0.src == GamePics.src) NoCorrect++;
}
if (NoCorrect == 8) {
document.forms[0].startButton.value = "Start Game!";
PlayingGame = 0;
AccClick = 0;
alert("Congratulations, You have completed the game! It
took"+document.forms[0].timeTaken.value+" seconds!");
document.forms[0].timeTaken.value = 0;
}
}
//-->
</script>
</head>
<body bgcolor="#000000">
<center>
<table class="content">
<tr>
<td class="innerContent">
<table width="300" border="0" cellspacing="1" cellpadding="1"
height="300">
<tr>
<td><A border='0' href="javascript:Clicked(1)"><img border='0'
name="image0" src="images/1.gif"></A></td>
<td><A border='0' href="javascript:Clicked(2)"><img border='0'
name="image1" src="images/2.gif"></A></td>
<td><A border='0' href="javascript:Clicked(3)"><img border='0'
name="image2" src="images/3.gif"></A></td>
</tr>
<tr>
<td><A border='0' href="javascript:Clicked(4)"><img border='0'
name="image3" src="images/4.gif"></A></td>
<td><A border='0' href="javascript:Clicked(5)"><img border='0'
name="image4" src="images/5.gif"></A></td>
<td><A border='0' href="javascript:Clicked(6)"><img border='0'
name="image5" src="images/6.gif"></A></td>
</tr>
<tr>
<td><A border='0' href="javascript:Clicked(7)"><img border='0'
name="image6" src="images/7.gif"></A></td>
<td><A border='0' href="javascript:Clicked(8)"><img border='0'
name="image7" src="images/8.gif"></A></td>
<td><A border='0' href="javascript:Clicked(9)"><img border='0'
name="image8" src="images/Blank.gif"></A></td>
</tr>
</table>
</td>
</tr>
</table>
<form>
<p>
<input type='button' type='button' name='startButton' value='Start
Game!' onclick='Shuffle()'>
<font color="#FFFFFF" size="3" face="Geneva, Arial, Helvetica,
sans-serif">You have taken &nbsp;
<input class='box' type='text' name='timeTaken' readonly size='2'
value='0'> seconds</font><br>
</p>
<p>&nbsp; </p>
<table width="100%" border="0">
<tr>
<td width="72%"><font color="#FFFFFF" size="3" face="Verdana, Arial,
Helvetica, sans-serif">The completed picture looks like this: </font></td>
<td width="28%"><font face="Verdana, Arial, Helvetica,
sans-serif"><img src="Images/Complete.gif" width="200" height="200"
border="2"></font></td>
</tr>
</table>
<p>&nbsp;</p>
</form>
</body>
</html>
Click to expand...


Yep, that's what I tried. It was the document.image and document.forms[0]
that I was missing. Thanks to everyone for taking the time to help, it is
really appreciated
Thanks again
Andy
 
B

Brian Genisio

Also note that your method of choosing random numbers doesn't really
work the way you (and possibly your instructor) think it does:
ind=Math.round(8*Math.random());
chooses 0 and 8 half as often as any of the other values.
Click to expand...

Ah ha! When I first looked at it, I couldnt figure out what you
meant... Then I tried it, and sure enough, you are right.

I understand what the problem is now... only 0 -> 0.049 will evaluate to
0, where 0.05 -> 0.149 will evalutate to 1, and 7.5->7.99 will evalutate
to 8, giving 9 possible values (not 8), where 0 and 8 split the likelyess.

For the OP, here is a more even distribution for your random function:

Math.floor(Math.random() * 8)

This will give values between 0 and 7 (8 values). If you need between 0
and 8 (9 values), change the 8 to a 9.

Brian
 
R

Richard Cornford

function Swap(Pic1, Pic2) {
var imag1 = eval("document.image" + Pic1);
var imag2 = eval("document.image" + Pic2);
Click to expand...
<snip>

It is *never* necessary (or desirable) to construct a dot notation
property accessor as a string and then use - eval - to resolve it. It is
always possible to use a bracket notation property accessor instead, and
doing so is considerably more efficient and reliable in more contexts
(such as environments implementing the ECMAScript compact profile (ECMA
327)). In this case:-

var imag1 = document["image" + Pic1];
var imag2 = document["image" + Pic2];

- are directly equivalent property accessors to the constructed dot
notation versions and as they use a standard language syntax there is no
need to mess around with - eval - to use them.

<URL: http://jibbering.com/faq/#FAQ4_39 >

There are almost no circumstances under with the use of - eval - are
appropriate and generally its use is indicative of a failure to
recognise a much better alternative.

<URL: http://jibbering.com/faq/#FAQ4_40 >

Richard.
 
M

McKirahan

Richard Cornford said:
function Swap(Pic1, Pic2) {
var imag1 = eval("document.image" + Pic1);
var imag2 = eval("document.image" + Pic2);
Click to expand...
<snip>

It is *never* necessary (or desirable) to construct a dot notation
property accessor as a string and then use - eval - to resolve it. It is
always possible to use a bracket notation property accessor instead, and
doing so is considerably more efficient and reliable in more contexts
(such as environments implementing the ECMAScript compact profile (ECMA
327)). In this case:-

var imag1 = document["image" + Pic1];
var imag2 = document["image" + Pic2];

- are directly equivalent property accessors to the constructed dot
notation versions and as they use a standard language syntax there is no
need to mess around with - eval - to use them.

<URL: http://jibbering.com/faq/#FAQ4_39 >

There are almost no circumstances under with the use of - eval - are
appropriate and generally its use is indicative of a failure to
recognise a much better alternative.

<URL: http://jibbering.com/faq/#FAQ4_40 >

Richard.
Click to expand...


I knew I was going to catch flack for using eval().

Since you've shown how to use the preferred alternative, I'll sin no more.
 
D

Dr John Stockton

JRS: In article <[email protected]>, seen in
news:comp.lang.javascript said:
if (NumClicked == 1 && image[1].src == GamePics[8].src)
Swap(0,1);
Click to expand...

and lots more similar. Anything that repetitive should be done with a
function or array.


function Xx(a, b, c, d, e) { // done once
if (NumClicked == a && image.src == GamePics[c].src) Swap(d,e) }

Xx(1,1,8,0,1)
...

or

var D = [
[1,1,8,0,1],
...,
[...]]

for (j=0;j<d.length;j++) { T = D[j]
if (NumClicked == T[0] && image[T[1]].src == GamePics[T[2]].src)
Swap(T[3],T[4]) }
 
B

Bas Cost Budde

Andy B wrote:

[snip huge quote]

Welcome, but consider trimming your quotes. Especially when you say
"thank you" that is often the first thing people like to see ;-) don't
hide it below the quote.
 

Ask a Question

Want to reply to this thread or ask your own question?

You'll need to choose a username for the site, which only take a couple of moments. After that, you can post your question and our members will help you out.

Ask a Question

Similar Threads

Can someone tell me if this a real tracker? Or is it one designed to show you a different message at certain times, ie. acting like one? 0
Uncaught ReferenceError: item is not defined at HTMLButtonElement.onclick in the: <button onclick="item.inserir()">Inserir dados</button> 1
Help with Visual Lightbox: Scripts 2
Help with my responsive home page 2
How to have two html audio players on one page? 0
SendGrid email issue in responsive Gmail 1
Filter table rows based on multiple checkboxes value 2
Slideshow not working properly 2

Members online

No members online now.
Total: 23 (members: 1, guests: 22)
Robots: 154

Forum statistics

Threads
473,777
Messages
2,569,604
Members
45,234
Latest member
SkyeWeems

Latest Threads

Top