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']: