Das deutsche Python-Forum

EyDu hat geschrieben:[…], aber warum machst du eine Unterscheidung zwischen Groß- und Kleinbuchstaben. Das kann man alles in einem Rutsch erledigen, auch wenn das Ergebnis dann eventuell ein wenig anders ist. […]

Möchtest du deine Codierung beibehalten, dann solltest du doppelten Code in Funktionen auslagen.

Und die ganzen "j" wirst du durch die enumerate-Funktion los.

Und natürlich möchtest du nicht 26 als magische Zahl benutzen, sonder die Länge der Strings.

The lowercase letters 'abcdefghijklmnopqrstuvwxyz'. This value is not locale-dependent and will not change.

Außerdem hätte ich noch etwas zum Nachdenken:

Code: Alles auswählen

>>> alpha = string.ascii_lowercase
>>> offset = 3
>>> dict(zip(alpha, alpha[offset:]+alpha[:offset]))
{'a': 'd', 'c': 'f', 'b': 'e', 'e': 'h', 'd': 'g', 'g': 'j', 'f': 'i', 'i': 'l', 'h': 'k', 'k': 'n', 'j': 'm', 'm': 'p', 'l': 'o', 'o': 'r', 'n': 'q', 'q': 't', 'p': 's', 's': 'v', 'r': 'u', 'u': 'x', 't': 'w', 'w': 'z', 'v': 'y', 'y': 'b', 'x': 'a', 'z': 'c'}
>>> 

Im string-Modul gibt es übrigens auch eine translate-Funktion.

Sebastian

derdon hat geschrieben:Am einfachsten bleibt aber immer noch str.encode

Code: Alles auswählen

>>> import string
>>> string.printable
'0123456789abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ!"#$%&\'()*+,-./:;<=>?@[\\]^_`{|}~ \t\n\r\x0b\x0c'
>>> string.printable.encode('rot13')
'0123456789nopqrstuvwxyzabcdefghijklmNOPQRSTUVWXYZABCDEFGHIJKLM!"#$%&\'()*+,-./:;<=>?@[\\]^_`{|}~ \t\n\r\x0b\x0c'

derdon hat geschrieben:

Code: Alles auswählen

>>> import string
>>> string.printable
'0123456789abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ!"#$%&\'()*+,-./:;<=>?@[\\]^_`{|}~ \t\n\r\x0b\x0c'
>>> string.printable.encode('rot13')
'0123456789nopqrstuvwxyzabcdefghijklmNOPQRSTUVWXYZABCDEFGHIJKLM!"#$%&\'()*+,-./:;<=>?@[\\]^_`{|}~ \t\n\r\x0b\x0c'

/me hat geschrieben:Hat schon mal jemand ROT-26 implementiert?

/me hat geschrieben:

derdon hat geschrieben:

Code: Alles auswählen

>>> import string
>>> string.printable
'0123456789abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ!"#$%&\'()*+,-./:;<=>?@[\\]^_`{|}~ \t\n\r\x0b\x0c'
>>> string.printable.encode('rot13')
'0123456789nopqrstuvwxyzabcdefghijklmNOPQRSTUVWXYZABCDEFGHIJKLM!"#$%&\'()*+,-./:;<=>?@[\\]^_`{|}~ \t\n\r\x0b\x0c'

ROT-13 ist mir eigentlich zu unsicher. Ich hatte eher an die doppelte Verschlüsselungsstärke gedacht. Hat schon mal jemand ROT-26 implementiert?

def encrypt(plain_text):
    cipher = plain_text
    return cipher

def decrypt(cipher):
    plain_text = cipher
    return plain_text

text = raw_input('Type in your text! ')
while text != '':
    text += raw_input('> ')
print "Encrypted:", encrypt(text)
print "Decrypted:", decrypt(text)

EyDu hat geschrieben: Dabei ändert sich dann natürlich die Codierung, da, bei einer Verschiebung von 1, das "z" nicht mehr auf das "a" fällt, sondern auf das "A".

Das du "len" statt einer magischen Zahlen benutzen solltest hat ganz einfache Gründe: Die 26 sagt erstmal gar nichts aus und bei etwas komplexeren Programmen wirst du in zwei Wochen nicht mehr wissen, für was die Zahl steht […]

Der nächste Schritt bei dir ist, von der for-Schleife weg zu kommen und ein Dictionary einzusetzen (oder zunächst einmal die find-Methode auf Strings).

Dann solltest du dir noch die get-Methode auf Dictionaries anschauen, mit etwas Überlegung kannst du dir auch den Sonderfall für nicht zu kodierende Zeichen einsparen.

Und zum translate:

Code: Alles auswählen

>>> translation = string.maketrans("abcdefghij", "0123456789")
>>> string.translate("Spam and Eggs", translation)
'Sp0m 0n3 E66s'

EyDu hat geschrieben:Hat es einen Grund, dass du zwei Dictionaries verwendest?

EyDu hat geschrieben:

Code: Alles auswählen

translation = dict(zip(l+u, l[shift:]+l[:shift]+u[shift:]+u[:shift]))

EyDu hat geschrieben:Und jetzt erinnerst du dich noch einmal an "dict.get" ;-)

Code: Alles auswählen

    l = string.ascii_lowercase
    u = string.ascii_uppercase

    cipher = []
    translation = dict(zip(l+u, l[shift:]+l[:shift]+u[shift:]+u[:shift]))
    for char in plain:
        cipher.append(translation.get(char, char)

return "".join(cipher)[/code]

Wo bei du die Schleife gleich als List Comprehension schreiben könntest:

Code: Alles auswählen

cipher = [translation.get(char, char) for char in plain]

Oder mit map:

Code: Alles auswählen

cipher = map(lambda x: translation.get(x, x), plain)

Oder gleich zum return:

Code: Alles auswählen

return "".join(translation.get(char, char) for char in plain)