-
Notifications
You must be signed in to change notification settings - Fork 54
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
Conversation
There was a problem hiding this 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: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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) |
There was a problem hiding this 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.
Size changes📦 Next.js Bundle Analysis for react-devThis analysis was generated by the Next.js Bundle Analysis action. 🤖 This PR introduced no changes to the JavaScript bundle! 🙌 |
Co-authored-by: Jakub Drozdek <[email protected]>
Co-authored-by: Jakub Drozdek <[email protected]>
Co-authored-by: Jakub Drozdek <[email protected]>
Co-authored-by: Jakub Drozdek <[email protected]>
Co-authored-by: Jakub Drozdek <[email protected]>
To nie czepianie się, jeśli chcesz dobrze i masz rację. Jestem otwarty na nawet najmniejsze uwagi 😄 |
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ę ;) |
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. |
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. |
No description provided.