'ErrorSection' is assigned a value but never used
192);
193
194// Error Section Component
195const ErrorSection = ({ error }) => <div className="error">{error}</div>;196
197export default Navbar;
'HamburgerSection' is assigned a value but never used
177);
178
179// Hamburger Section Component
180const HamburgerSection = ({ toggleMenu, menuOpen, handleKeyPress }) => (181 <div
182 className="hamburger"
183 onClick={toggleMenu}
'MenuSection' is assigned a value but never used
139);
140
141// Menu Section Component
142const MenuSection = ({ user, handleSignOut, menuOpen }) => (143 <div className={`menu ${menuOpen ? "show" : ""}`}>
144 <ul>
145 <MenuItem href="#">Top Universities</MenuItem>
'LogoSection' is assigned a value but never used
132};
133
134// Logo Component
135const LogoSection = () => (136 <div className="logo">
137 <img src={Logo} alt="Logo" />
138 </div>
'handleKeyPress' is assigned a value but never used
63 }, [setMenuOpen, menuOpen]);
64
65 // Define handleKeyPress outside of JSX
66 const handleKeyPress = useCallback((event) => { 67 if (event.key === 'Enter' || event.key === ' ') {
68 toggleMenuCallback();
69 }
'handleSignOutCallback' is assigned a value but never used
53 };
54
55 // Define callbacks using useCallback
56 const handleSignOutCallback = useCallback(() => { 57 signOutUser(navigate, setError);
58 }, [navigate, setError]);
59
'error' is assigned a value but never used
28 const navigate = useNavigate();
29 const [user, setUser] = useState(null);
30 const [menuOpen, setMenuOpen] = useState(false);
31 const [error, setError] = useState(null); 32
33
34 useEffect(() => {
'user' is assigned a value but never used
26// Navbar Component
27const Navbar = () => {
28 const navigate = useNavigate();
29 const [user, setUser] = useState(null); 30 const [menuOpen, setMenuOpen] = useState(false);
31 const [error, setError] = useState(null);
32
'setUser' is assigned a value but never used
26// Navbar Component
27const Navbar = () => {
28 const navigate = useNavigate();
29 const [user, setUser] = useState(null); 30 const [menuOpen, setMenuOpen] = useState(false);
31 const [error, setError] = useState(null);
32
'NavLink' is defined but never used
1import { useEffect, useState, useCallback } from 'react';
2import { signOut } from "firebase/auth";
3import { useNavigate, NavLink, Link } from "react-router-dom"; 4import Logo from "../../assets/logo.webp";
5
6import "./Navbar.css";
'toggleMenu' is assigned a value but never used
33
34 const [menuOpen, setMenuOpen] = useState(false);
35
36 const toggleMenu = () => { 37 setMenuOpen(!menuOpen);
38 };
39 return (
'handleSignOut' is assigned a value but never used
21 });
22 }, []);
23
24 const handleSignOut = () => { 25 signOut(auth)
26 .then(() => {
27 navigate("/");
'Logo' is defined but never used
5import Footer from "../Footer/Footer";
6import Navbar from '../Navbar/Navbar';
7import './Contact.css'
8import Logo from '../../assets/logo.webp' 9
10const Contact = () => {
11 const navigate = useNavigate();
'toggleMenu' is assigned a value but never used
44 });
45 };
46
47 const toggleMenu = () => { 48 setMenuOpen(!menuOpen);
49 };
50
'handleSignOut' is assigned a value but never used
34
35 const [menuOpen, setMenuOpen] = useState(false);
36
37 const handleSignOut = () => { 38 signOut(auth)
39 .then(() => {
40 navigate("/");
'Logo' is defined but never used
3import studentsData from './students.json';
4import './CollegePage.css';
5import { useParams, useNavigate } from 'react-router-dom';
6import Logo from '../../assets/logo.webp'; 7import { signOut } from "firebase/auth";
8import Navbar from "../Navbar/Navbar";
9import { auth } from "../../firebase/auth";
Description
Found variables that are declared but not used anywhere.
NOTE: In browser applications, DeepSource recommends the use of ESModules over regular
text/javascript
scripts. 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;
}