[RESOLVED] Javascript headache PLEASE HELP

[SirEvenhand]

I am working on a lottery for my web site. The code below works just fine on FireFox but keeps pulling an error in IE.

Can anyone examine the code and help me fix the error so that it works in IE also.

Thanks in advance.

The Javascript error that IE gives me is
ERROR: 'document.lp.text' is null or not an object

here is the js code for the head section and below is the html

<script type="text/javascript">


function Timer()
{
    setTimeout('',10000);
    /*
    for(i=0;i<10000;i++) { ; } // null
    */
}

function ClearBoard()
{
    for (i = 1; i <= 70; i++)
    {
        if ( document.lp["num" + i] ) 
            document.lp["num" + i].checked = 0;
    }
    RefreshText();
}

function SelectNumber(number)
{
    var IsChecked = 0;
    if (!document.lp["num" + number]) 
    { return; }
    for( j=1;j<number;j++)
    { 
        if (document.lp["num" + j].checked) 
        { 
            continue; 
            Timer();
        }
        document.lp["num" + j].checked =1;
        Timer();
        document.lp["num" + j].checked =0;
    }
    document.lp["num" + j].checked=1;
    RefreshText();
}     


function readCookie(name)
{
    var nameEQ = name + "=";
    var ca = document.cookie.split(';');
    for(var i=0;i < ca.length;i++)
    {
        var c = ca[i];
        while (c.charAt(0)==' ') c = c.substring(1,c.length);
        if (c.indexOf(nameEQ) == 0) return c.substring(nameEQ.length,c.length);
    }
    return null;
}


function RefreshText()
{
    var IsNotFirst = 0;
    document.lp["text"].value = "";
    for (i = 1; i <= 70; i++)
    {
        if (document.lp["num" + i]) 
        {
            var cb = document.lp["num" + i].checked;
            if (cb)
            {
                if ( IsNotFirst )
                    document.lp["text"].value = document.lp["text"].value + ",";
                document.lp["text"].value = document.lp["text"].value + i;
                IsNotFirst = 1;
            }
        } 
    }
}

function countChecked()
{
    var count = 0;
    for (i = 1; i <= 70; i++)
    {
        if (document.lp["num" + i]) 
        {
            var cb = document.lp["num" + i].checked;
            if (cb)
                count++;
        }
    }
    return count;
}

function checkClick(el)
{
    if (countChecked() > 6)
    {
        alert(" Please choose six numbers ")
        el.checked = false;
    } 
    else 
    {
        document.getElementById('clickedhotpicks').value = 0;
    }
    RefreshText();
}    

function RefreshText()
{
    var IsNotFirst = 0;
    document.lp["text"].value = "";
    for (i = 1; i <= 70; i++)
    {
        if (document.lp["num" + i]) 
        {
            var cb = document.lp["num" + i].checked;
            if (cb)
            {
                if ( IsNotFirst )
                    document.lp["text"].value = document.lp["text"].value + ",";
                document.lp["text"].value = document.lp["text"].value + i;
                IsNotFirst = 1;
            }
        } 
    }
}

function countChecked()
{
    var count = 0;
    for (i = 1; i <= 70; i++)
    {
        if (document.lp["num" + i]) 
        {
            var cb = document.lp["num" + i].checked;
            if (cb)
                count++;
        }
    }
    return count;
}


function GetTextData()
{
    var IsNotFirst = 0;
    document.lp["text"].value = "";
    for (i = 1; i <= 70; i++)
    {
        if (document.lp["num" + i]) 
        {
            var cb = document.lp["num" + i].checked;
            if (cb)
            {
                if ( IsNotFirst )
                    document.lp["text"].value = document.lp["text"].value + ",";
                document.lp["text"].value = document.lp["text"].value + i;
                IsNotFirst = 1;
            }
        } 
    }
    return document.lp["text"].value;
}




function checkClick(el)
{
    numChecked = countChecked();
    numpicked = GetTextData();
    if (numChecked > 6)
    {
        alert(" Please choose six numbers only. ")
        el.checked = false;
    } else if (numChecked == 6) {

    var agree=confirm(" You have selected "+numpicked+"")
if (agree)
document.lp.submit();


} else {
    document.getElementById('clickedhotpicks').value = 0;
}
RefreshText();
}    

function checkform(f) {
        var numbers;
        var chosen = 0;
        for (var i=1;i<=70;i++) {
            if (f['num'+i] && f['num'+i].checked) {
               chosen++;
            }
        }
        var needed = 6;
        if (chosen != needed) {
           alert("Please choose "+needed+" numbers");
           return false;
        }
        return true;
}
</script>

And below is the html

<form METHOD="POST" NAME="lp" ACTION="./lottery.php" onsubmit="return checkform(this)">
<input type="hidden" name="clickedhotpicks" id="clickedhotpicks" value="0">

<table width="400" align="center" border="1" bgcolor="#FF0000">

  <tr>
    <td width="400" style="vertical-align: bottom; padding: 0;">

  <span style="display: block; margin: 2px auto; font-family: Arial, Helvetica, sans-serif; font-size: 13px; font-weight: bold; text-align: center;">
    Pick Any 6 Numbers.
  </span>

  <table width="400">

    <tr>

      <td width="400" height="124">
        <table class="board" width="400" height="124">
          <tbody>
            <tr>

              <td class="boardcell">
<input onClick="checkClick(this);" type="checkbox" name="num1" />1
                  </td>

              <td class="boardcell">
<input onClick="checkClick(this);" type="checkbox" name="num2" />2
                  </td>


              <td class="boardcell">
<input onClick="checkClick(this);" type="checkbox" name="num3" />3
                  </td>

              <td class="boardcell">
<input onClick="checkClick(this);" type="checkbox" name="num4" />4
                  </td>

              <td class="boardcell">
<input onClick="checkClick(this);" type="checkbox" name="num5" />5
                  </td>

              <td class="boardcell">

<input onClick="checkClick(this);" type="checkbox" name="num6" />6
                  </td>

              <td class="boardcell">
<input onClick="checkClick(this);" type="checkbox" name="num7" />7
                  </td>

	</tr><tr>


              <td class="boardcell">
<input onClick="checkClick(this);" type="checkbox" name="num8" />8
                  </td>


              <td class="boardcell">

<input onClick="checkClick(this);" type="checkbox" name="num9" />9
                  </td>

              <td class="boardcell">
<input onClick="checkClick(this);" type="checkbox" name="num10" />10
                  </td>

              <td class="boardcell">
<input onClick="checkClick(this);" type="checkbox" name="num11" />11
                  </td>

              <td class="boardcell">

<input onClick="checkClick(this);" type="checkbox" name="num12" />12
                  </td>


              <td class="boardcell">
<input onClick="checkClick(this);" type="checkbox" name="num13" />13
                  </td>

              <td class="boardcell">
<input onClick="checkClick(this);" type="checkbox" name="num14" />14
                  </td>

	</tr>


</form>
                <tr>
                  <td colspan="13" align="center" valign="middle">

                <label>
                  Your numbers are: <input name="text" type="text" readonly>
                </label>
              </td>
            </tr>
          </tbody>
        </table>

      </td>

    </tr>
  </table>

</td>

  </tr>


</table>

[bpat1434]

Moved to the appropriate forum.

[johanafm]

Step 1: Validate your code.

Step 2: Remove redundant function definitions (e.g. function RefreshText())

Step 3: Clear up the code. Code that does nothing should not be there to begin with.
window.SetTimeout(f, t)
t is the time to wait before running function f. But this "waiting" is done in a separate thread. Your code does not actually wait for the timeout to complete before continuing (if that was what you were after).
This means that SetTimeout('', t) will do nothing after t milliseonds. So why not do nothing straight away, or in other words remove the function Timer() and all calls to it.

Since the debugging features are greately improved in IE8 over previous IE versions, always test your code in IE8, even if you eventually want the code to run in IE 5, 6, 7 as well.
Tools->Developer Tools->Script (tab)->Start Debugging
Refresh page.
click a checkbox

document.lp.text is null or not an object.

The reason should be obvious. And if it isn't, then it should be when you validate your code. If it still isn't, just validate the code and fix the errors.

[SirEvenhand]

Thanks johanafm

Everything worked once I got it cleaned up and found out what was calling the undeclared vars.