Caesar Shift Decoder in python

Question

I can't figure out what I'm doing wrong here. The shift works flawlessly excpet for it skips the first letter in the cipher code. What gives? It should translate to WELCOMETOCRYPTOGRAPHY but does ELCOME....

    cipher = 'ZGXJHZOJXMTKOJBMVKCT'
    letters = 'ABCDEFGHIJKLMNOPQRSTUVWXYZ'

    plain = ""
    for x in cipher:
            if x in letters:
                    pos = (letters.find(x) - 21) % 26
            plain += letters[pos]
    print ("Key #%s: %s") % (c, plain)

Are you sure your cipher is right? Because `WELCOMETOCRYPTOGRAPHY` has 21 characters, and `ZGXJHZOJXMTKOJBMVKCT` only has 20. — Kevin, Sep 24 '18 at 17:23
Your code is correct. Your input is wrong, you wanted to input `'RZGXJHZOJXMTKOJBMVKCT'` — Olivier Melançon, Sep 24 '18 at 17:23

score -1 · Answer 1 · edited Sep 24 '18 at 17:41

-1

Your code is fine. Your cipher itself is missing the first letter. The cipher should look like RZGXJHZOJXMTKOJBMVKCT not ZGXJHZOJXMTKOJBMVKCT

edited Sep 24 '18 at 17:41

NaruS

168
1
15

answered Sep 24 '18 at 17:26

Gunnar

19
6

Caesar Shift Decoder in python

1 Answers1