Skip to content

Commit 259c3ec

Browse files
committed
Validation added.
If user enters an empty array, they will be welcomed by a pop up which tells them to enter a value with spaces.
1 parent c60a55e commit 259c3ec

File tree

1 file changed

+28
-25
lines changed

1 file changed

+28
-25
lines changed

js/script.js

Lines changed: 28 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -33,37 +33,40 @@ function myExcelFuns() {
3333
var numberArr = numberStr.split(" ");
3434
var result = 0;
3535

36-
if (document.getElementById("arrSum").checked) {
36+
if (numberArr != "") {
37+
if (document.getElementById("arrSum").checked) {
3738

38-
for (var i = 0; i < numberArr.length; i++)
39-
if (numberArr[i] != "") {
40-
result = result + parseFloat(numberArr[i]);
41-
}
42-
} else if (document.getElementById("arrAvg").checked) {
39+
for (var i = 0; i < numberArr.length; i++)
40+
if (numberArr[i] != "") {
41+
result = result + parseFloat(numberArr[i]);
42+
}
43+
} else if (document.getElementById("arrAvg").checked) {
4344

44-
for (var i = 0; i < numberArr.length; i++)
45-
if (numberArr[i] != "") {
46-
result = result + parseFloat(numberArr[i]);
47-
}
48-
result = result / numberArr.length;
49-
} else if (document.getElementById("arrMax").checked) {
45+
for (var i = 0; i < numberArr.length; i++)
46+
if (numberArr[i] != "") {
47+
result = result + parseFloat(numberArr[i]);
48+
}
49+
result = result / numberArr.length;
50+
} else if (document.getElementById("arrMax").checked) {
5051

51-
for (var i = 0; i < numberArr.length; i++)
52-
if (numberArr[i] != "") {
53-
result = Math.max(...numberArr);
54-
}
55-
} else if (document.getElementById("arrMin").checked) {
52+
for (var i = 0; i < numberArr.length; i++)
53+
if (numberArr[i] != "") {
54+
result = Math.max(...numberArr);
55+
}
56+
} else if (document.getElementById("arrMin").checked) {
5657

57-
for (var i = 0; i < numberArr.length; i++)
58-
if (numberArr[i] != "") {
59-
result = Math.min(...numberArr);
60-
}
61-
} else {
58+
for (var i = 0; i < numberArr.length; i++)
59+
if (numberArr[i] != "") {
60+
result = Math.min(...numberArr);
61+
}
62+
} else {
6263

63-
return 0;
64+
return 0;
65+
}
66+
document.getElementById("result").innerHTML = result;
67+
} else {
68+
alert("Please enter a numeric value with spaces to proceed.");
6469
}
65-
document.getElementById("result").innerHTML = result;
66-
6770
return false;
6871
};
6972

0 commit comments

Comments
 (0)