Buenos días!
Quería preguntarles o mejor dicho quería saber cual es su opinión sobre este pequeño código en javascript/jquery.

Antes de mostrarles el código voy a explicar para que sirve, digamos que tengo el tradicional botón de "Cargar más resultados" al momento de presionarlo se me van a mostrar lógicamente más datos (Ajax), entre el momento que doy el click hasta el momento en que se completa y recibo los resultados dependiendo que tan lenta sea la web (que tan optimizada, que tantos usuarios estén navegando en ella de forma simultánea, etcétera) va a tardar más o menos tiempo. Por tanto lo que hago en ese "tiempo de espera" es ejecutar un acción, o mostrar algo, un gif, pre-loader como quieran llamarlo.

con Ajax ya lo hice hace tiempo y siempre use success pero aveces cuando muestro los otros datos estos no cargan inmediatamente por más que se haya ejecutado lo que yo tenga en el success, así que hice esto:

                document.body.style.cursor = 'progress';
                document.getElementById('cargando').style.display = 'block';
		document.getElementById('contenido').style.display = 'none';
 
                $.ajax({
			url: 'ajax/fichero_ajax.php',
			type: 'POST',
			dataType: 'html',
			data: {parametro},
 
			success: function()
			{
				$(window).ready(function()
				{
					document.body.style.cursor = 'default';
					document.getElementById('cargando').style.display = 'none';
					document.getElementById('contenido').style.display = 'block';
				});
			}
		})
 

Antes estaba sólo "success: function()" pero ahora agregue una linea más de código "$(window).ready(function()", Quería saber sus opiniones sobre esto.

Gracias!

Pues no se para que agregaste ese listener. Lo único que se me ocurre que estás haciendo es una petición AJAX inmediatamente antes de cargar el documento y quieres estar seguro que el código se ejecute antes que se cargue el documento.

Si esa petición se hace después de cargar el documento, ese listener no sirve para nada.

Edit: Oh y ahora que me doy cuenta si es totalmente redundante. Porque si haces esto:

document.body.style.cursor = 'progress';
document.getElementById('cargando').style.display = 'block';
document.getElementById('contenido').style.display = 'none';

Y esto funciona en ese punto, significa que los elementos están cargados en el DOM y no hay necesidad del listener.

Hasta donde yo tengo entendido el evento se dispara una sola vez, en la carga inicial de la pagina. En el código que estás mostrando, los elementos ya existen y están cargados. En ese código no estás cargando ningún elemento al DOM. Y si estuvieras cargando los elementos al DOM con jQuery o innerHTML o similar, esas APIs no son asincronas. Lo único que pudiera necesitar un evento es si tu codigo HTML carga recursos de otras partes. Por ejemplo, tu código HTML puede incluir IMG tags que cargan imagenes de algún lado. Tu puedes insertar las etiquetas IMG en el DOM inmediatamente, pero el navegador puede o no estar cargando las imagenes todavía. Si quieres mostrar la imagen cargada vas a tener que escuchar al evento load de las imagenes. Pero eso ya es otro tema.