Mejor forma de validar formuario

[rodrypaladin]

Buenas a todos, me encuentro con un dilema el cual mi opción no creo que sea la más eficiente para realizar la validación.

Mi opción es la siguiente:

Código PHP:

if($_POST["edad"] == "0" or $_POST["altura"] == "0" or $_POST["peso"] == "0" or $_POST["genero"] == "" or $_POST["fumar"] == "" or $_POST["beber"] == ""
or $_POST["drogas"] == "" or $_POST["alimentacion"] == "" or $_POST["ejercicio"] == "") {
 // Muestro formulario otra vez diciendo que falta algo por responder
} else {
// Ejecuta y envia el formulario }
 

El caso es que uso dos páginas, la del formulario y donde ejecuta ese formulario, y me encuentro con que he puesto 2 veces el código del funcionario, que no es corto que digamos, y me queda demasiado código y la picha echa un lio.

La estructura la tengo así

index.php
Muestra el formulario.

resultado.php
En caso de que no estén los datos rellenos muestra otra vez el formulario diciendo que falta algo por rellenar. En caso de que esté bien relleno lo envia.

Esto es una práctica normal ?? puedo reutilizar código ? ¿ Como lo hariáis vosotros ?

Un saludo y gracias.

[pateketrueke]

Es buena practica, además de mostrar el formulario de nuevo indicar qué campos están mal y también mantener los datos previamente introducidos.

[rodrypaladin]

Pues algo hago mal ya que algo va mal.

Si rellenan bien el formulario va bien. si lo rellenan mal, aparece que falta algo por rellenar. hast ahí bien, pero ya no se encuentran en el index.php si no en validado.php que es donde estaba probandolo, y si ahi lo rellenan bien, ya no funciona..

[Alexis88]

Seguramente (es lo más probable) estás redireccionando hacia 'validado.php' en lugar de hacia 'index.php' y si llenando los datos en el formulario de 'validado.php' no funciona bien, entonces la falla la tienes en dicho archivo.

Saludos

[ocp001a]

El index (o la pagina que sea) podría contener tanto el formulario como la validación, algo así:

Código PHP:

if(isset($_POST['enviar'])
  //realizar validaciones.
 
//...código del formulario, donde en los values e podría indicar el valor obtenido por post, para el caso de fallo.
 

[rodrypaladin]

El action de index.php redirecciona a validado.php, si todo va bien envia la información, si va mal, validado.php muestra el formulario que tiene un action a validado.php también, ya que ahí se utiliza la información

[rodrypaladin]

Lo estoy probando @ocp001a

Código PHP:

if(isset($_POST['enviar'])) {
// codigo php 
} else {
//formulario
}
 

Pero cuando envio el formulario no hace absolutamente nada, no salta ningun error, si no que parece que no haga nada.

[Triby]

La mejor forma de validar un formulario es incluir validaciones y mensajes en Javascript, igual que en PHP.

El algoritmo más común es:

Código PHP:

<?php
// Inicialización de variables
$nombre = '';
$edad = '';
$errores = array();
// .........
 
// Verificar si se va a procesar el formulario
if(isset($_POST['nombre'])) {
        // Asignación desde $_POST
        $nombre = htmlentities(trim($_POST['nombre']));
        $edad = intval($_POST['edad']);
        // .............
 
        // Validación de datos
        if(strlen($nombre) < 3) {
                $errores['nombre'] = 'Teclea tu nombre, usando 3 o más caracteres';
        }
        if($edad < 5) {
                $errores['edad'] = '¿En serio sabes escribir sin haber cumplido 5 años?';
        }
        // ...............
 
        if(count($errores) = 0) {
               // Aquí procesas el formulario
        }
}
// Aquí va el formulario:
?>
<form method="post" action="index.php">
Nombre: <input type="text" name="nombre" value="<?php echo $nombre; ?>" /><br />
<?php
if(isset($errores['nombre'])) {
       echo $errores['nombre'];
}
?>
</form>
 

Harías algo similar para todos los campos.

Ese sería el flujo a seguir para poder mantener valores. Si programas con algún patrón como MVC solo se necesitarían algunas adaptaciones; igual si prefieres usar variables de sesión para mantener los valores; eso ya es cosa de gustos.

[rodrypaladin]

Lo siento @triby pero con el ejemplo que me has puesto no me entero al pasarlo a mi código.

Pongo un trozo de formulario que tengo, que creo que si lo consigo con este trozo podré conseguirlo con todo lo demás.

Código PHP:

<FORM METHOD="POST" ACTION="" ENCTYPE="x-www-form-urlencoded">
        <h3><terror>Sobre tu físico</terror></h3><br/>
        <table border="0" align="left">
            <tr>
            <td><enun>Edad</enun></td>
            <td><SELECT name="edad">
                <OPTION VALUE="0">0</OPTION>
            <?php
                for($i=14;$i<=100;$i++) {
                    echo "<OPTION VALUE=\"$i\">$i</OPTION>";
                }
                ?>
            </SELECT></td>
            </tr>
            <tr>
                <td><enun>Altura</enun></td>
                <td><SELECT name="altura">
                    <OPTION VALUE="0">0</OPTION>
                <?php
                    for($i=130;$i<=250;$i++) {
                        echo "<OPTION VALUE=\"$i\">$i cm</OPTION>";
                    }
                    ?>
                    </SELECT></td>
            </tr>
            <tr>
                <td><enun>Peso</enun></td>
                <td><SELECT name="peso">
                    <OPTION VALUE="0">0</OPTION>
                <?php
                    for($i=30;$i<=200;$i++) {
                        echo "<OPTION VALUE=\"$i\">$i kg</OPTION>";
                    }
                    ?>
                    </SELECT></td>
            </tr>
            <tr>
                <td><enun>Género</enun></td>
                <td><INPUT type=radio name="genero" value="masculino">Masculino
                    <INPUT type=radio name="genero" value="femenino">Femenino</td>
            </tr>
        </table>
        <input type="submit" class="button" />
 

En caso de que cualquiera de los select esté a 0, y que los radio no estén definidos, muestre que faltan datos y cual.

[Triby]

Primero que nada: Todas las etiquetas y atributos HTML deben estar en minúsculas.

Un ejemplo con tu código (se me olvidó agregar la altura):

Código PHP:

<?php
// Una función útil para crear selects
function html_select($name, $id, $value, $options = array(), $start = 0, $end = 0) {
    echo "<select name=\"$name\" id=\"$id\">\n";
    if(count($options) == 0) {
        // Creamos el array con inicio y fin
        if($start > 0) {
            // Agregamos opción cero
            $options[0] = 'Selecciona...';
        }
        for($i = $start; $i <= $end; $i++) {
            $options[$i] = $i;
        }
    }
    // Agregamos las opciones
    foreach($options as $key => $val) {
        // Para establecer la opción preseleccionada
        $selected = ($key == $value) ? ' selected="selected"' : '';
        echo "\t<option value=\"$key\"$sel>$val</option>\n";
    }
    echo "</select>\n";
}
// Primero los valores iniciales
$edad = 0;
$peso = 0;
$genero = "";
$errores = array();
 
if(isset($_POST['edad'])) {
    $edad = intval($_POST['edad']);
    $peso = intval($_POST['peso']);
    $genero = $_POST['genero'];
    // Validamos con los límites propuestos
    if($edad < 14 || $edad > 100) {
        $errores['edad'] = 'Selecciona tu edad';
    }
    if($peso < 30 || $peso > 200) {
        $errores['peso'] = 'Selecciona tu peso';
    }
    if(!in_array($genero, array('masculino', 'femenino'))) {
        $errores['genero'] = 'Selecciona tu género';
    }
    if(count($errores) = 0) {
        // OK, todo bien, aquí terminas de procesar
    }
    // Si hubo errores se mostrará nuevamente el formulario
}
?>
        <form method="post" action="index.php" enctype="x-www-form-urlencoded">
            <h3><terror>Sobre tu físico</terror></h3><br/>
            <table border="0" align="left">
                <tr>
                <td><enun>Edad</enun></td>
                <td>
<?php
// Crear select con opción predefinida
html_select('edad', 'edad', $edad, array(), 14, 100);
if(isset($errores['edad'])) {
    echo "<span class=\"error\">{$errores['edad']}</span>";
}
?>
                </td>
                </tr>
                <tr>
                    <td><enun>Peso</enun></td>
                    <td>
<?php
// Crear select con opción predefinida
html_select('peso', 'peso', $peso, array(), 30, 200);
if(isset($errores['peso'])) {
    echo "<span class=\"error\">{$errores['peso']}</span>";
}
?>
                    </td>
                </tr>
                <tr>
                    <td><enun>Género</enun></td>
                    <td>
<?php
// Crear select con opción predefinida
html_select('genero', 'genero', $genero, array("" => 'Selecciona', 'masculino' => 'Masculino', 'femenino' => 'Femenino'));
if(isset($errores['genero'])) {
    echo "<span class=\"error\">{$errores['genero']}</span>";
}
?>
                    </td>
                </tr>
            </table>
            <input type="submit" value="Enviar" class="button" />
        </form>
 

[rodrypaladin]

Gracias @triby, funciona muy bien.

Está muy bien crear el formulario por medio de una función, deja el código más limpio la verdad. Por otro lado, cuando todo está correcto y tiene que procesar el formulario, aun procesándolo aparece el formulario abajo.

He probado metiendole un else a

Código PHP:

if(isset($_POST['edad'])) {
// validación
} else { 
// aquí muestra formulario 
}
 

Y ahora si lo rellenas todo procesa el formulario, muestra el echo y nada más, hasta ahí bien. Pero en cambio si falta algo por rellenar, no muestra el formulario con el error. si no que la página se queda en blanco.

Ni una ni otra. Que lio!!!

[Triby]

Es que debes detener el flujo del script; en la parte donde revisas que no hay errores:

Código PHP:

if(count($errores) = 0) {
        // OK, todo bien, aquí terminas de procesar
        // Debes mostrar mensaje, opciones para compartir, etc.
        // Finalizas la ejecución con exit o die()
        exit;
    }
 

[rodrypaladin]

Ahora si que si @Triby, Mil gracias, voy a intentar ahora modificar y añadir los radio que tengo en el formulario entero, ya que con select lo tengo claro, crearé la función para los radio y a ver que pasa. Si no lo consigo seguiré este tema si no es molestia. Mil gracias.

[rodrypaladin]

Perdonen de nuevo.

He añadido el código para procesar el formulario y realizar las operaciones que hago con los datos obtenidos del formulario, me salta un error que no entiendo.

Parse error: syntax error, unexpected end of file in /homepages/0/d513175986/htdocs/clickandbuilds/WordPress/MyBlog/ejemplo.php on line 392

Al ir a esa linea me encuentro con que es la última línea de código de la página

392: </html>

¿ Qué significa ?

[pateketrueke]

Los errores de PHP son poco descriptivos, la linea que te indica no es ni de lejos la linea del error.

Un error tipo "unexpected end of file" implica que alguna estructura de control está sin cerrar apropiadamente, por eso el analizador de PHP te indica error en la última linea de código PHP posible donde debería existir el cierre de dicha estructura, etc.

Eso es todo, revisa bien tus estructuras lógicas.

[rodrypaladin]

El formulario antes de poner el código que procesa el formulario no da errores y funciona bien, pero al añadir el código sale eso, he revisado el código que he añadido pero no encuentro que falte algun cierre de }. Volveré a revisar.

EDITO: Encontré donde poner un corchete de cierre y al ponerlo ya no salta el error y ejecuta la página, pero en cambio no realiza la ejecución del formulario. Seguiré revisando.

Código PHP:

if(count($errores) == 0) {
        echo "Se procesa el formulario.";
    }
 

Muestra el echo correctamente. Pero si añado el código que procesa el formulario aparece lo que comenté antes.

Código PHP:

if($genero == "masculino") {
    $esperanza_de_vida = rand(75,85);
} else { $esperanza_de_vida = rand(80,90); }
//calcular imc
$altura_en_metros = $altura / 100;
$imc = $altura_en_metros * $altura_en_metros;
$imc_final = round($peso / $imc,1);
 
if($imc_final < 18) {
    $estado_imc = 1;
} elseif($imc_final >= 18 and $imc_final < 24.9) {
    $estado_imc = 2; 
} elseif($imc_final >= 25 and $imc_final < 26.9) {
    $estado_imc = 3;
} elseif($imc_final >= 27 and $imc_final < 29.9) {
    $estado_imc = 4;
} elseif($imc_final >= 30 and $imc_final < 39.9) {
    $estado_imc= 5; 
} elseif($imc_final > 40) {
    $estado_imc = 6;
}
 
$edad_restada_imc = 0;
if($estado_imc == 1) {
    $edad_restada_imc = $edad_restada_imc + 5;
} elseif($estado_imc == 3) {
    $edad_restada_imc = $edad_restada_imc + 5;
} elseif($estado_imc == 4) {
    $edad_restada_imc = $edad_restada_imc + 7;
} elseif($estado_imc == 5) {
    $edad_restada_imc = $edad_restada_imc + 10;
} elseif($estado_imc == 6) {
    $edad_restada_imc = $edad_restada_imc + 15;
 
if($_POST['fumar'] == "si") {
    $cigarrillos_al_ano = $_POST['cigarrillos'] * 365;
    $cigarrillos_totales_fumados = $cigarrillos_al_ano * $_POST['tiempofumando'];
    $los_anos_que_quedan_por_fumar = ($esperanza_de_vida - $edad);
    $cigarrillos_que_quedan_por_fumar = $los_anos_que_quedan_por_fumar * $cigarrillos_al_ano;
    $suma_ambos = $cigarrillos_totales_fumados + $cigarrillos_que_quedan_por_fumar;
    $pasar_a_minutos = $suma_ambos * 11;
    $pasar_a_horas = $pasar_a_minutos / 60;
    $pasar_a_dias = $pasar_a_horas / 24;
    $pasar_a_anos = round($pasar_a_dias / 365);
} else { $pasar_a_anos = 0; }
 
$edad_restada_alcohol = 0;
if($_POST['beber'] == "apenas") {
    $edad_restada_alcohol = $edad_restada_alcohol + 1;
} elseif($_POST['beber'] == "fines") {
    $edad_restada_alcohol = $edad_restada_alcohol + 2;
} elseif($_POST['beber'] == "bastante") {
    $edad_restada_alcohol = $edad_restada_alcohol + 4;
} elseif($_POST['beber'] == "todoslosdias") {
    $edad_restada_alcohol = $edad_restada_alcohol + 5;
} elseif($_POST['beber'] == "alcoholico") {
    $edad_restada_alcohol = $edad_restada_alcohol +15;
}
 
if($_POST['drogas'] == "si") {
    if($_POST['tipodrogas'] == "blandas" and $_POST['cadacuantodrogas'] == "apenas") {
    $restado_por_drogas = 2; } 
    elseif($_POST['tipodrogas'] == "blandas" and $_POST['cadacuantodrogas'] == "amenudo") {
    $restado_por_drogas = 4; }
    elseif($_POST['tipodrogas'] == "blandas" and $_POST['cadacuantodrogas'] == "demasiado") {
    $restado_por_drogas = 6; }
    elseif($_POST['tipodrogas'] == "duras" and $_POST['cadacuantodrogas'] == "apenas") {
    $restado_por_drogas = 6; }
    elseif($_POST['tipodrogas'] == "duras" and $_POST['cadacuantodrogas'] == "amenudo") {
    $restado_por_drogas = 8; }
    elseif($_POST['tipodrogas'] == "duras" and $_POST['cadacuantodrogas'] == "demasiado") {
    $restado_por_drogas = 10; }
    elseif($_POST['tipodrogas'] == "ambas" and $_POST['cadacuantodrogas'] == "apenas") {
    $restado_por_drogas = 8; }
    elseif($_POST['tipodrogas'] == "ambas" and $_POST['cadacuantodrogas'] == "amenudo") {
    $restado_por_drogas = 10; }
    elseif($_POST['tipodrogas'] == "ambas" and $_POST['cadacuantodrogas'] == "demasiado") {
    $restado_por_drogas = 15; }
    } else { $restado_por_drogas = 0; }
 
if($_POST['alimentacion'] == "excelente") {
    $alimentacion_alarga = 14;
} elseif ($_POST['alimentacion'] == "decente") {
    $alimentacion_alarga = 8;
} elseif ($_POST['alimentacion'] == "decente") {
    $alimentacion_alarga = 8;
} elseif ($_POST['alimentacion'] == "normal") {
    $alimentacion_alarga = 6;
} elseif ($_POST['alimentacion'] == "decente") {
    $alimentacion_alarga = 8;
} elseif ($_POST['alimentacion'] == "bastantemala") {
    $alimentacion_alarga = 0;
} elseif ($_POST['alimentacion'] == "undesastre") {
    $alimentacion_alarga = 0;
}
 
if($_POST['ejercicio'] == "nunca") {
    $ejercicio_alarga = 0;
} elseif($_POST['ejercicio'] == "raravez") {
    $ejercicio_alarga = 1;
} elseif($_POST['ejercicio'] == "semanalmente") {
    $ejercicio_alarga = 2;
} elseif($_POST['ejercicio'] == "casitodoslosdias") {
    $ejercicio_alarga = 4;
}
//suma total
$cosasquerestan = $edad_restada_imc + $pasar_a_anos + $edad_restada_alcohol + $restado_por_drogas;
$cosasquesuman = $alimentacion_alarga + $ejercicio_alarga;
$resultadototal = ($esperanza_de_vida + $cosasquesuman) - $cosasquerestan;
//imprimimos datos para comprobar
echo "Esperanza de vida : <b>" . $esperanza_de_vida . "</b><br/>";
echo "IMC : <b>" . $estado_imc . "</b> Edad restada por IMC: <b>" . $edad_restada_imc . "</b><br/>";
echo "Fumar :<b> " . $fumar . "</b> Edad restada por FUMAR: <b>" . $pasar_a_anos . "</b><br/>";
echo "Beber : <b>" . $beber . "</b> Edad restada por BEBER: <b>" .  $edad_restada_alcohol . "</b><br/>";
echo "Drogas : <b>" . $drogas - "</b> Tipo : <b>" . $tipodrogas . "</b> Cada cuanto : <b>" . $cadacuantodrogas . "<b> Restado por drogas: <b>" . $restado_por_drogas . "</b><br/>";
echo "Alimentación : <b>" . $alimentacion . "</b> Edad sumada por alimentación <b>" . $alimentacion_alarga . "</b><br/>";
echo "Ejercicio <b>" . $ejercicio . "</b> Edad sumada por ejercicio <b>" . $ejercicio_alarga . "</b><br/>";
echo "<br/><br/";
echo "<b>" . $resultadototal . "</b>";
exit;
 

No he encontrado ahí errores de cierres de } pero al añadir el código en el condicional anterior, salta el error que comenté

Si no hay errores antes de poner el código que procesa el formulario, y no hay errores en el código que meto dentro, ¿ Donde está el error? ¿ Entra algo en conflicto ?

[Triby]

Generalmente ese error lo obtienes por que falta una llave de cierre, o sobra una de apertura, depende cómo lo veas.

Código PHP:

$edad_restada_imc = 0;
    if($estado_imc == 1) {
        $edad_restada_imc = $edad_restada_imc + 5;
    } elseif($estado_imc == 3) {
        $edad_restada_imc = $edad_restada_imc + 5;
    } elseif($estado_imc == 4) {
        $edad_restada_imc = $edad_restada_imc + 7;
    } elseif($estado_imc == 5) {
        $edad_restada_imc = $edad_restada_imc + 10;
    } elseif($estado_imc == 6) {
        $edad_restada_imc = $edad_restada_imc + 15;
    // Aquí no aparece la llave de cierre
 

[rodrypaladin]

Gracias Triby, ahí estaba el error.

Por fin he validado el formulario casi completo, pero me encuentro con otro problema. ( si si, otro, increible pero cierto )

Imaginar que el formulario tiene 10 preguntas, y se te olvida poner 2, te salta el error diciendo que dos preguntas te faltan por responder, PERO, las otras 8 que si que has respuesto vuelven a su estado de "selecciona" es decir, tienen que volver a responder todas, en vez de solo responder las 2 que faltan.

¿¿ Alguna recomendación para ello ?? Gracias de nuevo chicos !!

[Triby]

Pero, se supone que estabas incluyendo los valores para cada input; habría que ver tu código completo para ver lo que falta.

[rodrypaladin]

Aquí dejo el código del formulario si es el que necesitas:

Código PHP:

<form method="post" action="ejemplo.php" enctype="x-www-form-urlencoded">
            <h3><terror>Sobre tu físico</terror></h3><br/>
            <table border="0" align="left">
                <tr>
                <td><?php if(isset($errores['edad'])) { echo "<errores> Edad </errores>"; } else { echo "<enun> Edad </enun>"; } ?></td>
                <td>
<?php
// Crear select con opción predefinida
html_select('edad', '20', 'edad', $edad, array(), 14, 100);
?>
                </td>
                <td>
                <enunleft>años</enunleft>
                </td>
                </tr>
                <tr>
                    <td><?php if(isset($errores['peso'])) { echo "<errores> Peso </errores>"; } else { echo "<enun> Peso </enun>"; } ?></td>
                    <td>
<?php
// Crear select con opción predefinida
html_select('peso', '70', 'peso', $peso, array(), 30, 200);
 
?>
                    </td>
                     <td>
                <enunleft>Kg</enunleft>
                </td>
                </tr>
                <tr>
                    <td><?php if(isset($errores['altura'])) { echo "<errores> Altura </errores>"; } else { echo "<enun> Altura </enun>"; } ?></td>
                    <td>
<?php
// Crear select con opción predefinida
html_select('altura', '70', 'altura', $altura, array(), 130, 250);
 
?>
 
 
                    </td>
                     <td>
                <enunleft>Cm</enunleft>
                </td>
                </tr>
                <tr>
                    <td><?php if(isset($errores['genero'])) { echo "<errores> Genero </errores>"; } else { echo "<enun> Genero </enun>"; } ?></td>
                    <td>
 
<input type=radio name="genero" value="masculino">Masculino
<input type=radio name="genero" value="femenino">Femenino
 
                    </td>
                </tr>
            </table>
            <div class="separador">
        </div>
        <h3><terror>Sobre tus hábitos</terror></h3><br/>
        <table border="0" align="left">
            <tr>
                <td class="celda"><?php if(isset($errores['fumar'])) { echo "<errores> ¿Fumas tabaco? </errores>"; } else { echo "<enun> ¿Fumas tabaco? </enun>"; } ?></td>
                <td><INPUT type=radio name="fumar" value="si">Si
                    <INPUT type=radio name="fumar" value="no">No</td>
                <td><preg><enun>¿Cúantos cigarrillos al día?</enun></preg></td>
                <td>
               <SELECT name="cigarrillos">
                <?php
                    for($i=1;$i<=60;$i++) {
                        echo "<OPTION VALUE=\"$i\">$i</OPTION>";
                    }
                    ?>
                </SELECT>
                </td>
                    <td><enun>¿Cúanto años llevas fumando?</enun></td>
               <SELECT name="tiempofumando">
                <?php
                    for($i=1;$i<=50;$i++) {
                        echo "<OPTION VALUE=\"$i\">$i</OPTION>";
                    }
                    ?></SELECT>
                </td>
            </tr>
        </table>
        <table border="0" align="left">
        <tr>
            <td><?php if(isset($errores['beber'])) { echo "<errores> ¿Bebes alcohol? </errores>"; } else { echo "<enun> ¿Bebes alcohol? </enun>"; } ?></td>
            <td><INPUT type=radio name="beber" value="nunca">Nunca</td>
            <td><INPUT type=radio name="beber" value="apenas">Apenas</td>
            <td><INPUT type=radio name="beber" value="fines">Los fines de semana</td>
            <td><INPUT type=radio name="beber" value="bastante">Bastante</td>
            <td><INPUT type=radio name="beber" value="todoslosdias">Todos los días</td>
            <td><INPUT type=radio name="beber" value="alcoholico">Muchisimo</td>
        </tr>
        </table>
        <table border="0" align="left">
        <tr>
            <td><?php if(isset($errores['drogas'])) { echo "<errores> ¿Tomas drogas? </errores>"; } else { echo "<enun> ¿Tomas drogas? </enun>"; } ?></td>
            <td><INPUT type=radio name="drogas" value="si">Si
                <INPUT type=radio name="drogas" value="no">No</td>
            <td><preg><enun>¿De qué tipo?</enun></preg></td>
            <td><SELECT name="tipodrogas">
                    <OPTION VALUE="Ninguna">Ninguna</OPTION>
                    <OPTION VALUE="blandas">Blandas</OPTION>
                    <OPTION VALUE="duras">Duras</OPTION>
                    <OPTION VALUE="ambas">Ambas</OPTION>
                    </SELECT></td>
            <td><enun>¿Cada cúanto?</enun></td>
            <td><SELECT name="cadacuantodrogas">
                    <OPTION VALUE="Ninguna">Ninguna</OPTION>
                    <OPTION VALUE="apenas">Apenas</OPTION>
                    <OPTION VALUE="amenudo">A menudo</OPTION>
                    <OPTION VALUE="demasiado">Demasiado</OPTION>
                    </SELECT></td>
        </tr>
        </table>
        <table border="0" align="left">
        <tr>
            <td><?php if(isset($errores['alimentacion'])) { echo "<errores> ¿Como es tu alimentación? </errores>"; } else { echo "<enun> ¿Como es tu alimentación? </enun>"; } ?></td>
            <td><INPUT type=radio name="alimentacion" value="nunca">Excelente</td>
            <td><INPUT type=radio name="alimentacion" value="apenas">Decente</td>
            <td><INPUT type=radio name="alimentacion" value="fines">Normal</td>
            <td><INPUT type=radio name="alimentacion" value="bastante">Bastante mala</td>
            <td><INPUT type=radio name="alimentacion" value="todoslosdias">Un desastre</td>
        </tr>
        </table>
        <table border="0" align="left">
        <tr>
            <td><?php if(isset($errores['ejercicio'])) { echo "<errores> ¿Haces ejercicio? </errores>"; } else { echo "<enun> ¿Haces ejercicio? </enun>"; } ?></td>
            <td><INPUT type=radio name="ejercicio" value="nunca">Nunca</td>
            <td><INPUT type=radio name="ejercicio" value="raravez">Rara vez</td>
            <td><INPUT type=radio name="ejercicio" value="semanalmente">Semanalmente</td>
            <td><INPUT type=radio name="ejercicio" value="casitodoslosdias">Casi todos los días</td>
        </tr>
        </table>
        <br/><br/>
        <div class="separador">
        </div>
        <h3><terror>Sobre tus gustos</terror></h3><br/>
        <table border="0" align="left" style="width: 900px;">
        <tr style="width: 300px;">
            <TD><input type="checkbox" name="tomarelsol" value="tomarelsol">Opcion 1</TD> 
            <TD><input type="checkbox" name="conduccionrapida" value="conduccionrapida">Opción 2</TD> 
            <TD><input type="checkbox" name="deportesextremos" value="deportesextremos">Opción 3</TD> 
            <TD><input type="checkbox" name="cometerdelitos" value="cometerdelitos">Opción 4</TD>
        </tr style="width: 300px;">
        <tr>
            <TD><input type="checkbox" name="viajarmucho" value="viajarmucho">Opción 5</TD>
            <TD><input type="checkbox" name="nousarpreservativos" value="nousarpreservativos">Opción 6</TD>
            <TD><input type="checkbox" name="sexuales" value="sexuales">Opción 7</TD>
            <TD><input type="checkbox" name="salirtodoslosfines" value="salirtodoslosfines">Opción 8</TD> 
        </tr style="width: 300px;">
        <tr> 
            <TD><input type="checkbox" name="frecuentardiscotevas" value="frecuentardiscotecas">Opción 9</TD>
            <TD><input type="checkbox" name="trasnocharmucho" value="trasnocharmucho">Opción 10</TD>
            <TD><input type="checkbox" name="trabajarmucho" value="trabajarmucho">Opción 11</TD>
            <TD><input type="checkbox" name="tomarmuchocafe" value="tomarmuchocafe">Opción 12</TD>
        </tr>
        </table>
        <input type="submit" class="button" value="enviar"/><br/>
        </form>
 

[Triby]

Bueno, solo los selectes tienen función y variables para conservar los valores; falta hacerlo también con los radios y checkbox.

[rodrypaladin]

Pero tampoco en los select conserva el valor para cuando saltan los errores recordando que faltan preguntas por responder.

EDITO: Supuestamente esto es lo que haría que se quedara guardado el valor para no tener que volver a seleccionarlo en los select, pero no lo realiza.

Código PHP:

foreach($options as $key => $val) {
        // Para establecer la opción preseleccionada
        $selected = ($key == $value) ? ' selected="selected"' : '';
        echo "\t<option value=\"$key\"$sel>$val</option>\n";
    }
 

[Triby]

Y cómo quedo la función?, porque me parece que agregaste un parámetro o hiciste otra modificación.

Originalmente era esto:

Código PHP:

<?php
// Una función útil para crear selects
function html_select($name, $id, $value, $options = array(), $start = 0, $end = 0) {
    echo "<select name=\"$name\" id=\"$id\">\n";
    if(count($options) == 0) {
        // Creamos el array con inicio y fin
        if($start > 0) {
            // Agregamos opción cero
            $options[0] = 'Selecciona...';
        }
        for($i = $start; $i <= $end; $i++) {
            $options[$i] = $i;
        }
    }
    // Agregamos las opciones
    foreach($options as $key => $val) {
        // Para establecer la opción preseleccionada
        $selected = ($key == $value) ? ' selected="selected"' : '';
        echo "\t<option value=\"$key\"$sel>$val</option>\n";
    }
    echo "</select>\n";
}
 

Y la ejecutas así:

Código PHP:

html_select('peso', '70', 'peso', $peso, array(), 30, 200);
/*
    'peso' <<< OK, corresponde a $name
    '70'     <<< Este no sé de dónde salió **********
    'peso' <<< OK $id
    $peso <<< OK $value
    30       <<< OK $start
    200     <<< OK $end
*/
 

[rodrypaladin]

Si, perdona, le metí un size, para cambiar el ancho del select, pero lo quité hace nada porque al final no lo necesitaba. pero no era por eso, el valor desaparece igualmente.