-1

Me gustaría saber de qué manera puedo mejorar este código de jQuery, ya que en ocasiones no me recupera los valores, o incluso me recupera los valores de un dato anterior, es decir: en ocasiones no recupera los valores correspondientes. ¿Existe una mejor forma de lo que yo realice para recuperar los valores de un formulario?

Espero me puedan orientar de acuerdo a su experiencia, se los agradecería. Me gustaría mejorar mi código de acuerdo a sus experiencias. Para ir mejorando.

El siguiente código es de un crud que estoy realizando

 **$("#formReportes").submit(function (e) {
    e.preventDefault(); //evita el comportambiento normal del submit, es decir, recarga total de la página
    folio = $.trim($("#folio").val());
    nombre = $.trim($("#nombre").val());
    apellido = $.trim($("#apellido").val());
    direccion = $.trim($("#direccion").val());
    descripcion_domicilio = $.trim($("#descripcion_domicilio").val());
    telefono = $.trim($("#telefono").val());
    num_contrato = $.trim($("#num_contrato").val());
    fecha_inicio = $.trim($("#fecha_inicio").val());
    fecha_final = $.trim($("#fecha_final").val());
    cuadrilla = $.trim($("#cuadrilla").val());
    estatus = $.trim($("#estatus").val());
    observaciones = $.trim($("#observaciones").val());
    encargado_cuadrilla = $.trim($("#encargado_cuadrilla").val());
    otra_descripcion = $.trim($("#otra_descripcion").val());
    $.ajax({
      url: "../controlador/reportes.php",
      type: "POST",
      datatype: "json",
      data: {
        id: id,
        folio: folio,
        nombre: nombre,
        apellido: apellido,
        direccion: direccion,
        descripcion_domicilio: descripcion_domicilio,
        telefono: telefono,
        num_contrato: num_contrato,
        fecha_inicio: fecha_inicio,
        fecha_final: fecha_final,
        cuadrilla: cuadrilla,
        estatus: estatus,
        observaciones: observaciones,
        encargado_cuadrilla: encargado_cuadrilla,
        otra_descripcion: otra_descripcion,
        opcion: opcion,
      },
      success: function (data) {
        console.log(data);
        tablaReportes.ajax.reload(null, false);
      },
    });
    $("#modalCRUD").modal("hide");
  });
  //para limpiar los campos antes de dar de Alta una Persona
  $("#btnNuevo").click(function () {
    opcion = 1; //alta
    id = null;
    $("#formReportes").trigger("reset");
    $(".modal-header").css("background-color", "#17a2b8");
    $(".modal-header").css("color", "white");
    $(".modal-title").text("Alta Reportes");
    $("#modalCRUD").modal("show");
  });
    //Editar
      $(document).on("click", ".btnEditar", function () {
        opcion = 2; //editar
        fila = $(this).closest("tr");
        id = parseInt(fila.find("td:eq(0)").text()); //capturo el ID
        folio = fila.find("td:eq(1)").text();
        nombre = fila.find("td:eq(2)").text();
        apellido = fila.find("td:eq(3)").text();
        direccion = fila.find("td:eq(4)").text();
        descripcion_domicilio = fila.find("td:eq(5)").text();
        telefono = fila.find("td:eq(6)").text();
        num_contrato = fila.find("td:eq(7)").text();
        fecha_inicio = fila.find("td:eq(8)").text();
        fecha_final = fila.find("td:eq(9)").text();
        cuadrilla = fila.find("td:eq(10)").text();
        estatus = fila.find("td:eq(11)").text();
        observaciones = fila.find("td:eq(12)").text();
        encargado_cuadrilla = fila.find("td:eq(13)").text();
        otra_descripcion = fila.find("td:eq(14)").text();
        $("#folio").val(folio);
        $("#nombre").val(nombre);
        $("#apellido").val(apellido);
        $("#direccion").val(direccion);
        $("#descripcion_domicilio").val(descripcion_domicilio);
        $("#telefono").val(telefono);
        $("#num_contrato").val(num_contrato);
        $("#fecha_inicio").val(fecha_inicio);
        $("#fecha_final").val(fecha_final);
        $("#cuadrilla option:contains(" + cuadrilla + ")").attr("selected", "");
        $("#estatus option:contains(" + estatus + ")").attr("selected", "");
        $("#observaciones").val(observaciones);
        $("#encargado_cuadrilla").val(encargado_cuadrilla);
        $("#otra_descripcion").val(otra_descripcion);
        $(".modal-header").css("background-color", "#007bff");
        $(".modal-header").css("color", "white");
        $(".modal-title").text("Editar Reporte");
        $("#modalCRUD").modal("show");
      });**
13
  • Si bien es una biblioteca de JavaScript, lo que estás usando aquí es jQuery. ¿Dónde estás llamando al script? No olvides mostrar un ejemplo mínimo verificable. Por cierto, no preguntes por "la mejor forma de" ya que estas preguntas están basadas en opiniones, y terminan cerradas.
    – padaleiana
    el 14 sep. 2022 a las 15:47
  • Lo agregué a mi código. el 14 sep. 2022 a las 16:22
  • Hola, He mirado tu post y te comento lo siguiente:, Puedes leer este Articulo Cómo preguntar para que tu pregunta sea mejor recibida, Puedes leer este Articulo ejemplo mínimo verificable, También es importante que sepas que preguntas son mal vistas centro de ayuda.
    – user46896
    el 14 sep. 2022 a las 16:30
  • 1
    @Yussef no se puede mejorar un cadigo a medias y sin saber que se quiere mejorar... es en base a cada quien que se quiera mejorar... si esta pregunta que tu dices que tiene que ver con refactorizar ... en code review puede ser el sitio en donde hacerla; por lo general las refactorizacion que se hacen aqui son muy especificas...
    – user46896
    el 15 sep. 2022 a las 13:14
  • 1
    @Yussef tampoco tenemos el codigo HTML no podemos reproducir el error que tiene o que menciona el OP, por lo tanto hay que partir desde un inicio con que el OP aclare que sucede realmente, por eso estamos especulando y opinando de todo menos de lo que realmente quiere el Op... esta pregunta esta basada en opiniones y debe ser cerrada. hasta que se tengan los elementos para dar una rerspuesta de calidad para el sitio.
    – user46896
    el 15 sep. 2022 a las 13:17

1 respuesta 1

5

Aunque como comentan esta pregunta esta basada en opiniones te dejo un ejemplo, ya que todos estamos aprendiendo y se que aveces es complicado entender cuando vamos comenzando en el mundo de la programación, te recomiendo investigar mas del tema.

Si hay una forma mejor de obtener los valores de un form, veo que estas utilizando JQuery, te dejo un ejemplo:

//FUNCION QUE ENVIA EL FORMULARIO MEDIANTE AJAX, ESTE ESCRIPT DEBE IR AL FINAL DEL DOM 

//DETECTAMOS EL SUBMIT DEL FORM
$('#formulario').on('submit', function(ev) {

  $.ajax({
    //TOMA LOS ATRIBUTOS DEL FORM CON JQUERY
    //METODO ESPECIFICADO EN TU FORM, POST ,GET
    type: $('#formulario').attr('method'),
    //DIRECCION DE ENVIO DE LA DATA DEL FORM
    url: $('#formulario').attr('action'),
    //TODOS LOS CAMPOS QUE CONTIENE EL FORM
    data: $('#formulario').serialize(),
    beforeSend: function() {
      //ACCION ANTES DE RESOLVER LA PETICION
    },
    success: function(data) {
      //ACCION SI LA PETICION SE RESOLVIO CORRECTAMENTE
    },
    error: function() {
      //ACCION O MENSAJE EN CASO DE FALLAR LA PETICION
    }
  });
  //EVITAMOS EL EVENTO DE REDIRECCION AL ENVIAR FORMULARIO
  ev.preventDefault();
  //LIMPIAMOS EL FORMULARIO, SOLO CAMPOS DE TEXTO
  $("#formulario")[0].reset();
});

Este es un pequeño ejemplo, lo puedes adaptar, realiza tus intentos y en base a eso realizas posteriores preguntas.

7
  • no estoy de acuerdo con tu respuesta. primero que nada no implementas un selector delegado... si el form cambia tu ejemplo deja de funcionar
    – user46896
    el 14 sep. 2022 a las 17:14
  • 1
    Acaso el autor de la pregunta indico que su formulario estaba expuesto a cambios?, gracias por tu observación, pero si tienes una mejor respuesta puedes generarla o corregir lo que esta mal de la mía, creo que de eso se trata SO, no solo de opiniones, estamos para apoyarnos y seguir aprendiendo. el 14 sep. 2022 a las 17:27
  • 1
    si pero no respondo preguntas basadas en opiniones por este mismo asunto ... que se prestan para debates de que esta bien y que esta mal...
    – user46896
    el 14 sep. 2022 a las 17:34
  • 1
    otra cosa serialize no leera los checkbox no seleccionados.
    – user46896
    el 14 sep. 2022 a las 17:35
  • 3
    Hola, antes de responder puedes sugerir al OP aclarar su pregunta, las preguntas deben basarse en Cómo preguntar, ayúdanos a mantener la calidad de la información en el sitio, saludos.
    – Jorgesys
    el 14 sep. 2022 a las 20:34

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.