2015-11-03 86 views
-1

我在加密大写字母时遇到了问题,例如如果消息是COMPUTING IS FUN,关键字是GCSE,我应该得到JRFUBWBSN LL KBQ,但是我的实际结果是xftipkpgb zz ype。这个结果既不是正确的字母也不是资本。任何帮助表示赞赏加密大写字母python vigenere

    message = input('\nenter message: ') 
        keyword = input('enter keyword: ') 
        def chr_to_inta(char): 
         return 0 if char == 'Z' else ord(char)-64 
        def int_to_chra(integer): 
         return 'Z' if integer == 0 else chr(integer+64) 
        def add_charsa(msg, key): 
         return int_to_chr((chr_to_int(msg) + chr_to_int(key)) % 26) 


        def chr_to_int(char): 
         return 0 if char == 'z' else ord(char)-96 
        def int_to_chr(integer): 
         return 'z' if integer == 0 else chr(integer+96) 
        def add_chars(msg, key): 
         return int_to_chr((chr_to_int(msg) + chr_to_int(key)) % 26) 

        def vigenere(message, keyword): 

         keystream = cycle(keyword) 
         new = '' 
         for msg in message: 
          if msg == ' ': # adds a space 
           new += ' ' 
          elif 96 < ord(msg) < 123: # if lowercase 
           new += add_chars(msg, next(keystream)) 

          else: # if uppercase 
           new += add_charsa(msg, next(keystream)) 

         return new 

        new = vigenere(message, keyword) 
        print('your encrypted message is: ',new) 
+2

didnt我们已经做到这一点... HTTP://stackoverflow.com/questions/33442220/itertools-cycle-in-vigenere-密码导致的问题与空间-python/33442864#33442864 –

+0

我们上次添加空格,现在我需要做大写字母。我已经尝试添加三个与其他类似的新功能,但现在它们返回一个大写字母'Z'而不是小写字母'',它不起作用,并且我被卡住了,首先想到 – MrBeard

+0

,你有两个定义'add_chars'并且没有'add_charsa'的定义。第二,你需要确保'add_charsa'调用你的新函数,而不是你的旧函数,即它应该返回int_to_chra((chr_to_inta(msg)+ chr_to_inta(key))' –

回答

0

,因为你不似乎让我说的话:

def add_charsa(msg, key): 
    return int_to_chr((chr_to_int(msg) + chr_to_int(key)) % 26) 

是你目前有。这一点,你会得到你的坏输出:

>>> vigenere('COMPUTING IS FUN','GCSE') 
'xftipkpgb zz ype' 

这是因为你没有改变return语句此函数来调用新的大写功能。如果你改变了return语句:

def add_charsa(msg, key): 
    return int_to_chra((chr_to_inta(msg) + chr_to_inta(key)) % 26) 
#notice the change in function calls from int_to_chr -> int_to_chra, and chr_to_int -> chr_to_inta 

然后你会得到预期:

>>> vigenere('COMPUTING IS FUN','GCSE') 
'JRFUBWBSN LL KBQ' 

这是值得的,要知道,如果你的钥匙有大写和小写字母的混合,这不管用。很好。我反而会改变你的keystream是:keystream = cycle(keyword.lower())那么你的函数是:

def add_charsa(msg, key): 
     return int_to_chra((chr_to_inta(msg) + chr_to_int(key)) % 26) 
    #notice the call to chr_to_int(key) because key will always be lower case 
+0

哎呀,我现在感觉有点傻。非常感谢 – MrBeard