El problema es que vas haciendo cada replace "por turnos". Entonces primero reemplazas el 0 por 3. Pero más adelante reemplazas los 3 por 0, por lo que no obtienes el resultado buscado. De hecho, eso mismo debería pasarte conel cifrado de las letras tambiéntampoco funcionará correctamente, ya que por ejemplo primero cambias la "a" por "m", pero más adelante cambias la "m" por "j", por lo que al final las "aes" quedan como "jotas" en vez de "emes".
Lo que necesitas es una forma de hacer todos los cambios "a la vez" en un solo replace por así decir. Esta funcionalidad te la da str.translate()
. Para usarlo tienes que crear antes una "tabla de traducción" mediante str.maketrans()
.
Usando estas cosas, tu código se reduce a un par de líneas:
def encriptar(txt):
cifra = str.maketrans("abcdefghijklmnopqrstuvwxyz0123456789",
"mkwayxhpqfsdjtvrfuoiegcnlb3790285146")
return txt.translate(cifra)
Y puedes verificar que funciona:
encriptar("1234567890")
Sale:
7902851463
Bonus
La función desencriptar es análoga, intercambiando las cadenas usadas para crear la tabla de traducción.
def desencriptar(txt):
cifra = str.maketrans("mkwayxhpqfsdjtvrfuoiegcnlb3790285146",
"abcdefghijklmnopqrstuvwxyz0123456789")
return txt.translate(cifra)