Conversation
Deploying utcode-learn with
|
| Latest commit: |
29747f6
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://dfdf9c6a.utcode-learn.pages.dev |
| Branch Preview URL: | https://fix-loop-exercise.utcode-learn.pages.dev |
There was a problem hiding this comment.
Pull request overview
「繰り返し」章(ループ)の演習問題1について、解答例を複合代入演算子を使う版に統一し、別解として載せていた非複合代入の例を削除するPRです。
Changes:
- 演習問題1(10の階乗)の解答から「別解」セクションを削除し、表示する ViewSource を
_samples/productに統一 _samples/productの実装をi += 1/product *= iを用いた内容に更新_samples/product-altサンプル一式を削除
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| docs/1-trial-session/10-loop/index.mdx | 演習問題1の解答例を複合代入演算子の版に一本化し、参照サンプルも統一 |
| docs/1-trial-session/10-loop/_samples/product/script.js | ループ更新式と積の更新を複合代入演算子ベースに変更 |
| docs/1-trial-session/10-loop/_samples/product-alt/script.js | 別解サンプル削除 |
| docs/1-trial-session/10-loop/_samples/product-alt/index.html | 別解サンプル削除 |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
|
#927 がマージされた場合、複合代入演算子の初出はここということになりますか。そうすると複合代入演算子の説明がなくなってしまいませんか。 |
今回の差分の箇所よりも前の箇所である cf. |
|
確かに複合代入演算子の今回の修正よりも前に出てきているけど、#927 で複合代入演算子の説明を消すと複合代入演算子がいきなり登場してしまう気がします。
|

複合代入演算子を使用した例を別解として載せていましたが、for文を導入する際の説明で複合代入演算子が使用されているため、解答例でも複合代入演算子を使用することにし別解については削除しました。