Problema "tonto" con función de manejo de array

[freesoftwarrior]

Buenas tardes:
Estoy seguro que la solución es simple, pero estoy bloqueado, lo admito, y no ato ni desato. Verán, en Dynamicdrive encontré un script para el efecto "tip del día" que funciona bien, pero sólo tiene la opción de avanzar en los tips pero no retroceder, es decir, una vez cargada la ventana, los tips sólo puedes "verlos siempre hacia adelante" pero no retrocer.

El array lo forma

todmsg=new Array();

y la función para avanzar es ntip. Decidí agregarle la función ptip para retroceder pero no funciona bien. Primero porque al retroceder el primer elemento sale "undefined" y luego si funciona bien, pero si esta en el registro 14, por ejemplo, y luego le doy "siguiente", se salta como 3 registros, no es secuencial, se va al 17. Igual es si estoy avanzando hasta el 15 y le doy "previo" salta hasta el 11 y a veces aparece -1.

La primera función es la que viene con el script, la segunda es la que he creado para el botón "previo".

function ntip(){
if (cnt<todmsg.length-1)
cnt++;
else
cnt=0;
mtxt.innerHTML=todmsg[cnt];
cntr.innerHTML=(cnt+1)+"/"+todmsg.length;
}


function ptip(){
if (cnt>0)
cnt--;
else
cnt=todmsg.length;
mtxt.innerHTML=todmsg[cnt];
cntr.innerHTML=(cnt-1)+"/"+todmsg.length
}

Si alguien puede darme una ayuda por favor, les quedaré agradecido. Estoy seguro que he cometido un error tonto pero la verdad no lo logro ver.

Saludos desde Lima, Perú

[laratik]

Buenas tardes freesoftwarrior, te falto restarle 1 al largo del array (Recuerda el index de un array va desde 0 hasta length-1), ademas no habia necesidad de colocar:

Código:

cntr.innerHTML=(cnt-1)+"/"+todmsg.length

Con dejarlo como estaba funcionaba igual, ya que lo que hace es dar el valor del indice del array+1, en fin quedaria algo como esto:

Código Javascript:

function ntip(){
if (cnt<todmsg.length-1)
cnt++;
else
cnt=0;
mtxt.innerHTML=todmsg[cnt];
cntr.innerHTML=(cnt+1)+"/"+todmsg.length;
}
 
function ptip(){
if (cnt>0) {
    cnt--;
}
else {
    cnt=todmsg.length-1;
}
mtxt.innerHTML=todmsg[cnt];
cntr.innerHTML=(cnt+1)+"/"+todmsg.length
}
 

Espero haberte sido de ayuda.

[freesoftwarrior]

Me siento un burro jajaja..... bien decía que era una cosa de nada, ¿cómo no pude verlo?. Lo de la última línea si lo comprendo: como estoy "retrocediendo" pues siempre debería de restar... pero el detalle de la línea 15 ese sí no lo vi ni en broma...... mil mil mil gracias!!!!!!