[Tutor] Beginner problem: name 'convertToFahrenheit' is not defined

[Joseph Bae]

Thanks for the help!

I have managed to get a good temperature converter program working! I am
working on beefing it up a bit with some exception handling and an "and-or
trick". The error handling works okay but I am having problems using and-or.
Here's my updated code:

def main():
    true = 1
    while true:
        try:
            temp = int(raw_input("Enter A Number : "))
            break
        except ValueError:
            print "Invalid Input"
    while true:
        convertTo = raw_input("Convert To (F)ahrenheit or (C)elsius? : ")
        if not convertTo == "F" and not convertTo == "C":
            print "Invalid Input"
*        else:
            convertTo == "C" and convertToCelsius(temp) or
convertToFahrenheit(temp)
            break
*
def convertToCelsius(t):
    tC = (9.0/5.0) * (t - 32)
    print "%d Fahrenheit = %d Celsius" % (t, tC)

def convertToFahrenheit(t):
    tF = (9.0/5.0) * (t + 32)
    print "%d Celsius = %d Fahrenheit" % (t, tF)

if __name__=="__main__":
    main()

Sample Output (as of right now):

Enter A Number : 50
Convert to (F)ahrenheit or (C)elsius? C
50 Fahrenheit = 32 Celsius
32 Celsius = 147 Fahrenheit <-- shouldn't show up and 147 is too high ...

This only happens when I tell it to convert to C, if I say F it works
normally. I've debugged it with pdb.set_trace() many times but can't figure
out what's wrong. Help is much appreciated =)

Thanks,

Joe

On Thu, Aug 14, 2008 at 10:50 PM, Alan Gauld <alan.gauld at btinternet.com>wrote:

> "Joseph Bae" <joeturf at gmail.com> wrote
>
>  temp = input("Enter A Number : ")
>> convertTo = raw_input("Convert To (F)ahrenheit or (C)elsius? : ")
>>
>> if convertTo == "F":
>>   convertedTemp = convertToFahrenheit(temp)
>>   print "%d Celsius = %d Fahrenheit" % (temp, convertedTemp)
>> else:
>>   convertedTemp = convertToCelsius(temp)
>>   print "%d Fahrenheit = %d Celsius" % (temp, convertedTemp)
>>
>> def convertToFahrenheit(t):
>>   tF = (9.0/5.0) * (t + 32)
>>   return tF
>>
>> def convertToCelsius(t):
>>   tC = (9.0/5.0) * (t - 32)
>>   return tC
>>
>>         convertedTemp = convertToFahrenheit(temp)
>> NameError: name 'convertToFahrenheit' is not defined
>>
>> This is most likely a very simple error, but can someone please clarify
>> for
>> me why it's behaving this way?
>>
>
> Others have explained that you need to execute the function
> definitions before Python sees the name. You can do this in
> two ways depending on taste.
> 1) As suggested move your main code below the function definitions.
> 2) move your main code into a function - traditionally called main()
>   then call main as the last line of your code.
>
> The second method has two advantages:
> 1) It maintains the top-down design style if thats your preferred style
> 2) It makes it much easier to make the file into a reusable module.
>
> It has two minor disadvantages:
>
> 1) The extra function call (main() ) slows things down by a tiny amount
> 2) the extra indentation level of being inside a function reduces the page
>   width slightly
>
> HTH,
>
> --
> Alan Gauld
> Author of the Learn to Program web site
> http://www.freenetpages.co.uk/hp/alan.gauld
>
> _______________________________________________
> Tutor maillist  -  Tutor at python.org
> http://mail.python.org/mailman/listinfo/tutor
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.python.org/pipermail/tutor/attachments/20080815/9f2bf42b/attachment.htm>