Skip to content

Conversation

@smlkA
Copy link
Contributor

@smlkA smlkA commented Jul 6, 2020

Меня зацепил перевод оригинальной фразы:
Generally we recommend using componentDidUpdate() for such logic instead.

Было изначально переведено как:
Мы рекомендуем использовать такой подход в componentDidUpdate()

По моему мнению это не оч удачный перевод, так как в оригинальной фразе есть явное замещение, а в переводе можно понять, что setState с колбеком можно использовать в componentDidUpdate

@netlify
Copy link

netlify bot commented Jul 6, 2020

Deploy preview for ru-reactjs ready!

Built with commit f59e38e

https://deploy-preview-515--ru-reactjs.netlify.app

Как `state`, так и `props`, полученные функцией обновления, гарантированно будут обновлены. Результат функции поверхностно объединяется с `state`.

Второй параметр в `setState()` - дополнительный колбэк, который выполняется после того, как исполнится `setState` и произойдёт повторный рендер компонента. Мы рекомендуем использовать такой подход в `componentDidUpdate()`.
Второй необязательный параметр в `setState()` - колбэк, который вызывается после того, как будет выполнен `setState` и рендер компоненты завершиться. В общем случае мы рекомендуем использовать `componentDidUpdate()` вместо колбека для такой логики.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Спасибо за предложение. Может быть так?

Suggested change
Второй необязательный параметр в `setState()` - колбэк, который вызывается после того, как будет выполнен `setState` и рендер компоненты завершиться. В общем случае мы рекомендуем использовать `componentDidUpdate()` вместо колбека для такой логики.
Второй параметр в `setState()` – дополнительный колбэк, который вызывается после выполнения `setState` и повторного рендера компонента. Как правило, мы рекомендуем использовать `componentDidUpdate()` для такой логики.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

В принципе я согласен с вашим апдейтом, но есть два момента:

  1. Optional parameter - я перевел бы как 'необязательный', так как на мой взгляд это более распространенный термин. Если вы считаете, что это термины равнозначны, то давайте оставим "дополнительный"
  2. Мне не оч нравиться 'повторного рендера'. Мне кажется лучше бы звучало 'после выполнения setState и последующего рендера компонента'
Suggested change
Второй необязательный параметр в `setState()` - колбэк, который вызывается после того, как будет выполнен `setState` и рендер компоненты завершиться. В общем случае мы рекомендуем использовать `componentDidUpdate()` вместо колбека для такой логики.
Второй параметр в `setState()` - необязательный колбэк, который вызывается после выполнения `setState` и последующего рендера компоненты. Как правило, мы рекомендуем использовать `componentDidUpdate()` для такой логики.

Copy link
Collaborator

@gcor gcor Jul 8, 2020

Choose a reason for hiding this comment

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

  1. C «необязательный» хорошо.
  2. В этом разделе re-render везде переводился как повторный. Всё-таки последующий это немного другое.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok cool

Andrey Smolko added 2 commits July 9, 2020 13:28
Co-authored-by: Anton Ahatov <anton@gcor.ru>
@gcor gcor merged commit b2a761c into reactjs:master Jul 9, 2020
@gcor
Copy link
Collaborator

gcor commented Jul 9, 2020

Спасибо

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