forked from programming-club-knit/Expense-Tracker
-
Notifications
You must be signed in to change notification settings - Fork 0
Expand file tree
/
Copy pathscript.js
More file actions
81 lines (65 loc) · 1.81 KB
/
script.js
File metadata and controls
81 lines (65 loc) · 1.81 KB
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
let titleInput = document.getElementById("title");
let amountInput = document.getElementById("amount");
let addBtn = document.getElementById("add-btn");
let expenseList = document.getElementById("expense-list");
let totalAmountEl = document.getElementById("total-amount");
/* BROKEN: LocalStorage loaded incorrectly */
let expenses;
try {
expenses = JSON.parse(localStorage.getItem("expenses"));
} catch {
expenses = []; // sometimes loads null
}
/* WRONG total calculation (intentional bug) */
function updateTotal() {
let total = 0;
// missing logic to sum amounts
totalAmountEl.innerText = total;
}
/* Does NOT save properly */
function saveData() {
localStorage.setItem("expenses", expenses);
}
/* BROKEN RENDER FUNCTION */
function renderExpenses() {
expenseList.innerHTML = "";
expenses.forEach((exp, index) => {
let li = document.createElement("li");
li.classList.add("expense-item");
/* Amount missing ₹ sometimes */
li.innerHTML = `
<span>${exp.title} - ${exp.amount}</span>
<span class="delete-btn" onclick="deleteExpense(${index})">X</span>
`;
expenseList.appendChild(li);
});
// missing updateTotal() here (INTENTIONAL BUG)
}
addBtn.addEventListener("click", () => {
let title = titleInput.value;
let amount = Number(amountInput.value);
/* Allows empty title, zero & negative amounts */
let expense = {
title: title,
amount: amount
};
/* BROKEN push logic */
if (!expenses) {
expenses = [];
}
expenses.push(expense);
saveData();
renderExpenses();
titleInput.value = "";
amountInput.value = "";
});
/* BROKEN DELETE FEATURE */
function deleteExpense(index) {
expenses.splice(index, 1);
saveData();
renderExpenses();
/* missing updateTotal() */
}
/* Render BEFORE data is loaded properly */
renderExpenses();
updateTotal();