回答編集履歴

2

見直しキャンペーン中

2023/07/22 07:48

投稿

TN8001
TN8001

スコア9862

test CHANGED
@@ -1,125 +1,63 @@
1
1
  手元で動かしたところ、↓ボタンで下にずれました(`SetupForm`・`label2`に関連するところのコメント化、`Load += RubikCubeForm_Load`の追加)
2
2
 
3
-
4
-
5
3
  `CreateRubikCube()`もしくは`RubikCubeForm_Load()`を複数回呼んでいませんか?
6
-
7
4
  ラベルが重なって、変更されたラベルが見えていない可能性があります。
8
5
 
9
-
10
-
11
6
  あと、
12
-
13
- ```C#
7
+ ```cs
14
-
15
8
  for(var i = 0; i < _rubiksize; i++)
16
-
17
9
  for(var j = 0; j < _rubiksize; j++)
18
-
19
10
  {
20
-
21
11
  {
22
-
23
12
  //数字配置
24
13
 
25
-
26
-
27
14
  }
28
-
29
15
  //ボタン配置
30
16
 
31
-
17
+ }
18
+ ```
19
+ ここは、
20
+ ```cs
21
+ for(var i = 0; i < _rubikSize; i++)
22
+ {
23
+ for(var j = 0; j < _rubikSize; j++)
24
+ {
25
+ //数字配置
32
26
 
33
27
  }
34
-
35
- ```
36
-
37
- ここは、
38
-
39
- ```C#
40
-
41
- for(var i = 0; i < _rubikSize; i++)
42
-
43
- {
44
-
45
- for(var j = 0; j < _rubikSize; j++)
46
-
47
- {
48
-
49
- //数字配置
50
-
51
-
52
-
53
- }
54
-
55
28
  //ボタン配置
56
29
 
57
-
58
-
59
30
  }
60
-
61
31
  ```
62
-
63
32
  こうですよね?
64
-
65
33
  ボタンが_rubikSize倍、無駄に作られます。捨てているので動作自体には影響しないはずですが。
66
-
67
-
68
34
 
69
35
  ---
70
36
 
71
-
72
-
73
37
  コメントを受けての追記
74
-
75
38
  すでに作っていたら削除するのがお手軽でしょうか。
76
-
77
- ```C#
39
+ ```cs
78
-
79
40
  private void CreateRubikCube()
80
-
81
41
  {
82
-
83
42
  if(digitLabel != null)
84
-
85
43
  {
86
-
87
44
  //数字削除
88
-
89
45
  foreach(var label in digitLabel)
90
-
91
46
  {
92
-
93
47
  Controls.Remove(label);
94
-
95
48
  }
96
-
97
49
  }
98
-
99
50
  if(moveVerticalUpButton != null)
100
-
101
51
  {
102
-
103
52
  //上ボタン削除
104
-
105
53
  foreach(var button in moveVerticalUpButton)
106
-
107
54
  {
108
-
109
55
  button.Click -= new EventHandler(MoveUp_Click);
110
-
111
56
  Controls.Remove(button);
112
-
113
57
  }
114
-
115
58
  }
116
-
117
59
  //下左右ボタンも同様
118
60
 
119
-
120
-
121
61
  //以下元の作成コード
122
-
123
62
  }
124
-
125
63
  ```

1

追記

2020/06/19 05:24

投稿

TN8001
TN8001

スコア9862

test CHANGED
@@ -63,3 +63,63 @@
63
63
  こうですよね?
64
64
 
65
65
  ボタンが_rubikSize倍、無駄に作られます。捨てているので動作自体には影響しないはずですが。
66
+
67
+
68
+
69
+ ---
70
+
71
+
72
+
73
+ コメントを受けての追記
74
+
75
+ すでに作っていたら削除するのがお手軽でしょうか。
76
+
77
+ ```C#
78
+
79
+ private void CreateRubikCube()
80
+
81
+ {
82
+
83
+ if(digitLabel != null)
84
+
85
+ {
86
+
87
+ //数字削除
88
+
89
+ foreach(var label in digitLabel)
90
+
91
+ {
92
+
93
+ Controls.Remove(label);
94
+
95
+ }
96
+
97
+ }
98
+
99
+ if(moveVerticalUpButton != null)
100
+
101
+ {
102
+
103
+ //上ボタン削除
104
+
105
+ foreach(var button in moveVerticalUpButton)
106
+
107
+ {
108
+
109
+ button.Click -= new EventHandler(MoveUp_Click);
110
+
111
+ Controls.Remove(button);
112
+
113
+ }
114
+
115
+ }
116
+
117
+ //下左右ボタンも同様
118
+
119
+
120
+
121
+ //以下元の作成コード
122
+
123
+ }
124
+
125
+ ```