Skip to content

Commit c7435e8

Browse files
committed
feat(JS) refactor canUserAddDot(buttonValue)
1 parent 9a55705 commit c7435e8

File tree

1 file changed

+22
-20
lines changed

1 file changed

+22
-20
lines changed

index.js

Lines changed: 22 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -40,9 +40,9 @@ btns.forEach((btn) => {
4040
toggleSign();
4141
}
4242

43-
if (buttonValue === ".") {
44-
canUserAddDot();
45-
}
43+
44+
canUserAddDot(buttonValue);
45+
4646

4747
if (buttonValue === "=") {
4848
try {
@@ -119,26 +119,28 @@ function deteLastEntry() {
119119
}
120120
}
121121

122-
function canUserAddDot() {
123-
var dotAllowed = true;
124-
for (var i = data.length - 1; i >= 0; i--) {
125-
console.log("data > " + data[i]);
126-
if (data[i] === ".") {
127-
dotAllowed = false;
128-
break;
129-
} else if (/[\/*\-+]/.test(data[i])) {
130-
break;
122+
function canUserAddDot(button) {
123+
if(button === ".") {
124+
var dotAllowed = true;
125+
for (var i = data.length - 1; i >= 0; i--) {
126+
console.log("data > " + data[i]);
127+
if (data[i] === ".") {
128+
dotAllowed = false;
129+
break;
130+
} else if (/[\/*\-+]/.test(data[i])) {
131+
break;
132+
}
131133
}
132-
}
133-
if (dotAllowed) {
134-
if (data.length == 0) {
135-
data.push("0");
136-
} else if (/[\/*\-+]/.test(data[data.length - 1])) {
137-
data.push("0");
134+
if (dotAllowed) {
135+
if (data.length == 0) {
136+
data.push("0");
137+
} else if (/[\/*\-+]/.test(data[data.length - 1])) {
138+
data.push("0");
139+
}
140+
data.push(".");
138141
}
139-
data.push(".");
142+
screen.innerText = data.join(" ");
140143
}
141-
screen.innerText = data.join(" ");
142144
}
143145

144146
function deleteEverythingFromScreen(button) {

0 commit comments

Comments
 (0)