Some suggestions.

1. I hope you are using Option Explicit. If you are not, then start...now.

2. That will force you to properly declare variables. As it stands:[vba]
NumOne = InputBox ("Please enter the price of the car")
[/vba]will return a string. Yes, using Cdbl() will work, but what if NumOne is "123e4"? The user slipped and hit a text key?

In other words....

- declare proper variables
- get into the habit of at least basic error trapping