Skip to content

Commit e901790

Browse files
authored
fix: use const where applicable in examples for keeping components pure (#7819)
1 parent 06965de commit e901790

File tree

1 file changed

+13
-13
lines changed

1 file changed

+13
-13
lines changed

src/content/learn/keeping-components-pure.md

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -175,7 +175,7 @@ function Cup({ guest }) {
175175
}
176176

177177
export default function TeaGathering() {
178-
let cups = [];
178+
const cups = [];
179179
for (let i = 1; i <= 12; i++) {
180180
cups.push(<Cup key={i} guest={i} />);
181181
}
@@ -245,7 +245,7 @@ Rendering is a *calculation*, it shouldn't try to "do" things. Can you express t
245245

246246
```js src/Clock.js active
247247
export default function Clock({ time }) {
248-
let hours = time.getHours();
248+
const hours = time.getHours();
249249
if (hours >= 0 && hours <= 6) {
250250
document.getElementById('time').className = 'night';
251251
} else {
@@ -307,7 +307,7 @@ You can fix this component by calculating the `className` and including it in th
307307

308308
```js src/Clock.js active
309309
export default function Clock({ time }) {
310-
let hours = time.getHours();
310+
const hours = time.getHours();
311311
let className;
312312
if (hours >= 0 && hours <= 6) {
313313
className = 'night';
@@ -606,14 +606,14 @@ export default function StoryTray({ stories }) {
606606
import { useState, useEffect } from 'react';
607607
import StoryTray from './StoryTray.js';
608608

609-
let initialStories = [
609+
const initialStories = [
610610
{id: 0, label: "Ankit's Story" },
611611
{id: 1, label: "Taylor's Story" },
612612
];
613613

614614
export default function App() {
615-
let [stories, setStories] = useState([...initialStories])
616-
let time = useTime();
615+
const [stories, setStories] = useState([...initialStories])
616+
const time = useTime();
617617

618618
// HACK: Prevent the memory from growing forever while you read docs.
619619
// We're breaking our own rules here.
@@ -702,14 +702,14 @@ export default function StoryTray({ stories }) {
702702
import { useState, useEffect } from 'react';
703703
import StoryTray from './StoryTray.js';
704704

705-
let initialStories = [
705+
const initialStories = [
706706
{id: 0, label: "Ankit's Story" },
707707
{id: 1, label: "Taylor's Story" },
708708
];
709709

710710
export default function App() {
711-
let [stories, setStories] = useState([...initialStories])
712-
let time = useTime();
711+
const [stories, setStories] = useState([...initialStories])
712+
const time = useTime();
713713

714714
// HACK: Prevent the memory from growing forever while you read docs.
715715
// We're breaking our own rules here.
@@ -770,7 +770,7 @@ Alternatively, you could create a _new_ array (by copying the existing one) befo
770770
```js src/StoryTray.js active
771771
export default function StoryTray({ stories }) {
772772
// Copy the array!
773-
let storiesToDisplay = stories.slice();
773+
const storiesToDisplay = stories.slice();
774774

775775
// Does not affect the original array:
776776
storiesToDisplay.push({
@@ -794,14 +794,14 @@ export default function StoryTray({ stories }) {
794794
import { useState, useEffect } from 'react';
795795
import StoryTray from './StoryTray.js';
796796

797-
let initialStories = [
797+
const initialStories = [
798798
{id: 0, label: "Ankit's Story" },
799799
{id: 1, label: "Taylor's Story" },
800800
];
801801

802802
export default function App() {
803-
let [stories, setStories] = useState([...initialStories])
804-
let time = useTime();
803+
const [stories, setStories] = useState([...initialStories])
804+
const time = useTime();
805805

806806
// HACK: Prevent the memory from growing forever while you read docs.
807807
// We're breaking our own rules here.

0 commit comments

Comments
 (0)