-
Notifications
You must be signed in to change notification settings - Fork 0
Expand file tree
/
Copy pathformat-time.js
More file actions
49 lines (41 loc) · 1.74 KB
/
format-time.js
File metadata and controls
49 lines (41 loc) · 1.74 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
// This is the latest solution to the problem from the prep.
// Make sure to do the prep before you do the coursework
// Your task is to write tests for as many different groups of input data or edge cases as you can, and fix any bugs you find.
function formatAs12HourClock(time) {
const hours = Number(time.slice(0, 2));
if (hours > 12) {
return `${hours - 12}:00 pm`;
}
return `${time} am`;
}
const currentOutput = formatAs12HourClock("08:00");
const targetOutput = "08:00 am";
console.assert(
currentOutput === targetOutput,
`current output: ${currentOutput}, target output: ${targetOutput}`
);
const currentOutput2 = formatAs12HourClock("23:00");
const targetOutput2 = "11:00 pm";
console.assert(
currentOutput2 === targetOutput2,
`current output: ${currentOutput2}, target output: ${targetOutput2}`
);
// Test of existing fuction
console.log("=== TESTING BUGGY FUNCTION ===");
console.log(formatAs12HourClock("00:00"), "→ Expected: 12:00 am ❌"); // Bug: midnight
console.log(formatAs12HourClock("12:00"), "→ Expected: 12:00 pm ❌"); // Bug: noon
console.log(formatAs12HourClock("14:30"), "→ Expected: 02:30 pm ❌"); // Bug: minutes lost
console.log(formatAs12HourClock("08:15"), "→ Expected: 08:15 am ❌"); // Bug: minutes lost
console.log(formatAs12HourClock("23:59"), "→ Expected: 11:59 pm ❌"); // Bug: minutes lost
// Fixed function
function formatAs12HourClockFixed(time) {
const hours = Number(time.slice(0, 2));
const minutes = time.slice(3, 5);
if (hours === 0) return `12:${minutes} am`; // Midnight
if (hours === 12) return `12:${minutes} pm`; // Noon
if (hours > 12) {
const h = (hours - 12).toString().padStart(2, "0");
return `${h}:${minutes} pm`;
}
return `${time.slice(0, 5)} am`;
}