By using this site, you agree to our updated Privacy Policy and our Terms of Use. Manage your Cookies Settings.
440,172 Members | 762 Online
Bytes IT Community
+ Ask a Question
Need help? Post your question and get tips & solutions from a community of 440,172 IT Pros & Developers. It's quick & easy.

Conversion code not working properly

P: n/a
Why is the Farenheit to Celsius part not working properly? Instead of
showing a similar range of what the farenheit is listing, the celsius
portion is showing half the range of farenheit.

print "\nWelcome to the Temperature program"

while True:
low1 = raw_input("Please enter a low temperature: ").upper()
if low1 != 'I QUIT':
low = int(low1)
if low1 == 'I QUIT':
break
high = int(raw_input("Please enter a high temperature: "))
temp = raw_input('Please indicate Celsius or Fahrenheit ').upper()

while not temp or temp not in 'CF':
temp = raw_input('Please indicate Celsius or Fahrenheit
').upper()

if temp == 'C' and low <= high:

low_other_temp = (9 / 5 * low) + 32
high_other_temp = (9 / 5 * high) + 32
rnge = range(low, high + 1)
faren = range(low_other_temp, high_other_temp + 1)
print rnge
print faren

elif temp == 'F' and low <= high:

num1 = low - 32
num2 = high - 32
low_temp = num1 * 5/9
high_temp = num2 * 5/9
rnge = range(low, high + 1)
celsi = range(low_temp, high_temp + 1)
print rnge
print celsi

else:
print 'Invalid temperatures specified! The low must be less
than the high!'

exit = raw_input('Hit <enter> to continue. Type " I quit" in the
low temp field to exit.')
print 'Returning to main.'
Jul 18 '05 #1
Share this Question
Share on Google+
8 Replies


P: n/a
weasel wrote on Sun, 22 Feb 2004 19:07:01 GMT:
Why is the Farenheit to Celsius part not working properly? Instead of
showing a similar range of what the farenheit is listing, the celsius
portion is showing half the range of farenheit.
Try this in your interactive interpreter:
9/5
What's odd about the result? "/" does integer division. It's weird and
unexpected to lots of people, but it's the way it is - it will be fixed in
due time. However, due to some time machine Guido is said to possess, you
can already use the future behavior by doing this:

from __future__ import division

Then try 9/5 again. Integer division is "//" now, while "/" behaves as
floating-point division.

Alternatively, you could use a floating-point number instead of integers.
Try this in a clean interpreter session (without the from __future__
thing):
9.0/5
9/5.0
9.0/5.0


<snip> low_other_temp = (9 / 5 * low) + 32
high_other_temp = (9 / 5 * high) + 32
rnge = range(low, high + 1)
faren = range(low_other_temp, high_other_temp + 1)
print rnge
print faren

elif temp == 'F' and low <= high:

num1 = low - 32
num2 = high - 32
low_temp = num1 * 5/9
high_temp = num2 * 5/9
rnge = range(low, high + 1)
celsi = range(low_temp, high_temp + 1)
print rnge
print celsi

<snip>

--
Yours,

Andrei

=====
Real contact info (decode with rot13):
ce******@jnanqbb.ay. Fcnz-serr! Cyrnfr qb abg hfr va choyvp cbfgf. V ernq
gur yvfg, fb gurer'f ab arrq gb PP.
Jul 18 '05 #2

P: n/a
It's not working. I don't think the conversion is the problem. After
converting, the program shows the original entry you put in

i.e. Please enter low temp:
32
Please enter high temp:
42
Please enter a conversion letter (C, F):
F

Then, the display shows
32, 33, 34, 35, 36, 37, 38, 39, 40, 41, 42

However,

the conversion is only showing
0, 1, 2, 3, 4, 5.

What happened to the other half? I tried using 5.0/9.0, but got the
same results.

Any more ideas?
On Sun, 22 Feb 2004 20:32:50 +0100, Andrei <fa**@fake.net> wrote:
weasel wrote on Sun, 22 Feb 2004 19:07:01 GMT:
Why is the Farenheit to Celsius part not working properly? Instead of
showing a similar range of what the farenheit is listing, the celsius
portion is showing half the range of farenheit.


Try this in your interactive interpreter:
9/5
What's odd about the result? "/" does integer division. It's weird and
unexpected to lots of people, but it's the way it is - it will be fixed in
due time. However, due to some time machine Guido is said to possess, you
can already use the future behavior by doing this:

from __future__ import division

Then try 9/5 again. Integer division is "//" now, while "/" behaves as
floating-point division.

Alternatively, you could use a floating-point number instead of integers.
Try this in a clean interpreter session (without the from __future__
thing):
9.0/5
9/5.0
9.0/5.0


<snip>
low_other_temp = (9 / 5 * low) + 32
high_other_temp = (9 / 5 * high) + 32
rnge = range(low, high + 1)
faren = range(low_other_temp, high_other_temp + 1)
print rnge
print faren

elif temp == 'F' and low <= high:

num1 = low - 32
num2 = high - 32
low_temp = num1 * 5/9
high_temp = num2 * 5/9
rnge = range(low, high + 1)
celsi = range(low_temp, high_temp + 1)
print rnge
print celsi

<snip>


Jul 18 '05 #3

P: n/a

"weasel" <he******@hotmail.com> wrote in message
news:3u********************************@4ax.com...
Why is the Farenheit to Celsius part not working properly?
Specific examples of what you put in, what you get out, and what you
expected out and why often helps get answers.
Instead of
showing a similar range of what the farenheit is listing, the celsius
portion is showing half the range of farenheit.


Celcius ranges are 5/9ths, approximately 1/2, the corresponding Farenheit
ranges.

Terry J. Reedy


Jul 18 '05 #4

P: n/a
see my other post. I've got an example of what's going wrong.

On Sun, 22 Feb 2004 14:27:28 -0500, "Terry Reedy" <tj*****@udel.edu>
wrote:

"weasel" <he******@hotmail.com> wrote in message
news:3u********************************@4ax.com.. .
Why is the Farenheit to Celsius part not working properly?


Specific examples of what you put in, what you get out, and what you
expected out and why often helps get answers.
Instead of
showing a similar range of what the farenheit is listing, the celsius
portion is showing half the range of farenheit.


Celcius ranges are 5/9ths, approximately 1/2, the corresponding Farenheit
ranges.

Terry J. Reedy


Jul 18 '05 #5

P: n/a
I just figured it out. It has to do with range. The conversion is
working, but the range functionality is screwing it all up. Back to
the drawing board!
Jul 18 '05 #6

P: n/a
weasel wrote on Sun, 22 Feb 2004 19:45:30 GMT:
It's not working. I don't think the conversion is the problem. After
converting, the program shows the original entry you put in
You're right, I saw integer division where you were trying to convert
temperatures and automatically assumed that was the only problem, without
really reading the code.
What happened to the other half? I tried using 5.0/9.0, but got the
same results.
How did you get range() to work on floats without deprecation warnings?
Any more ideas?


Terry Reedy answered a short while ago to why a Celsius range is by
definition shorter than a Fahrenheit range. I'm not quite sure what the
point of the conversion is to be honest, more specifically why a range is
returned. You can't map a celsius temperature in the Celsius range to a
Fahrenheid temperature in the other range and the extreme values are
incorrect due to the integer division and/or the range() used with float
arguments. 42 F should be 5.556 C and your program returns 5.

Btw, there's also a Python list for beginners if you're interested
(python-tutor).

--
Yours,

Andrei

=====
Real contact info (decode with rot13):
ce******@jnanqbb.ay. Fcnz-serr! Cyrnfr qb abg hfr va choyvp cbfgf. V ernq
gur yvfg, fb gurer'f ab arrq gb PP.
Jul 18 '05 #7

P: n/a
Well, I know that the range command is the problem, because it just
keeps adding 1 which throws off the decimal entries. So, in removing
range, I need to take the beginning temp, and add up to the ending
temp.

The conversion seems to be working fine, even by using "float" for the
input.

On Sun, 22 Feb 2004 21:15:30 +0100, Andrei <fa**@fake.net> wrote:
weasel wrote on Sun, 22 Feb 2004 19:45:30 GMT:
It's not working. I don't think the conversion is the problem. After
converting, the program shows the original entry you put in


You're right, I saw integer division where you were trying to convert
temperatures and automatically assumed that was the only problem, without
really reading the code.
What happened to the other half? I tried using 5.0/9.0, but got the
same results.


How did you get range() to work on floats without deprecation warnings?
Any more ideas?


Terry Reedy answered a short while ago to why a Celsius range is by
definition shorter than a Fahrenheit range. I'm not quite sure what the
point of the conversion is to be honest, more specifically why a range is
returned. You can't map a celsius temperature in the Celsius range to a
Fahrenheid temperature in the other range and the extreme values are
incorrect due to the integer division and/or the range() used with float
arguments. 42 F should be 5.556 C and your program returns 5.

Btw, there's also a Python list for beginners if you're interested
(python-tutor).


Jul 18 '05 #8

P: n/a
On Sun, 22 Feb 2004 20:09:17 GMT, weasel <he******@hotmail.com> wrote:
I just figured it out. It has to do with range. The conversion is
working, but the range functionality is screwing it all up. Back to
the drawing board!

Just finished it. Thank you for the suggestions, it helped me move in
the right direction.
Jul 18 '05 #9

This discussion thread is closed

Replies have been disabled for this discussion.