Should not have unused variables JS-0128
Bug risk
4 months ago
—
5 months old
'validateLogin' is defined but never used. Allowed unused vars must match /^_/u
5 * it isn't, it adds the class "is-valid" to the input.
6 * @param input - The input element that is being validated.
7 */
8function validateLogin(input) { 9 let url = "/user/" + input.value + "/login";
10 $.get(url, function(data){
11 if (data === "True") {'confirmPassword' is defined but never used. Allowed unused vars must match /^_/u
76 * </code>
77 * @param input - The input element that is being validated.
78 */
79function confirmPassword(input) { 80 if (input.value !== $("#password").val()) {
81 $("#confirm_password").addClass("is-invalid");
82 $("#confirm_password").removeClass("is-valid");'validateEmail' is defined but never used. Allowed unused vars must match /^_/u
27 * "is-invalid" to the input, if it isn't, it adds the class "is-valid" to the input.
28 * @param input - The input element that is being validated.
29 */
30function validateEmail(input) { 31 let url = "/user/" + input.value + "/email";
32 $.get(url, function(data){
33 if (data === "True") {'validateCpfCnpj' is defined but never used. Allowed unused vars must match /^_/u
95 * the input, otherwise it adds the class "is-invalid" to the input.
96 * @param input - The input element that is being validated.
97 */
98function validateCpfCnpj(input) { 99 if (input.value.length === 14) {
100 if (validateCpf(input.value)) {
101 $("#cpfCnpj").addClass("is-valid");'validatePassword' is defined but never used. Allowed unused vars must match /^_/u
53 * after the password input, and set the custom validity of the password input to an empty string.
54 * @param input - The input element that is being validated.
55 */
56function validatePassword(input) { 57 if (input.value.length < 8) {
58 $("#password").addClass("is-invalid");
59 $("#password").removeClass("is-valid");
Description
Found variables that are declared but not used anywhere.
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.
var y = 10;
y = 5;
// A variable that modifies only itself isn't considered used.
var 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
var 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;
}