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

ループと反復処理 #1741

Closed
vb-tsukakoshi opened this issue Jun 7, 2024 · 1 comment · Fixed by #1742
Closed

ループと反復処理 #1741

vb-tsukakoshi opened this issue Jun 7, 2024 · 1 comment · Fixed by #1742
Labels
Type: Errata 誤記、誤字、表記揺れ

Comments

@vb-tsukakoshi
Copy link

URL : https://github.com/asciidwango/js-primer/blob/master/source/basic/loop/README.md

はじめまして、いつもこちら利用し学習させていただいております。
引数の表記が誤っているのではないかと思い、ご連絡です。

内容としては、「もちろん、次のようにcontinue文を使わずに「偶数ならresultsへ追加する」という書き方も可能です。」の以下に続くコードについて現在のコードでは引数が異なるためかうまく動作しませんでした。

初学者ゆえ、当方の書き損じや誤認識もあるかも知れませんが、ご確認いただければ幸いです。

現在
if (isEven(number)) { results.push(number); }
修正案
if(isEven(num)) { results.push(num); }

@azu azu added the Type: Errata 誤記、誤字、表記揺れ label Jun 7, 2024
@azu azu closed this as completed in #1742 Jun 7, 2024
@azu
Copy link
Collaborator

azu commented Jun 7, 2024

ありがとうございます!
num とするのが正しいので、#1742 で修正しました。

以前、 #829 で サンプルコードをまとめて numbernum に変更したときに、漏れていたようです。ご報告ありがとうございます!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Errata 誤記、誤字、表記揺れ
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants