Skip to content

Update noImplicitAny.md #80

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 4 commits into from
Nov 10, 2019
Merged

Update noImplicitAny.md #80

merged 4 commits into from
Nov 10, 2019

Conversation

tbotaq
Copy link

@tbotaq tbotaq commented Nov 9, 2019

noImplicitAny.md に関して、以下の変更をしています。

  • 翻訳の仕方を少し変更(好みの問題かもしれません)
  • テキストの装飾を original のものと同期
  • 全角のを半角の:に修正

Copy link
Collaborator

@yottahmd yottahmd left a comment

Choose a reason for hiding this comment

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

修正いただき、ありがとうございます。一点コメントさせていただきました。ご確認いただければ幸いです。

@@ -1,6 +1,6 @@
# noImplicitAny

推測できないものや推測できないものがあり、予期しないエラーが発生する可能性があります。良い例は関数の引数です。アノテーションを付けないと、何が有効であるべきか、または無効であるべきかが明確ではありません
推測ができない、もしくは推測する事により予期せぬエラーを発生しかねないものがあります。良い例は関数の引数です。アノテーションを付けないと、何が有効で、何が有効でないのかが不明確です
Copy link
Collaborator

@yottahmd yottahmd Nov 10, 2019

Choose a reason for hiding this comment

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

型を推測できない、あるいは型を推測した結果が予期せぬエラーとなるかもしれない場合があります。良い例は関数の引数です。アノテーションを付けなければ、どの型が有効であるべきか、そうでないかが不明確になります。

ではいかがでしょうか?「推測」の意味が不明瞭のため、わかりやすく「型推測」に変更しました。
また、最後の文については文意をわかりやすくしました。

原文

There are some things that cannot be inferred or inferring them might result in unexpected errors. A fine example is function arguments. If you don't annotate them, its unclear what should and shouldn't be valid e.g.

- Specify what is inferred.
- Tweak the last part.
@tbotaq tbotaq requested a review from yottahmd November 10, 2019 05:50
Copy link
Collaborator

@yottahmd yottahmd left a comment

Choose a reason for hiding this comment

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

修正いただき、ありがとうございます!
今後ともTypeScript DeepDive 日本語版を改善していただけますと幸いです🙏

@yottahmd yottahmd merged commit 547a0a1 into typescript-book-ja:master Nov 10, 2019
@tbotaq tbotaq deleted the fix-no-implicit-any-md branch November 10, 2019 05:56
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