「変数のスコープ」の項をリライト by nakaterm · Pull Request #927 · ut-code/utcode-learn

nakaterm · 2026-02-15T10:00:19Z

resolves #815

cloudflare-workers-and-pages · 2026-02-15T10:01:32Z

Deploying utcode-learn with Cloudflare Pages

Latest commit:	`5066c13`
Status:	✅ Deploy successful!
Preview URL:	https://b6200e1c.utcode-learn.pages.dev
Branch Preview URL:	https://rewrite-variable-scope.utcode-learn.pages.dev

View logs

nakaterm · 2026-02-15T10:09:24Z

docs/1-trial-session/09-functions/index.mdx


 function greet() {
-  guestCount += 1;
+  guestCount = guestCount + 1;


ここで初出の += が出てくるとノイズなので、ここでは普通に今まで扱った方法で書き、下のコラム内で「これを += に書き換えられる」という説明をするようにしています

nakaterm · 2026-02-15T10:09:57Z

docs/1-trial-session/09-functions/index.mdx

+
+greet();
+
+// document.write(message); これはエラーになる


エラーになるサンプルコードを提供しない、という方針に従いコメントアウトしている

Copilot

Pull request overview

「変数のスコープ」セクションを、具体例（関数内のローカル変数／外側で宣言した変数の参照）を用いて説明し直し、合わせて複合代入演算子の説明導線を整理するPRです。

Changes:

スコープの説明を、message のローカルスコープ例（関数内のみ参照可）で具体化
外側（関数外）で宣言した変数を関数内から参照・更新できる例を維持しつつ、説明文をリライト
複合代入演算子の説明を「guestCount = guestCount + 1 → guestCount += 1」の流れに整理

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

docs/1-trial-session/09-functions/index.mdx

chvmvd · 2026-02-21T06:06:00Z

docs/1-trial-session/09-functions/index.mdx

+
+```javascript
+function greet() {
+  let message = "Hello!";


ここでconstではなく、letを使った理由が分からなかったのですが、何かあったりしますか？

あー普通に忘れてました・・
ただ、下で関数外の変数を参照する例は let でこちらが const なのはノイズなのでどちらかに統一しようとは思います
まあ両方 const にして、+= の話は全部複合代入演算子のコラム内でやるようにしようかなぁ

こちらがconstで、下の例がletになってしまってもそんなに問題ないとは私は思いますが。いい例があれば、そのように統一しても良いと思います。

chvmvd · 2026-02-21T06:14:19Z

docs/1-trial-session/09-functions/index.mdx

+
+```javascript
+function greet() {
+  let message = "Hello!";


(optional)

上の説明では、メッセージが"Hello!"ではなく、"Hello World!"になっているため、そちらに統一すると認知負荷が少し小さくなるかもしれません。

utcode-learn/docs/1-trial-session/09-functions/index.mdx

Lines 11 to 20 in 27d5138

```javascript

// 関数を定義しておけば

function greet() {

document.write("Hello World!");

}

// 後から呼び出すことができる

greet();

greet();

```

確かにそうですね！

chvmvd · 2026-02-21T06:21:07Z

docs/1-trial-session/09-functions/index.mdx

+
+```javascript
+function greet() {
+  let message = "Hello!";


同様にlet → const？

chvmvd · 2026-02-21T06:34:59Z

docs/1-trial-session/09-functions/index.mdx

+
+greet();
+
+// document.write(message); これはエラーになる


(optional)

これを単にコメントアウトすると、document.write(message); これはエラーになるとなってしまい、これも構文エラーになってしまうため、単にコメントアウトしても構文エラーにならないようになっていると少し親切かもしれません。

改行したほうがいいか〜

個人的には次のようにエラーとなるところをコメントアウトせずに書くというのも悪くない気もしてきましたが。

utcode-learn/docs/4-advanced/03-typescript/index.mdx

Lines 102 to 104 in c8ad394

sum = "7"; // Type 'string' is not assignable to type 'number'.

add("3", "4"); // Argument of type 'string' is not assignable to parameter of type 'number'.

https://github.com/mdn/translated-content/blob/7fa1e352b7db8747d8f1050c1d5c4444343047a0/files/ja/web/javascript/guide/grammar_and_types/index.md?plain=1#L132-L135

あー別にエラーになるコードを書いちゃってる例もあるんですね
ならそれが一番わかりやすそう
(ただまあ今回のはランタイムエラーだし議論の余地はありそう？)

nakaterm

すみません、全体的に例を変えました！

nakaterm · 2026-02-21T15:37:09Z

docs/1-trial-session/09-functions/index.mdx


 ```javascript
-guestCount = guestCount + 1;
+const siteName = "田中のブログ";


この節の上の例で使われている greetingType や myName にあたるものを使うことも考えたのですが、そうすると同じユースケースで上では引数で渡しており、こちらでは渡していないということになって混乱を招くので例を変えました。

定数としておいていても変ではないものを探した結果、siteName にしました。

Copilot

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

nakaterm · 2026-02-21T15:47:32Z

一応 document.write() のコメントアウト外したコミットも作っておきました (賛否分かれそうなので微妙だったら戻しましょう)

nakaterm force-pushed the rewrite-variable-scope branch from bf5fed0 to 1153079 Compare February 15, 2026 10:01

「変数のスコープ」の項をリライト

1c4b02e

nakaterm force-pushed the rewrite-variable-scope branch from 1153079 to 1c4b02e Compare February 15, 2026 10:02

prettier-ignore を忘れていたので追加

e615ec5

nakaterm commented Feb 15, 2026

View reviewed changes

nakaterm requested review from chvmvd and Copilot and removed request for chvmvd February 15, 2026 11:22

Copilot started reviewing on behalf of nakaterm February 15, 2026 11:23 View session

Copilot AI reviewed Feb 15, 2026

View reviewed changes

docs/1-trial-session/09-functions/index.mdx Outdated Show resolved Hide resolved

docs/1-trial-session/09-functions/index.mdx Outdated Show resolved Hide resolved

let と const に限る旨を追加

a5ef48a

nakaterm force-pushed the rewrite-variable-scope branch from 3ee392d to a5ef48a Compare February 15, 2026 13:01

nakaterm requested review from Yokomi422, chelproc and chvmvd February 15, 2026 13:02

chvmvd reviewed Feb 21, 2026

View reviewed changes

const に統一して全体を整理

37c6674

nakaterm commented Feb 21, 2026

View reviewed changes

nakaterm requested review from chvmvd and Copilot and removed request for chvmvd February 21, 2026 15:38

Copilot started reviewing on behalf of nakaterm February 21, 2026 15:39 View session

Copilot AI reviewed Feb 21, 2026

View reviewed changes

nakaterm requested a review from chvmvd February 21, 2026 15:41

エラーになる行をコメントアウトしない

5066c13


		greet();

		// document.write(message); これはエラーになる

	```javascript
	// 関数を定義しておけば
	function greet() {
	document.write("Hello World!");
	}

	// 後から呼び出すことができる
	greet();
	greet();
	```

	sum = "7"; // Type 'string' is not assignable to type 'number'.

	add("3", "4"); // Argument of type 'string' is not assignable to parameter of type 'number'.

Comments

Conversation

nakaterm commented Feb 15, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

cloudflare-workers-and-pages bot commented Feb 15, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Deploying utcode-learn with Cloudflare Pages

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

chvmvd Feb 21, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

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 Feb 21, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

nakaterm Feb 21, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

nakaterm left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

nakaterm commented Feb 21, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

nakaterm commented Feb 15, 2026 •

edited

Loading

cloudflare-workers-and-pages bot commented Feb 15, 2026 •

edited

Loading

chvmvd Feb 21, 2026 •

edited

Loading

chvmvd Feb 21, 2026 •

edited

Loading

nakaterm Feb 21, 2026 •

edited

Loading

nakaterm commented Feb 21, 2026 •

edited

Loading