Cash Register Project

JavaScript
1

Hey everyone!
I’ve recently tried to build the Cash Register Project for the JavaScript algorithms course.
However, the code runs perfectly fine if I run it on my PC locally and it catches all the requirements. But the freeCodeCamp tests fail (4 of the last 5 requirements to be precise).

<!doctype html>
<html lang="en">
  <head>
    <meta charset="UTF-8">
    <meta name="viewport" content="width=device-width, initial-scale=1.0" >
    <link rel="stylesheet" href="styles.css">
    <script src="script.js"></script>
    <title>Cash Register</title>
  </head>
  <body>
    <script>
      let price = 3.26;
      let cash = 0.00;
      let cid = [
        ["PENNY", 1.01],
        ["NICKEL", 2.05],
        ["DIME", 3.1],
        ["QUARTER", 4.25],
        ["ONE", 90],
        ["FIVE", 55],
        ["TEN", 20],
        ["TWENTY", 60],
        ["ONE HUNDRED", 100]
      ];
      let amount = [
        100.00,
        20.00,
        10.00,
        5.00,
        1.00,
        0.25,
        0.10,
        0.05,
        0.01
      ]
      cid = cid.reverse();
    </script>
    <div id="content">
      <h1>Register</h1>
      <span id="span">Item price: $19.5</span><br/><br/>
      <input id="cash" type="number" placeholder="Enter paid value">
      <br/>
      <button id="purchase-btn" onclick="calculate(cid, price)">Purchase</button>
      <br/><br/>
      <div id="change-due"></div>
    </div>
  </body>
</html>

function calculate(cid, price){
    price = parseFloat((price).toFixed(2));
    let cash = 0.00;
    cash = parseFloat(document.getElementById("cash").value).toFixed(2);

    let diff = parseFloat(cash - price).toFixed(2);
    let sum = 0.00;

    let statusOpen = "Status: OPEN";
    let statusClosed = "Status: CLOSED";
    let statusIns = "Status: INSUFFICIENT_FUNDS";
    let changeDue = document.getElementById("change-due");
    
    for (let i = 0; i < cid.length; i++) {
        sum += parseFloat(parseFloat(cid[i][1]).toFixed(2));
    }
    //ALERTS
    if(diff < 0){
        alert("Customer does not have enough money to purchase the item");
        return;
    }
    else if(diff == 0){
        changeDue.innerText = ("No change due - customer paid with exact cash");
        return;
    }
    //STATUS
    if(sum > diff){
        changeDue.innerText = statusOpen;
    }
    else if(sum < diff || sum == 0){
        changeDue.innerText = statusIns;
        return;
    }
    else if(sum == diff){
        changeDue.innerText = statusClosed;
    }
    //ALGORITHM
    for(let i = 0; i < amount.length; i++){
        let count = 0;
        while(diff >= amount[i] && cid[i][1] >= amount[i]){
            if(diff == amount[i] && cid[i][1] == amount[i]){
                changeDue.innerText = statusClosed;
                diff = (diff - amount[i]).toFixed(2);
                cid[i][1] = (cid[i][1] - amount[i]).toFixed(2);
                count++;
            }
            else{
                diff = (diff - amount[i]).toFixed(2);
                cid[i][1] = (cid[i][1] - amount[i]).toFixed(2);
                count++;
            }
        }
        if(count > 0){
            if(sum < diff || i == (amount.length - 1) && diff > 0.00){
                changeDue.innerText = statusIns;
                return;
            }
            else{
                changeDue.innerText += `\n${cid[i][0]}: $${amount[i] * count}`;
            }
        }
    }
}

This is my entire code (without CSS). Hopefully someone can help me with that. Thanks in advance! :slight_smile:

2

put this before the closing body tag

also it makes more sense and it’s more ordered if you have all the code in the same file

how to try if your code can survive the tests:

make price and cid here

let price = 0;
let cid = [];

and instead go at the end of your code, after everything, and write there the values for cid and price

price = 3.26;
cid = [
        ["PENNY", 1.01],
        ["NICKEL", 2.05],
        ["DIME", 3.1],
        ["QUARTER", 4.25],
        ["ONE", 90],
        ["FIVE", 55],
        ["TEN", 20],
        ["TWENTY", 60],
        ["ONE HUNDRED", 100]
      ];

if it works well, you are good.

3

Thank you very much, I’ve been on this for about 8 hours now and the final solution appears to do the initialization at the very end.

Really glad it worked, thank you!

4

I think this piece of JS code is really useful for my project.
Idioms Thanks! I found it useful.
Emily

1 Like
5

I’m glad it was helpful for you :slight_smile:

6

Hey there, i hope your are well. I am working on the cash register project as well but cannot pass the last 2 test cases. Were you able to pass all the text cases with your code? I am partially convinced the issue is with the test case since I am getting the same output as the one describes in the test case but the test case still fails :frowning: I just wanted to see of someone else has passed all the test case

7

Hey there! To answer your question: yes, I have passed all the test cases with this code, although I didn’t at first. The change that made my code suitable was to declare the list of values before the function which calculates the output and initialize it afterwards. If you do not set the values before the function, the tests can assert their own values and it should work :slight_smile:
Hope that might help you!

8

Hello there. Thanks for the reply. I am still stuck with the same issue. I have created a new topic for it. If you have the time please check it out. Thanks