型の表記ゆれを統一しました。(例:「文字列型」→「string型」) #714
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
suin
reviewed
May 19, 2023
docs/learning-resources.md
Outdated
| @@ -213,7 +213,7 @@ HaskellやMLなどの関数型プログラミング(FP)の経験がある学習 | |||
| | [Everyday Types](https://www.typescriptlang.org/docs/handbook/2/everyday-types.html) | 日常的に使用するあらゆる型についての説明 | | |||
| | [Narrowing](https://www.typescriptlang.org/docs/handbook/2/narrowing.html) | 型の絞り込みについての説明 | | |||
| | [More on Functions](https://www.typescriptlang.org/docs/handbook/2/functions.html) | 関数の型についての詳細説明 | | |||
| | [Object Types](https://www.typescriptlang.org/docs/handbook/2/objects.html) | オブジェクト型についての詳細説明 | | |||
| | [Object Types](https://www.typescriptlang.org/docs/handbook/2/objects.html) | object型についての詳細説明 | | |||
There was a problem hiding this comment.
「Object Types」は「オブジェクト型」ではなく「オブジェクトの型」についてだと思うので、object型ではないと思われます。
| @@ -47,7 +47,7 @@ const bar = ([a, b]) => {}; | |||
|
|
|||
| ## 分割代入引数の型注釈 | |||
|
|
|||
| TypeScriptでオブジェクトを分割代入する場合、分割代入引数の右にオブジェクト型の型注釈を書きます。 | |||
| TypeScriptでオブジェクトを分割代入する場合、分割代入引数の右にobject型の型注釈を書きます。 | |||
There was a problem hiding this comment.
objectという型注釈があって、object型とオブジェクト型は違う意味なので、「オブジェクトの型」と表現したほうが、書籍全般通じて読者にとって誤解が少ないかと思います。
| @@ -54,15 +54,15 @@ function func({ x, y, z }) { | |||
|
|
|||
| ## Options Objectパターンの型注釈 | |||
|
|
|||
| TypeScriptでOptions Objectパターンを使うときには、引数の型注釈が必要になります。型注釈はオブジェクト型を書きます。 | |||
| TypeScriptでOptions Objectパターンを使うときには、引数の型注釈が必要になります。型注釈はobject型を書きます。 | |||
docs/reference/statements/unknown.md
Outdated
| @@ -106,9 +106,9 @@ function isNumberArray(value: unknown): value is number[] { | |||
| } | |||
| ``` | |||
|
|
|||
| ### unknown型をオブジェクト型に絞り込む | |||
| ### unknown型をobject型に絞り込む | |||
docs/reference/type-reuse/README.md
Outdated
| @@ -16,7 +16,7 @@ const keys1 = ["a", "b", "c"]; | |||
| const keys2 = Object.keys(obj); // keys1より保守性が高い | |||
| ``` | |||
|
|
|||
| 上は変数の再利用の例です。TypeScriptには変数の再利用の型バージョンがあります。それが型の再利用です。たとえば、あるオブジェクト型からキーの型を導き出すことができます。 | |||
| 上は変数の再利用の例です。TypeScriptには変数の再利用の型バージョンがあります。それが型の再利用です。たとえば、あるobject型からキーの型を導き出すことができます。 | |||
| @@ -23,7 +23,7 @@ box = { width: "1080", height: 720 }; | |||
| box = { width: 1080 }; | |||
| ``` | |||
|
|
|||
| プロパティの区切り文字には、オブジェクトリテラルのようにカンマ`,`も使えますが、セミコロン`;`を用いるほうを推奨します。理由は、コード整形ツール[Prettier](/tutorials/prettier)がオブジェクト型注釈を直すとき、カンマをセミコロンに置き換えるためです。 | |||
| プロパティの区切り文字には、オブジェクトリテラルのようにカンマ`,`も使えますが、セミコロン`;`を用いるほうを推奨します。理由は、コード整形ツール[Prettier](/tutorials/prettier)がobject型注釈を直すとき、カンマをセミコロンに置き換えるためです。 | |||
There was a problem hiding this comment.
もともと誤字で、「の」が抜けてました。ただしくは「オブジェクトの型注釈」
| @@ -97,7 +97,7 @@ calculator; | |||
|
|
|||
| ## `Record<Keys, Type>` | |||
|
|
|||
| 連想配列のようなキーバリューのオブジェクト型を定義する場合、ユーティリティ型の`Record`を使う方法もあります。 | |||
| 連想配列のようなキーバリューのobject型を定義する場合、ユーティリティ型の`Record`を使う方法もあります。 | |||
docs/symbols-and-keywords.md
Outdated
| @@ -1073,7 +1073,7 @@ function isDuck(animal: Animal): animal is Duck { | |||
|
|
|||
| ### `keyof` keyof型演算子 (keyof) ![ts] | |||
|
|
|||
| オブジェクト型からプロパティ名を型として返す型演算子です。 | |||
| object型からプロパティ名を型として返す型演算子です。 | |||
| @@ -9,7 +9,7 @@ | |||
|
|
|||
| - 事実: TypeScriptでは、interfaceとtype aliasのどちらでもオブジェクトの型を定義できる。 | |||
| - 見解: 書籍のサンプルコードはどちらかに統一するのが好ましい。 | |||
| - 論点: サンプルコードにおいて、オブジェクト型の型定義には、interfaceとtype aliasどちらを選択するか？ | |||
| - 論点: サンプルコードにおいて、object型の型定義には、interfaceとtype aliasどちらを選択するか？ | |||
| - expected: object型 | ||
| patterns: | ||
| - Object型 | ||
| - オブジェクト型 |
There was a problem hiding this comment.
Boolean, Number, BigInt, String, Objectはラッパーオブジェクトがあるため、それらを説明しようとしたときに小文字に直されるのは執筆事故になりそうなので、パターンからは除外したほうがいいかもしれません
There was a problem hiding this comment.
ラッパーオブジェクトを削除しました！
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
どの用語に統一したかは #704 を見てください。
やったこと
<!-- textlint-disable prh -->を使い置換しないようにしましたclose #704