Windowing System: Improvement Ideas
@SleeplessByte がすでに取り組んでいます。
2022年2月7日 から。
評価
この issue はまだ評価されていません。
説明
After seeing a couple of solutions to "Windowing System" during mentoring, I think the exercise could need some changes to better achieve it's learning objectives.
- For
PositionandSize, the constructor and the method have the same implementation. I saw a solution that used the method in the constructor which is not a very typical example. - There is no reason to use the methods to update values so students often just set the property values directly.
- The calculation for the window resize and move is rather involved which does not add value for the actual learning objectives but it seems students spend quite some time finding creative solutions there, e.g.
Some ideas how to improve the exercise:
- Maybe we should expect some/all properties to be marked as private with the underscore to discourage the direct assignment. That might also make it easier to pick up unwanted code with the analyzer later. (Not sure how to best test in this case though. Either the test disrespects the underscore which is not a good example or the test needs to fall back on some user-implemented serialization function. But if that function is wrong all the tests will fail which is also annoying as everything else the user did could be correct.)
- The methods on
PositionandSizecould change the values by some amount instead of just setting a value. That would make them distinct from the constructor. (Probably only needed if we stick with "public properties".) - Simplify the window resize/move method e.g. by saying some lower bound is always respected
- 主要言語
- JavaScript
- スター
- 652
- フォーク
- 658
- 平均マージ
- 13時間 46分
- マージ済み PR（30日）
- 6
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
exercism/javascript のほかの issue
-
難易度 3/5 1〜2日 初心者へのやさしさ 45/100
exercism/javascript#2847 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 42/100
exercism/javascript#2832 ·
-
exercism/javascript#2310 · コメント 22 件 · 担当者 1 名 ·
-
pinned :pushpin:
難易度 5/5 1週間以上 初心者へのやさしさ 5/100
exercism/javascript#2144 ·
-
難易度 1/5 1時間未満 初心者へのやさしさ 35/100
exercism/javascript#1882 · コメント 2 件 ·
exercism/javascript の issue をすべて見る
似ている issue
-
bot:ai-assisted component:compact-js status:untriaged
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
midnightntwrk/midnight-sdk#403 ·
-
難易度 1/5 1時間未満 初心者へのやさしさ 92/100
-
難易度 1/5 1〜3時間 初心者へのやさしさ 86/100
DavidAnson/markdownlint-cli2#940 ·
-
documentation
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
githubnext/gh-aw-workshop#3692 ·
-
agent/guide documentation hive/hosted-available-lke648397-260827-5n31
難易度 2/5 1〜3時間 初心者へのやさしさ 90/100