Python de chiffrement de vigenere d'aller plus loin que nécessaire?

Je ne me contenterai pas de résoudre le problème pour vous car il y a beaucoup d'autres choses à améliorer ici !

Mettre de l'espace autour des opérateurs: a=b devrait être a = b, même avec +, -, etc.

J'ai trouver de mieux utiliser la fonction params que input. Vous pouvez toujours avoir une deuxième fonction pour obtenir l'entrée et chiffrer l'entrée:

def encryption(plaintext, keyword):
    pass  # (do nothing)

Je vous laisse écrire la fonction auxiliaire.

J', et de plus, l'habitude de mettre des commentaires sur la ligne ci-dessus le code correspondant. Aussi, pas besoin d'écrire This tous les temps, et de l'impératif est généralement préféré.

Maintenant, nous allons jeter un oeil à votre while boucle. La condition est len(keyword) < len(plaintext), à l'intérieur de vous vérifiez len(keyword) > len(plaintext). Quand cela peut-il arriver ? Seulement au cours de la dernière itération. Ainsi, passer le code de la boucle.

Aussi, ce que vous faites à l'intérieur de la if n'a pas besoin d'un if:

any_string[:len(any_string) + n] == any_string (n positif int).

De Plus, vous n'utilisez jamais newkey!

Nous pouvons simplifier la boucle pour:

# Begin a while loop based on the fact that the length of the keyword is
# shorter than the length of the plaintext.
while len(keyword) < len(plaintext):
    # Repeat the keyword.
    keyword += keyword

# Cut the string length of the keyword
keyword = keyword[:len(plaintext)]

Qui est équivalent à:

# Do the operation once
txt_len = len(plaintext)
# Get a string just a bit longer than plaintext
# Otherwise we would have issues when plaintext is not a multiple
# of keyword
keyword *= txt_len // len(keyword) + 1
# Chop of the extra part
keyword = keyword[:txt_len]

Noter que les deux échouent lorsque len(keyword) == 0.

Maintenant à la for boucle:

Vous pouvez utiliser zip, comme polku vous a montré, mais je vais supposer que c'est trop compliqué pour l'instant et garder le range.
Vous pouvez également utiliser l'alphabet, mais simple arithmétique peut faire l'affaire:

alphabet.index(x) == ord(x) - ord('a'), si dans votre code:

char = ord(plaintext[c]) - ord('a')
temp = ord(keyword[c]) - ord('a')
newchar = char + temp
# Or simply:
newchar = ord(plaintext[c]) + ord(keyword[c]) - 194  # 2 * ord('a')

Si nous ignorons les lettres majuscules, nous pouvons substituer

if newchar > 25:
    newchar -= 25
# With
newchar %= 25

Enfin:
alphabet[i] == ord(i + ord('a')).

Voici tout cela mis ensemble:

def encryption(plaintext, keyword):
    # Do the operation once
    txt_len = len(plaintext)

    # Get a string just a bit longer than plaintext
    # Otherwise we would have issues when plaintext is not a multiple
    # of keyword
    keyword *= txt_len // len(keyword) + 1  # //is floor division
    # Chop of the extra characters (if there are any)
    keyword = keyword[:txt_len]

    # Create a string to store the encrypted message
    encoded = ""

    # Now you should change this to work with capital letters
    for c in range(txt_len):
        # 194 = 2 * ord('a')
        newchar = ord(plaintext[c]) + ord(keyword[c]) - 194
        newchar %= 25
        encoded += chr(newchar + 97)  # 97 = ord('a')

    return encoded


def encrypt_input():
    # This function should use input to get plaintext and keyword
    # Then use encryption with those strings and print the result
    pass