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

論理値の章に簡単な説明を追加 #718

Merged
merged 6 commits into from
May 3, 2024

Conversation

naka-12
Copy link
Contributor

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

  • 「論理値と論理演算子」の章で、論理演算子と比較演算子について例を交えて簡単な説明を追加しました。
  • 高尾山の課題をコラムにし、新しい課題を追加しました。

truefalse を直接 document.write() して画面に表示するのはやや不自然かもしれませんが、初学者にとっては「今この式が true になっているのか、false になっているのか」が目に見える形で試行錯誤することが必要と考えてこのような形にしました。

Copy link

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

Deploying utcode-learn with  Cloudflare Pages  Cloudflare Pages

Latest commit: 014e660
Status: ✅  Deploy successful!
Preview URL: https://3997cd03.utcode-learn.pages.dev
Branch Preview URL: https://update-boolean-description.utcode-learn.pages.dev

View logs

Comment on lines 51 to 54
document.write(3 === 3); // true
document.write(3 === 4); // false
document.write("Hello" === "Hello"); // true
document.write(3 !== 3); // false
Copy link
Contributor

Choose a reason for hiding this comment

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

これはなくてもいいんじゃない?
比較はたしか小学校で習っててかなり馴染みがあるものだから、抽象的な3 === 3より具体的な使用例の方がむしろ馴染みがあるような気がする。

論理演算子は、抽象的な形の方が馴染みがあると思うけど。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

bfdd27f で別の話に差し替えました。

@naka-12
Copy link
Contributor Author

naka-12 commented Apr 28, 2024

ちょうど話題に上ったので、d32e66e で課題を変更する話を追加しました (後からですみません……)

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.

例の変数名が気になる以外は非常に良いと思います。


:::tip

`+`, `-`, `*`, `/` などの算術演算子は、比較演算子よりも先に計算されます。
Copy link
Contributor

Choose a reason for hiding this comment

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

上で「優先順位」という言葉を導入しているので、使っても良いかもです。

```javascript
const myScore = 120;
const opponentScore = 80;
const iWon = /* ここに式を書いてください */;
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 29, 2024

Choose a reason for hiding this comment

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

isWinnerとか?myScoreopponentScoreと言ってるから、isWinnerは誰が勝ったのか明確なような気はする。他の意見も聞きたいですね。

Copy link
Contributor

Choose a reason for hiding this comment

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

これは Starter のコピペじゃだめなんでしたっけ?
だめなら、 amIWinner とかがいいんじゃないかなと思います

Copy link
Contributor

Choose a reason for hiding this comment

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

英語的に i が小文字なのも気になりますし。

Copy link
Contributor Author

@naka-12 naka-12 Apr 29, 2024

Choose a reason for hiding this comment

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

isWinnerとか?myScoreとopponentScoreと言ってるから、isWinnerは誰が勝ったのか明確なような気はする。

うーん、結局 isWinner だと自分が勝ったのか相手が勝ったのかわからないのでやめました。

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.

isWinnerとか?myScoreとopponentScoreと言ってるから、isWinnerは誰が勝ったのか明確なような気はする。

うーん、結局 isWinner だと自分が勝ったのか相手が勝ったのかわからないのでやめました。

そもそもmyopponentという変数を使っている時点で自分を中心に作ってしまっているので、どちらにしろ感はありますね。
自分のことだけが重要な文脈であると考えるとそこまで悪くは見えないような気がします。
それか、問題自体を変えるかですかね。

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

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.

変えました。

@naka-12 naka-12 requested a review from chvmvd April 29, 2024 14:07
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.

よさげ

@naka-12
Copy link
Contributor Author

naka-12 commented Apr 30, 2024

@chvmvd Approve ではないんですか……😢

@chvmvd
Copy link
Contributor

chvmvd commented Apr 30, 2024

@chvmvd Approve ではないんですか……😢

ごめんなさい。ボタン押し間違えたようです...

@naka-12 naka-12 merged commit 71b6fe4 into master May 3, 2024
3 checks passed
@naka-12 naka-12 deleted the update-boolean-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