10

Trabajo con SharpDevelop.

A la hora de hacer esta comprobación:

string aux1 = "123";
string aux2 = "456";

bool iguales = false;

if(aux1.equals(aux2))
    iguales = true;

El compilador, me sugiere cambiarlo por:

string aux1 = "123";
string aux2 = "456";

bool iguales = false;

iguales |= aux1.equals(aux2);

Si se analiza este código en concreto, lo hace bien, pero lo hace bien porque el booleano es false, pero si fuese true no funcionaría esta equivalencia.

¿Esto quiere decir que el compilador "sabe" que el bool antes de la comparación es false y por eso hace la conversión, y si fuese true no lo haría?

¿No sería más sencillo asignar directamente el valor de esa comparación al bool como en las líneas siguientes?

string aux1 = "123";
string aux2 = "456";

bool iguales = false;

iguales = aux1.equals(aux2);
8
  • ¿Has probado a poner iguales a true a ver si sugiere lo mismo?
    – Alejandro
    Commented el 19 feb. 2018 a las 12:54
  • Si pongo igual a true no sugiere ese cambio, por eso pongo "¿Esto quiere decir que el compilador 'sabe' que el bool antes de la comparación es false y por eso hace la conversión y si fuese true no lo haría?" Commented el 19 feb. 2018 a las 12:56
  • No habías especificado que lo hubieses probado. Entonces es que sí, el compilador entiende que al ser falsepuede usar el operador OR. Será una manera de reducir la cantidad de líneas de código. Al igual que podría sugerir que asignes directamente la variable iguales al resultado de la comparación. Dependerá del editor.
    – Alejandro
    Commented el 19 feb. 2018 a las 13:22
  • 1
    Hola. Te recomiendo una prueba. Usa tu código, compila y abre el ensamblado resultante con un disassembler como puede ser ildasm Luego, compila el código con la modificación que te propone y examina el resultado. Compara ambos códigos IL y seguro verás diferencia. Commented el 19 feb. 2018 a las 22:24
  • 3
    Buenas, creo que se debe a que un if va a realizar primero una comparación (testz probablemente) y posteriormente un salto en función de ese resultado. La instrucción por la que ha sido sustituida es un or a nivel de bit sobre un resultado, que va a funcionar (para este caso en el que vale false) exactamente igual que un or lógico, y por lo tanto se evita operaciones en código máquina. En cualquier caso, para confirmarlo, habría que seguir el consejo de @SergioParraGuerra
    – Gatchan00
    Commented el 9 mar. 2018 a las 12:54

2 respuestas 2

1

Totalmente de acuerdo contigo Aunque para este caso puedes hacer la asingación como te la voy a dejar a continuación.

string aux1 = "123";
string aux2 = "456";

bool iguales = aux1.equals(aux2);
0

El compilador te va a sugerir evitar anidaciones utilizando operadores logicos traducibles a operaciones lógicas más sencillas.

En tu caso dependerá del momento en el que hagas la asignación del valor, si esta encapsulado dentro de un método de una clase, etc. y en el punto posterior que hagas el cambio puede ser equivalente o no.

Referencia: Operador OR

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.