¿Por qué esta declaración IF devuelve falso?

Question

Tengo dos cuadros de texto y quiero saltar un bloque de código única cuando ambos están vacíos:

if (txtBox1.Text.Trim() != string.Empty && txtBox2.Text.Trim() != string.Empty) 
{ 
    // Do something 
} 

Si cualquiera de los cuadros de texto tiene algo, quiero que el // Hacer algo parte de ejecutar. Solo cuando ambos están vacíos, quiero omitir.

Sin embargo, el fragmento de código anterior no funciona. ¿Por qué?

Answer 1

Creo que tiene sus operadores lógicos mezclados. Lo que usted está buscando es

if (txtBox1.Text.Trim() != string.Empty || txtBox2.Text.Trim() != string.Empty) 
{ 
    // Do something 
} 

Answer 2

Debe reemplazar su && con ||. Actualmente, el código en el bloque if solo se ejecutará si ambos campos de texto tienen texto.

Answer 3

Quiere || en lugar de & &.

Answer 4

if (txtBox1.Text.Trim() != string.Empty || txtBox2.Text.Trim() != string.Empty) 
{ 
    // Do something 
} 

está usando el && que requieren tanto de ellos sea no está vacío. Debe usar || (O) para obtener el resultado deseado.

Answer 5

podría ser más fácil de entender:

if (!(txtBox1.Text.Trim() == string.Empty && txtBox2.Text.Trim() == string.Empty)) 
{ 
    // Do something 
} 

Answer 6

Los que señaló que necesita || en lugar de & & tienen razón. Si prefiere & &, también se puede utilizar:

if (!(txtBox1.Text.Trim() == string.Empty && txtBox2.Text.Trim() == string.Empty)) 
{ 
    // Do something 
} 

La diferencia es puramente estético, sino que equivale a comprobar (en Inglés): "Es no es cierto que los dos cuadros de texto están vacíos" en lugar de "Es cierto que al menos un cuadro de texto no está vacío". Mismo significado, diferente forma de expresarlo.

Para el registro, la forma en que lo tenía en su pregunta original: "Es cierto que los dos cuadros de texto no están vacíos."

Answer 7

he dividido el BothBoxesEmpty a cabo a una variable independiente para que sea más fácil de leer ... todos los soportes de convertirse en un lío ilegible lo contrario. Puede combinar las dos afirmaciones si lo desea:

bool BothBoxesEmpty = string.IsNullOrEmpty(TextBox1.Text.Trim()) && 
         string.IsNullOrEmpty(TextBox2.Text.Trim()); 
if (!BothBoxesEmpty) 
{ 
    /* Do your stuff */ 
} 

Usted puede igualmente reemplazar fácilmente el String.IsNullOrEmpty con TextBox1.Text.Trim() == string.Empty como TextBox1.Text siempre devolverá una cadena (vacío o no) y nunca volverá null.

Answer 8

Si estás en un método que podría hacer algo como esto, así:

if (txtBox1.Text.Trim() == string.Empty && txtBox2.Text.Trim() == string.Empty) 
     return; 
//Do Something 

Answer 9

Usted puede hacer que sea más concisa mediante la concatenación de las cadenas, luego las pruebas:

if ((txtBox1.Text.Trim() + txtBox2.Text.Trim()) != string.Empty) 
{ 
    // Do something 
} 

Dependiendo Sin embargo, lo que las cajas representan esto podría ser algo menos obvio.

Answer 10

Esto me recuerda una historia que escuché sobre un codificador junior al que se le dijo que implementara reglas para calcular los beneficios del seguro, donde "todos menores de 18 y mayores de 65" no calificaron. Así que se codificó como:

if (age < 18 and age > 65) ... 

La historia cuenta que sus compañeros de trabajo trataron repetidamente de convencerlo de que su lógica no funcionaría correctamente, pero él se negó obstinadamente a entender o cambiarlo.