Unas cuantas observaciones:
La idea es iterar sobre cada carácter y comprobar si cumple una de las condiciones. Ahora iteras tantas veces como caracteres tiene la variable clave, pero no iteras sobre los caracteres.
Si iteras con:
for i in range(len(clave))
debes obtener cada carácter indizando:
caracter = clave[i]
de todas formas, en Python itera siempre directamente sobre el iterable con un for in
si no necesitas modificar sus items, es más eficiente y más legible.
for caracter in clave:
En cada iteración debes comprobar si el carácter es una @ o (or
) es una letra mayúscula, no si son ambas cosas (and
).
Para comprobar si un carácter es una letra mayúscula o no puedes usar el método str.isupper()
Por eficiencia, combine que rompas el ciclo en cuanto encuentres una @ o una mayúscula. Para ello puedes usar break
.
clave = input("Por favor, introduce la clave: ")
caracter_presente = False
for caracter in clave:
if caracter == "@" or caracter.isupper():
caracter_presente = True
break
if caracter_presente:
print("La clave es válida")
else:
print("La clave es incorrecta")
Realmente no necesitas la variable boleana, puedes valerte de la cláusula else
junto a un for
:
clave = input("Por favor, introduce la clave: ")
for caracter in clave:
if caracter == "@" or caracter.isupper():
print("La clave es válida")
break
else:
print("La clave es incorrecta")
El contenido del else
solo se ejecuta si el for
iteró al completo sobre el iterable, es decir, si no se ejecutó el break
.
Especificas que la clave debe tener "una "@" o una letra mayúscula". Si la condición fuera "una "@" y una letra mayúscula", entonces si necesitarás tener dos variables boleanas, una implementación con una sola iteración y también con cortocircuito puede ser:
clave = input("Por favor, introduce la clave: ")
arroba_presente = False
mayuscula_presente = False
for caracter in clave:
if caracter == "@":
arroba_presente = True
if mayuscula_presente:
print("La clave es válida")
break
elif caracter.isupper():
mayuscula_presente = True
if arroba_presente:
print("La clave es válida")
break
else:
print("La clave es incorrecta")