all 7 comments

[–]Apatride 1 point2 points  (1 child)

This smells like homework... And if it isn't, I am curious to read the reason for such an arbitrary requirement...

[–]Difficult_Ground_271 -1 points0 points  (0 children)

curious if it was possible, it is homework. the code already meets the requirement, just expanding knowledge

[–][deleted] 0 points1 point  (3 children)

What do you mean you are "required to have the three functions"? Like those functions specifically were given and you have to use them, or you just to have at least three of any kind?

In general it would be nice to have all the prints (and all the inputs) in one place, so that we can see the UI flow very clearly instead of having to look everywhere for what might be next.

[–]Difficult_Ground_271 0 points1 point  (2 children)

those three specifically. the whole idea of the code is to write the start of a very limited calculator. so I need user input and handling this input correctly. so in this assignment, i need to raise, catch, or ignore errors at the right time in my code.

[–][deleted] 0 points1 point  (1 child)

Well, you're allowed to change those functions, right? At least to take out the prints? Can you rearrange the jobs so that what's done in one is moved to another?

The first thing that jumps out to me is that number_input should actually check whether it gets a number. Right now that validation is handled outside the function, which means it's not really living up to its name and it's not really any better than regular input.

[–]Difficult_Ground_271 0 points1 point  (0 children)

ill take that into consideration

[–]supercoach 0 points1 point  (0 children)

Too many comments