Skip to content
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

無名関数の章のこれ以前の説明と不整合な部分を修正 #719

Merged
merged 5 commits into from
May 3, 2024

Conversation

naka-12
Copy link
Contributor

@naka-12 naka-12 commented Apr 28, 2024

No description provided.

@naka-12 naka-12 changed the title これ以前の説明と不整合な部分を修正 無名関数の章のこれ以前の説明と不整合な部分を修正 Apr 28, 2024
@naka-12 naka-12 marked this pull request as ready for review April 28, 2024 07:30
@naka-12 naka-12 requested review from chvmvd and aster-void April 28, 2024 07:30
Copy link

cloudflare-workers-and-pages bot commented Apr 29, 2024

Deploying utcode-learn with  Cloudflare Pages  Cloudflare Pages

Latest commit: 618ddda
Status: ✅  Deploy successful!
Preview URL: https://3c31006f.utcode-learn.pages.dev
Branch Preview URL: https://update-anonymus-function-des.utcode-learn.pages.dev

View logs

Copy link
Contributor

@chvmvd chvmvd left a comment

Choose a reason for hiding this comment

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

よいですね。

@@ -75,7 +75,7 @@ const add3 = function (a, b) {

## 無名関数を用いると便利な関数

[`Array#map` メソッド](https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Global_Objects/Array/map)は、関数オブジェクトの引数をひとつとり、配列の各要素に適用した上で、その戻り値からなる新しい配列を返します。
[`Array#map` メソッド](https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Global_Objects/Array/map)は、引数として値としての関数をひとつとり、配列の各要素に適用した上で、その戻り値からなる新しい配列を返します。
Copy link
Contributor

Choose a reason for hiding this comment

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

「値としての関数」だと値としての関数ではないものの可能性を示唆してしまいそう。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

うーん、どういうことですか? 値としての関数ではないものって例えばどういうものですかね?

Copy link
Contributor

Choose a reason for hiding this comment

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

「値としての関数」ということばを使うと、読者に「値としてではない関数」があるんだと思われそうという感じです。

Copy link
Contributor Author

@naka-12 naka-12 Apr 30, 2024

Choose a reason for hiding this comment

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

うーん、そうするとそもそもイベントの項で「値としての関数」という導入をしているところを見直さないといけない、という話になりませんかね?

そうしたら、とりあえずこの部分に関しては「関数の引数をひとつとり」で十分ですかね?

Copy link
Contributor

Choose a reason for hiding this comment

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

そっち側は値として関数を見るというように読み取っていたので、気にはなっていませんでした。こっちだと「値としての関数」しか渡せないのじゃないかみたいに少し思っちゃったという感じです。

Copy link
Contributor

@chvmvd chvmvd Apr 30, 2024

Choose a reason for hiding this comment

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

そうしたら、とりあえずこの部分に関しては「関数の引数をひとつとり」で十分ですかね?

それでいいのかなとは思ってます。

@naka-12 naka-12 requested a review from chvmvd April 30, 2024 11:26
@@ -75,7 +75,7 @@ const add3 = function (a, b) {

## 無名関数を用いると便利な関数

[`Array#map` メソッド](https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Global_Objects/Array/map)は、関数オブジェクトの引数をひとつとり、配列の各要素に適用した上で、その戻り値からなる新しい配列を返します。
[`Array#map` メソッド](https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Global_Objects/Array/map)は、関数の引数をひとつとり、配列の各要素に適用した上で、その戻り値からなる新しい配列を返します。
Copy link
Contributor

Choose a reason for hiding this comment

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

関数の引数だと意味を勘違いされそう。
ただArray#reduceが引数を1つとらないと言っているように見える気がする。つまり、
関数をArray#reduceのことだと思われそう。

Copy link
Contributor

Choose a reason for hiding this comment

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

Array#map ですよ。自分には合ってるように見えます。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

言われてみればそうですね(reduceではないですけど)
じゃあ「引数として関数をひとつとり」とかにしておきます。

Copy link
Contributor

Choose a reason for hiding this comment

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

言われてみればそうですね(reduceではないですけど)

じゃあ「引数として関数をひとつとり」とかにしておきます。

ごめん。いろいろ間違ってるけど、そういうことです。

@naka-12
Copy link
Contributor Author

naka-12 commented Apr 30, 2024

なおしました

@naka-12 naka-12 requested a review from chvmvd April 30, 2024 11:46
@naka-12 naka-12 merged commit c70e292 into master May 3, 2024
3 checks passed
@naka-12 naka-12 deleted the update-anonymus-function-description branch May 3, 2024 03:43
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.

3 participants