From af56e3b4de815d618e267fc9ab1fd860a760ea59 Mon Sep 17 00:00:00 2001 From: Sy Traore Date: Wed, 23 Jul 2025 14:45:20 -0400 Subject: [PATCH] Refactor variable declarations to use const Replaced 'let' with 'const' for variable declarations in server.js, animation.jsx, animationNoCircleDraw.jsx, and trayGenerators.js where variables are not reassigned. This improves code clarity and enforces immutability where appropriate. --- apps/backend/src/server.js | 2 +- apps/frontend/src/components/animation.jsx | 8 ++++---- apps/frontend/src/components/animationNoCircleDraw.jsx | 8 ++++---- apps/frontend/src/helpers/trayGenerators.js | 2 +- 4 files changed, 10 insertions(+), 10 deletions(-) diff --git a/apps/backend/src/server.js b/apps/backend/src/server.js index 6ecdd05..991ce4d 100644 --- a/apps/backend/src/server.js +++ b/apps/backend/src/server.js @@ -132,7 +132,7 @@ app.post("/register", async (req, res) => { const user = jwt.verify(token, JWT_SECRET); const username = user.name; - let updateField = {}; + const updateField = {}; updateField[`answers.${pageType}`] = answers; const updatedUser = await User.findOneAndUpdate( diff --git a/apps/frontend/src/components/animation.jsx b/apps/frontend/src/components/animation.jsx index b6ce339..b67fc87 100644 --- a/apps/frontend/src/components/animation.jsx +++ b/apps/frontend/src/components/animation.jsx @@ -27,10 +27,10 @@ function Animation({ onAnimationFinish }) { // update progress ellipse based on percentage useEffect(() => { - let progressEllipse = document.querySelector('.progress'); // select element - let rx = parseFloat(progressEllipse.getAttribute('rx')); - let ry = parseFloat(progressEllipse.getAttribute('ry')); - let perimeter = Math.PI * (3 * (rx + ry) - Math.sqrt((3 * rx + ry) * (rx + 3 * ry))); // calculate + const progressEllipse = document.querySelector('.progress'); // select element + const rx = parseFloat(progressEllipse.getAttribute('rx')); + const ry = parseFloat(progressEllipse.getAttribute('ry')); + const perimeter = Math.PI * (3 * (rx + ry) - Math.sqrt((3 * rx + ry) * (rx + 3 * ry))); // calculate progressEllipse.style.strokeDasharray = perimeter; // set stroke dash array to perimeter progressEllipse.style.strokeDashoffset = perimeter - (percent * perimeter) / 100; // update offset based on percentage diff --git a/apps/frontend/src/components/animationNoCircleDraw.jsx b/apps/frontend/src/components/animationNoCircleDraw.jsx index aef1b1b..d3a89f6 100644 --- a/apps/frontend/src/components/animationNoCircleDraw.jsx +++ b/apps/frontend/src/components/animationNoCircleDraw.jsx @@ -27,10 +27,10 @@ function AnimationNoCircle({ onAnimationFinish }) { // useEffect hook to update progress ellipse based on percentage useEffect(() => { - let progressEllipse = document.querySelector('.progress'); - let rx = parseFloat(progressEllipse.getAttribute('rx')); - let ry = parseFloat(progressEllipse.getAttribute('ry')); - let perimeter = Math.PI * (3 * (rx + ry) - Math.sqrt((3 * rx + ry) * (rx + 3 * ry))); + const progressEllipse = document.querySelector('.progress'); + const rx = parseFloat(progressEllipse.getAttribute('rx')); + const ry = parseFloat(progressEllipse.getAttribute('ry')); + const perimeter = Math.PI * (3 * (rx + ry) - Math.sqrt((3 * rx + ry) * (rx + 3 * ry))); progressEllipse.style.strokeDasharray = perimeter; progressEllipse.style.strokeDashoffset = perimeter - (percent * perimeter) / 100; diff --git a/apps/frontend/src/helpers/trayGenerators.js b/apps/frontend/src/helpers/trayGenerators.js index c94b0ab..0f2b4cd 100644 --- a/apps/frontend/src/helpers/trayGenerators.js +++ b/apps/frontend/src/helpers/trayGenerators.js @@ -80,7 +80,7 @@ const cookiePositions = { // Function to generate DynamicTray export function generateDynamicTray(numCookies, trayW, trayH, cookieW, cookieH, padding, minGap) { - let placedCookies = []; + const placedCookies = []; for (let i = 0; i < numCookies; i++) { let newCookie;