You Deserved It!


#1

https://www.codecademy.com/courses/close-the-super-makert/0/7?curriculum_id=506324b3a7dffd00020bf661#

I get the message "Your bill is NaN" on my code. I have tried several variations, this one runs, but doesn't come out a number for a total. The error messag is that it is false.

Thanks!


function StaffMember(name,discountPercent){
    this.name = name;
    this.discountPercent = discountPercent;
}

var sally = new StaffMember("Sally",5);
var bob = new StaffMember("Bob",10);

// Create yourself again as 'me' with a staff discount of 20%
var me = new StaffMember("Mickey", 20);

var cashRegister = {
    total:0,
    lastTransactionAmount: 0,
    add: function(itemCost){
        this.total += (itemCost || 0);
        this.lastTransactionAmount = itemCost;
    },
    scan: function(item,quantity){
        switch (item){
        case "eggs": this.add(0.98 * quantity); break;
        case "milk": this.add(1.23 * quantity); break;
        case "magazine": this.add(4.99 * quantity); break;
        case "chocolate": this.add(0.45 * quantity); break;
        }
        return true;
    },
    voidLastTransaction : function(){
        this.total -= this.lastTransactionAmount;
        this.lastTransactionAmount = 0;
    },
    // Create a new method applyStaffDiscount here
    applyStaffDiscount : function() {
        this.total -= this.total * (StaffMember.discountPercent / 100);
    }
    
};

cashRegister.scan('eggs',1);
cashRegister.scan('milk',1);
cashRegister.scan('magazine',3);
// Apply your staff discount by passing the 'me' object 
// to applyStaffDiscount
cashRegister.applyStaffDiscount('me');

// Show the total bill
console.log('Your bill is '+cashRegister.total.toFixed(2));

https://www.codecademy.com/courses/close-the-super-makert/0/7?curriculum_id=506324b3a7dffd00020bf661#


#2

here:

cashRegister.applyStaffDiscount('me');

string 'me' doesn't have discount percentage? You should use the me object you created at line. Now lets look at your actually applyStaffDiscount method:

    applyStaffDiscount : function() {
        this.total -= this.total * (StaffMember.discountPercent / 100);
    }

we have multiply staffmembers with different discount percentages, using StaffMember just doesn't work

give your method a parameter so we actually know which staffmember we are dealing with.


#3

Your function should take one argument employee,

your employee argument should take the place of StaffMember on this line,

this.total -= this.total * (StaffMember.discountPercent / 100);


#4

Thanks to both of you! I think I get myself turned around and then just confuse myself more.


#5

This topic was automatically closed 7 days after the last reply. New replies are no longer allowed.