fix: 「クラス」における誤字修正や説明の改善 #1598
fix: 「クラス」における誤字修正や説明の改善 #1598
Conversation
source/basic/class/README.md
Outdated
| また、コンストラクタは初期化処理を書く場所であるため、`return`文で値を返すべきではありません。 | ||
| JavaScriptでは、コンストラクタ関数が任意のオブジェクトを返すことが可能ですが、行うべきではありません。 |
There was a problem hiding this comment.
この箇所について、完全に同じではありませんが似たようなことが続いて書かれており不自然に感じたため、後半の文を削除しました。
There was a problem hiding this comment.
微妙に異なることを言っていて、どちらかという後者の方がストレートな表現な気がします。
前者は、仕様的にできないようにも読めます。コードの対応を考えると後者をベースにしたほうが良い気がしました
また、コンストラクタ関数は`return`文で任意のオブジェクトを返すことが可能ですが、行うべきではありません。
なぜなら、クラスを`new`演算子で呼び出し、その評価結果はクラスのインスタンスを期待するのが一般的であるためです。
source/basic/class/README.md
Outdated
| // 現在要素数より小さな`newLength`が指定された場合、指定した要素数となるように末尾を削除する | ||
| if (newLength < currentItemLength) { | ||
| // 現在要素数より小さな`newLength`が指定された場合、指定した要素数となるように末尾を削除する |
There was a problem hiding this comment.
直下の else if ブロックに合わせ、コメント位置を修正しました。
There was a problem hiding this comment.
これは元の位置にあるのが通常な気がします。
コメントに条件の説明が入ってるので、条件より後に説明があるのは不自然に見えます。
✅ Deploy Preview for js-primer ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
source/basic/class/README.md
Outdated
| // 現在要素数より小さな`newLength`が指定された場合、指定した要素数となるように末尾を削除する | ||
| if (newLength < currentItemLength) { | ||
| // 現在要素数より小さな`newLength`が指定された場合、指定した要素数となるように末尾を削除する |
There was a problem hiding this comment.
これは元の位置にあるのが通常な気がします。
コメントに条件の説明が入ってるので、条件より後に説明があるのは不自然に見えます。
source/basic/class/README.md
Outdated
| また、コンストラクタは初期化処理を書く場所であるため、`return`文で値を返すべきではありません。 | ||
| JavaScriptでは、コンストラクタ関数が任意のオブジェクトを返すことが可能ですが、行うべきではありません。 |
There was a problem hiding this comment.
微妙に異なることを言っていて、どちらかという後者の方がストレートな表現な気がします。
前者は、仕様的にできないようにも読めます。コードの対応を考えると後者をベースにしたほうが良い気がしました
また、コンストラクタ関数は`return`文で任意のオブジェクトを返すことが可能ですが、行うべきではありません。
なぜなら、クラスを`new`演算子で呼び出し、その評価結果はクラスのインスタンスを期待するのが一般的であるためです。
Co-authored-by: azu <azu@users.noreply.github.com>
source/basic/class/README.md
Outdated
| @@ -986,7 +985,7 @@ example.instanceMethod(); | |||
| example.prototypeMethod(); | |||
| ``` | |||
|
|
|||
| しかしこの2つのメソッドの定義方法は、メソッドを定義先となるオブジェクトが実際に異なります。 | |||
| しかしこの2つのメソッドの定義方法は、メソッドの定義先となるオブジェクトが実際は異なります。 | |||
There was a problem hiding this comment.
この"実際に"は、強調的な意味合いだと思うので、"実際は"にすると少し不自然な気がしました。(どっちも"実際には"の短縮のように感じるかも)
📝 何に対して 実際は なのかが気になる。
何かを否定する文な気がするけど、否定する対象の文がない。
(同じオブジェクトに定義しているように見えます とかの前提がない)
Co-authored-by: azu <azu@users.noreply.github.com>
|
ありがとうございました |
「クラス」を読んでいて見つけた誤りや文の乱れなどを修正しました。