Ошибка из за яваскрипта, не работает в эксплорере

[virgin]

я решил использовать скрипт в блоггере, который заменяет слово на другое. Вместо слово Анонимный, я заменяю на другое слово, когда оставляют комментарии без имени.

Скрипт не хочет работать в эксплорере, и не заменяет слово Анонимный на нужно, остаётся всё по прежднему, и плюс к этому в эксплорере появилась ошибка на странице, из за чего это происходит?

скрипт такой:

<script type='text/javascript'> 
	var authors = document.getElementById('comments').getElementsByClassName('comment-author')
	for (key in authors) {
		var author = authors[key]
		if (author.innerHTML) {
			author.innerHTML = author.innerHTML.replace(/Анонимный/g, 'Онанимое создание')
		}
	}
</script>

ошибка такая:

Сведения об ошибке на веб-странице

Сообщение: 'document.getElementById(...)' -  есть null или не является объектом
Строка: 1451
Символ: 5
Код: 0

[stars]

getElementsByClassName() - IE не поддерживает эту функцию решение вашей проблемы может выглядеть так:

if(document.getElementsByClassName == undefined) { 
   document.getElementsByClassName = function(cl) { 
	  var retnode = []; 
	  var myclass = new RegExp('\\b'+cl+'\\b'); 
	  var elem = this.getElementsByTagName('*'); 
	  for (var i = 0; i < elem.length; i++) { 
		 var classes = elem[i].className; 
		 if (myclass.test(classes)) { 
			retnode.push(elem[i]); 
		 } 
	  } 
	  return retnode; 
   } 
};

Суть его в том что если браузер не поддерживает getElementsByClassName, эта функция объявляется и работает по принцепу перебрать все элементы через getElementsByTagName('*') и просмотреть потом у них всех атрибут className, возвращает массив, элементов.. На практике если на странице достаточно много элементов то я бы советовал воспользоваться jquery будет работать намного быстрей...

Edited September 16, 2010 by stars

[psywalker]

Можно решить эту проблему с помощью регулярных выражений.

[Great Rash]

Еще возможно, что вы скрипт подключаете наверху из-за этого он начинает отрабатывать до загрузки DOM и элемента с id="comments" еще не существует.

Можно решить эту проблему с помощью регулярных выражений.

author.innerHTML.replace(/Анонимный/g, 'Онанимое создание') <-- А это что по твоему?

[stars]

Еще возможно, что вы скрипт подключаете наверху из-за этого он начинает отрабатывать до загрузки DOM и элемента с id="comments" еще не существует.

кстати да енто я чегото просмотрел ) хотя раз в других браузерах работает, то...

Я бы вообще пробовал решать эту проблему на пхп...

Edited September 16, 2010 by stars

[psywalker]

author.innerHTML.replace(/Анонимный/g, 'Онанимое создание') <-- А это что по твоему?

Да, но для начала к этому элементу нужно ещё подобраться getElementsByClassName('comment-author')

[Great Rash]

Ах ты об этом! Тогда да.

[virgin]

getElementsByClassName() - IE не поддерживает эту функцию решение вашей проблемы может выглядеть так:

if(document.getElementsByClassName == undefined) { 
   document.getElementsByClassName = function(cl) { 
	  var retnode = []; 
	  var myclass = new RegExp('\\b'+cl+'\\b'); 
	  var elem = this.getElementsByTagName('*'); 
	  for (var i = 0; i < elem.length; i++) { 
		 var classes = elem[i].className; 
		 if (myclass.test(classes)) { 
			retnode.push(elem[i]); 
		 } 
	  } 
	  return retnode; 
   } 
};

Суть его в том что если браузер не поддерживает getElementsByClassName, эта функция объявляется и работает по принцепу перебрать все элементы через getElementsByTagName('*') и просмотреть потом у них всех атрибут className, возвращает массив, элементов.. На практике если на странице достаточно много элементов то я бы советовал воспользоваться jquery будет работать намного быстрей...

у меня не получается вставить этот код в блог, как правильно его надо вставлять?

я пробовал его отдельно вставить в теге <script type='text/javascript'> перед прошлым скриптом, но так не работает, и пробовал добавить его в конце прошлого скрипта, так тоже не работает, или его надо исправлять сначала?

[stars]

у меня не получается вставить этот код в блог, как правильно его надо вставлять?

я пробовал его отдельно вставить в теге <script type='text/javascript'> перед прошлым скриптом, но так не работает, и пробовал добавить его в конце прошлого скрипта, так тоже не работает, или его надо исправлять сначала?

А вы понимаете что вы получаете массив?

вот смотрите пример накидал (html код взят с этого сайта "Краткие рецепты по CSS и HTML" ) )

<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01//EN" "http://www.w3.org/TR/html4/strict.dtd">
<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=windows-1251">
<title>Слои по горизонтали</title>
<style type="text/css">
#layer1 {
 background-color: #fc0;
 border: 1px solid #000;
 padding: 5px;
 float: left;
 width: 200px;
}
#layer2 {
 background-color: #c0c0c0;
 border: 1px solid #000;
 padding: 5px;
 width: 300px;
 float: left;
}
.clear {
 clear: both;
}
</style>
<script type='text/javascript'>
if(document.getElementsByClassName == undefined) { 
   document.getElementsByClassName = function(cl) { 
	  var retnode = []; 
	  var myclass = new RegExp('\\b'+cl+'\\b'); 
	  var elem = this.getElementsByTagName('*'); 
	  for (var i = 0; i < elem.length; i++) { 
		 var classes = elem[i].className; 
		 if (myclass.test(classes)) { 
			retnode.push(elem[i]); 
		 } 
	  } 
	  return retnode; 
   } 
};
window.onload = function() { 
document.getElementsByClassName('color')[2].style.backgroundColor="#FFCC80";;
};
</script>
</head>
<body>

<div id="layer1" class="color">
Lorem ipsum dolor sit amet, consectetuer adipiscing elit, sed diem nonummy nibh euismod tincidunt ut lacreet dolore magna aliguam erat volutpat.
</div>

<div id="layer2" class="color">
Ut wisis enim ad minim veniam, quis nostrud exerci tution ullamcorper suscipit lobortis nisl ut aliquip ex ea commodo consequat. Duis te feugifacilisi. 

</div>

<div class="clear"></div>
<p class="color">Duis te feugifacilisi. Duis autem dolor in hendrerit in vulputate velit esse molestie consequat.</p>

</body>
</html>

у меня IE 8 на ноуте другой версии нету потестить суть скрипта в том что получается массив тэгов с классом = "color" из массива берется [2] - третий элемент(счет идет с нуля) и изменяется его backgroundColor

Edited September 16, 2010 by stars