for loop in function calling another function

[zimitry]

Hi

i currently have this code for an assignment i'm working my way through

var flightTimes = new Array ('07:15', '08:05', '08:30', '09:30', '09:50', '10:00', '10:20', '10:45', '11:00', '12:00', '13:00', '13:05', '13:15', '13:35', '14:10', '15:00', '16:00', '16:40', '16:55', '17:00', '19:30', '21:00', '22:00', '23:20', '23:59');
       var flightDestinations = new Array ('Arkham', 'Hensonville', 'Stepford', 'Maycomb', 'Bay Port', 'Denton', 'Waldon City', 'Denton', 'Denton', 'Waldon City', 'Arkham', 'Maycomb', 'Bay Port', 'Stepford', 'Denton', 'Maycomb', 'Waldon City', 'Hensonville', 'Denton', 'Denton', 'Stepford', 'Bay Port', 'Hensonville', 'Waldon City', 'Arkham');
       var flightOperators = new Array ('Miskatonic', 'SuperSwift', 'Aalto', 'DynaJet', 'Oceanic', 'Aalto', 'Aalto', 'MidSouthCentral Air', 'SuperSwift', 'SuperSwift', 'Miskatonic', 'DynaJet', 'Miskatonic', 'SuperSwift', 'MidSouthCentral Air', 'Aalto', 'SuperSwift', 'Aalto', 'MidSouthCentral Air', 'Aalto', 'DynaJet', 'Oceanic', 'SuperSwift', 'Aalto', 'Miskatonic');
var flightFares = new Array (300, 100, 150, 75, 250, 150, 75, 125, 100, 25, 600, 75, 400, 250, 300, 250, 150, 200, 125, 150, 75, 100, 100, 50, 300);
 
function displayMessage(aMessage)
       {
                 document.getElementById('theFlights').innerHTML = aMessage;
       }
 
function showTimetable()
       {
 
           for (var i = 0; i <= flightTimes.length; i = i + 1)
 
               {
                   if (i <= flightTimes.length - 1)
                        {
                        displayMessage((i + 1) + ' ' + flightTimes[i] + ' to ' + flightDestinations[i] + ' operated by ' + flightOperators[i] + '. $' + flightFares[i] + '<BR>');
                        }
               }
 
 
           /*for (var i = 0; i <= flightTimes.length; i = i + 1)
 
               {
                   if (i <= flightTimes.length - 1)
                        {
                        document.write((i + 1) + ' ' + flightTimes[i] + ' to ' + flightDestinations[i] + ' operated by ' + flightOperators[i] + '. $' + flightFares[i] + '<BR>');
                        }
               }*/
 
        }
 
 

the trouble i'm having is getting the displayMessage to write down all of the Array's in a <DIV>

it works fine when you do document.write but when you do display message it works it way through the arrays only displaying the last one?

i can't really think of which way to do it if i'm doing it correct in the first place?

any help is much appreciated

z

[Logician]

the trouble i'm having is getting the displayMessage to write down all of the Array's in a <DIV>

it works fine when you do document.write but when you do display message it works it way through the arrays only displaying the last one?

i can't really think of which way to do it if i'm doing it correct in the first place?

any help is much appreciated

z

Try

document.getElementById('theFlights').innerHTML += aMessage;

[zimitry]

Cool that worked!

i did have a workaround of this

var myString1 = ('1 ' + flightTimes[0] + ' to ' + flightDestinations[0] + ' operated by ' + flightOperators[0] + '. $' + flightFares[0] + '<BR>' + 
           '2 ' + flightTimes[1] + ' to ' + flightDestinations[1] + ' operated by ' + flightOperators[1] + '. $' + flightFares[1] + '<BR>' + 
           '3 ' + flightTimes[2] + ' to ' + flightDestinations[2] + ' operated by ' + flightOperators[2] + '. $' + flightFares[2] + '<BR>' +
           '4 ' + flightTimes[3] + ' to ' + flightDestinations[3] + ' operated by ' + flightOperators[3] + '. $' + flightFares[3] + '<BR>' +
           '5 ' + flightTimes[4] + ' to ' + flightDestinations[4] + ' operated by ' + flightOperators[4] + '. $' + flightFares[4] + '<BR>' );
 
           var myString2 = ('6 ' + flightTimes[5] + ' to ' + flightDestinations[5] + ' operated by ' + flightOperators[5] + '. $' + flightFares[5] + '<BR>' + 
           '7 ' + flightTimes[6] + ' to ' + flightDestinations[6] + ' operated by ' + flightOperators[6] + '. $' + flightFares[6] + '<BR>' + 
           '8 ' + flightTimes[7] + ' to ' + flightDestinations[7] + ' operated by ' + flightOperators[7] + '. $' + flightFares[7] + '<BR>' +
           '9 ' + flightTimes[8] + ' to ' + flightDestinations[8] + ' operated by ' + flightOperators[8] + '. $' + flightFares[8] + '<BR>' +
           '10 ' + flightTimes[9] + ' to ' + flightDestinations[9] + ' operated by ' + flightOperators[9] + '. $' + flightFares[9] + '<BR>' );
 
           var myString3 = ('11 ' + flightTimes[10] + ' to ' + flightDestinations[10] + ' operated by ' + flightOperators[10] + '. $' + flightFares[10] + '<BR>' + 
           '12 ' + flightTimes[11] + ' to ' + flightDestinations[11] + ' operated by ' + flightOperators[11] + '. $' + flightFares[11] + '<BR>' + 
           '13 ' + flightTimes[12] + ' to ' + flightDestinations[12] + ' operated by ' + flightOperators[12] + '. $' + flightFares[12] + '<BR>' +
           '14 ' + flightTimes[13] + ' to ' + flightDestinations[13] + ' operated by ' + flightOperators[13] + '. $' + flightFares[13] + '<BR>' +
           '15 ' + flightTimes[14] + ' to ' + flightDestinations[14] + ' operated by ' + flightOperators[14] + '. $' + flightFares[14] + '<BR>' );
 
           var myString4 = ('16 ' + flightTimes[15] + ' to ' + flightDestinations[15] + ' operated by ' + flightOperators[15] + '. $' + flightFares[15] + '<BR>' + 
           '17 ' + flightTimes[16] + ' to ' + flightDestinations[16] + ' operated by ' + flightOperators[16] + '. $' + flightFares[16] + '<BR>' + 
           '18 ' + flightTimes[17] + ' to ' + flightDestinations[17] + ' operated by ' + flightOperators[17] + '. $' + flightFares[17] + '<BR>' +
           '19 ' + flightTimes[18] + ' to ' + flightDestinations[18] + ' operated by ' + flightOperators[18] + '. $' + flightFares[18] + '<BR>' +
           '20 ' + flightTimes[19] + ' to ' + flightDestinations[19] + ' operated by ' + flightOperators[19] + '. $' + flightFares[19] + '<BR>' );
 
           var myString5 = ('21 ' + flightTimes[20] + ' to ' + flightDestinations[20] + ' operated by ' + flightOperators[20] + '. $' + flightFares[20] + '<BR>' + 
           '22 ' + flightTimes[21] + ' to ' + flightDestinations[21] + ' operated by ' + flightOperators[21] + '. $' + flightFares[21] + '<BR>' + 
           '23 ' + flightTimes[22] + ' to ' + flightDestinations[22] + ' operated by ' + flightOperators[22] + '. $' + flightFares[22] + '<BR>' +
           '24 ' + flightTimes[23] + ' to ' + flightDestinations[23] + ' operated by ' + flightOperators[23] + '. $' + flightFares[23] + '<BR>' +
           '25 ' + flightTimes[24] + ' to ' + flightDestinations[24] + ' operated by ' + flightOperators[24] + '. $' + flightFares[24] + '<BR>' );
 
          displayMessage(myString1 + '<BR>' + myString2 + '<BR>' + myString3 + '<BR>' + myString4 + '<BR>' + myString5);
 

is there anything i could insert to put a gap every 5 lines? ie

if (flightTimes[i] == 5)
{
<BR>
}
 

[zimitry]

on Adding the + sign to the code everything i click a link button the text never clears the screen it just keeps adding to it? is this right

z

[pbmods]

on Adding the + sign to the code everything i click a link button the text never clears the screen it just keeps adding to it? is this right

Yup. If you want to clear the text from it, put a line similar to this one before your loop executes:

var $theFlights = document.getElementById('theFlights');
while($theFlights.firstChild)
    $theFlights.removeChild($theFlights.firstChild);
 

[zimitry]

Is there no way in which this can be done without chaing the actual function as it's used for other tasks?

z

[acoder]

Is there no way in which this can be done without chaing the actual function as it's used for other tasks?

z

You mean showTimetable()? Does it matter if "theFlights" div is cleared before you show the timetable?

[acoder]

is there anything i could insert to put a gap every 5 lines? ie

if (flightTimes[i] == 5)
{
<BR>
}
 

if (i % 5 == 0)

% is the modulus operator. It gives the remainder.

[zimitry]

What i mean is the there no way in doing this without chaning the function displayMessage?

 
function displayMessage(aMessage)
       {
                 document.getElementById('theFlights').innerHTML = aMessage;
       }
 
 

[acoder]

Just define and call another function if you don't want to change the function.