Skip to main content
se añadieron 140 caracteres en el cuerpo
Origen Enlace
aeportugal
  • 10.6k
  • 2
  • 12
  • 24

Hay 2 problemas en está línea:

if i != 0 or i != 1:

(1) Estás comprando una cadena '1' o '0' contra un número 1 o 0. Así que ese if siempre dará True.

(2) En un or, se evalúa True inmediatamente donde se encuentre una condición verdadera, ignorando las demás, entonces:

Por ejemplo:

Para i = '1' --> if i != '0' ¡verdadero!, entrará al bloque
Para i = '0' --> if i != '1' ¡verdadero!, entrará al bloque
Para i = '3' --> if i != '0' ¡verdadero!, entrará al bloque

En cualquier caso entrará al if, resultando el mensaje que siempre ves.

Solución: Si bien, la lógica negativa suele ser elegante, algunas veces confunde a los más nuevos. Si lo cambias a lógica positiva se entiende más:

    # ...
    validacion = True # <-- Se cambió a True
    for i in numero_binario:
      if i == '0' or i == '1': # <-- Cambio
        validacion = False # <-- cambió a False
        break
      else:
        pass #None <-- Usar pass en vez de None si no se va hacer nada
    # ...

Otra solución es dejando todo el código tal cual, y solo cambiando la línea problemática por:

if i not in ['0', '1']:

Hay 2 problemas en está línea:

if i != 0 or i != 1:

(1) Estás comprando una cadena '1' o '0' contra un número 1 o 0. Así que ese if siempre dará True.

(2) En un or, se evalúa True inmediatamente donde se encuentre una condición verdadera, ignorando las demás, entonces:

Por ejemplo:

Para i = '1' --> if i != '0' ¡verdadero!, entrará al bloque
Para i = '0' --> if i != '1' ¡verdadero!, entrará al bloque
Para i = '3' --> if i != '0' ¡verdadero!, entrará al bloque

En cualquier caso entrará al if, resultando el mensaje que siempre ves.

Solución: Si bien, la lógica negativa suele ser elegante, algunas veces confunde a los más nuevos. Si lo cambias a lógica positiva se entiende más:

    # ...
    validacion = True # <-- Se cambió a True
    for i in numero_binario:
      if i == '0' or i == '1': <-- Cambio
        validacion = False <-- cambió a False
        break
      else:
        pass #None <-- Usar pass en vez de None si no se va hacer nada
    # ...

Hay 2 problemas en está línea:

if i != 0 or i != 1:

(1) Estás comprando una cadena '1' o '0' contra un número 1 o 0. Así que ese if siempre dará True.

(2) En un or, se evalúa True inmediatamente donde se encuentre una condición verdadera, ignorando las demás, entonces:

Por ejemplo:

Para i = '1' --> if i != '0' ¡verdadero!, entrará al bloque
Para i = '0' --> if i != '1' ¡verdadero!, entrará al bloque
Para i = '3' --> if i != '0' ¡verdadero!, entrará al bloque

En cualquier caso entrará al if, resultando el mensaje que siempre ves.

Solución: Si bien, la lógica negativa suele ser elegante, algunas veces confunde a los más nuevos. Si lo cambias a lógica positiva se entiende más:

    # ...
    validacion = True # <-- Se cambió a True
    for i in numero_binario:
      if i == '0' or i == '1': # <-- Cambio
        validacion = False # <-- cambió a False
        break
      else:
        pass #None <-- Usar pass en vez de None si no se va hacer nada
    # ...

Otra solución es dejando todo el código tal cual, y solo cambiando la línea problemática por:

if i not in ['0', '1']:
Origen Enlace
aeportugal
  • 10.6k
  • 2
  • 12
  • 24

Hay 2 problemas en está línea:

if i != 0 or i != 1:

(1) Estás comprando una cadena '1' o '0' contra un número 1 o 0. Así que ese if siempre dará True.

(2) En un or, se evalúa True inmediatamente donde se encuentre una condición verdadera, ignorando las demás, entonces:

Por ejemplo:

Para i = '1' --> if i != '0' ¡verdadero!, entrará al bloque
Para i = '0' --> if i != '1' ¡verdadero!, entrará al bloque
Para i = '3' --> if i != '0' ¡verdadero!, entrará al bloque

En cualquier caso entrará al if, resultando el mensaje que siempre ves.

Solución: Si bien, la lógica negativa suele ser elegante, algunas veces confunde a los más nuevos. Si lo cambias a lógica positiva se entiende más:

    # ...
    validacion = True # <-- Se cambió a True
    for i in numero_binario:
      if i == '0' or i == '1': <-- Cambio
        validacion = False <-- cambió a False
        break
      else:
        pass #None <-- Usar pass en vez de None si no se va hacer nada
    # ...