amirho/react.dev
1
0
Fork 0
mirror of https://github.com/reactjs/react.dev.git synced 2026-02-24 04:33:10 +00:00
Code Issues Packages Projects Releases Wiki Activity

stricter wording

Browse Source
This commit is contained in:
Dan Abramov
2022-09-15 06:19:35 +01:00
parent 176b93c964
commit 8e422e03e9
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
beta/src/content/apis/react/useCallback.md
View File

@@ -706,14 +706,14 @@ function ChatRoom({ roomId }) {
// ...
```
This ensures that the `createOptions` function is the same between re-renders if the `roomId` is the same. **However, it's even better to remove the need for a function dependency** by pulling the function *inside* the Effect:
This ensures that the `createOptions` function is the same between re-renders if the `roomId` is the same. **However, it's even better to remove the need for a function dependency.** Move your function *inside* the Effect:
```js {5-10,16}
function ChatRoom({ roomId }) {
const [message, setMessage] = useState('');
useEffect(() => {
function createOptions() {
function createOptions() { // No need for useCallback or function dependencies!
return {
serverUrl: 'https://localhost:1234',
roomId: roomId
@@ -728,7 +728,7 @@ function ChatRoom({ roomId }) {
// ...
```
[Read more about removing unnecessary Effect dependencies.](/learn/removing-effect-dependencies)
**When possible, avoid function dependencies.** [Read more about removing unnecessary Effect dependencies.](/learn/removing-effect-dependencies#move-dynamic-objects-and-functions-inside-your-effect)
---