Infinite Loop!!!!

kosmokramer · Sep 28, 2002

Can someone tell me what is wrong with this script? I keep getting an infinite loop.

//This is the calendar data.js file
if (null==dataSet) dataSet = new Array();

dataSet[dataSet.length] = "August";
dataSet[dataSet.length] = "Meeting ,1st,9:00,Hankamer";
dataSet[dataSet.length] = "Service Rush,5th,7:00,Place TBA";
dataSet[dataSet.length] = "Social Rush ,8th,6:00,Place TBA";
dataSet[dataSet.length] = "Informal Rush ,11th,8:00,Place TBA";
dataSet[dataSet.length] = "September";
dataSet[dataSet.length] = "Something Else,1st,9:00,Hankamer";
dataSet[dataSet.length] = "Another Thing ,5th,7:00,Place TBA";
dataSet[dataSet.length] = "Blah,8th,6:00,Place TBA";
dataSet[dataSet.length] = "eworuewr,11th,8:00,Place TBA";
dataSet[dataSet.length] = "October";
dataSet[dataSet.length] = "Meething 3,28th,9:00,Hankamer";
dataSet[dataSet.length] = "eworuewewrwe,5th,7:00,Place TBA";
dataSet[dataSet.length] = "Yet another thing,8th,6:00,Place TBA";
dataSet[dataSet.length] = "Something else 2,11th,8:00,Place TBA";

//This is the regular page
<html>
<head>
<script language="JavaScript">
var months = new makeArray("January","February","March","April","May","June","July","August","September","October","November","December&quot

;
var month = new Date()
var tempMonth = month.getMonth()

function makeArray() //This function just sets up the necessary space for the month array
{
for (var i = 0; i<makeArray.arguments.length; i++)
this = makeArray.arguments;
}

dataSet = new Array() //This is a seperate object. This is creating an array for use with the calendar data.js file
</script>

<script language="JavaScript" type="text/javascript" src="calendar data.js"></script>
</head>

<body>
<table border=0 cellpadding=0 width=65% style="margin-left:30pt">
<script language="JavaScript" type="text/javascript">
//////////////////////////////////////////////////////////////
//This if statement writes out the table for the current month
//////////////////////////////////////////////////////////////
var doMonthLoop = false
var indexOfFirstMonth = dataSet.length;

document.write("doMonthLoop= " + doMonthLoop + " " //Testing variables
document.write("IndexofFirstMonth= " + indexOfFirstMonth + " "
document.write("dataset.length= " + dataSet.length + " "
for(var n=0; n<dataSet.length || doMonthLoop == true; n++)
{
document.write("doMonthLoop2= " + doMonthLoop + " " //Testing variables again
document.write("IndexofFirstMonth2= " + indexOfFirstMonth + " "
document.write("dataset.length2= " + dataSet.length + " "

if (dataSet[n] == months[tempMonth])
{
doMonthLoop = true
indexOfFirstMonth = n
}
}
document.write("doMonthLoop3= " + doMonthLoop + " " //Third testing of variables
document.write("IndexofFirstMonth #3= " + indexOfFirstMonth + ""
if(doMonthLoop == true && indexOfFirstMonth != dataSet.length)
{
document.write("<tr>" +
"<td colspan=4 style=background:yellow height=36>"+
"" +
"" + dataSet[indexOfFirstMonth] + ""+
"" +
"</td>" +
"</tr>" +

"<tr>" +
"<td style=background:#9999CC height=22>"+
"Event"+
"</td>" +

"<td style=background:#9999CC height=22>"+
"Day"+
"</td>" +

"<td style=background:#9999CC height=22>"+
"Time"+
"</td>" +

"<td style=background:#9999CC height=22>"+
"Place"+
"</td>"+
"</tr>"

for(var i=indexOfFirstMonth+1; i<dataSet.length && dataSet != months[tempMonth+1]; i++)
{
var tempString = dataSet .split(',');
if(i%2 == 1)
{
document.write("<tr>"+
"<td style=background:#CCCCFF height=19>"+
tempString[0] +
"</td>"+

"<td style=background:#CCCCFF height=19>"+
tempString[1] +
"</td>"+

"<td style=background:#CCCCFF height=19>"+
tempString[2] +
"</td>"+

"<td style=background:#CCCCFF height=19>" +
tempString[3] +
"</td>" +
"</tr>"
}
else
{
document.write("<tr>"+
"<td style=background:#9999CC height=19>"+
tempString[0] +
"</td>"+

"<td style=background:#9999CC height=19>"+
tempString[1] +
"</td>"+

"<td style=background:#9999CC height=19>"+
tempString[2] +
"</td>"+

"<td style=background:#9999CC height=19>" +
tempString[3] +
"</td>" +
"</tr>"
}
}
}
//////////////////////////////////////////////////////
//If it is not the current month, this will print out
//////////////////////////////////////////////////////
var doMonthLoop2 = false
var indexOfNextMonth = dataSet.length;

document.write("doMonthLoop2= " + doMonthLoop + " " //These are testing variables
document.write("IndexofNextMonth= " + indexOfNextMonth + " "
document.write("dataset.length= " + dataSet.length + " "
for(var n=0; n<dataSet.length || doMonthLoop2 == false; n++)
{
document.write("doMonthLoop2 #2= " + doMonthLoop2 + " " //Testing variables again
document.write("IndexofNextMonth2= " + indexOfNextMonth + " "
document.write("dataset.length2= " + dataSet.length + " "
if (dataSet[n] == months[tempMonth+1])
{
doMonthLoop2 = true
indexOfNextMonth = n
}
}
document.write("doMonthLoop2 #3= " + doMonthLoop2 + " " //Third testing of variables
document.write("IndexofNextMonth3= " + indexOfNextMonth + " "
document.write("dataset.length3= " + dataSet.length + " "
if(doMonthLoop2 == true && indexOfNextMonth != dataSet.length)
{
document.write("<tr>" +
"<td colspan=4 style=background:yellow height=36>"+
"" +
"" + dataSet[indexOfNextMonth] + ""+
"" +
"</td>" +
"</tr>"

for(var i=indexOfNextMonth+1; i<dataSet.length && dataSet != months[tempMonth+2]; i++)
{
var tempString2 = dataSet .split(',');
if(i%2 == 1)
{
document.write("<tr>"+
"<td style=background:#CCCCFF height=19>"+
tempString2[0] +
"</td>"+

"<td style=background:#CCCCFF height=19>"+
tempString2[1] +
"</td>"+

"<td style=background:#CCCCFF height=19>"+
tempString2[2] +
"</td>"+

"<td style=background:#CCCCFF height=19>" +
tempString2[3] +
"</td>" +
"</tr>"
}
else
{
document.write("<tr>"+
"<td style=background:#9999CC height=19>"+
tempString2[0] +
"</td>"+

"<td style=background:#9999CC height=19>"+
tempString2[1] +
"</td>"+

"<td style=background:#9999CC height=19>"+
tempString2[2] +
"</td>"+

"<td style=background:#9999CC height=19>" +
tempString2[3] +
"</td>" +
"</tr>"
}

}
document.write("<tr>"+
"<td style=background:#CCCCFF height=34>"+
"* FBC – on 5th St., Room #103 "+
"**Columbus Ave. Baptist Church Activities Center "+
"</td>"+

"<td style=background:#CCCCFF height=34>"+
"</td>"+

"<td style=background:#CCCCFF height=34>"+
"</td>"+

"<td style=background:#CCCCFF height=34>"+
"</td>"+
"</tr>"+
"</table>&quot
}
else
{
document.write("<tr>" +
"<td colspan=4 style=background:yellow height=36>"+
"" +
"There are no events currently scheduled!"+
"" +
"</td>" +
"</tr>"+
"</table>"
}
</script>

</body>
</html>

BodieBoots · Sep 29, 2002

This looks like an infinite loop to me because if the doMonthLoop == true when this starts, nothing then sets it to false and so this will keep looping.

for(var n=0; n<dataSet.length || doMonthLoop == true; n++)
{
document.write("doMonthLoop2= " + doMonthLoop + " &quot

; //Testing variables again
document.write("IndexofFirstMonth2= " + indexOfFirstMonth + " &quot

;
document.write("dataset.length2= " + dataSet.length + " &quot

;

if (dataSet[n] == months[tempMonth])
{
doMonthLoop = true
indexOfFirstMonth = n
}
}

kosmokramer · Sep 30, 2002

Thanks. I tried that earlier actually, and it did not seem to work. I have tried all sorts of data in the calendar data.js file and with some it will work fine, but with others it keeps looping (for example if I have the currrent month, skip a month, and put in the following month).

Tek-Tips is the largest IT community on the Internet today!

Members share and learn making Tek-Tips Forums the best source of peer-reviewed technical information on the Internet!

Infinite Loop!!!!

kosmokramer

Programmer

BodieBoots

Programmer

kosmokramer

Programmer

Similar threads

Part and Inventory Search

Sponsor