Asignar variable en la declaración de condición de condición, ¿buenas prácticas o no?

Question

Me mudé hace un año de los lenguajes OO clásicos, como Java a JavaScript. es, sin duda no se recomienda el siguiente código (o incluso no corrige) en Java:

if(dayNumber = getClickedDayNumber(dayInfo)) 
{ 
    alert("day number found : " + dayNumber); 
} 
function getClickedDayNumber(dayInfo) 
{ 
    dayNumber = dayInfo.indexOf("fc-day"); 
    if(dayNumber != -1) //substring found 
    { 
     //normally any calendar month consists of "40" days, so this will definitely pick up its day number. 
     return parseInt(dayInfo.substring(dayNumber+6, dayNumber+8)); 
    } 
    else return false; 
} 

Básicamente, sólo descubrí que puedo asignar una variable a un valor en un estado de condición si, y comprobar inmediatamente el valor asignado como si es booleano

Para una apuesta más segura, generalmente lo separo en dos líneas de código, primero asigno y luego verifico la variable, pero ahora que encontré esto, me pregunto si es una buena práctica o no a los ojos de un JavaScript experimentado. desarrolladores?

Answer 1

No lo recomendaría. El problema es que parece un error común cuando intenta comparar valores, pero usa un solo = en lugar de == o ===. Por ejemplo, cuando vea esto:

if (value = someFunction()) { 
    ... 
} 

usted no sabe si eso es lo que querían hacer, o si se trataba de escribir esto:

if (value == someFunction()) { 
    ... 
} 

Si realmente quiere hacer la asignación en el lugar, yo recomendaría hacer una comparación explícita también:

if ((value = someFunction()) === <whatever truthy value you are expecting>) { 
    ... 
} 

Answer 2

Puede hacerlo en Java también. Y no, no es una buena práctica. :)

(Y el uso de la === en Javascript por la igualdad escrito leído el libro Buenas piezas de Crockford en JS..)

Answer 3

Me parece que es más bien un estilo de la vieja escuela C; en realidad no es una buena práctica en JavaScript, así que debes evitarla.

Answer 4

Lo hice muchas veces. Para pasar por alto la advertencia de JavaScript, añado dos parens:

if ((result = get_something())) { } 

Debe evitar que, si realmente desea utilizarlo, escribir un comentario sobre ella diciendo lo que está haciendo.

Answer 5

No es una buena práctica. Pronto te confundirás al respecto. Se parece a un error común: uso indebido de operadores "=" y "==".

Debe dividirlo en 2 líneas de códigos. No solo ayuda a que el código sea más claro, sino que también es fácil de refactorizar en el futuro. ¿Imagine que cambia la condición IF? Puede eliminar accidentalmente la línea y su variable ya no obtendrá el valor asignado.

Answer 6

También puede realizar asignaciones en sentencias if en Java. Un buen ejemplo podría ser la lectura y la escritura algo en usted mismo:

http://www.exampledepot.com/egs/java.io/CopyFile.html?l=new

El código:

// Copies src file to dst file. 
// If the dst file does not exist, it is created 
void copy(File src, File dst) throws IOException 
{ 
    InputStream in = new FileInputStream(src); 
    OutputStream out = new FileOutputStream(dst); 

    // Transfer bytes from in to out 
    byte[] buf = new byte[1024]; 
    int len; 
    while ((len = in.read(buf)) > 0) { 
     out.write(buf, 0, len); 
    } 
    in.close(); 
    out.close(); 
} 

Answer 7

no veo ninguna prueba de que no es una buena práctica. Sí, puede parecer un error, pero eso se remedia fácilmente mediante comentarios juiciosos.Tomemos, por ejemplo:

if (x = processorIntensiveFunction()) { // declaration inside if intended 
    alert(x); 
} 

¿Por qué se debe permitir que la función para ejecutar una segunda vez con:

alert(processorIntensiveFunction()); 

Debido a que la primera versión se ve mal? No puedo estar de acuerdo con esa lógica.

Answer 8

¡Si tuviera que consultar el libro de Martin Fowlers Refactoring improving the design of existing code! Luego hay varios casos en que sería una buena práctica, por ejemplo. condicionales complejas largos para utilizar una llamada de función o método para hacer valer su caso:

"Motivation

One of the most common areas of complexity in a program lies in complex conditional logic. As you write code to test conditions and to do various things depending on various conditions, you quickly end up with a pretty long method. Length of a method is in itself a factor that makes it harder to read, but conditions increase the difficulty. The problem usually lies in the fact that the code, both in the condition checks and in the actions, tells you what happens but can easily obscure why it happens.

As with any large block of code, you can make your intention clearer by decomposing it and replacing chunks of code with a method call named after the intention of that block of code. > With conditions you can receive further benefit by doing this for the conditional part and each of the alternatives. This way you highlight the condition and make it clearly what you > are branching on. You also highlight the reason for the branching."

Y sí su respuesta también es válido para las implementaciones de Java. Sin embargo, no asigna la función condicional a una variable en los ejemplos.

Answer 9

Hay un caso cuando lo haces, con while bucles.
Al leer archivos, usualy hace así:

void readFile(String pathToFile) { 
    // Create a FileInputStream object 
    FileInputStream fileIn = null; 
    try { 
     // Create the FileInputStream 
     fileIn = new FileInputStream(pathToFile); 
     // Create a variable to store the current line's text in 
     String currentLine; 
     // While the file has lines left, read the next line, 
     // store it in the variable and do whatever is in the loop 
     while((currentLine = in.readLine()) != null) { 
      // Print out the current line in the console 
      // (you can do whatever you want with the line. this is just an example) 
      System.out.println(currentLine); 
     } 
    } catch(IOException e) { 
     // Handle exception 
    } finally { 
     try { 
      // Close the FileInputStream 
      fileIn.close(); 
     } catch(IOException e) { 
      // Handle exception 
     } 
    } 
} 

Mira el -loop while en la línea 9. Allí, una nueva línea se lee y se almacena en una variable y, a continuación, el contenido del bucle es corrió. Sé que esta no es una declaración if, pero supongo que también se puede incluir un ciclo while en su pregunta.

La razón para esto es que cuando se utiliza un FileInputStream, cada vez que llame FileInputStream.readLine(), se lee la siguiente línea en el archivo, por lo que si lo hubiera llamado desde el bucle con sólo fileIn.readLine() != null sin asignar la variable, en lugar de llamando al (currentLine = fileIn.readLine()) != null, y luego lo llamó desde el interior del bucle también, solo obtendría cada segunda línea.

Espero que lo entiendas, y buena suerte!

Answer 10

se podría hacer algo así:

if (value = /* sic */ some_function()){ 
    use_value(value) 
}