Validación Js para formularios generados con do-while

[malmonacid]

Amigos del Foro;
Estoy aquí en busca de ayuda, ya que en mi ignorancia no he podido validar el mismo formulario, pero que se repite n veces en la misma página (lo replico usando la sentencia do-while). Los usuarios pueden submit solo uno cada vez.
Sé que necesito darles un identificador único a cada uno para que el js sepa cual es el que debe validar o alomejor un id a la función Js?.

Este es el código de los formularios

Código PHP:

$editFormAction = $_SERVER['PHP_SELF'];
    if (isset($_SERVER['QUERY_STRING'])) {
      $editFormAction .= "?" . htmlentities($_SERVER['QUERY_STRING']);
    }
    if ((isset($_POST["MM_update"])) && ($_POST["MM_update"] == "form3")) {
      $updateSQL = sprintf("UPDATE tblcoment_caractpoli SET intActivo=%s, dateEliminar=%s WHERE idComent_caractpoli=%s",
                           GetSQLValueString($_POST['intActivo'], "int"),
                GetSQLValueString($_POST['dateEliminar'], "timestamp"),
                           GetSQLValueString($_POST['idComent_caractpoli'], "int"));
      mysql_select_db($database_conexionproject_politics, $conexionproject_politics);
      $Result1 = mysql_query($updateSQL, $conexionproject_politics) or die(mysql_error());
      $updateGoTo = "";
      if (isset($_SERVER['QUERY_STRING'])) {
        $updateGoTo .= (strpos($updateGoTo, '?')) ? "&" : "?";
        $updateGoTo .= $_SERVER['QUERY_STRING'];
      }
      header(sprintf("Location: %s", $updateGoTo));
    }
     
    do {?>
    <form action="<?php echo $editFormAction; ?>" method="POST" name="form3" id="form3" >
            <a onclick="return validarFormulario();" href="javascript:document.form3.submit();"><span>Eliminar</span></a>
             <input type="hidden" name="MM_update" value="form3" />
              <input type="hidden" name="intActivo" value="0" />
              <input type="hidden" name="dateEliminar" value="now()" />
              <input type="hidden" name="idComent_caractpoli" value="<?php echo $opinion; ?>" />
            </form>
    <?php } while ($row_DatosOpiniones = mysql_fetch_assoc($DatosOpiniones));
 

Aquía va el código Js que debería validar los form's
Código Javascript

Código Javascript:

function validarFormulario()
{
    valid = true;
 
    document.form3.intActivo.style.border='1px solid #CCC';
    
    if ( document.form3.intActivo.value == "0" )
    {
        alert ( "Está seguro que quiere eliminar esta opinión?" );
        valid = false;
    }
        
    if ( document.form3.intActivo.value == "0" )
         document.form3.intActivo.style.border='2px solid red';
 
    return valid;
}
 

:

No está de más decir que AGRADECERÍA enormemente cualquier ayuda para poder arreglar este código y hacerlo que pueda validar cualquiera de los formularios.

[Alexis88]

Podrías asignar un ID distinto en cada iteración a cada formulario creado, así ya no tomaría al formulario por su nombre sino por su ID, aunque, particularmente, preferiría tener una lista de datos con un botón de edición y que al pulsarlo, muestre un formulario con los datos de dicho registro. Eso de tener una lista de N formularios no me parece práctico ni estético.

Saludos

[malmonacid]

Gracias por responder Alexis88.

Podrías explicarme un poco, por favor, como puedo implementar el ID de manera automática. El ID para el formulario, lo entiendo así.....

Código HTML:

<form action="<?php echo $editFormAction; ?>" method="POST" name="form<?php echo $i++; ?>" id="form<?php echo $i++; ?>" >
 

Pero debo implementar algo similar en el codigo que envía los datos. El "form3" debe cuadrar con el form<?php echo $i++; ?> que definí arriba.

Código PHP:

if ((isset($_POST["MM_update"])) && ($_POST["MM_update"] == "form3")) {
      $updateSQL = sprintf("UPDATE tblcoment_caractpoli SET intActivo=%s, dateEliminar=%s WHERE idComent_caractpoli=%s",
                           GetSQLValueString($_POST['intActivo'], "int"),
                GetSQLValueString($_POST['dateEliminar'], "timestamp"),
                           GetSQLValueString($_POST['idComent_caractpoli'], "int"));
 

Y además la función Js que valida el formulario debería también tener el mismo índice.

Creo que entiendo la idea general?, pero me falla el conocimiento técnico para implementarlo.

Gracias de antemano.

[Alexis88]

Creo que en lugar de seguir complicándote con este camino, mejor toma el consejo que te di en la segunda parte de mi anterior respuesta. Es una manera más limpia y ordenada de hacerlo, a pesar de que, para la forma en que lo haces, también hay solución.

Saludos

[malmonacid]

El listado que quiero mostrar son opciones secundarias, que ya estoy desplegando en una nueva ventana. Estoy asociando un botón "Eliminar" a cada opción para que si el usuario desea, pueda eliminar las opciones que quiera, una por una.

Básicamente lo que me sugieres, entiendo, sería desplegar el formulario en otra ventana, para evitarme la recursividad. Pero no quiero usar una tercera ventana, eso sería demasiado......

[Alexis88]

¿Tercera ventana? Lo que te decía es que listes a todos los registros y que por cada registro, le añadas al final dos botones: Uno para editar los datos de dicho registro y otro para eliminarlos, para lo cual solo tendrías que pasar el ID de dicho registro en la ruta de la opción. Al pulsar uno de dichos botones, bien podrías cargar el formulario de edición en una ventana emergente, en una ventana modal o en una ventana/pestaña nueva. Insisto, tener a todos los formularios juntos, puede resultar incómodo, confuso y poco estético.

Saludos

[malmonacid]

Alexis,

Disculpa por que no fui claro en mi anterior comentario. Sería una tercera ventana, por que el listado de datos ya lo estoy desplegando en una ventana emergente y quiero que en esta ventana sólo tengan que pulsar el botón asociado al dato que quieran Eliminar, bueno en realidad puse eliminar para simplicar el tema, supuestamente. En realidad cada formulario tiene un menu desplegable, los usuarios eligen una opción y la envían; quiero que la función Js verifique que seleccionaron una opción.

Mi tema es que no quiero incrementar el proceso redireccionando a los usuarios a una nueva página
especifica para el dato que les interese. Además que una nueva página se justificaría si aportara más información del dato que les interese, pero no va a ser así, sería sólo el dato aislado, ya no listado con otros.

Bueno, conociendo mejor el contexto, ahora entenderás por que tu recomendación no la he implementado.

[Alexis88]

En lugar de usar un enlace, podrías usar un botón de tipo submit. Al ejecutar el proceso de envío, muestras una ventana de confirmación; si el usuario acepta eliminar los datos que correspondan al formulario del botón pulsado, entonces se ejecuta el respectivo proceso.

Código HTML:

<form action="<?php echo $editFormAction; ?>" method="POST" name="form<?php echo $i++; ?>" id="form<?php echo $i++; ?>" onsubmit = "return confirm('¿Está seguro de que desea eliminar estos datos?')" >
    <!-- Resto de controles -->
    <input type = "submit" value = "Eliminar" />
</form>
 

Saludos

[malmonacid]

Realmente agradezco tu interés por ayudar. Claramente tratando de simplicar mi problema, al final lo he complicado más.

La acción que los usuarios pueden hacer no es "Eliminar", lo usé como ejemplo tratando de simpliquicar el asunto. Para Eliminar tu solución es perfecta.

Lo que necesito realmente es validar que no me manden el formulario vacío. El form es un menu desplegable con un botón tipo "submit". Pueden mandar uno a la vez de los n de la lista.

En definitiva necesito una ventana de "warnning" que aparezca si no ha seleccionado ninguna opción y lo devuelva.

El formulario que debí haber puesto desde un principio es este:

Código PHP:

do {?>
    <form action="<?php echo $editFormAction; ?>" method="post" name="form1" id="form1">
      <select name="intEvaluacion" id="intEvaluacion">
      <option value="" <?php if (!(strcmp('', htmlentities(ObtenerEvalUsuario_CaractPoli($row_Recordset2['idCaract']), ENT_COMPAT, 'iso-8859-1')))) {echo "SELECTED";} ?>>Seleccionar</option>
        <option style="color:#03F" value="10" <?php if (!(strcmp(10, htmlentities(ObtenerEvalUsuario_CaractPoli($row_Recordset2['idCaract']), ENT_COMPAT, 'iso-8859-1')))) {echo "SELECTED";} ?>>100%</option>
        <option style="color:#03F" value="5" <?php if (!(strcmp(5, htmlentities(ObtenerEvalUsuario_CaractPoli($row_Recordset2['idCaract']), ENT_COMPAT, 'iso-8859-1')))) {echo "SELECTED";} ?>> 50%</option>
        <option style="color:#03F" value="0" <?php if (!(strcmp(0, htmlentities(ObtenerEvalUsuario_CaractPoli($row_Recordset2['idCaract']), ENT_COMPAT, 'iso-8859-1')))) {echo "SELECTED";} ?>>  0%</option>
 
      <input onclick="return validarFormulario('menu<?php echo $row_Recordset2['idCaract'];?>');" type="submit" value="Evaluar" />
 
<input type="hidden" name="dateFecha" value="now()" />
  <input type="hidden" name="intActivo" value="1" />
  <input type="hidden" name="MM_insert" value="form1" />
   <input type="hidden" name="idCaract" value="<?php echo $row_Recordset2['idCaract']; ?>" />
   <input type="hidden" name="idUser" value="<?php echo $_SESSION['MM_IdUser']; ?>" />
</form>
 
    <?php } while ($row_DatosOpiniones = mysql_fetch_assoc($DatosOpiniones));
 

Saludos.

[Alexis88]

Te estás complicando mucho para hacer algo tan sencillo. Lo que debes de hacer es enviar un identificador del formulario, buscar en él al combo y verificar su valor; si este es nulo, se muestra el mensaje de alerta, caso contrario, se procede a enviar los datos del formulario.

Código HTML:

<form action="<?php echo $editFormAction; ?>" method="post" name="form1" id="form1" onsubmit = "return validar(this)">
 

Código Javascript:

function validar(form){
    var combo = form.getElementsByTagName("select")[0];
 
    if (!combo.value.length){
        alert ("Debe de elegir una opción");
        return false;
    }
    else{
        return true;
    }
}
 

Otra manera de validar pero con HTML5 de que se elija una opción del combo es mediante el uso del atributo required.

Código HTML:

<select name="intEvaluacion" id="intEvaluacion" required>
 

De esta forma, si el usuario no seleccionó una opción válida, se cancelará el envío de datos y se mostrará un mensaje en un pequeño recuadro debajo del combo en el que se le indicará al usuario que debe de elegir una opción de la lista. La desventaja está en que no funcionará en navegadores viejos que son los que no soportan HTML5. Ojo, estás haciendo mal al asignar el mismo id a todos los combos que se generen en el bucle. Corrige eso.

Saludos

[malmonacid]

Muchas gracias, ahora funciona para ese fin específico.

Lo de complicarme, tienes razón en parte. Lo que sucede es que quería matar varios pájaros de un tiro. Me refiero a que con la orientación que recibiera poder adaptarlo a varios problemas que tengo.

Básicamente no se con hacer que una función Js, que es llamada con PHP, funcione cuando los datos los genero mediante un bucle. Por lo tanto esa función debe poder reconocer en que registro actuar. No se como implementar los id únicos y automáticamente. Por ejemplo la funcion Js (Mostrar y después cambia a Ocultar), que te permite mostrar un comentario completo o una parte de el; cuando los comentarios se generan mediante un bucle, no encuentro información o la solución para poder implementarlo.

Nuevamente muchas gracias por tu ayuda Alexis88.