Skip to main content
se añadieron 169 caracteres en el cuerpo
Origen Enlace
abulafia
  • 58.7k
  • 3
  • 54
  • 89

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)

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 con las letras también.

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)

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, el cifrado de las letras tampoco 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)
se añadieron 318 caracteres en el cuerpo
Origen Enlace
abulafia
  • 58.7k
  • 3
  • 54
  • 89

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 con las letras también.

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)

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 con las letras también.

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

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 con las letras también.

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)
Origen Enlace
abulafia
  • 58.7k
  • 3
  • 54
  • 89

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 con las letras también.

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