Problema insertando datos desde un array con checkboxes en MySQL

loki_gz · #1 (**permalink**) 19/12/2011, 12:06

Tengo un problema con un formulario con el que envío datos a una base de datos mysql utilizando un array.

Todo va perfecto salvo cuando en el formulario introduzco un checkbox para pasarle el valor 0 ó 1 a la BD.

He probado de dos maneras (con dos resultados distintos) pero no consigo obtener lo que quiero.

El código ha sido generado con DW, pero ha sido ligeramente adaptado a mis necesidades

Este es el código que uso:

Código PHP:

 
 $editFormAction = $_SERVER['PHP_SELF']; 
if (isset($_SERVER['QUERY_STRING'])) { 
  $editFormAction .= "?" . htmlentities($_SERVER['QUERY_STRING']); 
} 
 
if ((isset($_POST["MM_insert"])) && ($_POST["MM_insert"] == "form")) { 
  foreach($_POST['input'] as $key => $val){ 
  $insertSQL = sprintf("INSERT INTO tabla (input, check) VALUES ($val, %s)", 
                       GetSQLValueString(isset($_POST['check']) ? "true" : "", "defined","1","0")); 
 
  mysql_select_db($database_db, $db); 
  $Result1 = mysql_query($insertSQL, $db) or die(mysql_error()); 
} 
} 
 
<body> 
 
<form  id="form" name="form" method="post" action="<?php echo $editFormAction; ?>" 
<table> 
    <tr> 
        <td>input 1</td> 
        <td> 
            <input type="text" name="input[]" value="" /> 
        </td> 
        <td align="center" valign="middle"> 
            <input name="check[]" type="checkbox"/> 
        </td> 
    </tr> 
    <tr> 
        <td>input 2</td> 
        <td> 
            <input type="text" name="input[]" value="" /> 
        </td> 
        <td align="center" valign="middle"> 
            <input name="check[]" type="checkbox"/> 
        </td> 
 
    </tr> 
    <tr> 
        <td>input 3</td> 
        <td> 
            <input type="text" name="input[]" value="" /> 
        </td> 
        <td align="center" valign="middle"> 
            <input name="check[]" type="checkbox"/> 
        </td> 
    </tr> 
    <tr> 
        <input type="hidden" name="MM_insert" value="form" /> 
    <input type="submit" value="Insertar" /> 
    </tr> 
</table> 
</form> 
 
</body>

Con este código el resultado que obtengo es una BD con tres nuevos registros. Pero si en el formulario había activado el checkbox en alguno de los tres inputs, siempre le pasará el valor 1 al primero de los registros. Da igual que active el primero, el segundo o el tercero, siempre se lo pasará al primer registro.

Con esta pequeña modificación que le hice al código, el resultado es distinto, pero tampoco es válido:

Código PHP:

 
 $editFormAction = $_SERVER['PHP_SELF']; 
if (isset($_SERVER['QUERY_STRING'])) { 
  $editFormAction .= "?" . htmlentities($_SERVER['QUERY_STRING']); 
} 
 
if ((isset($_POST["MM_insert"])) && ($_POST["MM_insert"] == "form")) { 
  foreach($_POST['input'] as $key => $val){ 
  $insertSQL = sprintf("INSERT INTO tabla (input, check) VALUES ($val, %s)", 
                       GetSQLValueString($_POST['check'][$key], "int")); 
 
  mysql_select_db($database_db, $db); 
  $Result1 = mysql_query($insertSQL, $db) or die(mysql_error()); 
} 
} 
 
<body> 
 
<form  id="form" name="form" method="post" action="<?php echo $editFormAction; ?>" 
<table> 
    <tr> 
        <td>input 1</td> 
        <td> 
            <input type="text" name="input[]" value="" /> 
        </td> 
        <td align="center" valign="middle"> 
            <input name="check[]" type="checkbox" value="1"/> 
        </td> 
    </tr> 
    <tr> 
        <td>input 2</td> 
        <td> 
            <input type="text" name="input[]" value="" /> 
        </td> 
        <td align="center" valign="middle"> 
            <input name="check[]" type="checkbox" value="1"/> 
        </td> 
    </tr> 
    <tr> 
        <td>input 3</td> 
        <td> 
            <input type="text" name="input[]" value="" /> 
        </td> 
        <td align="center" valign="middle"> 
            <input name="check[]" type="checkbox" value="1"/> 
        </td> 
    </tr> 
    <tr> 
        <input type="hidden" name="MM_insert" value="form" /> 
    <input type="submit" value="Insertar" /> 
    </tr> 
</table> 
</form> 
 
</body>

De esta manera, lo que consigo es que sólo crea un nuevo registro y le asigna el valor 1 al campo check si activo cualquiera de los 3 checkboxes. Es decir, mantiene el mismo problema con los checkboxes que con el código anterior, pero ahora aparece otro problema, ya que no genera 2 de los 3 registros que debiera crear.

Espero haberme explicado bien y que sepáis ayudarme a solucionar este problema. He buscado bastante por internet, pero no consigo dar con la solución.

No dudéis en pedirme cualquier aclaración sobre lo que he escrito aquí.

truman_truman · #2 (**permalink**) 19/12/2011, 19:31

yo creo que el problema está en que si han dejado vacio el input text, quizás no se recupera la variable por POST, por eso probé con un for:

for($i=0;$i<sizeof($_POST['input']);$i++){
$insertSQL = sprintf("INSERT INTO tabla (input, check) VALUES ('".$_POST['input'][$i]."', %s)",
GetSQLValueString($_POST['check'][$i], "int"));

Supongo que fue un error de copiar y pegar, pero la etiqueta de apertura del form no está cerrada

toledo_nocturno · #3 (**permalink**) 20/12/2011, 01:32

Si, es justo lo que comenta Truman. Tienes que ir contando cada vuelta del bucle para que compare si ese valor está a cero o a uno, y que haga lo que deba en cada caso.

Saludos.

loki_gz · #4 (**permalink**) 20/12/2011, 05:05

O sea que el problema está en que el loop ha de hacerse con un for, no con un foreach.

Lo probaré ahora mismo y os cuento

loki_gz · #5 (**permalink**) 20/12/2011, 05:37

Nada, he probado a cambiar el loop, pero sigue dando el mismo problema.

Si lo hago siguiendo el primero de los códigos, crea tres nuevos registros pero le asigna siempre el valor 1 al campo checkbox del primero de ellos, independientemente de cual sea el checkbox que haya activado.

Si sigo el segundo ejemplo, sólo me crea un registro y le asigna el valor 1 al campo checkbox, aunque en el formulario hubiera activado el checkbox 2.

Por cierto, detecté un error en el código del primer ejemplo:

donde puse

Código PHP:

  GetSQLValueString(isset($_POST['check']) ? "true" : "", "defined","1","0"));

debe poner

Código PHP:

  GetSQLValueString(isset($_POST['check'][$key]) ? "true" : "", "defined","1","0"));

SirDuque · #6 (**permalink**) 20/12/2011, 06:08

Por favor ante pone este script en tu codigo, ejecutalo y mostrame tus resultados.

Código PHP:

  <?php 
$input = $_POST['check']; 
echo "<pre>"; 
print_r($input); 
echo "</pre>"; 
//despues 
for( $i = 0; $i<count($input); $i++) 
{ 
echo $input[$i]."<br>"; 
} 
?>

loki_gz · #7 (**permalink**) 20/12/2011, 06:26

Me devuelve lo siguiente:

Cita:

Array
(
[0] => 1
)

1

SirDuque · #8 (**permalink**) 20/12/2011, 06:35

Fijate ahora:

Código PHP:

  <?php  
if(isset($_POST['check'])) 
{ 
$check = $_POST['check'];  
echo "<pre>";  
print_r($check);  
echo "</pre>";  
//despues  
for( $i = 0; $i<count($check); $i++)  
{  
echo $check[$i]."<br>";  
}  
} 
if(isset($_POST['input'])) 
{ 
$input = $_POST['input'];  
echo "<pre>";  
print_r($input);  
echo "</pre>";  
//despues  
for( $i = 0; $i<count($input); $i++)  
{  
echo $input[$i]."<br>";  
}  
} 
?>

test.php

Código HTML:

Ver original<!DOCTYPE html>
<html lang="es">
<head>
<title>Foros del web</title>
<meta charset="iso-8859-1"> 
</head>
<body>
<form  id="form" name="form" method="post" action="test.php" >
<table> 
    <tr> 
        <td>input 1</td> 
        <td> 
            <input type="text" name="input[]" value="" /> 
        </td> 
        <td align="center" valign="middle"> 
            <input name="check[]" type="checkbox"/> 
        </td> 
    </tr> 
    <tr> 
        <td>input 2</td> 
        <td> 
            <input type="text" name="input[]" value="" /> 
        </td> 
        <td align="center" valign="middle"> 
            <input name="check[]" type="checkbox"/> 
        </td> 
 
    </tr> 
    <tr> 
        <td>input 3</td> 
        <td> 
            <input type="text" name="input[]" value="" /> 
        </td> 
        <td align="center" valign="middle"> 
            <input name="check[]" type="checkbox"/> 
        </td> 
    </tr> 
    <tr> 
        <input type="hidden" name="MM_insert" value="form" /> 
    <input type="submit" value="Insertar" /> 
    </tr> 
</table> 
</form> 
</body>
</html>

loki_gz · #9 (**permalink**) 20/12/2011, 06:44

Lo sigue haciendo mal, ¿no?

El resultado que me da es este:

Cita:

Array
(
[0] => on
)

on

Array
(
[0] => prueba 1
[1] => prueba 2 (on)
[2] => prueba 3
)

prueba 1
prueba 2 (on)
prueba 3

Es decir, no interpreta 3 checkboxes distintos, sino que para él es uno sólo, y en cuanto activamos uno de los tres, para él ya está en on

SirDuque · #10 (**permalink**) 20/12/2011, 07:05

Cita:

Iniciado por loki_gz

Lo sigue haciendo mal, ¿no?

El resultado que me da es este:

Es decir, no interpreta 3 checkboxes distintos, sino que para él es uno sólo, y en cuanto activamos uno de los tres, para él ya está en on

Algo parecido, en realidad cuando pones corchetes en el name del input, le estas diciendo a PHP que es un array, por defecto lo interpreta como tal.
los checkbox , solo es un valor Boleano. y por defecto solo ingresa en el array los True, no los false.
Mi consejo:

check1
check2
check3

No utilices corchetes en con los checkbox

loki_gz · #11 (**permalink**) 20/12/2011, 07:10

Ok. Intentaré implementar un código siguiendo tu consejo.

Ya te contaré qué tal me fue

loki_gz · #12 (**permalink**) 23/12/2011, 06:57

Bueno, pues por fin he dado en el clavo y he conseguido solucionarlo. El tema está en utilizar los corchetes, pero asignándole un número de serie consecutivo que luego evaluaría el loop cuando inserta (o actualiza) los registros.

Según el código de muestra que colgué al principio y con las modificaciones que he hecho, la cosa quedaría así:

prueba.php

Código PHP:

  <?php
$editFormAction = $_SERVER['PHP_SELF'];
if (isset($_SERVER['QUERY_STRING'])) {
  $editFormAction .= "?" . htmlentities($_SERVER['QUERY_STRING']);
}

if ((isset($_POST["MM_insert"])) && ($_POST["MM_insert"] == "form")) {
  for($key=0;$key<sizeof($_POST['input']);$key++){
  $insertSQL = sprintf("INSERT INTO tabla (input, check) VALUES (%s, %s)",
                       GetSQLValueString($_POST['input'][$key], "int"),
                       GetSQLValueString(isset($_POST['check'][$key]) ? "true" : "", "defined","1","0"));

  mysql_select_db($database_db, $db);
  $Result1 = mysql_query($insertSQL, $db) or die(mysql_error());
}
}
?>

form.html

Código HTML:

<html>
<body>

<form  id="form" name="form" method="post" action="prueba.php"
<table>
    <tr>
        <td>input 1</td>
        <td>
            <input type="text" name="input[]" value="" />
        </td>
        <td align="center" valign="middle">
<!-- Al nombre de cada check le asigno un número de serie correlativo comenzando por 0" -->
            <input name="check[0]" type="checkbox" value="1"/>
        </td>
    </tr>
    <tr>
        <td>input 2</td>
        <td>
            <input type="text" name="input[]" value="" />
        </td>
        <td align="center" valign="middle">
            <input name="check[1]" type="checkbox" value="1"/>
        </td>
    </tr>
    <tr>
        <td>input 3</td>
        <td>
            <input type="text" name="input[]" value="" />
        </td>
        <td align="center" valign="middle">
            <input name="check[2]" type="checkbox" value="1"/>
        </td>
    </tr>
    <tr>
        <input type="hidden" name="MM_insert" value="form" />
    <input type="submit" value="Insertar" />
    </tr>
</table>
</form>

</body>
</html>

Salvo que me haya equivocado a la hora de copiar y pegar código, la cosa funciona, ¡probado!

Gracias por vuestra ayuda