problema con array y chekbutton

[lauramiracle]

Hola...
Estoy haviendo un panel de administrador donde inserto noticias, y las borro...
El problema lo tengo al borrarlas. Lo que hago es un formulario donde las muestro con un checkbutton para elegir las que deseo eliminar de la base de datos. Luego las envio a un php donde elimino las seleccionadas...la verdad es que estoy un poco liada...Cuando selecciono uno sí lo borra, pero cuando selecciono 3 o 4, me borra uno de ellos...puede ser que el problema esté en el value que le doy a los checkbutton?

Si alguien me puede hechar una mano, lo agradecería...

Código PHP:

 <div id="contenedor">
  <form name="frm" id="frm" action="elimnot2.php" method="post">
  <table width="603" border="0" cellpadding="0" cellspacing="0">
    <tr>
      <td colspan="2" class="noticias">NOTICIAS PCI </td>
    </tr>
    <?php
    $n=0;
            while($n<$totaltitulos1)
            {
                if($titulo1[$n]!=NULL)
                {
                ?>    
    <tr>
      <td width="52"><input type="checkbox" name="borradopci[<?php echo $n ?>]" 
                                            value="<?php echo $titulo1[$n] ?>" ></td>
      <td width="551" class="titulo"><?php echo $titulo1[$n] ?></td>
    </tr>
   <?php
                }
                    $n++;
            }
     ?>   

elimnot2.php

Código PHP:

      $pci=$_POST['borradopci'];
    
//conexion    
    $di="localhost";
    $usuario="localhost";
    $clave="localhost";
    $conexion=mysql_connect($di,$usuario,$clave);
    if($conexion==NULL)
    {
        echo ("Error");
    }
    $db='deinsa';
    mysql_select_db($db,$conexion);
 
 
$arraypci=count($pci);
 
 
$n=0;
while($n<$arraypci)
{
    $borrar1="DELETE FROM noticias WHERE titulo='".$pci[$n]."'";
        $consultamysql=mysql_query($borrar1,$conexion);
    $n++;
} 
   

[José Molina]

Saludos

No hace falta que en el nombre del Check le coloques el ID dentro del arrego:

Código:

name="borradopci[<?php echo $n ?>]"

Puedes colocar simplemente:

Código:

name="borradopci[]"

Sugerencias:

1) No hagas una ejecucion del query con cada opción seleccionada, consumes muchos recursos.

Cambia esto:

Código PHP:

  while($n<$arraypci) 
{ 
    $borrar1="DELETE FROM noticias WHERE titulo='".$pci[$n]."'"; 
        $consultamysql=mysql_query($borrar1,$conexion); 
    $n++; 
} 
   

por esto:

Código PHP:

  if ($arraypci > 0) {

$borrar1="DELETE FROM noticias WHERE (";

while($n<$arraypci) 
{ 
    $borrar1.=" titulo='".$pci[$n]."'"; 
    
    if ($n<($arraypci-1)) {
        $borrar1.=" OR ";
    }
    $n++; 
}

$borrar1.=")";
$consultamysql=mysql_query($borrar1,$conexion); 
} 
   

2) No hagas el mach por el TITULO de la noticia. Debes hacerlo por el ID autonumerico y unico de la tabla noticias

Suerte

[lauramiracle]

Iniciado por José Molina

Saludos

No hace falta que en el nombre del Check le coloques el ID dentro del arrego:

Código:

name="borradopci[<?php echo $n ?>]"

Puedes colocar simplemente:

Código:

name="borradopci[]"

Sugerencias:

1) No hagas una ejecucion del query con cada opción seleccionada, consumes muchos recursos.

Cambia esto:

Código PHP:

  while($n<$arraypci) 
{ 
    $borrar1="DELETE FROM noticias WHERE titulo='".$pci[$n]."'"; 
        $consultamysql=mysql_query($borrar1,$conexion); 
    $n++; 
} 
   

por esto:

Código PHP:

  if ($arraypci > 0) {
 
$borrar1="DELETE FROM noticias WHERE (";
 
while($n<$arraypci) 
{ 
    $borrar1.=" titulo='".$pci[$n]."'"; 
    
    if ($n<($arraypci-1)) {
        $borrar1.=" OR ";
    }
    $n++; 
}
 
$borrar1.=")";
$consultamysql=mysql_query($borrar1,$conexion); 
} 
   

2) No hagas el mach por el TITULO de la noticia. Debes hacerlo por el ID autonumerico y unico de la tabla noticias

Suerte

Lo solucioné....muchas gracias.
El error exactamente estaba en name="borradopci[]