Tu función contiene varios errores, uno de ellos especialmente grave:
Internamente llama a validate_pin()
, que es ella misma, por lo que lo primero que hará será llamarse (y en esa llamada lo primero que hará será llamarse, etc...) sin llegar a retornar nunca. Estás creando un bucle de recursión infinita que acaba por llenar la pila de Python y causando el error que has mostrado.
Y es que no necesitas para nada llamar a la propia función. Simplemente quieres saber si la cadena pin
es o no es una secuencia de dígitos (y no si el resultado de llamar a validate_pin()
es o no es una cadena)
Para mirar si la cadena pin
contiene sólo dígitos intentas mirar si su tipo es int
. Lo haces mal porque de nuevo llamas recursivamente a validate_pin()
, cuando lo que deberías hacer para ver si el tipo es int
sería if type(pin)==int
. Pero es que eso tampoco sería correcto porque no es lo mismo mirar si un dato es un entero, que mirar si una cadena contiene sólo dígitos. En tu caso pin
va a ser siempre una cadena, nunca un int
, pero puedes mirar si contiene solo dígitos usando pin.isdigit()
De la misma forma, mirar si es de tipo str
no equivale a mirar "si tiene letras". Pero esto se resuelve por si solo usando pin.isdigit()
ya que si esta función te da False
es que no son sólo dígitos lo que hay en el pin, y por tanto que habrá letras u otros símbolos.
Por último las funciones no deberían imprimir resultados, sino retornarlos, ya que esto las hace más útiles. El programa que llama a la función recibiría el valor retornado que puede usar para imprimirlo, o para otras cosas. Luego daré un ejemplo.
La función que buscas sería por tanto así:
def validate_pin(pin):
if pin.isdigit():
if len(pin) == 4 or len(pin)==6:
return True
return False
Es decir, primero miras si la cadena está compuesta sólo por dígitos y en ese caso que la longitud sea 4 ó 6. Sólo si todo ello se cumple retornas True
. Si no, retornas False
. En vez de dos if
anidados podría haber usado uno solo con una condición que utilice and
.
Ahora puedes usarlo así para imprimir el resultado:
print(validate_pin(input("introduce pin")))
pero el h echo de que la función te lo haya retornado la hace más útil, pues también puedes hacer algo como lo siguiente:
ok = False
while not ok:
pin = input("Introduce pin")
if validate_pin(pin):
print("OK, pin válido")
ok = True
else:
print("El pin no es válido, introdúzcalo de nuevo")
validate_print()
dentro devalidate_print()
. Esto es lo que se llama recursividad, y aunque puede usarse para resolver ciertos tipos de problemas, en tu caso es simplemente un error, y causa que la función se llame a sí misma en bucle infinito hasta agotar el tamaño de la pila de Python. Supongo que querías hacerif type(pin)==str
yif type(pin)==int
. Eso evitará la recursión infinita.pin
es de tipoint
, no tiene sentido mirar luego su ´len()` ya que esa función sólo existe para cadenas. Supongo que en realidad no quieres mirar si esstr
o si esint
, sino mirar si contiene sólo dígitos o contiene algo que no sean dígitos. En ambos casos sería sin embargo una cadena. Puedes usarpin.isdigit()
para ver si tiene sólo dígitos.