Hi Roger, and welcome. See my comments below.

On Wed, Apr 18, 2018 at 04:39:27PM -0700, Roger Lea Scherer wrote:

> def encryptor(address):
>     encrypted = ""
>     for char in address:
>         if char != alphabet:
>             encrypted += char
>         else:
>             pos = alphabet.index(char)
>             encrypted += encryption[pos]
>     print(encrypted)

For each character in the Gettysburg address, you compare the individual 
single character "F", "o", "u", "r", etc against the ENTIRE alphabet 
"abcde...xyz".

Since a single letter is never equal to 26 letters, you always get

    char != alphabet  # they are never equal

and so you always add the char unchanged to the encrypted text.

Try this instead:

    if char not in alphabet:
        encrypted += char

Then once you have that working, write back (with a shorter extract of 
the Gettysburgh address please!) and we'll see how else we can improve 
your code.



-- 
Steve
_______________________________________________
Tutor maillist  -  Tutor@python.org
To unsubscribe or change subscription options:
https://mail.python.org/mailman/listinfo/tutor

Reply via email to