5. Bleep bleep - code review


#1

I can't seem to figure out what is wrong here. Happy early new year!

var cashRegister = {
    total:0,
    lastTransactionAmount: 0,
    //Dont forget to add your property
    add: function(itemCost) {
        this.total +=  itemCost;
        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;
    },
    //Add the voidLastTransaction Method here
        voidLastTransaction: function() {
            this.total -= this.lastTransactionAmount;
        };
    
};

cashRegister.scan('eggs',1);
cashRegister.scan('milk',1);
cashRegister.scan('magazine',1);
cashRegister.scan('chocolate',4);

//Void the last transaction and then add 3 instead
cashRegister.voidLastTransaction();
cashRegister.scan("chocolate", 3);

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

#2

@dreamthink i will find the solution soon.


#3

Hey @dreamthink first of all Happy Early New Year! :fireworks: Second your problem is in the code. I posted something in the code locating your error. So look at your code carefully :smile:

 voidLastTransaction: function() {
            this.total -= this.lastTransactionAmount;
        };  <<-- Here your not suppose to have a semi-colon. Not used when you create stuff in objects.

#4

Hi @amanuel2

Checked your reply and did the same. Still prompts as "Unexpected Token" ):

Here's my code:

var cashRegister = {
total:0,
//Dont forget to add your property
add: function(itemCost) {
this.total += 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;
},
//Add the voidLastTransaction Method here
voidLastTransaction: function(); {
this.total -= this.lastTransactionAmount;
}

};

cashRegister.scan('eggs',1);
cashRegister.scan('milk',1);
cashRegister.scan('magazine',1);
cashRegister.scan('chocolate',4);

//Void the last transaction and then add 3 instead
cashRegister.voidLastTransaction();
cashRegister.scan("chocolate", 3);
//Show the total bill
console.log('Your bill is '+cashRegister.total);

Happy New Year! Thanks in advance. (:


#5

You did not add the property lastTransactionAmount to the add method. As indicated in Hint, it should equal itemCost.
this.lastTransactionAmount = itemCost;


#6

Did you mean to add a semi-colon after the initialization of the function?


#7

when you make an object with literal notation, you should not use semi-colon after variables and methods.

voidLastTransaction: function() {
this.total -= this.lastTransactionAmount;
};

this should be

voidLastTransaction: function() {
this.total -= this.lastTransactionAmount;
}


#8

Here is the code, If anyone needs -

var cashRegister = {
total:0,
//Dont forget to add your property
add: function(itemCost) {
this.total += itemCost;
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;
},
//Add the voidLastTransaction Method here
voidLastTransaction: function() {
this.total = this.total - this.lastTransactionAmount;
}
};
cashRegister.scan('eggs',1);
cashRegister.scan('milk',1);
cashRegister.scan('magazine',1);
cashRegister.scan('chocolate',4);

//Void the last transaction and then add 3 instead
cashRegister.voidLastTransaction();
cashRegister.scan('chocolate',3);

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