JavaScript Forum

**simster** · 11-30-2007, 07:58 AM

can someone tell me whats wrong with my code and how to fix it

Code:

<html>
<head>
    
 <title>Christmas Tree</title>

</head>

<body>
<center>
<script>
var width=1;
for (i=o, i <= 5, i++)
{
for (x=0, x<=4, x++)
{
for (y=0, y<=width, y++)
{
var Number=Math.random()*10;
var Ornament.Math.round(Number);
if (Ornament<=1)
{
document.write("O");
}
if (Ornament<=2)
{
document.write("X");
}
}
document.write("<br>");
width=width+1
}
width=width-1
}

</script>
</center>
</body>
</html>

**tripy** · 11-30-2007, 08:51 AM

And what if you'd rather tell us what is wrong and what you want to achieve ?
Rather than having us try to understand what you wanted to do ?

**simster** · 11-30-2007, 10:59 PM

its the christmas tree script from "learn how to program using any web browser" and i copied it from the book to see if it works (before i edit it), and it doesnt work.

**chrishirst** · 12-01-2007, 04:06 AM

Mainly typing errors by the look of it

1/ for (i=o ... ) should be for (i=0

2/ the commas in the for loops should be semi-colons

3/ var Ornament.Math.round(Number); should be var Ornament = Math.round(Number);

4/ missing type in the <script> tag

**Vivi** · 12-03-2007, 11:36 AM

for (i=o, i <= 5, i++)

It is o should be is zero?? not a alphabert "o"??