¿Es una mala práctica de Javascript lo que estoy haciendo aquí?

Question

por alguna razón hago eso cada vez porque lo encuentro limpio. Declaro variables en la parte superior para usarlas a continuación. Lo hago incluso si los uso solo una vez.

Aquí es un ejemplo (usando jQuery marco):

$("#tbListing").delegate("a.btnEdit", "click", function(e) { 
    var storeId = $(this).closest("tr").attr("id").replace("store-", ""), 
     storeName = $(this).closest("tr").find("td:eq(1)").html(), 
     $currentRow = $(this).closest("tr"); 

    $currentRow.addClass("highlight"); 

    $("#dialogStore") 
     .data("mode", "edit") 
     .data("storeId", storeId) 
     .data("storeName", storeName) 
     .dialog("open"); 

    e.preventDefault(); 
}); 

que tienden a hacer que en PHP también. ¿Tengo razón si creo que no es muy eficiente en la memoria para hacer eso?

Edit: Gracias por todas las respuestas. Todos ustedes han dado buenas respuestas. Acerca de esa optimización de código ahora. ¿Eso es mejor ahora?

  $("#tbListing").delegate("a.btnEdit", "click", function(e) { 
       var $currentRow = $(this).closest("tr"), 
        storeId = this.rel, /*storing the storeId in the edit button's rel attribute now*/ 
        storeName = $currentRow.find("td:eq(1)").html(); 

       $currentRow.addClass("highlight"); 

       $("#dialogStore") 
        .data("info", { 
         "mode" : "edit", 
         "storeId" : storeId, 
         "storeName" : storeName 
        }) /*can anyone confirm that overusing the data isn't very efficient*/ 
        .dialog("open"); 


       e.preventDefault(); 
      }); 

Answer 1

Lo sentimos, ¿está preguntando si está bien para declarar las variables incluso si las está utilizando una vez?

¡Absolutamente! Hace que el código sea un millón de veces más legible si nombra las cosas correctamente con una variable. La legibilidad debería ser su principal preocupación. La eficacia de la memoria solo debería ser una preocupación si resulta problemático.

Como dijo Knuth,

Debemos olvidarnos de pequeñas eficiencias, dicen que alrededor del 97% del tiempo: la optimización prematura es la raíz de todo mal.

Si preguntas más sobre la declaración de las variables al comienzo de la función, en lugar de en los que se utilizan en primer lugar, a continuación, Emmett has it right - Crockford recomienda hacer esto en JavaScript para evitar la confusión relacionados con el ámbito. Si vale la pena en PHP es una pregunta puramente subjetiva, diría, pero no hay nada de malo en mantener sus estilos de codificación PHP y JS similares.

---

Una cita más CS (de Abelson y Sussman SICP de):

programas deben ser escritos para que la gente lea, y sólo incidentalmente para máquinas para su ejecución.

Answer 2

No es una mala práctica.

Los enunciados var deben ser los primeros enunciados en el cuerpo de la función.

---

JavaScript no tiene ámbito de bloque, por lo que definir variables en bloques pueden confundir a los programadores que experimentaron con otros familiares lenguajes C. Defina todas las variables en la parte superior de la función .

http://javascript.crockford.com/code.html

Answer 3

Declaración de variables en la parte superior es una buena cosa que hacer. Hace que el código sea más legible. En su ejemplo particular, podría reemplazar $ (this) .closest ('tr') con una variable, como se sugiere en los comentarios, pero en general encuentro el código con nombres de variables descriptivos, todo en un solo lugar, muy legible.

Answer 4

nah, yo diría que estás haciendo exactamente lo correcto.

Como dice @Caspar, podría simplificar su código configurando primero $currentRow y usando eso en lugar de $(this).closest("tr") en las otras dos líneas. Y puede haber algunas otras cosas que podrías mejorar. Pero establecer vars al comienzo de una función de la manera que lo has hecho es absolutamente una buena cosa.

Especialmente bueno porque lo has hecho dentro de la función, por lo que son variables locales, lo que significa que se descartan al final de la función, por lo que no hay problemas de uso de memoria.

Si los hubiera establecido como variables globales, podría haber sido un problema mayor, aunque para ser honesto incluso entonces, ya que solo está configurando punteros a un objeto existente, no usaría un gran cantidad de memoria incluso entonces (aunque estaría contaminando el espacio de nombres global, que no es bueno)