Fix/drawing/etch a skethc/layout

[sokahiri]

やったこと

• キャンバスを超えてポインターが移動することの無いように修正しました。
• directionのkeyの命名を変更しました。(isUp => up, ...)

説明

• キャンバスを超えるかどうかのチェック、超えていた場合キャンバス内に収まるように修正を行うメソッドを用意しました。ポインターがキャンバスを超える可能性のある操作を行うタイミングで、このメソッドを挿入しています。
• limitプロパティは、キャンバスサイズを所得し、その値を保存しておくためのものです。上記メソッドの実装に必要なため導入しました。キャンバスサイズ変更の度に更新されます。
• limitプロパティのkey名に合わせて、directionのkey名も変更してみましたが、これは変えなくてもいいですね。

備考

ポインターを動かしながらキャンバスサイズを変更すると、ポインターが動いた軌跡を線で描いてしまうバグを確認していますが、修正に手間がかかりそうなことと、実際にはそう起こらないバグであると思ったことから、放置しています。

やってほしいこと

挙動テストと、余裕がありましたらコードレビューをお願いしたいです。

質問

• directionのkey名変更について、以前の方が良いでしょうか？良い命名というのがよくわかっていないので、意見をお伺いしたいです。
• 備考で述べたバグにつきまして、修正案がございましたらお願いします。また、どれだけマイナーであろうと、やはりバグは全て修正するべきでしょうか？

[mcnLeandro]

directionのkey名変更について、以前の方が良いでしょうか？良い命名というのがよくわかっていないので、意見をお伺いしたいです。 今回の変更でOKだと思います！

備考で述べたバグにつきまして、修正案がございましたらお願いします。また、どれだけマイナーであろうと、やはりバグは全て修正するべきでしょうか？

66

自分も優先度は低いと思います！