[Tutor] Script won't run for no apparent reason

[Dave Angel]

On 08/10/2012 03:33 PM, Selby Rowley Cannon wrote:
> On 10/08/12 20:07, Joel Goldstick wrote:
>> On Fri, Aug 10, 2012 at 2:57 PM, Selby Rowley Cannon
>> <selbyrowleycannon at ymail.com> wrote:
>>> On 10/08/12 18:17, Joel Goldstick wrote:
>>>> On Fri, Aug 10, 2012 at 1:05 PM, Selby Rowley Cannon
>>>> <selbyrowleycannon at ymail.com> wrote:
>>>>> I have written a small application to encrypt some text. The
>>>>> script looks
>>>>> fine to me, but it won't run and I can't figure out why. I have
>>>>> attached
>>>>> it,
>>>>> if anyone knows why it doesn't work please let me know!
>>>>>
>>>> What do you mean 'it won't run'?  Do you get an error with Traceback?
>>>>
>>>> I glanced at your code, and your dictionary ends like this: , 'X':'A',
>>>> 'Y':'B', 'Z':'C',
>>>>
>>>> There is no closing brace
>>> OK,
>>>
>>>    File "./crypto.py", line 6
>>>      def encrypt():
>>>        ^
>>> SyntaxError: invalid syntax
>>>
>> First, don't reply to me, reply to the group.  That might mean
>> choosing reply all.
>>
>> So, your first problem is that the dictionary isn't closed.  This is
>> causing the error at line 6
>>
>> fix that and find your next error.  It looks like there are lots of them
>>
>> With such a small file you would do better to just post the code
>> directly.  That way if people see problems they can point them out in
>> the body of the reply
>>
>> good luck
> #!/usr/bin/env python3
>
> import random
> values = {'a':'d', 'b':'e', 'c':'f', 'd':'g', 'e':'h', 'f':'i',
> 'g':'j', 'h':'k', 'i':'l', 'j':'m', 'k':'n', 'l':'o', 'm':'p',
> 'n':'q', 'o':'r', 'p':'s', 'q':'t', 'r':'u', 's':'v', 't':'w',
> 'u':'x', 'v':'y', 'w':'z', 'x':'a', 'y':'b', 'z':'c', 'A':'D',
> 'B':'E', 'C':'F', 'D':'G', 'E':'H', 'F':'I', 'G':'J', 'H':'K',
> 'I':'L', 'J':'M', 'K':'N', 'L':'O', 'M':'P', 'N':'Q', 'O':'R',
> 'P':'S', 'Q':'T', 'R':'U', 'S':'V', 'T':'W', 'U':'X', 'V':'Y',
> 'W':'Z', 'X':'A', 'Y':'B', 'Z':'C'}
> def encrypt():
>     textInputE = input('Please enter the text you wish to encrypt: ')
>     textInputE.list()
>     for Eletter in textInputE.list():
>         try:
>             print (values[Eletter])
>         except KeyError:
>             print ('Sorry, that input couldn\'t be parsed as text. Try
> again.')
>             input('Press Enter')
> def decrypt():
>     textInputD = input('Please enter the numbertext you wish to decrypt')
>     textInputD.list()
>     for Dletter in textInputD.list():
>         try:
>             print (values[Dletter])
>         except KeyError:
>             print ('Sorry, that input couldn\'t be parsed as
> numbertext from our cipher. Please try again.')
>             input('Press Enter')
>
> while True:
>     EorD = input('Encrypt or Decrypt: ')
>     if EorD == 'Encrypt' or EorD == 'encrypt':
>         encrypt()
>     elif EorD == 'Decrypt' or EorD == 'decrypt':
>         decrypt()
>     else:
>         print('Encrypt or Decrypt?')
>
> Thanks, I am not quite used to this client yet. The next error is:
>
> Traceback (most recent call last):
>   File "crypto.py", line 25, in <module>
>     EorD = input('Encrypt or Decrypt: ')
>   File "<string>", line 1, in <module>
> NameError: name 'Encrypt' is not defined
>

Did you play with the script as it stands now? Did you try typing things
other than Encrypt at the input prompt?  You'll notice that the error
message is complaining about what YOU typed as the user.

Looks like you're running Python 3 code on a Python 2 system.  If you're
running Python 2, you want to use raw_input() which gets a string,
rather than input(), which tries to evaluate an expression.


-- 

DaveA