Use
const declarations for variables that are never reassigned JS-0242
Anti-pattern
Minor
5 months ago
—
5 months old
'newMail' is never reassigned. Use 'const' instead
109 showDeleteModal.value = true;
110}
111
112let newMail = (item: any) => {113 userToNewMail.value = item;
114 showNewLinkModal.value = true;
115}'deleteUser' is never reassigned. Use 'const' instead
104let userToDelete = ref({});
105let userToNewMail = ref({});
106
107let deleteUser = (item: any) => {108 userToDelete.value = item;
109 showDeleteModal.value = true;
110}'userToNewMail' is never reassigned. Use 'const' instead
102let showUserCreationModal = ref(false);
103
104let userToDelete = ref({});
105let userToNewMail = ref({});106
107let deleteUser = (item: any) => {
108 userToDelete.value = item;'userToDelete' is never reassigned. Use 'const' instead
101let showNewLinkModal = ref(false);
102let showUserCreationModal = ref(false);
103
104let userToDelete = ref({});105let userToNewMail = ref({});
106
107let deleteUser = (item: any) => {'showUserCreationModal' is never reassigned. Use 'const' instead
99let showEditModal = ref(false);
100let showDeleteModal = ref(false);
101let showNewLinkModal = ref(false);
102let showUserCreationModal = ref(false);103
104let userToDelete = ref({});
105let userToNewMail = ref({});
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);