teratail header banner
teratail header banner
質問するログイン新規登録

回答編集履歴

1

箇条書きをリストに修正

2015/06/09 14:41

投稿

ina
ina

スコア127

answer CHANGED
@@ -4,10 +4,10 @@
4
4
  これで意図した動作になると思いますが、
5
5
  その他で気になったところも書いておきます。
6
6
 
7
- mousemoveはclickするたびに設定する必要はないので、click関数の外に書いたほうがよさそう。
7
+ - mousemoveはclickするたびに設定する必要はないので、click関数の外に書いたほうがよさそう。
8
- clientX、cliengYはウィンドウ上の座標のため、ページにスクロールがあると位置がずれるのでpageX、pageYを使うほうがよさそう。
8
+ - clientX、cliengYはウィンドウ上の座標のため、ページにスクロールがあると位置がずれるのでpageX、pageYを使うほうがよさそう。
9
- flagはグローバル変数にしないほうよさそう。
9
+ - flagはグローバル変数にしないほうよさそう。
10
- $("#idou")は何度か使うので、毎回取得せずに変数に入れて使うほうがよさそう。
10
+ - $("#idou")は何度か使うので、毎回取得せずに変数に入れて使うほうがよさそう。
11
11
 
12
12
  参考までに修正したコードを貼っておきます。
13
13
  ```lang-javascript