Problema con for each

Integer78 · #1 (**permalink**) 31/07/2015, 11:23

Estimados tengo un problema que debe ser simple, pero no logro resolverlo:
Tengo una tabla de productos (MySQL) y uno de sus campos (numérico) indica con un 0 o un 1 si el producto debe mostrarse al público en la página o no.
Para que el administrador de la página muestre / oculte productos le armé una pantalla donde selecciona un rubro, le listo los productos de ese rubro con un checkbox al lado y simplemente tilda los productos que deben ocultarse.
Con un for each recorro todos los checkbox (que en su propiedad value me traen el id del producto) y ejecuto una instrucción SQL UPDATE para modificar el campo en cuestión.
El problema es que para los primeros productos funciona todo bien, pero cuando se tilda algún producto, digamos, de la mitad del listado para abajo no pasa nada, no llega la variable $_POST. El rubro que más productos tiene tendrá unos 340 no se si serán muchos.

Código HTML del listado de productos:

Código PHP:

Ver original<input type="checkbox" name="chkocultar[]" id="chkocultar[]" value="<? echo $reg['codigo'];?>">
 
Código PHP que procesa los productos "tildados":
 
 foreach($_POST['chkocultar'] as $codigo)
          {
            // echo $codigo."<br>";
          mysql_query("UPDATE productos SET kpasivo=1 WHERE codigo='$codigo'",$link);
          $i++;     
          }

Agradeceré cualquier orientación.

Triby · #2 (**permalink**) 31/07/2015, 13:44

Incluye la siguiente línea antes de procesar los datos:

Código PHP:

Ver originalvar_dump($_POST);

Pega aquí el resultado.

Tienes un error en HTML, la ID de cada elemento debe ser única y la estás repitiendo para cada checkbox, además de que no se permiten corchetes.

Otra cosa, hay un límite para la cantidad de datos enviados por formulario, aunque se me hace raro que lo superes si solo estás enviando los checkbox; de cualquier forma, sería más fácil para el usuario si en vez de un listado de tantos productos manejas de 30 o 50 por página.

chronos682 · #3 (**permalink**) 31/07/2015, 13:46

Traté de simular tu código poniendo 340 checkbox y lo único que tuve que cambiar fue el órden de los parámetros en la función mysql_query, pero de resto todo funciona bien. Aquí te dejo el código que usé y funciona bien:

Código PHP:

Ver original<?php
    $mysqli = new mysqli("localhost", "root", "", "prueba");
?>
<?php
    if (isset($_POST['enviar']))
    {
        foreach($_POST['chkocultar'] as $codigo)
        {
            echo $codigo."<br>";
            $result = mysqli_query($mysqli, "UPDATE prueba SET kpasivo=1 WHERE codigo='{$codigo}'");
        } 
        exit;
    }
?>
<html>
<head></head>
<body>
<form action="" method="post">
<?php
    for ($i=1;$i<=340;$i++)
    {
        echo "<input type='checkbox' name='chkocultar[]' value='{$i}'>{$i}<br>";
    }
?>
<input type="submit" name = "enviar" value="Enviar">
</form>
</body>
</html>

Como ves, no hay problema con el número de checkbox.

Corrijo el código. Ya la costumbre de usar mysqli. Así es como debe quedar. Y no hay que cambiar el órden en mysql_query:

Código PHP:

Ver original<?php
    $mysql = mysql_connect("localhost", "root", "");
    mysql_select_db("prueba", $mysql) or die(mysql_error());
?>
<?php
    if (isset($_POST['enviar']))
    {
        foreach($_POST['chkocultar'] as $codigo)
        {
            echo $codigo."<br>";
            $result = mysql_query("UPDATE prueba SET kpasivo=1 WHERE codigo='{$codigo}'",$mysql);
        } 
        exit;
    }
?>
<html>
<head></head>
<body>
<form action="" method="post">
<?php
    for ($i=1;$i<=340;$i++)
    {
        echo "<input type='checkbox' name='chkocultar[]' value='{$i}'>{$i}<br>";
    }
?>
<input type="submit" name = "enviar" value="Enviar">
</form>
</body>
</html>