Use
const declarations for variables that are never reassigned JS-0242
Anti-pattern
Minor
a year ago
—
a year old
'seconds' is never reassigned. Use 'const' instead.
37 let days = Math.floor(client.uptime / 86400000)
38 let hours = Math.floor(client.uptime / 3600000) % 24
39 let minutes = Math.floor(client.uptime / 60000) % 60
40 let seconds = Math.floor(client.uptime / 1000) % 6041
42 // Latency Check
43 let webLatency = new Date() - interaction.createdAt'minutes' is never reassigned. Use 'const' instead.
36 // Function Uptime
37 let days = Math.floor(client.uptime / 86400000)
38 let hours = Math.floor(client.uptime / 3600000) % 24
39 let minutes = Math.floor(client.uptime / 60000) % 6040 let seconds = Math.floor(client.uptime / 1000) % 60
41
42 // Latency Check'hours' is never reassigned. Use 'const' instead.
35 try {
36 // Function Uptime
37 let days = Math.floor(client.uptime / 86400000)
38 let hours = Math.floor(client.uptime / 3600000) % 2439 let minutes = Math.floor(client.uptime / 60000) % 60
40 let seconds = Math.floor(client.uptime / 1000) % 60
41 'days' is never reassigned. Use 'const' instead.
34 async execute(client, interaction) {
35 try {
36 // Function Uptime
37 let days = Math.floor(client.uptime / 86400000)38 let hours = Math.floor(client.uptime / 3600000) % 24
39 let minutes = Math.floor(client.uptime / 60000) % 60
40 let seconds = Math.floor(client.uptime / 1000) % 60'apiLatency' is never reassigned. Use 'const' instead.
41
42 // Latency Check
43 let webLatency = new Date() - interaction.createdAt
44 let apiLatency = client.ws.ping45 let totalLatency = webLatency + apiLatency
46
47 // Emoji
Description
Variables that are never re-assigned a new value after their initial declaration should be declared with the const keyword.
This prevents the programmer from erroneously re-assigning to a read-only variable, and informs those reading the code that a variable is a constant value.
Bad Practice
let pi = Math.PI
for (let x of xs) {
use(x);
}
let { a, b } = object;
use(a, b);
Recommended
const pi = Math.PI
for (const x of xs) {
use(x);
}
const { a, b } = object;
use(a, b);