Tag names or possibly class names.

I tried with TagNames, but not working as expected

IDs are unique, i.e. only one element can have the same ID. Try to approach it some other way then.

only the first occurance is replaced, but the subsequent occurances are not replaced. Any pointers on this is highly appreciated

It’s just a matter of having a variable and appending the string content for each loop iteration. You need to reference the element with document.getElementById(“element-id”).innerHTML = yourStringVariableWithHTML

This is a report which pops out in the preview mode. So, in this case how do I write using .innherHTML.

Also, I’m storing all the values in the array, so without the for loop am not sure how to put them to the report.

Any pointers would be really appreciated

Deepak

I’m sure the problem lies in document.write, where it for each instance has to write to the document. Try adding the values to a variable, and then, after the loop, write to the document (and rather by using .innerHTML on the element where you want to insert content).

Also, that many lines with such a huge number is destined to become slow in a web browser.

Good luck!

I have used all the different types of looping suggested by Robert, but the results have not changed.

The Code is

***************************************
temp = (totalexists==1)? bookinggridcols+1 : bookinggridcols-1;
for (i=0;i<=temp;i++)
document.write("” + bookinggridvalues[i] + “”);
****************************************

This spits out a huge table of 3,654 rows with each row containing 35 cells.

I find that the above code, initially starts spitting out each row in 0.015 secs but as the file starts to grow, eventually it takes 1.5 secs for each row (after it has done around 1000 rows), and magically the last 600 odd rows becomes faster at around less 0.2 secs for each row.

The time I have quoted is for actually printing each row, and there is no calculation involved as the values are stored in an array. The array size is always maintained at 35 in this example, but is free to grow if required. After each for loop, there is other code which populates the array, and that time is not included in the above times I have mentioned.

Any help in this regard will be hugely appreciated.

I personally have decided to switch to using example three except reversed and counting down to zero. I am not sure if there is any speed difference, but if there is then I need to boost as much performance as I can out of the loop for my application of it.

I have no concrete example here, but from what I know it’s not faster. Another thing to generally think about with for (var i in) loops is that it goes through all data types and members in that collection, often resulting in unexpected behavior and/or values.

Therefore, you would need to add a check to each iteration (for instance, to make sure it’s really a string you’re trying to manipulate), and that would definitely hurt the performance.

for (var i in items) {
// Do something.
}

Generally, looping backwards is very fast, but not always suitable in all scenarios.

Absolutely, that can be a good option.

Suresh,

Glad to hear that it helped!

Thanks
Baba

Thanks for your comment!
While I don’t necessarily agree about readability, you have some thought-worthy points.