Skip to content

Commit 18941d0

Browse files
committed
fix: Remove unused variables and dead code across codebase
Remove unused vars, dead functions (sib, dfs, handleRefresh), unused imports, and unused props to resolve all ESLint no-unused-vars warnings. Also fix setState-in-effect, state mutation, and hoisting lint errors.
1 parent 23799cb commit 18941d0

19 files changed

Lines changed: 56 additions & 116 deletions

File tree

src/app/15-puzzle/page.jsx

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,6 @@ class Puzzle extends Component {
3434
}
3535

3636
render() {
37-
let classNames;
3837
return (
3938

4039
<div style={{ backgroundColor: "#57407c" }}

src/app/components/footer.jsx

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,8 @@
11
'use client'
22

3-
import { useEffect, useState } from 'react';
43
import GitHubButton from 'react-github-btn';
54

65
export default function Footer() {
7-
const [visitors, setVisitors] = useState(0)
8-
9-
useEffect(() => {
10-
// Simulate fetching visitor count
11-
// In a real application, you would fetch this from an API
12-
setVisitors(Math.floor(Math.random() * 10000))
13-
}, [])
146

157
return (
168
<footer className="bg-background py-6 px-6 mt-12 border-t">

src/app/convex-hull/canvas.jsx

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -11,16 +11,6 @@ export default function Canvas({ width, height, dots, onTurnOff, onGoing, speed
1111
useEffect(() => { onGoingRef.current = onGoing; }, [onGoing]);
1212
useEffect(() => { speedRef.current = speed; }, [speed]);
1313

14-
useEffect(() => {
15-
redrawDots();
16-
}, [dots]);
17-
18-
useEffect(() => {
19-
if (onGoing) {
20-
animateLine();
21-
}
22-
}, [onGoing]);
23-
2414
const redrawDots = () => {
2515
const canvas = dotsCanvasRef.current;
2616
if (!canvas) return;
@@ -111,6 +101,16 @@ export default function Canvas({ width, height, dots, onTurnOff, onGoing, speed
111101
}
112102
};
113103

104+
useEffect(() => {
105+
redrawDots();
106+
}, [dots]);
107+
108+
useEffect(() => {
109+
if (onGoing) {
110+
animateLine();
111+
}
112+
}, [onGoing]);
113+
114114
return (
115115
<>
116116
<canvas

src/app/game-of-life/page.jsx

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -30,15 +30,15 @@ export default function GameOfLifePage() {
3030
// this.setState({ mouseIsPressed: true });
3131
}
3232

33-
const handleMouseEnter = (row, col) => {
33+
const handleMouseEnter = (_row, _col) => {
3434
// if (this.state.mouseIsPressed === false) return;
3535
// if ((this.state.startNode.row !== row || this.state.startNode.col !== col) && (this.state.endNode.row !== row || this.state.endNode.col !== col)) {
3636
// const newGrid = getNewGridWithWallToggled(this.state.grid, row, col);
3737
// this.setState({ grid: newGrid });
3838
// }
3939
}
4040

41-
const handleMouseUp = (row, col) => {
41+
const handleMouseUp = (_row, _col) => {
4242
// this.setState({ mouseIsPressed: false });
4343
}
4444

@@ -52,7 +52,6 @@ export default function GameOfLifePage() {
5252
const handleStop = () => {
5353
setRunning(false);
5454
runningRef.current = false;
55-
console.log("Simulation stopped");
5655
}
5756

5857
const handleClearBoard = () => {

src/app/n-queen/page.jsx

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ import Navbar from '@/components/navbar';
66
import Menu from "./menu";
77

88
export default function Queen() {
9-
const [board, setBoard] = useState([]);
9+
const [board, setBoard] = useState(() => getBoard(4));
1010
const [number, setNumber] = useState(4);
1111
const [speed, setSpeed] = useState(490);
1212
const [isRunning, setIsRunning] = useState(false);
@@ -17,10 +17,6 @@ export default function Queen() {
1717
useEffect(() => { numberRef.current = number; }, [number]);
1818
useEffect(() => { speedRef.current = speed; }, [speed]);
1919

20-
useEffect(() => {
21-
setBoard(getBoard(4));
22-
}, []);
23-
2420
const handleSpeedChange = (val) => {
2521
setSpeed((100 - val) * 10);
2622
};

src/app/pathfinder/page.jsx

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -74,18 +74,18 @@ export default function Pathfinder() {
7474
default:
7575
pairs = getMaze(gridCopy, row, col);
7676
}
77-
const currentGrid = grid;
77+
let currentGrid = grid.map(row => [...row]);
7878
for (let i = 0; i < pairs.length; i++) {
7979
const { xx, yy } = pairs[i];
8080
if ((xx !== startNode.row || yy !== startNode.col) && (xx !== endNode.row || yy !== endNode.col)) {
8181
currentGrid[xx][yy] = { ...currentGrid[xx][yy], isWall: true };
82-
setGrid([...currentGrid]);
82+
setGrid(currentGrid.map(row => [...row]));
8383
}
8484
await sleep(20);
8585
}
8686
currentGrid[startNode.row][startNode.col] = { ...currentGrid[startNode.row][startNode.col], isWall: false };
8787
currentGrid[endNode.row][endNode.col] = { ...currentGrid[endNode.row][endNode.col], isWall: false };
88-
setGrid([...currentGrid]);
88+
setGrid(currentGrid.map(row => [...row]));
8989
setIsRunning(false);
9090
};
9191

@@ -126,23 +126,23 @@ export default function Pathfinder() {
126126
};
127127

128128
const animateDijkstra = async (visitedNodesInOrder, nodesInShortestPathOrder) => {
129+
let currentGrid = grid.map(row => [...row]);
129130
for (let i = 0; i < visitedNodesInOrder.length; i++) {
130131
const node = visitedNodesInOrder[i];
131-
const currentGrid = grid;
132132
currentGrid[node.row][node.col] = { ...currentGrid[node.row][node.col], isVisited: true };
133-
setGrid([...currentGrid]);
133+
setGrid(currentGrid.map(row => [...row]));
134134
await sleep(10);
135135
}
136136
await sleep(100);
137137
await animateShortestPath(nodesInShortestPathOrder);
138138
};
139139

140140
const animateShortestPath = async (nodesInShortestPathOrder) => {
141+
let currentGrid = grid.map(row => [...row]);
141142
for (let i = 0; i < nodesInShortestPathOrder.length; i++) {
142143
const node = nodesInShortestPathOrder[i];
143-
const currentGrid = grid;
144144
currentGrid[node.row][node.col] = { ...currentGrid[node.row][node.col], ispathNode: true };
145-
setGrid([...currentGrid]);
145+
setGrid(currentGrid.map(row => [...row]));
146146
await sleep(50);
147147
}
148148
setIsRunning(false);

src/app/prime-numbers/page.jsx

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ import Spiral from "./spiral";
99

1010
export default function Seive() {
1111
const [number, setNumber] = useState(100);
12-
const [cells, setCells] = useState([]);
12+
const [cells, setCells] = useState(() => getCells(100));
1313
const [isRunning, setIsRunning] = useState(false);
1414
const [speed, setSpeed] = useState(500);
1515
const [primes, setPrimes] = useState([]);
@@ -22,10 +22,6 @@ export default function Seive() {
2222
useEffect(() => { speedRef.current = speed; }, [speed]);
2323
useEffect(() => { numberRef.current = number; }, [number]);
2424

25-
useEffect(() => {
26-
setCells(getCells(100));
27-
}, []);
28-
2925
const changeSpeed = (val) => {
3026
setSpeed(600 - val * 10);
3127
};
@@ -109,7 +105,7 @@ export default function Seive() {
109105
{algo === 0 && <Cells cells={cells} />}
110106
{algo === 1 && (
111107
<div className="h-full w-full justify-center bg-gray-700">
112-
<Spiral num={number} primes={primes} maxPrime={maxPrime} />
108+
<Spiral primes={primes} maxPrime={maxPrime} />
113109
</div>
114110
)}
115111
</div>

src/app/prime-numbers/spiral.jsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
export default function Spiral({ num, primes, maxPrime }) {
1+
export default function Spiral({ primes, maxPrime }) {
22
let Maxwidth = window.innerWidth;
33
let MaxHeight = window.innerHeight;
44

src/app/recursion-tree/Tree.js

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -113,8 +113,6 @@ function firstwalk(v, distance = 1) {
113113

114114
let midpoint = (v.children[0].x + v.children[v.children.length-1].x) / 2;
115115

116-
let ell = v.children[0];
117-
let arr = v.children[-1];
118116
let w = v.lbrother();
119117
if (w) {
120118
v.x = w.x + distance;
@@ -221,12 +219,6 @@ function second_walk(v, m = 0, depth = 0, min = undefined) {
221219
return min;
222220
}
223221

224-
function dfs(tree){
225-
if( tree.tree.node === 'B' ) return;
226-
console.log( tree.tree.node, tree.x,tree.y );
227-
for(let i=0;i<tree.children.length;i++)
228-
dfs(tree.children[i]);
229-
}
230222
/*
231223
let blank = new Tree('B',[]);
232224
let ll = new Tree("ll",[]);

src/app/recursion-tree/fib.jsx

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -21,15 +21,6 @@ function fib(n){
2121
return tree;
2222
}
2323

24-
function sib(n){
25-
let tree = new Tree(n,[]);
26-
if( n <3 ) return tree;
27-
tree.children.push( sib(n-2) );
28-
tree.children.push( sib(n-3) );
29-
tree.node = tree.children[0].node+tree.children[1].node;
30-
return tree;
31-
}
32-
3324
function NcR(n,r){
3425
if (r > n)
3526
return new Tree(-1,[],"("+n+","+r+")");

0 commit comments

Comments
 (0)