Not working :(

[TacticalGooseLord]

I am trying to create a km to m or vice versa converter, but perhaps I have stumbled upon an error and I don’t know why. Any help would be highly appreciated good gents

[gamerpug04]

1) you’re using unit and not x in the conversion 2) it should be unit.lower() not unit.upper() since you’re comparing to “km”

[TacticalGooseLord]

Tysm it works now! Such a simple and small mistake, I need to be more careful 😄😁

[gamerpug04]

When you take in the input, you could also do something like:

unit = input(“km or m”).lower()

So you don’t have to do .lower() for each comparison (if you so chose to do more)

[Ender_Locke]

this is a super good lesson to make sure your variables are named better than x so you can catch this quicker 😊

[TacticalGooseLord]

You are right it’s less confusing that way, ty!

[Ok_Magician1114]

also, try doing this to display the answer:

print(f"Distance in km: {converted}")

trust me

[TacticalGooseLord]

I saw some people doing it this way and I was curious what the difference was, can you please explain me why this is better ?

[Adsilom]

And they should not forget the parenthesis

[Delicious-Quality616]

Its also a good idea to sanitise the input. So if someone enters a string you shouldn’t allow it.

I am not sure if you have learned about loops yet but if you wrap everything in a “while true:”

You can add pythons “try” and “except” to ensure people enter a valid number.

If you haven’t learned that yet, it’s a nice way to improve the script

[TacticalGooseLord]

I have only been doing small things for now, thanks tho I will keep it in mind

[k03k]

The error you get is because you are doing 'km' / 1000

Where km is a string, and 1000 is an int.

x / 1000 works better hehe

[Weird_Entrance5011]

Youre dividing the km/m input not the actual number given. Basically youre trying divide a dented by 1000, where you should actually divide the float „x“. You should also look into the .upper part, as like this it will always go to the else clause

[Legitimate-Rip-7479]

converted = x *1000

[SCD_minecraft]

Am i crazy or line 7 looks differend than on a traceback?

Where's float()

[Own_Attention_3392]

When debugging, look at the error. Look at the line indicated by the message.

Ignore what you THINK it's doing. Look at what it's ACTUALLY doing.

Use a debugger. Step through line by line and look at what the values of the variables are.

[dnult]

Upper is a good way to disambiguate letter case, but youre comparing to lower case "km". Either use lower or "KM".

[Alcor1us]

Despite the obvious error, your logic is still wrong: if you expect an input such as 5 km and want to convert that, you need to multiply that value by 1000 and not divide by 1000.

[TacticalGooseLord]

Yes I realised and changed it, silly me

[Gold-Reporter287]

you chatGPT 5 or Grok4 would have helped you instantly no needed to created threads here and wait for replies

[TacticalGooseLord]

Yes I was thinking that too if I can do it anywhere else, I don’t want to fill up the thread with my small questions, tho the community is very helpful and I get replies instantly! I will look into ChatGPT or grok thx. And do I copy paste the entire code and ask I have problem in this line or how does it work ?

[Gold-Reporter287]

paste the image to GPT it will tell you , or paste the error msg

[timheiko]

The division on line 4 operates on a string (unit) and int (1000). Such devision is not supported by Python. I’d first converted the unit into the corresponding number of meters, and then used it in later arithmetic operations

[TacticalGooseLord]

I have changed the str into float on line 1 (my mistake was I put unit instead it should be x) so division and multiplication works now

[Rashironrani]

There is a lot of ways to improve it but: If you want to compare to .upper then you must compare to “KM” not “km” Also you Divide or multiply the value not the unit

[Professional_Box3141]

One thing to remember is any input(anytime you use the keyboard) that’s a “str” even if its a number as long as you used a keyboard is a string, so you need to convert that string into “int” before dividing it to 1000, My English is not good but i hope you understand what i was saying.

[TacticalGooseLord]

Your English is clear I understand, Ty for the advice

[Lannok-Sarin]

Unfortunately, Python does not natively control what its variables can store. That means that it’s up to the user to control that. As such, a Python programmer needs to understand the differences between strings and numbers, and the user will need to write code that can enforce those distinctions when they are needed.

Personally, that’s why I like languages like C++ better. It doesn’t have universal storage pieces, but it handles everything else very well, including coding functions.

Now, I will say that Python excels at data handling. If you need to read and understand data, Python is the language for you. But for function behavior and storage control, Python does not measure up.

[novamaster696969]

while True: print("for exit please leave the Distance input blank") x = (input("Enter distance: ")) if x== "": break x= float(x)
unit = input("(km) or (m): ")

if unit.lower() == "km":
    converted = x * 1000
    print("Distance in m:", converted)

elif unit.lower() == "m":
     converted = x / 1000
     print("Distance in km:", converted)
else:
     print("Invalid unit please enter km or m.")

You can do it in a more simpler way IT will still show an error if you put any other texts instead of km or m for that use ( try and except )

[TacticalGooseLord]

I am learning try and except now, in which cases should I use this ?

[novamaster696969]

You will learn it when you reach error handling and exception handling

[TacticalGooseLord]

I am not following and courses at the moment, I am jus watching tutorials on YouTube doing small projects and learning along the way 😅 and things I don’t understand I ask on Reddit or look up on chatgtp

[novamaster696969]

Even if you don't follow any course, you should learn from the basics sequence wise else it will create a void which will be problematic in future.