logica en función checbox (identificar el error)

[oscarbt]

Hola a todos, tengo un problema con una funcion que utilizo para verificar y pasar los ids de uns checkbox:

La funcion recibe el nnumero de filas de una consulta, luego hago un for y en una cadena envio los id de esos checkbox, separando por comas esos ids, el problema es que siempre a lo ultimo queda una coma

Entonces por ejemplo genera el resultado asi:

'870001 ','902208 ',

Y la ultima coma, no deberia ir

la función es la siguiente:

Código Javascript:

function verifichek()
 {
    var numfilas=document.getElementById("numfilas").value; 
   var cad="";
   var cad2="";
    for($i=1; $i<=numfilas; $i++)
    {
        x=document.getElementById("ap"+$i).value;
       if(document.getElementById(x).checked)
      {
        cad=cad+"\'"+x+"\'"+",";
        //alert(cad);
       document.getElementById("examenes").value=cad;
      }
    
    }
}
 

y el codigo de la consulta y donde tengo una caja oculta para pasar lso valores es:

Código PHP:

$sql="SELECT * FROM .... WHERE .....";
        $obj->consulta($sql);
        $nfilas=$obj->numFilas();
        
        echo "<input type='hidden' id='numfilas' value='".$nfilas."'>";
      
        echo "<table border='1' width='400'> ";
 
        echo "<tr><th colspan='2' align='center'>Examenes del folio ".$folio."</th>";
        $nfilas=0;
        while($row = sqlsrv_fetch_array($obj->result)){
            
            $nfilas++;
            //echo $row[3];
            $descripcion=$row[7];
        echo "<tr><td align='center'><input  type='hidden' id='ap".$nfilas."' value='".$row[3]."' >".$descripcion."</td><td align='center'><input type='checkbox' id='".$row[3]."' ></td></tr>";
            
            }
            echo "<input type='hidden' id='examenes' value='' size='50' >";
            echo "</table></br></br>";
 

Espero me puedan ayudar a identificar el error, lo publico aca porque creo que es problema de logica en la función, muchas gracias

[jahepi]

Hola oscarbt !

¿ Por que no eliminar el último caracter de la cadena con el método slice ?

Código javascript:

cad = cad.slice(0, -1)
 

Un saludo y suerte !

[oscarbt]

Ok muchas gracias....

[oscarbt]

El problema es que asi me le corta la coma a cada parte de la cadena
d=cad+"\'"+x+"\'"+",";
cad = cad.slice(0, -1)

Y me muestra resultados asi:

'870001 ''902208 '

Donde deberia ser asi:

'870001 ','902208 '

[jahepi]

Iniciado por oscarbt

El problema es que asi me le corta la coma a cada parte de la cadena
d=cad+"\'"+x+"\'"+",";
cad = cad.slice(0, -1)

Y me muestra resultados asi:

'870001 ''902208 '

Donde deberia ser asi:

'870001 ','902208 '

Hola oscarbt !

Eso pasa porque estás utilizando el método slice dentro del ciclo for, lo tendrías que hacer al finalizar:

Código javascript:

var cad = "";
for($i=1; $i<=numfilas; $i++) {
    x=document.getElementById("ap"+$i).value;
    if(document.getElementById(x).checked) {
        cad = cad + "\'" + x + "\'" + ",";
    }
}
// Al salir del ciclo eliminamos el último carácter
cad = cad.slice(0, -1)
 

Un saludo y suerte !