-
Notifications
You must be signed in to change notification settings - Fork 180
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
Translate hooks-rules #64
Conversation
#91 でフックに関わる差分がマージされます。翻訳を修正するまでマージはお待ちください。 |
ef9a175
to
f36df4f
Compare
直しました。変わっているのは「フック (hook) は React 16.8 で追加された新機能です。」云々のトップの部分だけです |
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.
@smikitky 細かい点についてコメントしたので確認お願いします 🙏 imoなのでApproveしてます。
|
||
By following this rule, you ensure that all stateful logic in a component is clearly visible from its source code. | ||
このルールを守ることで、コンポーネント内のすべての state を使うロジックがソースコードから間違いなく参照可能になります。 |
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.
[imo] clearly visible
なので、ソースコードから可視化されていることが言いたいのかなと思ったのですがいかがでしょう?参照だとアクセス可能であることと取られる可能性があるかなと思いました。
このルールを守ることで、コンポーネント内のすべての state を使うロジックがソースコードから間違いなく参照可能になります。 | |
このルールを守ることで、コンポーネント内のすべての state を使うロジックがソースコードから明確になります。 |
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.
「ロジックが明確になります」はちょっと意味が変わってしまうように思います。「ロジックが不明確」とは主に、変数名の付け方が下手などの理由で読みづらいけど正しく動くコードに対して使われる表現のような感じがします。
stateful logic が「コンポーネントのソースコードから可視化されている」と「コンポーネントのソースコードから参照できる」であれば、個人的にはどちらでもいいのですが、そもそもあまり意味の違いがあるような気がしません。ループ内やコンポーネント外で useState を使っても、その返り値の state/setState はコンポーネントの他の部分から正しく利用できませんよ、ということを "not visible" と表現しているのだと思うので、まさに「アクセス可能」のことだと思いますし、「可視化」でも「参照可能」でも同じのような気がしているのですが、如何でしょうか。
f36df4f
to
09bc581
Compare
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.
すみません、見た後に Approve 忘れていました 🙇
良きタイミングでマージしてください!
5/8 of the Hooks articles