problema al eliminar registro de un array (delete mensajesp[i])

[CHuLoSoY]

Hola amigos, estoy creando un script para enviar los elementos a borrar de una base de datos a una página php, pero seleccionando y deselecionando con javascript. El código funciona bien, pero cuando trato de borrar un elemento del array (deseleccionando un mensaje) en la línea delete mensajesp[i]; me borra dos registros, o eso parece, con el for.

Si seleccionas dos registros va bien. Si a esos dos registros le quitas uno, en vez de aparecer el que queda, no aparece ninguno.

Agradezco cualquier ayuda.

Un saludo.

Código PHP:

  var mensajesp=new Array();
function seleccionaMensajePrivadoParaBorrar(id) {

    donde=document.getElementById("aseleccion"+id);
    if(donde.value==0) {

        total=document.getElementById("cuantosMensajesSeleccionados");
        ese=Number(total.value)+1;

        mensajesp[ese]=id;
        total.value=ese;
        donde.value=1;

        document.getElementById("selecsionostia"+id).style.backgroundColor="yellow";
    } else {

        total=document.getElementById("cuantosMensajesSeleccionados");

        for(i=1;i<=total.value;i++) {
            if(mensajesp[i]==id){
                delete mensajesp[i];
            }
        }
        document.getElementById("selecsionostia"+id).style.backgroundColor="";
        total.value=Number(total.value)-1;
        donde.value=0;

    }

}


function borrarMensajesSeleccionados() {

    total=document.getElementById("cuantosMensajesSeleccionados");

    if (total.value==0) { alert('No hay mensajes seleccionados'); } else {
        var todostoditos="";

        for(i=1;i<=total.value;i++) {
            if(mensajesp[i]) { todostoditos+=mensajesp[i]+","; }
        }

        var user=top.document.getElementById("nombreUsuario").value;
    }

} 
   

He puesto el highlight en php porque el code no colorea javascript, el cual es una pena.

Espero que alguien me pueda ayudar, gracias y un saludo.

[JavierB]

Hola CHuLoSoY

Muevo tu tema al foro de Javascript desde CSS.

Saludos,

[CHuLoSoY]

Alguien puede ayudarme?

for(i=1;i<=total.value;i++) {
if(mensajesp[i]==id){
delete mensajesp[i];
}
}

y

for(i=1;i<=total.value;i++) {
if(mensajesp[i]) { todostoditos+=mensajesp[i]+","; }
}

Gracias!

[jackson666]

Me suena a que hay una confusion con el for... Los arrays secuenciales tienen indices numericos que empiezan desde cero, o sea

Código PHP:

   for(i=0;i<total.value;i++) {
            if(mensajesp[i]==id){
                delete mensajesp[i];
            }
        } 
   

Una pregunta: el valor guardado en la variable id se puede repetir? porque sino te convendria poner un break despues del delete

[zerokilled]

yo le daria otro enfoque al codigo, especialmente a la inicializacion del array. yo tu me olvido de comenzar el indice por 1 porque trae mas problema que beneficio. intenta la siguiente modificacion.

Código:

var mensajesp=new Array();
function seleccionaMensajePrivadoParaBorrar(id) {
    var donde=document.getElementById("aseleccion"+id);
    var total=document.getElementById("cuantosMensajesSeleccionados");
    if(donde.value==0) {
        mensajesp.push(id);
        total.value=mensajesp.length;
        donde.value=1;
        document.getElementById("selecsionostia"+id).style.backgroundColor="yellow";
    } else {
        for(i=0;i<mensajesp.length;i++) {
            if(mensajesp[i]==id){
            mensajesp.splice(i, 1);
            break; // como menciona jackson666, si los IDs son unicos, no hay porque seguir recorriendo el array;
            }
        }
        document.getElementById("selecsionostia"+id).style.backgroundColor="";
        total.value=mensajesp.length;
        donde.value=0;
    }

}


function borrarMensajesSeleccionados() {
    var total=document.getElementById("cuantosMensajesSeleccionados");
    if (!mensajesp.length) { alert('No hay mensajes seleccionados'); } else {
        var todostoditos=mensajesp.join(",");
        var user=top.document.getElementById("nombreUsuario").value;
    }

}

[CHuLoSoY]

He descubierto que el error está en el total.value, ya que si pone que el total es 1 y el registro existente es i=2, no llega a él aunque exista. ¿Cómo podría solucionarlo? ¿Cómo plantearlo?

Gracias :)

Edito, he visto el código anterior y creo que voy a probarlo. No sabía eso del .lenght para saber la longitud del array. Yo lo hacía con un campo oculto que sumaba o restaba.

Muchas gracias de nuevo.

[CHuLoSoY]

Iniciado por zerokilled

yo le daria otro enfoque al codigo, especialmente a la inicializacion del array. yo tu me olvido de comenzar el indice por 1 porque trae mas problema que beneficio. intenta la siguiente modificacion.

Código:

var mensajesp=new Array();
function seleccionaMensajePrivadoParaBorrar(id) {
    var donde=document.getElementById("aseleccion"+id);
    var total=document.getElementById("cuantosMensajesSeleccionados");
    if(donde.value==0) {
        mensajesp.push(id);
        total.value=mensajesp.length;
        donde.value=1;
        document.getElementById("selecsionostia"+id).style.backgroundColor="yellow";
    } else {
        for(i=0;i<mensajesp.length;i++) {
            if(mensajesp[i]==id){
            mensajesp.splice(i, 1);
            break; // como menciona jackson666, si los IDs son unicos, no hay porque seguir recorriendo el array;
            }
        }
        document.getElementById("selecsionostia"+id).style.backgroundColor="";
        total.value=mensajesp.length;
        donde.value=0;
    }

}


function borrarMensajesSeleccionados() {
    var total=document.getElementById("cuantosMensajesSeleccionados");
    if (!mensajesp.length) { alert('No hay mensajes seleccionados'); } else {
        var todostoditos=mensajesp.join(",");
        var user=top.document.getElementById("nombreUsuario").value;
    }

}

Wow! Tu código funciona de miedo. Le he añadido todo el código que no he puesto (por irrelevante en el problema) y funciona a las mil maravillas. No conocía el .join con el que me ahorro el for.
Tampoco conocía splice, que estoy buscando ahora mismo en google para saber qué carajos hace.
Y tampoco el .push, que veo que añade un elemento en el array.
Con este código todo se hace de manera ordenada, eres un fenómeno.Mil millones de gracias!

[zerokilled]

http://www.w3schools.com/jsref/jsref_obj_array.asp