No sé si la pregunta en sí es correcta, pero aquí hay una explicación.
Por ejemplo, tengo la función load_window() y toneladas de eventos de clic, cambio (etc.).
¿Cómo crear correctamente la variable custom_data en eventos y usarla en la función load_window() ? Básicamente, después de un evento exitoso, quiero almacenar esa variable en función y cargar ajax con esa variable de datos personalizada.
¿Estoy cerca de este, o necesito reescribirlo todo?
function load_window() { $.ajax({ type: "POST", url: url+'/window', dataType: "html", data: {custom_data}, success: function(data) { $('#window').html(data); return data; } }); } $('body').on('click', '#div', function(event) { event.stopPropagation(); var this_id = $(this).data('id'); var custom_data = "'id':"+this_id; load_window(); }); $('body').on('change', '#div > .div > select', function(event) { event.stopPropagation(); var custom_id = $(this).data('id'); var custom_data = "'id':"+custom_id; load_window(); }); // etcLo siento por mal inglés y gracias por cualquier respuesta.
Es solo mi opinión, pero yo haría algo como esto:
function load_window(id) { $.ajax({ type: "POST", url: url+'/window', dataType: "html", data: {id:id}, success: function(data) { $('#window').html(data); return data; } }); } $('select').on('change', '#div > .div > select', function(event) { event.stopPropagation(); var custom_id = $(this).data('id'); load_window(custom_id); });esto reducirá ligeramente la cantidad de líneas de código sin afectar la legibilidad, pero esta es solo mi opinión
Estás cerca. Probablemente quieras pasar las variables a la función.
function load_window(custom_id, custom_data) { $.ajax({ type: "POST", url: url+'/window', dataType: "html", data: {custom_data}, success: function(data) { $('#window').html(data); return data; } }); }Entonces cuando lo llamas
load_window(custom_id, custom_data);Podrías resolver el problema de dos maneras. En primer lugar, podría pasar los datos a su función load_window() .
Tenga en cuenta que la forma que está utilizando actualmente para crear el objeto no es sintácticamente correcta y dará un resultado diferente al que espera. Debe proporcionar la clave y el valor por separado, no como una sola cadena proporcionada al objeto. También tenga en cuenta que no puede return nada desde un controlador asíncrono. Si necesita usar los data devueltos, hágalo en la devolución de llamada. Prueba esto:
function load_window(requestData) { $.ajax({ type: "POST", url: url + '/window', dataType: "html", data: requestData, success: function(data) { $('#window').html(data); // do something with data here, don't 'return' it. } }); } $('body').on('click', '#div', function(e) { e.stopPropagation(); load_window({ id: $(this).data('id') }); }); $('body').on('change', '#div > .div > select', function(e) { e.stopPropagation(); load_window({ id: $(this).data('id') }); }); Alternativamente, puede proporcionar la referencia de la función load_window() a los controladores de eventos y obtener los data('id') usando this referencia dentro de la función:
function load_window() { $.ajax({ type: "POST", url: url + '/window', dataType: "html", data: { id: $(this).data('id') }, success: function(data) { $('#window').html(data); // do something with data here, don't 'return' it. } }); } $('body') .on('click', '#div', load_window) .on('change', '#div > .div > select', load_window);