Error en getElementById

[lsobpla]

Hola,

Tengo esta función en javascript, que me va poniendo los números de página para una tabla que voy paginando. El caso es que esta función va incluyendo el número de página, y tb le voy dando estilo a la paginación, de forma que a la página actual que va creando le doy el estilo especial 'current' (está en una hoja css), y al resto de las páginas quiero quitarle ese estilo mediante la instrucción removeAttribute, para que sólo vaya resaltando la página actual.
La función es esta:

Código:

function naveg_pag(){
	var pagerHtml = document.getElementById('pagination');
	pagerHtml.setAttribute('className','pagination');
	var nuevoSpan = document.createElement('span');
	nuevoSpan.className='pagination';
	var a = document.createElement('a');
	a.id='pg'+pagina;
	a.setAttribute('href','javascript:mostrar_pag('+pagina+')');
	for(i=1;i<=pagina;i++){
		if(pagina_actual==i){
			alert("pongo atributo");
			document.getElementById('pg'+i).setAttribute('className','current');
		}
		else{
			alert("borro atributo");
			document.getElementById('pg'+i).removeAttribute('className');
		}
	}
	
	a.appendChild(document.createTextNode(pagina));
	nuevoSpan.appendChild(a);
	pagerHtml.appendChild(nuevoSpan);

	return;
}

Pero cuando intento ejecutar esto me da el error 'document.getElementById(...) es nulo o no es un objeto, en la línea:
document.getElementById('pg'+i).setAttribute('clas sName','current');
y no sé cómo solucionarlo para que sólo vaya aplicando o borrando el estilo a cada elemento...
Alguna ayudita pliss...

Gracias y un saludo!!

[ZiTAL]

Código:

for(i=1;i<=pagina;i++){

cambia por:

Código:

for(var i=1;i<=pagina;i++){

suerte!!!

[agressor]

Iniciado por lsobpla

Hola,

Código:

document.getElementById('pg'+i).setAttribute('className','current');
		
document.getElementById('pg'+i).removeAttribute('className');

Creo que deberias colcoar 'pg'+i cocantenada en otra variables para que no hagas referencia de ese modo cree que ahi puede estar tu error...

Prueba coloca

'pg'+i dentro de una variable y luego solo haces referencia a la variable en el getelementbyid..

Espero te sirva..

Salu2..!!

[lsobpla]

Hola de nuevo!

He probado las opciones que me dais pero sigo teniendo el mismo error...
Incluso he llegado a meter el bucle for en una función a parte para ver si así se solucionaba, pero nada...no consigo encontrar la solución... Me parece superraro que no me deje hacer eso, ya que no veo que el código esté mal
¿Alguna ayuda más?

[ZiTAL]

veo que:

Código:

...
a.id='pg'+pagina;
...

pagina no esta declarado, de todas formas pon alert's para debugear.

[lsobpla]

Ya he conseguido solucionarlo...
Resulta que los nuevos elementos que iba creando (<span>, <a>) no se habían agregado todavía a la página cuando yo intentaba acceder al id de uno de ellos. Poniendo el bucle for justo después de la instrucción:

Código:

pagerHtml.appendChild(nuevoSpan);

Funciona correctamente.

Gracias por las respuestas!!