Skip to main content
3 de 4
se eliminaron 146 caracteres en el cuerpo
ElChiniNet
  • 3.3k
  • 12
  • 26

Lo que sucede es que las dos llamadas Ajax se ejecutan a la vez, por lo tanto no importa lo que devuelva la primera llamada, la segunda siempre se ejecutará. Para que la segunda se ejecute solo cuando ocurra una de las condiciones deberías ejecutar la segunda dentro de dicha condición.

Voy a reescribir tu código siguiendo algunas buenas prácticas que te ayudarán a que el mismo esté más optimizado y más ordenado:

function obtenerSancionado () {

    var cedulaInput = $("#ccedula");

    var cedula = { rcedula: cedulaInput.val() };

    var limpiarCedula = function () {

        cedulaInput.val("").focus();

    };

    var metodos = {

        "metodo0" : function () {

            alert("Este funcionario se encuentra disponible para la sanción");

            metodos.obtener();

        },

        "metodo1" : function () {

            alert("Este funcionario NO se encuentra disponible por estar sancionado");

            limpiarCedula();

        },

        "metodo2" : function () {

            alert("Este n\u00famero de C\u00e9dula no está registrado en la base de datos");

            limpiarCedula();

        },

        "metodo3" : function () {

            alert("Este n\u00famero de C\u00e9dula no está registrado en la base de datos");

            limpiarCedula();

        },

        "metodo4" : function () {

            // ejemplo modificado con el json devuelto
            $("#papellido").val(respuesta.papellidox);
            $("#sapellido").val(respuesta.sapellidox);
            $("#pnombre").val(respuesta.pnombrex);
            $("#snombre").val(respuesta.snombrex);
            $("#grados").val(respuesta.gradof);
            $("#valor_grados").val(respuesta.gradoc);
            $("#fascenso").val(respuesta.fult_ascenso);
            $("#cargo").val(respuesta.cargo);
            $("#ccedula").prop("disabled", true);
            $("#restablecer1").prop("hidden", false);

        },

        "obtener" : function () {

            $.post("./funciones_php_obtenerDatosBasicos.php", cedula , function (respuesta) {

                metodos[respuesta.pnombrex == 99999 ? "metodo3" : "metodo4"]();

            }, "json" );

        }

    };

    //---Primera llamada Ajax
    $.post("./funciones_php_verificarSancion.php", cedula , function (respuesta) {

        metodos["metodo" + respuesta.sancionado]();

    }, "json" );

}

Te explico los cambios en tu código:

  1. Si vas a utilizar el elemento $("#ccedula") muchas veces, lo mejor es cachearlo en una variable.
  2. Si vas a repetir las mismas líneas varias veces en tu código, lo mejor es situar estas líneas en una función y llamar a esta cada vez que lo necesites (esto lo he aplicado con la función limpiarCedula que lo que hace es limpiar el valor del input y darle foco al mismo).
  3. En vez de crear varios if y else if, sitúa todos tus métodos dentro de un objeto y llama a estos métodos cuando sea necesario.
  4. Uno de los métodos es la segunda llamada Ajax (le he situado como nombre al método, obtener). Cuando se llame al método metodo0 se llama al método obtener, si no se llama al metodo0 la segunda llamada Ajax nunca se ejecutará.
ElChiniNet
  • 3.3k
  • 12
  • 26