var profitPerUnit = 213.38 / 47:

var profitPerUnit = 213.38 / 47;

You don't have to do the math yourself.

Just put

var profitPerUnit = profit / quantity;

I assume you're on challenge 3 of 3? If so, the code checker shouldn't have let you passed the first two challenges. You should be using the variables in every expression, not the actual values. Variables are there to store the values for you, for future use.

Consider if the whole sale price changes. The way you've written your solutions, you'd have ton manually search for, and change the price everywhere you've written $5.45. That'd be even more tedious to do, if your wholesale and retail price were the same from the beginning. How would you know which represented which?

Your code should look like this:

var wholesalePrice = 5.45,
retailPrice = 9.99,
quantity = 47,
salesTotal = retailPrice * quantity,
profit = salesTotal - wholesalePrice * quantity,
profitPerUnit = profit / quantity;

With your program written like this, changing a value at the top will be reflected at the bottom. You make a change in one place, and it changes everywhere else. For example:

var wholesalePrice = 5.45,
retailPrice = 7.99, //retail price dropped by two dollars
quantity = 50, //quantity goes up by 3; anywhere you see "quantity, the value 50 is stored there
salesTotal = retailPrice * quantity, //this is now 7.99 * 50
profit = salesTotal - wholesalePrice * quantity, 
profitPerUnit = profit / quantity; 

Note: I'm not sure if you're familiar with the way I've written the variables. you can use a comma to separate multiple variable declarations, if they're all the same type. It does nothing but save a few keystrokes -- typing var multiple times.

Thanks...something so simple..:) Just love the treehouse help forum :)