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

回答編集履歴

3

安易にMouseMoveを編集するべきではなかったので訂正

2017/04/12 04:58

投稿

haru666
haru666

スコア1593

answer CHANGED
@@ -71,9 +71,9 @@
71
71
 
72
72
  //マウス座標で矩形を選択する
73
73
  this.recIter.SelectByMouse(mymouse);
74
+
75
+ //ピクチャーボックスの再表示を依頼する
76
+ this.pictureBox1.Invalidate();
74
77
  }
75
-
76
- //ピクチャーボックスの再表示を依頼する
77
- this.pictureBox1.Invalidate();
78
78
  }
79
79
  ```

2

名詞の誤りの修正

2017/04/12 04:58

投稿

haru666
haru666

スコア1593

answer CHANGED
@@ -12,14 +12,14 @@
12
12
  ```
13
13
  void MoveSelected(Point from, Point to)
14
14
  {
15
- // 選択されている形のMoveを呼び出す
15
+ // 選択されている形のMoveを呼び出す
16
16
  }
17
17
  ```
18
18
 
19
19
  ---
20
20
 
21
- **②形の移動開始地点を取得する**
21
+ **②形の移動開始地点を取得する**
22
- 形がどこから移動を始めたのか記憶しなければいけません。
22
+ 形がどこから移動を始めたのか記憶しなければいけません。
23
23
  マウスがクリックされた時から移動が始まるならクリック位置が始点になります。
24
24
 
25
25
  マウスのクリックが開始されたのを知るには`OnMouseDown`イベントを使います。
@@ -32,35 +32,35 @@
32
32
  ```
33
33
  OnMouseDown
34
34
  ->MouseEventArgsから最初の座標を得る
35
- 形を移動する状態にする
35
+ 形を移動する状態にする
36
36
  OnMouseMove
37
- ->形を移動する状態なら、MouseEventArgsから現在の座標を得る
37
+ ->形を移動する状態なら、MouseEventArgsから現在の座標を得る
38
- 1回前の座標と組み合わせて形を移動する(※RecObjIterator.MoveSelectedを使う)
38
+ 1回前の座標と組み合わせて形を移動する(※RecObjIterator.MoveSelectedを使う)
39
39
  現在の座標を記憶する
40
40
  OnMouseUp
41
41
  ->MouseEventArgsから現在の座標を得る
42
- 1回前の座標と異なれば、形を移動する(※必要ならRecObjIterator.MoveSelectedを使う)
42
+ 1回前の座標と異なれば、形を移動する(※必要ならRecObjIterator.MoveSelectedを使う)
43
- 形を移動する状態を終了する
43
+ 形を移動する状態を終了する
44
44
  ```
45
45
 
46
- ※余談ですが、マウス移動時になめらかに形を移動させるにはPictureBoxコントロールの`Capture`プロパティをtrueにします。`OnMouseDown`でtrueにし、`OnMouseUp`でfalseにすると良いのですが、これがどういう意味かは自分で調べてみてください。
46
+ ※余談ですが、マウス移動時になめらかに形を移動させるには`PictureBox`コントロールの`Capture`プロパティをtrueにします。`OnMouseDown`でtrueにし、`OnMouseUp`でfalseにすると良いのですが、これがどういう意味かは自分で調べてみてください。
47
47
 
48
48
  ---
49
49
 
50
- **③選択された形を維持する**
50
+ **③選択された形を維持する**
51
- 今のままだとMouseMoveが呼ばれるたびに選択される図形が変わってしまいますね。
51
+ 今のままだと`MouseMove`が呼ばれるたびに選択される図形が変わってしまいますね。
52
- 座標の移動時にもMouseMoveイベントを経由する必要があります。
52
+ 座標の移動時にも`MouseMove`イベントを経由する必要があります。
53
- Mouseがクリックされている間は同じ図形を全て移動したいのですよね。
53
+ マウスがクリックされている間は同じ図形を全て移動したいのですよね。
54
54
  であれば、クリックされている間は今の処理が再度呼び出されないようにする必要があります。
55
55
  ```
56
56
  private void pictureBox1_MouseMove(object sender, MouseEventArgs e)
57
57
  {
58
- if (/* 形が移動中なら */)
58
+ if (/* 形が移動中なら */)
59
59
  {
60
60
  // ここに移動用の処理を書く
61
61
 
62
62
  // 過去の処理で選択状態が作られているわけだから
63
- // 選択された形のMoveを呼び出せばよい
63
+ // 選択された形のMoveを呼び出せばよい
64
64
  }
65
65
  else {
66
66
  // 今までの処理...

1

追記

2017/04/11 09:00

投稿

haru666
haru666

スコア1593

answer CHANGED
@@ -34,12 +34,12 @@
34
34
  ->MouseEventArgsから最初の座標を得る
35
35
  菱形を移動する状態にする
36
36
  OnMouseMove
37
- ->MouseEventArgsから菱形移動状態なら現在の座標を得る
37
+ ->菱形移動する状態なら、MouseEventArgsから現在の座標を得る
38
- 1回前の座標と組み合わせて菱形を移動する
38
+ 1回前の座標と組み合わせて菱形を移動する(※RecObjIterator.MoveSelectedを使う)
39
39
  現在の座標を記憶する
40
40
  OnMouseUp
41
41
  ->MouseEventArgsから現在の座標を得る
42
- 座標が1回前と異なれば、菱形を移動する
42
+ 1回前の座標と異なれば、菱形を移動する(※必要ならRecObjIterator.MoveSelectedを使う)
43
43
  菱形を移動する状態を終了する
44
44
  ```
45
45