Skip to content

Translation of useCallback reference #423

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 14 commits into from
Aug 25, 2023
Merged

Conversation

dawidsabat
Copy link
Collaborator

No description provided.

Copy link
Member

@jakubdrozdek jakubdrozdek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dobra robota. Ja się czepiam, bo muszę i koniec 😄, ale większość błędów była raczej małego kalibru.

// ...
```

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:
To zapewnia, że funkcja `createOptions` pozostaje taka sama między przerenderowaniami, jeśli `roomId` jest taki sam. **Jednakże jeszcze lepiej jest usunąć potrzebę zależności funkcji.** Przenieś swoją funkcję do *wnętrza* efektu:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
To zapewnia, że funkcja `createOptions` pozostaje taka sama między przerenderowaniami, jeśli `roomId` jest taki sam. **Jednakże jeszcze lepiej jest usunąć potrzebę zależności funkcji.** Przenieś swoją funkcję do *wnętrza* efektu:
Taka zmiana zapewnia, że funkcja `createOptions` pozostaje taka sama między renderowaniami, jeśli `roomId` jest taki sam. **Jednakże jeszcze lepiej jest usunąć potrzebę używania funkcji jako zależności.** Przenieś swoją funkcję do *wnętrza* efektu:


Make sure you've specified the dependency array as a second argument!
Upewnij się, że podałeś tablicę zależności jako drugi argument!
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Upewnij się, że podałeś tablicę zależności jako drugi argument!
Upewnij się, że podajesz tablicę zależności jako drugi argument!

// ...
```

If this doesn't help, then the problem is that at least one of your dependencies is different from the previous render. You can debug this problem by manually logging your dependencies to the console:
Jeśli to nie pomaga, problem może wynikać z tego, że przynajmniej jedna z twoich zależności zmieniła się od poprzedniego renderowania. Możesz rozwiązać ten problem, dodając ręcznie logowanie twoich zależności do konsoli:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Jeśli to nie pomaga, problem może wynikać z tego, że przynajmniej jedna z twoich zależności zmieniła się od poprzedniego renderowania. Możesz rozwiązać ten problem, dodając ręcznie logowanie twoich zależności do konsoli:
Jeśli to nie pomaga, problem może wynikać z tego, że przynajmniej jedna z twoich zależności zmieniła się od poprzedniego renderowania. Możesz rozwiązać ten problem dodając ręcznie logowanie twoich zależności do konsoli:

```

When you find which dependency is breaking memoization, either find a way to remove it, or [memoize it as well.](/reference/react/useMemo#memoizing-a-dependency-of-another-hook)
Kiedy znajdziesz, która zależność psuje zapamiętywanie, albo znajdź sposób, aby ją usunąć, albo [również ją zapamiętaj.](/reference/react/useMemo#memoizing-a-dependency-of-another-hook)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Kiedy znajdziesz, która zależność psuje zapamiętywanie, albo znajdź sposób, aby ją usunąć, albo [również ją zapamiętaj.](/reference/react/useMemo#memoizing-a-dependency-of-another-hook)
Kiedy znajdziesz zależność psującą memoizację, znajdź sposób, aby ją usunąć albo [również ją zapamiętaj.](/reference/react/useMemo#memoizing-a-dependency-of-another-hook)

Copy link
Member

@jakubdrozdek jakubdrozdek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dobra robota. Ja się czepiam, bo muszę i koniec 😄, ale większość błędów była raczej małego kalibru.

Główna rzecz do poprawki: pisz bezosobowo lub "my". Dokumentacja powinna być bezosobowa, a zwłaszcza nie powinna "gadać" do czytelnika jak do faceta.

@github-actions
Copy link

github-actions bot commented Aug 24, 2023

Size changes

📦 Next.js Bundle Analysis for react-dev

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

@dawidsabat
Copy link
Collaborator Author

Dobra robota. Ja się czepiam, bo muszę i koniec 😄, ale większość błędów była raczej małego kalibru.

Główna rzecz do poprawki: pisz bezosobowo lub "my". Dokumentacja powinna być bezosobowa, a zwłaszcza nie powinna "gadać" do czytelnika jak do faceta.

To nie czepianie się, jeśli chcesz dobrze i masz rację. Jestem otwarty na nawet najmniejsze uwagi 😄
Zapamiętam twoje uwagi, dzięki!

@dawidsabat
Copy link
Collaborator Author

dawidsabat commented Aug 25, 2023

Przy okazji: wcześniej korzystałem z Githuba tylko jako zdalene repo, nie miałem okazji tworzyć issuesów, robić i przeglądać tu PRów itp. więc mogę za bardzo nie czuć jaki tu jest "workflow".

W związku z tym proszę o odrobinę wyrozumiałości pod tym kątem. Postaram się samemu to ogarnąć, ale też śmiało zwracaj mi na takie rzeczy uwagę ;)

@dawidsabat
Copy link
Collaborator Author

Czy dobrze rozumiem, żeby nie stosować słowa "przerenderować" i po prostu pisać "renderować"? Poprawić to wszędzie?

@ghost
Copy link

ghost commented Aug 25, 2023

Czy dobrze rozumiem, żeby nie stosować słowa "przerenderować" i po prostu pisać "renderować"? Poprawić to wszędzie?

Renderować oznacza mniej więcej tyle, że funkcja (komponent) jest uruchamiana i zwraca drzewo elementów. Przerenderować to wyrenderować ponownie z innym wynikiem (zazwyczaj).

Skojarz sobie: malować a przemalować (pokój), kopać a przekopać (ogródek). Kojarzy się to z czynnością, której zamiarem jest jakaś zmiana lub efektem jest inny stan czegoś. Przynajmniej ja tak to sobie skojarzyłem 🙂

Jednak w przypadku tego pliku, analogicznie, można coś zrobić "pomiędzy malowaniami pokojów" (np. iść na papierosa), ale trochę dziwnie brzmi "pomiędzy przemalowaniami pokojów". Bardziej zwracamy uwagę na czas i konsekwencje, a i trudniej sobie przy tej prostszej formie połamać język. Dodatkowo, komponent może być renderowany ponownie z różnych przyczyn, i nie zawsze wynika to ze zmiany własnego stanu.

@ghost
Copy link

ghost commented Aug 25, 2023

Przy okazji: wcześniej korzystałem z Githuba tylko jako zdalene repo, nie miałem okazji tworzyć issuesów, robić i przeglądać tu PRów itp. więc mogę za bardzo nie czuć jaki tu jest "workflow".

W związku z tym proszę o odrobinę wyrozumiałości pod tym kątem. Postaram się samemu to ogarnąć, ale też śmiało zwracaj mi na takie rzeczy uwagę ;)

To nie ma lepszego miejsca na naukę tego typu rzeczy niż otwarte repo z tłumaczeniem dokumentacji 🙂 widzę, że wrzuciłeś poprawki z moich komentarzy, czyli ogarniasz tyle, ile trzeba.

@jakubdrozdek jakubdrozdek merged commit 3d3d80f into reactjs:main Aug 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants