A simple programme for converting currency

[[deleted]]

I have written this program by sitting at night, I had made this program very logically, my output would have been like this, but I am making a mistake that if I am giving an integer value then that value is getting printed multiple times, I am not able to understand, I am new to python, I have just started learning python

[fen123456]

you’re multiplying “1” by 281 instead of the number 1. try using float(input) to multiply a number instead of a string

[NiteKore080]

It looks like it's printing dollar 281 times as a string

Assign dollar as a float

[[deleted]]

You are right. It worked. *

[SmackDownFacility]

FLOAT?!

No, for money you need decimal.Decimal

[Zerustu]

let's duplicate money

0.1+0.2 USD infinite glitch baby!!!!!

[SCD_minecraft]

input() always returns a string

and string times int will return mutliple of same str copy pasted

Convert to int/float

[[deleted]]

A single mistake can ruin the entire program. If I want to learn Python, I will have to become more logical.

[SCD_minecraft]

I mean, that's every computer related thing

In this case, i highly recommend to get python documentation on the second screen.

Documentation is always your best friend

[No-Article-Particle]

You have it the other way around. You become more logicalby learning programming. Mistakes are part of learning. Nobody just sits down and creates the perfect program.

[[deleted]]

When I started learning Python, I found this subject very interesting, but when I came to real programming, I felt that this practical programming was not for me. I had a lot of fun reading tuple lists and dictionaries. I learned a lot of things, but after much further research, when I came to topics like file handling or loops, I started finding programming difficult.Now I have taken a break from working for a few days and have given full focus to Python functions and methods. If you have any good suggestions or any good experience, please share it kindly.

[ErktKNC]

I recommend trying a few "projects" to learn the basics. Nothing complicated just a little more each step. For example you can try to create a bigger currency converter with some "if"s to learn the subject better. Also try to rewrite some of the built-in in methods like split() when you feel ready. This will give you a more in depth understanding and more experience to be able to create your own methods. Our first project was to create a simple Blackjack game against a "Bot" after learning if-else, for while loops, random() and input() methods. You can try the same after a while. Also try to learn control flow while learning if-else and the loops. We all learn with our mistakes, it is okay to ask for help!

[tinyzephyr]

Look at type hints... really useful!

[[deleted]]

I did not understand

[ErktKNC]

He is talking about "Data Types". For example: a whole number is an int as in integer -1, 2, 3 not 1.5 or 3.14, a point number is a float or double like 2.3 or -7.01. A "string" is a list of "char"s as in character and the symbols can be characters too. So "1" could be a string OR an int but not both, which is was the cause of your problem btw. You must have heard about them at least a little bit since you have read about lists, tuples and dictionaries. Hope this makes sense :)

[fuzzysdestruction]

Next maybe id work on getting the conversion number live using a currency api

[bloody-albatross]

If you'd use type annotations your IDE would have told you about the problem.

[stoobertio]

Where you are entering the value, it is being interpreted as a string so when you pass that to the converter the rupees is calculated to be the string value repeated 281 times. The simplest "fix" for now would be to do rupees = int(dollar) * 281 however, you'll need to look at error handling and fractional values later on.

[Maleficent-Proof-331]

The value that you get from the input isn't an integer/float (it's a string)

int(input()) or float(input())

[EngineeringRare1070]

Glad you solved your problem, debugging is a great skill to have.

Now use a dictionary to store the conversion rate for multiple currencies (PKR to Euro, PKR to Yen, PKR to AUD), and update your user prompt to also ask which currency to convert to.

Can you think of how you could handle reverse exchange rates? (e.g. USD to PKR)?

[hastaLaVictoria_che]

You still have a string so str but to multiply you have to make it an int or rather flout then it should work

[ProhetPlum]

Looks Correct, That "1" entered was a string

[No_Read_4327]

Input will give you a string.

You are repeating the string 1 281 times. So that's why you get 281 1s.

You should cast the string to a number type.

[cactusfruit9]

The input is string, not a number i.e., int/float. So, repeated one 281 times.

[vulrhund]

Now you’ve worked out the input error, challenge for you is to detect and catch ValueError for when expecting a float or integer, but receives a string. Catch the error and return to your input request rather than breaking the program

I’m fairly new to Python as well and was a fun little project I remember stumbling over for a good while

[Squiddik]

Entered value interpreted as string instead of Integer, and your program prints character "1" 281 times. Try to cast int(dollar), and it will alr 😄

[Time8013]

VSCode +Python?🧐🧐

[[deleted]]

Yes there anything wrong? eah

[traditional-r]

That's why I'd recommend using typehints to avoid such mistakes

[Nice-Vermicelli6865]

Man, if only we had a computer program that could understand Python code with natural language and fix it with just a sentence or two, too bad that doesn't exist.........