'info' is never reassigned. Use 'const' instead
39 pass: process.env.EMAIL_SERVER_PASSWORD,
40 },
41 });
42 let info = await transporter.sendMail({43 from: `"The-Caverns" <${process.env.EMAIL_FROM}>`,
44 to: body.email,
45 subject: "Sign up for The Caverns",
'transporter' is never reassigned. Use 'const' instead
30 ex: 86400
31 });
32 // Send email
33 let transporter = nodemailer.createTransport({34 host: "mail.ee",
35 port: 465,
36 secure: true,
'magicNumber' is never reassigned. Use 'const' instead
36 loadImages();
37 }
38 useEffect(() => {
39 let magicNumber = 3000;40 let interval = setInterval(() => {
41 setTime((new Date()).getTime());
42 if (((Math.floor(time / magicNumber) * magicNumber) % magicNumber) == 0) {
'interval' is never reassigned. Use 'const' instead
37 }
38 useEffect(() => {
39 let magicNumber = 3000;
40 let interval = setInterval(() => {41 setTime((new Date()).getTime());
42 if (((Math.floor(time / magicNumber) * magicNumber) % magicNumber) == 0) {
43 forward();
'imageSrcs' is never reassigned. Use 'const' instead
3
4export default function ImageCarousel(props) {
5 const [index, setIndex] = useState(props.index ? props.index : 0);
6 let imageSrcs = props.images ? props.images.split(";") : []; 7 const [images, setImages] = useState([]);
8 const [time, setTime] = useState((new Date()).getTime());
9 const loadImages = () => {
'l' is never reassigned. Use 'const' instead
7 const [images, setImages] = useState([]);
8 const [time, setTime] = useState((new Date()).getTime());
9 const loadImages = () => {
10 let l = [];11 imageSrcs.forEach((imageSrc, i) => {
12 l.push(
13 <img className={`flex-1 rounded w-auto h-auto ml-auto mr-auto max-w-sm md:max-w-md lg:max-w-lg xl:max-w-xl 2xl:max-w-2xl ${(index == i) ? styles.imageCarouselActiveImage : styles.imageCarouselInactiveImage}`} src={imageSrc} key={i}/>
Description
Variables that are never re-assigned a new value after their initial declaration should be declared with the const
keyword.
This prevents the programmer from erroneously re-assigning to a read-only variable, and informs those reading the code that a variable is a constant value.
Bad Practice
let pi = Math.PI
for (let x of xs) {
use(x);
}
let { a, b } = object;
use(a, b);
Recommended
const pi = Math.PI
for (const x of xs) {
use(x);
}
const { a, b } = object;
use(a, b);