Rewrite between html and if statement by sYakusHinineko · Pull Request #392 · ut-code/utcode-learn

sYakusHinineko · 2023-09-26T05:35:47Z

教材の該当箇所を修正。特に変数（05-variables）の章の構成を大きく変更。

cloudflare-workers-and-pages · 2023-09-26T05:35:49Z

Deploying with Cloudflare Pages

Latest commit:	`44eed19`
Status:	✅ Deploy successful!
Preview URL:	https://4249c0d4.utcode-learn.pages.dev
Branch Preview URL:	https://rewrite-between-html-and-if.utcode-learn.pages.dev

View logs

要検討

chelproc · 2023-09-26T11:56:02Z

 ```

-1 行目の `let myGreatName;` で、`myGreatName` という名前の<Term type="javascriptVariable">変数</Term>を<Term type="javascriptDeclaration">宣言</Term>しています。`let` は、<Term type="javascriptVariable">変数</Term>を<Term type="javascriptDeclaration">宣言</Term>するためのキーワードです。
+`let` は、<Term type="javascriptVariable">変数</Term>を<Term type="javascriptDeclaration">宣言</Term>するためのキーワードです。 1 行目では、`myGrateName` という名前の<Term type="javascriptVariable">変数</Term>を<Term type="javascriptDeclaration">宣言</Term>し、そこに`"Becky Jones"` という文字列を保存しています。<Term type="javascriptVariable">変数</Term>に`=` を使って<Term type="javascriptValue">値</Term>を保存することを<Term type="javascriptAssignment">代入</Term>と呼びます。


代入の定義に = があるのではなく、代入という操作があってその演算子として = を使う

chelproc · 2023-09-26T11:56:46Z

+2 行目では、<Term type="javascriptVariable">変数</Term> `myGreatName` が<Term type="javascriptEvaluation">評価</Term>され、<Term type="javascriptAssignment">代入</Term>されていた<Term type="javascriptString">文字列</Term><Term type="javascriptValue">値</Term> `"Becky Jones"` が画面に表示されます。

-3 行目では、<Term type="javascriptVariable">変数</Term> `myGreatName` が<Term type="javascriptEvaluation">評価</Term>され、<Term type="javascriptAssignment">代入</Term>されていた<Term type="javascriptString">文字列</Term><Term type="javascriptValue">値</Term> `"Becky Jones"` が画面に表示されます。
+<p><Term type="javascriptVariable">変数</Term>を<Term type="javascriptDeclaration">宣言</Term>するキーワードには、<code>let</code> 以外にも<code>const</code> というものがあります。記法自体は<code>let</code> と同様です。</p>


「〜というもの」が口語的

chelproc · 2023-09-26T11:58:30Z

-```javascript title="script.js"
-let myGreatName = "Becky Jones";
-document.write(myGreatName);
+<Term type="javascriptVariable">変数</Term>や<Term type="javascriptFunction">関数</Term>は適切に命名することが大切です。以下の例を見てみましょう。


関数はまだ登場していない

chelproc · 2023-09-26T12:02:51Z

+const pricePerItem = 500;
+const numberOfItems = 3;
+const paymentTotal = pricePerItem * numberOfItems;
+document.write(paymentTotal);


変数名の付け方があまり良くなさそう

chelproc · 2023-09-26T12:17:56Z

+```

-<p><Term type="javascriptVariable">変数</Term>には、最後に<Term type="javascriptAssignment">代入</Term>された<Term type="javascriptValue">値</Term>のみを保持する性質があります。</p>
+この二つでは、前者は何を言いたいのかさっぱりですが、後者はコードを通じて書きたいことが分かりやすく、後から読み返しても困りません。他人と開発を行うときや、過去のコードを自分が書き換えるときには、コードの可読性が必要になります。変数名は多少長くなっても良いので、とにかく分かりやすいことを心がけましょう。


「何を言いたいのかさっぱり」が曖昧。「a, b, c」に何が入っているかわからない、と言えばいいと思います。
「コードの可読性が必要になります」、「必要」は言い過ぎで「重要」くらいの表現にするといいと思います。
挙げられている例で「可読性」が必要な理由は共通して「プログラムを書いた時に考えていたことを覚えていないから」なので、それは明示しておいてもいいかと思います。
「心掛ける」は自分の行動に対して使う言葉で「分かりやすい」に対応していないです。

chelproc · 2023-09-26T12:21:37Z

 ```

-変数 `mysteriousNumber` は 3 回<Term type="javascriptAssignment">代入</Term>されていますが、最後に<Term type="javascriptAssignment">代入</Term>された `2` が表示されます。
+上の例では、<Term type="javascriptVariable">変数</Term>の<Term type="javascriptValue">値</Term>が次々と変化しています。`let` で<Term type="javascriptDeclaration">宣言</Term>された<Term type="javascriptVariable">変数</Term>は最後に<Term type="javascriptAssignment">代入</Term>された<Term type="javascriptValue">値</Term>のみを保持します。<Term type="javascriptVariable">変数</Term> `mysteriousNumber` は 3 回<Term type="javascriptAssignment">代入</Term>されていますが、最後に<Term type="javascriptAssignment">代入</Term>された `2` が表示されます。


「変数は最後に格納された値のみを保持する」は宣言方法によらず「変数」という概念そのものの性質なので、あえて「letで」という必要はないと思います。
あとは3文の接続が薄いので適当な接続士を挟むと良いと思います。

chelproc · 2023-09-26T12:25:42Z

+## constとletの用途
+
+`const` と`let` は用途によって使い分けがあります。基本的には、意図せず<Term type="javascriptValue">値</Term>を書き換えてしまうことを防ぐために`const` が使われます。`let` が必要な場合は様々ですが、状態を保存しておくための<Term type="javascriptVariable">変数</Term>を<Term type="javascriptDeclaration">宣言</Term>することに使われることが多いです。[繰り返し](./../08-loop/index.md)の章にあるカウンタ<Term type="javascriptVariable">変数</Term>がその一例です。他にも、<Term type="javascriptVariable">変数</Term>の値を踏まえて条件分岐を行うなど、様々な用途があります。
+
+```javascript title="script.js"
+let i = 0;
+while (i < 5) {
+  i += 1;
+  document.write("Hello world!");
+}
+```


「基本的には const を使おう」とだけ言えれば十分だと思います。

chelproc · 2023-09-26T12:30:30Z

 - 18 歳以上 ～ 25 歳未満なら `投票に行けます` と表示する
 - 25 歳以上なら `衆議院議員に立候補できます` と表示する

+<Details summary={<summary>課題が解けたら…</summary>}>


解答 -> 「こんな書き方してないですか」の順番の方が良さそう

chvmvd · 2023-10-05T09:21:08Z

@sYakusHinineko

This reverts commit 475768d.

…flict-of-#392

chvmvd · 2023-10-07T11:59:53Z

とりあえず、すべて直しました！！！レビューお願いします。

chvmvd

LGTM!

aster-void · 2023-10-07T12:30:35Z

諸事情でスマホから編集してるのでデバッグしてません、ごめんなさい🙏
他はLGTMです

aster-void

ビルド通ったので 👍

chvmvd · 2023-10-07T14:16:02Z

@fridge0 対応しました！

sYakusHinineko added 7 commits September 25, 2023 18:54

05-variablesのページにconstの説明を追加。

5eacd0e

変数名の命名の重要さのtipsを追加

a2dfc0f

if文の処理順の解説を課題のフォローアップとして追加

a078349

03-javascriptの軽微な修正

93a6cb4

変数の章にconstの説明追加、構成の大規模な変更

f2cf5f1

軽微な修正

26c5482

軽微な修正

8815594

sYakusHinineko requested review from chelproc and chvmvd September 26, 2023 05:36

sYakusHinineko added 2 commits September 26, 2023 14:42

htmlの章のcmdをcommandに置き換え

475768d

要検討

変数の章のリンク切れを修正

130bd07

chvmvd reviewed Sep 26, 2023

View reviewed changes

chelproc reviewed Sep 26, 2023

View reviewed changes

aster-void added a commit that referenced this pull request Oct 7, 2023

Resolve Conflict of PR #392

1d8e41c

chvmvd added 3 commits October 7, 2023 19:46

Revert "htmlの章のcmdをcommandに置き換え"

3c571a5

This reverts commit 475768d.

Fix typos

e3de36e

Add inline code in title

dcc0291

aster-void added a commit that referenced this pull request Oct 7, 2023

Merge branch 'rewrite-between-html-and-if-statement' into resolve-con…

8804455

…flict-of-#392

chvmvd added 2 commits October 7, 2023 20:43

Rewrite section of variables

2fb46cd

Rewrite description of answer of the right to vote

cb5e063

chvmvd requested a review from chelproc October 7, 2023 11:59

chvmvd approved these changes Oct 7, 2023

View reviewed changes

Add more Term tags

44eed19

aster-void approved these changes Oct 7, 2023

View reviewed changes

Comment thread docs/1-trial-session/03-javascript/index.md

Comment thread docs/1-trial-session/05-variables/index.md Outdated

Comment thread docs/1-trial-session/05-variables/index.md Outdated

chvmvd mentioned this pull request Oct 7, 2023

p タグを p 要素に変更 #431

Closed

chvmvd merged commit fb9fb61 into master Oct 8, 2023

chvmvd deleted the rewrite-between-html-and-if-statement branch October 8, 2023 07:40

Conversation

sYakusHinineko commented Sep 26, 2023

Uh oh!

cloudflare-workers-and-pages Bot commented Sep 26, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Deploying with Cloudflare Pages

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

chvmvd commented Oct 5, 2023

Uh oh!

chvmvd commented Oct 7, 2023

Uh oh!

chvmvd left a comment

Choose a reason for hiding this comment

Uh oh!

aster-void commented Oct 7, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

aster-void left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

chvmvd commented Oct 7, 2023

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

cloudflare-workers-and-pages Bot commented Sep 26, 2023 •

edited

Loading

aster-void commented Oct 7, 2023 •

edited

Loading