teratail
回答率
85.35%
teratail header banner
teratail header banner
質問するログイン新規登録
トップに関する質問if文を使用すると、値が割り当てられない

編集履歴

質問編集履歴

3

簡潔にまとめる

2020/12/08 23:41

投稿

rekutasu
rekutasu

スコア13

test CHANGED
File without changes
test CHANGED
@@ -60,7 +60,7 @@
60
60
 
61
61
  {
62
62
 
63
- PlayFirework();
63
+ Play();
64
64
 
65
65
  }
66
66
 
@@ -68,19 +68,11 @@
68
68
 
69
69
 
70
70
 
71
- void PlayFirework()
71
+ void Play()
72
72
 
73
73
  {
74
74
 
75
- var rand = Random.Range(0, 2);
76
-
77
-
78
-
79
- var r2 = Random.Range(0, 256);//redランダム
75
+ var r2 = Random.Range(0, 256);//r2ランダム
80
-
81
- var g2 = Random.Range(0, 256);//greenランダム
82
-
83
- var b2 = Random.Range(0, 256);//blueランダム
84
76
 
85
77
 
86
78
 
@@ -96,23 +88,7 @@
96
88
 
97
89
 
98
90
 
99
- //redを255固定でgreen,blueを0-255でランダム
100
-
101
- if (rand == 0)
102
-
103
- {
104
-
105
91
  newColor = new Color32(255, (byte)g2, (byte)b2, 255);
106
-
107
- }
108
-
109
-
110
-
111
- //greenを255固定でred,blueをランダム
112
-
113
- //blueを255固定でred,greenをランダム
114
-
115
-
116
92
 
117
93
 
118
94
 
@@ -142,8 +118,6 @@
142
118
 
143
119
  /*
144
120
 
145
- //redを255固定でgreen,blueを0-255でランダム
146
-
147
121
  if (rand == 0)
148
122
 
149
123
  {
@@ -154,15 +128,7 @@
154
128
 
155
129
 
156
130
 
157
- //greenを255固定でred,blueを0-255でランダム
158
-
159
- //blueを255固定でred,greenを0-255でランダム
160
-
161
-
162
-
163
131
  */
164
-
165
-
166
132
 
167
133
  newColor = new Color32(255, (byte)g2, (byte)b2, 255);
168
134
 

2

ソースコードの改善

2020/12/08 23:41

投稿

rekutasu
rekutasu

スコア13

test CHANGED
File without changes
test CHANGED
@@ -76,11 +76,19 @@
76
76
 
77
77
 
78
78
 
79
- var r2 = Random.Range(0, 255);//redランダム
79
+ var r2 = Random.Range(0, 256);//redランダム
80
80
 
81
- var g2 = Random.Range(0, 255);//greenランダム
81
+ var g2 = Random.Range(0, 256);//greenランダム
82
82
 
83
- var b2 = Random.Range(0, 255);//blueランダム
83
+ var b2 = Random.Range(0, 256);//blueランダム
84
+
85
+
86
+
87
+ 　　　　/*Randomについてご助言をいただき変更を加えました。
88
+
89
+ Random.Rangeはmax値が結果に含まれないので0-255のランダムにする場合は、Random.Range(0, 256)になる。
90
+
91
+ */
84
92
 
85
93
 
86
94
 

1

見やすくまとめるために編集を行った

2020/11/06 03:50

投稿

rekutasu
rekutasu

スコア13

test CHANGED
File without changes
test CHANGED
@@ -14,7 +14,7 @@
14
14
 
15
15
 
16
16
 
17
- 58行目： ps.startColor = newColor;//合わせて195075通り
17
+ 58行目： ps.startColor = newColor;
18
18
 
19
19
 
20
20
 
@@ -84,57 +84,31 @@
84
84
 
85
85
 
86
86
 
87
- var r1 = 255;//red固定
87
+ Color32 newColor;
88
-
89
- var g1 = 255;//greem固定
90
-
91
- var b1 = 255;//blue固定
92
88
 
93
89
 
94
90
 
95
- Color32 newColor;//C#にはスコープという概念があるので、if文の外で定義する
96
-
97
-
98
-
99
- //redを255固定でgreen,blueを0-255でランダム（65025通り）
91
+ //redを255固定でgreen,blueを0-255でランダム
100
92
 
101
93
  if (rand == 0)
102
94
 
103
95
  {
104
96
 
105
- newColor = new Color32((byte)r1, (byte)g2, (byte)b2, 255);
97
+ newColor = new Color32(255, (byte)g2, (byte)b2, 255);
106
98
 
107
99
  }
108
100
 
109
101
 
110
102
 
111
- //greenを255固定でred,blueを0-255でランダム（65025通り）
103
+ //greenを255固定でred,blueをランダム
112
104
 
113
- else if (rand == 1)
114
-
115
- {
116
-
117
- newColor = new Color32((byte)r2, (byte)g1, (byte)b2, 255);
118
-
119
- }
120
-
121
-
122
-
123
- //blueを255固定でred,greenを0-255でランダム(65025通り)
105
+ //blueを255固定でred,greenをランダム
124
-
125
- else if (rand == 2)
126
-
127
- {
128
-
129
- newColor = new Color32((byte)r2, (byte)g2, (byte)b1, 255);
130
-
131
- }
132
106
 
133
107
 
134
108
 
135
109
 
136
110
 
137
- ps.startColor = newColor;//合わせて195075通り
111
+ ps.startColor = newColor;//
138
112
 
139
113
 
140
114
 
@@ -160,45 +134,29 @@
160
134
 
161
135
  /*
162
136
 
163
- //redを255固定でgreen,blueを0-255でランダム（65025通り）
137
+ //redを255固定でgreen,blueを0-255でランダム
164
138
 
165
139
  if (rand == 0)
166
140
 
167
141
  {
168
142
 
169
- newColor = new Color32((byte)r1, (byte)g2, (byte)b2, 255);
143
+ newColor = new Color32(255, (byte)g2, (byte)b2, 255);
170
144
 
171
145
  }
172
146
 
173
147
 
174
148
 
175
- //greenを255固定でred,blueを0-255でランダム（65025通り）
149
+ //greenを255固定でred,blueを0-255でランダム
176
150
 
177
- else if (rand == 1)
178
-
179
- {
180
-
181
- newColor = new Color32((byte)r2, (byte)g1, (byte)b2, 255);
151
+ //blueを255固定でred,greenを0-255でランダム
182
-
183
- }
184
152
 
185
153
 
186
-
187
- //blueを255固定でred,greenを0-255でランダム(65025通り)
188
-
189
- else if (rand == 2)
190
-
191
- {
192
-
193
- newColor = new Color32((byte)r2, (byte)g2, (byte)b1, 255);
194
-
195
- }
196
154
 
197
155
  */
198
156
 
199
157
 
200
158
 
201
- newColor = new Color32((byte)r2, (byte)g2, (byte)b1, 255);
159
+ newColor = new Color32(255, (byte)g2, (byte)b2, 255);
202
160
 
203
161
  ```
204
162