Should not have unused variables JS-0128
Bug risk
Major
a year ago
—
a year old
'deleteRecord' is defined but never used. Allowed unused vars must match /^_/u
199 table.tBodies[0].innerHTML = '';
200}
201
202function deleteRecord(id){203 colRecords.doc(id).delete().then(() => {
204 console.log("Document successfully deleted!");
205 updateRecords();'reload' is assigned a value but never used. Allowed unused vars must match /^_/u
12const app = firebase.initializeApp(firebaseConfig);
13const db = app.firestore();
14let colRecords = db.collection("records");
15let reload = false; 16
17
18$(document).ready(() => {'r' is defined but never used. Allowed unused args must match /^_/u
16let totDebit = 0;
17let totCredit = 0;
18
19loadRecords().then(r => addTotalRow());20
21
22'OtherReceivables' is assigned a value but never used. Allowed unused vars must match /^_/u
32const Property = document.getElementById("Property");
33const Allowance = document.getElementById("Allowance");
34const Value = document.getElementById("Value");
35const OtherReceivables = document.getElementById("OtherReceivables"); 36const Goodwill = document.getElementById("Goodwill");
37const WebsiteAndDomains = document.getElementById("WebsiteAndDomains");
38const OtherIntangibleAssets = document.getElementById("OtherIntangibleAssets");'cash' is assigned a value but never used. Allowed unused vars must match /^_/u
13const db = app.firestore();
14let colRecords = db.collection("records");
15
16const cash = document.getElementById("Cash"); 17const ShortInvestments = document.getElementById("ShortInvestments");
18const AccountsReceivable = document.getElementById("AccountsReceivable");
19const PrepaidRent = document.getElementById("PrepaidRent");
Description
Found variables that are declared but not used anywhere.
NOTE: In browser applications, DeepSource recommends the use of ESModules over regular
text/javascriptscripts. Currently, we don't support variables that are not explicitly exported, and are injected into other scripts by being included in an HTML file
Unused variables are most often the result of incomplete refactoring. They can lead to confusing code and minor performance hitches.
NOTE: If you have intentionally left a variable unused, we suggest you to prefix the variable name with a _ to prevent them from being flagged by DeepSource.
Bad Practice
// Write-only variables are not considered as used.
let y = 10;
y = 5;
// A variable that modifies only itself isn't considered used.
let z = 0;
z = z + 1;
// Unused argument
(function(x) {
return 5;
})();
// Unused recursive functions also raise this issue.
function fact(n) {
if (n < 2) return 1;
return n * fact(n - 1);
}
// When a function definition destructures an array,
// unused entries from the array also cause warnings.
function getY([x, y]) {
return y;
}
Recommended
let x = 10;
alert(x);
((arg1) => {
return arg1;
})();
let myFunc;
myFunc = (n) => {
// this is legal
if (n < 0) myFunc();
};
// this is also considered legal
console.log(declaredLater);
var declaredLater;
// Only the second argument from the descructured array is used.
function getY([, y]) {
return y;
}