Skip to content

Translate 3 playground files to Japanese #494

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
Apr 10, 2020

Conversation

satomixx
Copy link
Contributor

Part of #220,
I translated 3 playground files to Japanese.

Big number literals.ts
Const to let.ts
Infer From Usage Changes.ts

@satomixx satomixx requested a review from orta as a code owner April 10, 2020 01:39
@msftclas
Copy link

msftclas commented Apr 10, 2020

CLA assistant check
All CLA requirements met.

@satomixx satomixx changed the title Translate/ja 3 playground files to Japanese Translate 3 playground files to Japanese Apr 10, 2020
Copy link
Contributor

@Quramy Quramy left a comment

Choose a reason for hiding this comment

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

いくつかnitsよりのコメントを書きましたー。

const maxHighValue = 9007199254740991
const maxLowValue = -9007199254740991

// これらの数字を上回る/下回ることは、大変危険です。
Copy link
Contributor

Choose a reason for hiding this comment

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

IMOですが、 dangerous territory とあるので、↓みたいな感じかな、と思いました。

Suggested change
// これらの数字を上回る/下回ることは、大変危険です。
// この数字を上回る/下回ったら、
// それは危険な領域に足を踏み入れることになります。

あと、極力原文と訳文で行の数は一致するように心がけています(対応関係をわかりやすくするため

9007199254740994;
-9007199254740994

// 六進数
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
// 六進数
// 16進数

const oneBelowMin = -9007199254740992

// このサイズの数値を扱うための解決策は、
// これらの数値を、number の代わりに BigInts に変換することです:
Copy link
Contributor

Choose a reason for hiding this comment

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

原文は複数形なので BigInts で正しいですが、日本語は BigInt としてよいと思います。

// 既知の重要な型 (string, number, array, Promise) のリストを用いて、
// 型の使用方法がこれらのオブジェクトのAPIと一致するかどうかを推論する様になりました。

// 次のいくつかの例では、関数のパラメータを選択し、電球をクリックして "Infer Parameter types..." を選択してください。
Copy link
Contributor

Choose a reason for hiding this comment

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

[IMO]

Suggested change
// 次のいくつかの例では、関数のパラメータを選択し、電球をクリックして "Infer Parameter types..." を選択してください。
// 以降の例で、関数のパラメータを選択し、
// 電球マークをクリックして
// "Infer Parameter types..." を選択してください。

Copy link
Contributor

@sasurau4 sasurau4 left a comment

Choose a reason for hiding this comment

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

翻訳ありがとうございます 👍
ちょっとだけ指摘しました

//
// https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/BigInt

// TypeScriptは、2の52乗以上の数値リテラル(正/負)の修正機能を提供するようになり、
Copy link
Contributor

@sasurau4 sasurau4 Apr 10, 2020

Choose a reason for hiding this comment

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

aboveはmore thanと同じなので以上ではないんですよね。。。

Suggested change
// TypeScriptは、2の52乗以上の数値リテラル(正/負)の修正機能を提供するようになり、
// TypeScriptは、2の52乗より大きい数値リテラル(正/負)の修正機能を提供するようになり、

// https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/BigInt

// TypeScriptは、2の52乗以上の数値リテラル(正/負)の修正機能を提供するようになり、
// 接尾辞 "n"を追加してJavaScriptにBigInt型であることを知らせるようになりました。
Copy link
Contributor

Choose a reason for hiding this comment

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

spaceを空けないに統一

Suggested change
// 接尾辞 "n"を追加してJavaScriptにBigInt型であることを知らせるようになりました。
// 接尾辞"n"を追加してJavaScriptにBigInt型であることを知らせるようになりました。


// 3.7では、const変数の値が再代入されたときに、素早くletに変換する機能が追加されました。

// 以下のエラーをハイライトして、クイックフィックスを実行することで試すことができます。
Copy link
Contributor

Choose a reason for hiding this comment

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

IMO: このhighlighting、popupを表示することを指している可能性がある気がしますが、原文をリスペクトしておきます

Suggested change
// 以下のエラーをハイライトして、クイックフィックスを実行することで試すことができます。
// ハイライトされている以下のエラーから、クイックフィックスを実行することで試すことができます。

@@ -0,0 +1,32 @@
//// { compiler: { noImplicitAny: false }, order: 2 }

// 3.7 で TypeScript 既存の「使用状況からの推論」によるコード修正はよりスマートになりました。
Copy link
Contributor

Choose a reason for hiding this comment

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

IMO

Suggested change
// 3.7 で TypeScript 既存の「使用状況からの推論」によるコード修正はよりスマートになりました
// 3.7 で TypeScript 既存の「使用状況からの推論」によるコード修正はより賢くなりました

return result
}

// 文字列配列に文字列が追加された為、文字列配列を推論します:
Copy link
Contributor

Choose a reason for hiding this comment

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

IMO: 「に」だと思ってますが、もしかして「を」の方が一般的です?
てにをは難しい

Suggested change
// 文字列配列に文字列が追加された為、文字列配列を推論します:
// 文字列配列に文字列が追加された為、文字列配列に推論します:

//// { compiler: { noImplicitAny: false }, order: 2 }

// 3.7 で TypeScript 既存の「使用状況からの推論」によるコード修正はよりスマートになりました。
// 既知の重要な型 (string, number, array, Promise) のリストを用いて、
Copy link
Contributor

Choose a reason for hiding this comment

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

IMO: かなり意訳ですが、このknownは一般的に知られているぐらいの意味だと思うので、考えられているにしてみました

Suggested change
// 既知の重要な型 (string, number, array, Promise) のリストを用いて、
// 重要だと考えられている型 (string, number, array, Promise) のリストを用いて、

@orta
Copy link
Contributor

orta commented Apr 10, 2020

Agree! Thanks

@orta orta merged commit cbed313 into microsoft:v2 Apr 10, 2020
@orta
Copy link
Contributor

orta commented Apr 10, 2020

oh wait, this is the wrong PR - doh sorry

@orta
Copy link
Contributor

orta commented Apr 10, 2020

I don't think it's worth the revert & rollback - @satomixx if you're happy with those changes ^ please make a new PR

@satomixx
Copy link
Contributor Author

satomixx commented Apr 11, 2020

@sasurau4 @Quramy @orta
Thank you for reviews!!
Since this PR got merged before I could reflect on the feedback, the next time I get around to it, I'll make a correction along with the translation of the other files.

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.

5 participants