I'm maintaining some code that has far too many document.write
instances. For example, the code would document.write
a div, then use the DOM to find that div, then append an <img>
to开发者_如何学JAVA it.
I've tried to refactor a relevant piece to look more like this:
var node_to_append = document.createElement("div");
//node_name is a hardcoded constant
node_to_append.id = node_name;
var i = 0;
for( i = 0; i < request_urls.length; i++) {
var image_to_append = document.createElement( "img" );
image_to_append.width = 1;
image_to_append.height = 1;
image_to_append.alt = "";
image_to_append.src = request_urls[ i ];
node_to_append.appendChild( image_to_append );
}
//finally, append the div to the HTML doc
document.body.appendChild( node_to_append );
This is throwing an error in IE, saying "Internet Explorer can not open the Internet site http://blah.com. Operation aborted."
I'm told this is because I needed to indeed use document.write
. I'm hoping there's an alternative that would allow me to use the above approach and not force me to turn my node into a string (e.g. "") and append it via document.write
. Can anyone suggest a solution to my problem?
If this code had document.writes in it, then it was being run while the page was being loaded/parsed, wasn't it? You should wrap your code in a window.onload event handler. Even better, use jQuery's ready event, or another lib's dom ready wrapper.
精彩评论