2017-10-14 13 views
1

J'essaie d'utiliser les définitions paires et impaires pour chiffrer mon message. La fonction def swap_letters(message) est ce que je suis en utilisant pour cela:Python - Message Encryption renvoie `NoneType` Erreur

  • boucle I à travers les listes de paires et impaires lettres avec ` « message = un message + « x » » '; Ensuite, j'ajoute la lettre suivante au message final avec "for counter in range(0, int(len(message)/2)): ...;
  • Enfin, j'ajouter la lettre suivante, même au message final à l'aide "letter_1.append(even_letters[counter])...

Je fais mon new_message en se joignant à la letter_1 puis le retourner

Quelle erreur dois-je faire et comment dois-je résoudre ce problème code pour le faire fonctionner correctement?

import tkinter as Tk 
from tkinter import messagebox, simpledialog, Tk 

def is_even(number): 
    return number % 2 == 0 

def get_even_letters(message): 
    even_letters = [] 
    for counter in range(0, len(message)): 
     if is_even(counter): 
      even_letters.append(message[counter]) 


def get_odd_letters(message): 
    odd_letters = [] 
    for counter in range(0, len(message)): 
     if not is_even(counter): 
      odd_letters.append(message[counter]) 
    return odd_letters 

def swap_letters(message): 
    letter_1 = [] 
    if not is_even(len(message)): 
     message = message + 'x' 
    even_letters = get_even_letters(message) 
    odd_letters = get_odd_letters(message) 
    for counter in range(0, int(len(message)/2)): 
     letter_1.append(odd_letters[counter]) 
     letter_1.append(even_letters[counter]) 
    new_message = ''.join(letter_1) 
    return new_message 

def get_message(): 
    message = simpledialog.askstring('Message', 'Enter the secret here: ') 
    return message 

def get_task(): 
    task = simpledialog.askstring('Task', 'Do you want to encrypt or decrypt?') 
    return task 

root = Tk() 

while True: 
    task = get_task() 
    if task == 'encrypt': 
     message = get_message() 
     encrypted = swap_letters(message) 
     messagebox.showinfo('Ciphertext of the secret message is:', encrypted) 
    elif task == 'decrypt': 
     message = get_message() 
     decrypted = swap_letters(message) 
     messagebox.showinfo('Plaintext of the secret message is:', decrypted) 
    else: 
     break 
root.mainloop() 

C'est l'erreur que je reçois:

Traceback (most recent call last): 
    File "T:\Sublime Text 3\shhhhh.py", line 47, in <module> 
    encrypted = swap_letters(message) 
    File "T:\Sublime Text 3\shhhhh.py", line 29, in swap_letters 
    letter_1.append(even_letters[counter]) 

TypeError: 'NoneType' object is not subscriptable 
[Finished in 21.2s with exit code 1] 
[shell_cmd: python -u "T:\Sublime Text 3\shhhhh.py"] 
[dir: T:\Sublime Text 3] 
[path: C:\Program Files (x86)\Python36-32\Scripts\;C:\Program Files `(x86)\Python36-32\;C:\WINDOWS\system32;C:\WINDOWS;C:\WINDOWS\System32\Wbem;C:\WINDOWS\System32\WindowsPowerShell\v1.0\;C:\Program Files (x86)\NVIDIA Corporation\PhysX\Common;C:\Program Files\dotnet\;C:\Program Files\Microsoft SQL Server\130\Tools\Binn\;C:\Program Files (x86)\Xoreax\IncrediBuild;C:\Users\mwliv\AppData\Local\Microsoft\WindowsApps;]` 

Répondre

1

Votre fonction get_even_letters ne retourne rien explicitement, il renvoie implicitement None, et donc l'erreur que vous voyez plus tard. Ajout d'une déclaration return devrait résoudre le problème:

def get_even_letters(message): 
    even_letters = [] 
    for counter in range(0, len(message)): 
     if is_even(counter): 
      even_letters.append(message[counter]) 

    return even_letters # Here! 
+1

merci beaucoup, c'était super gentil et très simple, pouces vers le haut de moi! (et honnêtement, quel imbécile bête ...) –

0

Juste modifier votre code comme ci-dessous: -

for counter in range(0, int(len(message)/2)): 
    # You need to verify whether values are None type or not before calling append 
    if(odd_letters != None): 
     letter_1.append(odd_letters[counter]) 
    if(even_letters != None): 
     letter_1.append(even_letters[counter])