Evitar envio de formulario con checkbox marcado con otro campo input vacio

[iovan]

Amigo, es que hay mucha redundancia en el código.

La solucion es tan sencilla, algunos ya se han acostumbrado a usar un monton de condicionales a mi en lo personal no me gusta.

Por ejemplo, En este caso el resultado lo podrias obtener en 200 lineas o mas utilizando un manojo de condicionales o en menos de 60 lineas, eso depende de tí.

A mi me gustaría ayudarte, lo único que yo quiero saber es a que le quieres dar control. Es todo.

Porque el código que te puse controla todo, y con una modificacion en 1 linea controlarias hasta el comportamiento de todos los checkbox.

Bueno, un saludo.
>>>>>>>>>>>>>>>>>>>>>>>>>>>>
Edito.

Que tal ésto:

formulario html

Código HTML:

<form action="" method="post">
    <p>check1: <input type="checkbox" name="check1" value="" /> importe-rotura:  
        <input type="text" name="importe-rotura" value="Incompleto"/></p>
 
    <p>check2: <input type="checkbox" name="check2" value="" />  importe-daños
        <input type="text" name="importe-daños" value="Incompleto"/></p>
 
    <p>check3: <input type="checkbox" name="check3" value="" />  importe-derrumbe
        <input type="text" name="importe-derrumbe" value="Incompleto"/></p>
 
    <p>check4: <input type="checkbox" name="check4" value="" /> importe-todo-riesgo
        <input type="text" name="importe-todo-riesgo" value="Incompleto"/></p>
 
    <input type="submit" name="submit">
</form>
 

El funcionamiento es, si bien entiendo es lo que buscas:

4 checkbox cada uno corresponde a una textfield. Es decir si se marca un checkbox se debe colocar un valor correcto en la textfield correspondiente, y si se coloca un valor correcto en la textfield se debe marcar el checkbox correspondiente.

Nota: el numero de checkbox y texfield pueden aumentar o disminuir tanto se necesite.
script PHP

Código PHP:

<?php
$cheakboxes = array("check1", "check2", "check3", "check4");
$textfields = array("importe-rotura", "importe-daños", "importe-derrumbe", "importe-todo-riesgo");
 
$error = $cont_cheakbox = $cont_textfields = 0;
 
function enviarForm() {
    echo "Mando el formulario";
}
 
foreach ($cheakboxes as $key => $valor) {
    if (isset($_POST[$valor]))
        $cont_cheakbox++;
}
foreach ($textfields as $key => $valor) {
    if (!empty($_POST[$valor]) && $_POST[$valor] != "Incompleto")
        $cont_textfields++;
}
 
if (isset($_POST['submit'])) {
//Todos los cheakbox estan marcados por lo tanto todos los textfield tambien deben estarlo, (validar)
    if ($cont_cheakbox == count($cheakboxes)) {
        //Valido que todas las textfield esten correctas, en caso de que haya una que no lo este muestro cual
        if ($cont_textfields == count($textfields)) {
            enviarForm();
        } else if ($cont_textfields > 0) {
            foreach ($textfields as $key => $value) {
                if(isset($_POST[$cheakboxes[$key]]) && (empty($_POST[$textfields[$key]]) || $_POST[$textfields[$key]] == "Incompleto"))
                    echo "<p>Has seleccionado el checkbox " . $cheakboxes[$key] . " por lo tanto tienes que escribir algo en " . $textfields[$key] . "</p>";
            }
        } else {
            echo "<p>Debes corregir todos los textfields</p>";
        }
    } else if ($cont_cheakbox == 0) {
        //Si ningun checkbox se ha seleccionado,
        if($cont_textfields == 0){
            echo "<p>Debe llenar el formulario</p>";
        }
        else if ($cont_textfields > 0) {
            //1.- Valido si se hay una textfield correcta cuyo checkbox no haya sido marcado
            foreach ($textfields as $key => $value) {
                if( !isset($_POST[$cheakboxes[$key]]) && (!empty($_POST[$textfields[$key]]) && $_POST[$textfields[$key]] != "Incompleto"))
                    echo "<p>Has escrito algo en " . $textfields[$key] . " por lo tanto debes marcar el checkbox " . $cheakboxes[$key]."</p>";
            }
        }
    } else {
        //Ahora si se selecciona por lo menos 1 checkbox 
        //Valido cual checkbox se ha seleccionado y verifico que su textfield tambien este correcto
        foreach ($cheakboxes as $key => $value){
            if(isset($_POST[$cheakboxes[$key]]) && (empty($_POST[$textfields[$key]]) || $_POST[$textfields[$key]] == "Incompleto")){
                echo "<p>Has seleccionado el checkbox " . $cheakboxes[$key] . " por lo tanto tienes que escribir algo en " . $textfields[$key] . "</p>";
                $error++;
            }
            //Verifico ahora si lo que no esta correcto es que se haya puesto correcto el textfield pero no se haya seleccionado el checkbox
            else if( !isset($_POST[$cheakboxes[$key]]) && (!empty($_POST[$textfields[$key]]) && $_POST[$textfields[$key]] != "Incompleto")){
                echo "Has escrito algo en " . $textfields[$key] . " por lo tanto debes marcar el checkbox " . $cheakboxes[$key];
                $error++;
            }
        }
        
        if($error==0){
            enviarForm();
        }
    }
}
?>
 

Las primeras 2 lineas tenemos

Código PHP:

$cheakboxes = array("check1", "check2", "check3", "check4");
$textfields = array("importe-rotura", "importe-daños", "importe-derrumbe", "importe-todo-riesgo");
 

Se tratan de 2 arreglos, uno para los nombres de los checkbox y el otro para guardar los nombres de las textfields

Aqui puedes aumentar o disminuir el numero de campos a validar, siempre y cuando se respete que cualquiera de los 2 arreglos es directamente proporcional al otro en tamaño.

Esta duda ya te ha tomado bastante tiempo y es muy sencilla como para que nosotros no la podamos resolver en una sentada haha. Yo me he tardado al rededor de 15 minutos espero esta vez si te haya ayudado por que si no, entonces me despido del tema ya hice todo lo que pude.

Te prometí menos de 60 lineas (por decir un número) pero te he dado 66. Supongo que no hay problema.

P.D: Si le quitamos las 7 de comentarios serian 59 lineas de codigo, curiosamente he cumplido con lo dicho

haha Saludos.