質問編集履歴

2

質問変更

2019/02/19 06:57

投稿

niconicodryyy
niconicodryyy

スコア36

test CHANGED
@@ -1 +1 @@
1
- コードレビューをお願いしたいです。
1
+ 解説をお願いしたいです。
test CHANGED
@@ -6,7 +6,11 @@
6
6
 
7
7
  また何故変換したものが規則正しく並ぶのでしょうか?
8
8
 
9
- 詳しい方コードレビューをお願いしたいです。
9
+ 詳しい方コードの解説お願いしたいです。
10
+
11
+ 解説ですが、右も左もわからない状態ですので1から100まで5歳児に教えるように「ここで文字列を取得してここでこういった動作をするからこうなる」と言ったように面倒でしょうが最初から最後まで教えて頂けると幸いです。わからないところはコードにコメントで書いておきました。
12
+
13
+ 今までのコメントは消しました。
10
14
 
11
15
 
12
16
 
@@ -116,10 +120,6 @@
116
120
 
117
121
 
118
122
 
119
-
120
-
121
- // ファイル選択画面へ
122
-
123
123
  private void FileSelect_Click(object sender, EventArgs e)
124
124
 
125
125
  {
@@ -138,13 +138,13 @@
138
138
 
139
139
 
140
140
 
141
- // 内容表示
141
+
142
142
 
143
143
  private void Display_Click(object sender, EventArgs e)
144
144
 
145
145
  {
146
146
 
147
- // ファイルエラーー---ファイル未選択
147
+
148
148
 
149
149
  if(String.IsNullOrEmpty(Fname.Text))
150
150
 
@@ -158,19 +158,19 @@
158
158
 
159
159
  {
160
160
 
161
- // 再度ファイルのバイナリ表示をするときはテキストボックスをリセット
161
+
162
162
 
163
163
  DataBox.ResetText();
164
164
 
165
165
 
166
166
 
167
- // ヘッダークラスの呼び出し
167
+
168
168
 
169
169
  header();
170
170
 
171
171
 
172
172
 
173
- // 表示内容クラスの呼び出し
173
+
174
174
 
175
175
  binaryviewer();
176
176
 
@@ -180,7 +180,7 @@
180
180
 
181
181
 
182
182
 
183
- // OKをクリックで閉じる
183
+
184
184
 
185
185
  private void Close_Click(object sender, EventArgs e)
186
186
 
@@ -202,9 +202,9 @@
202
202
 
203
203
  //ここからどいった動きをしているかわかりません。
204
204
 
205
-
205
+     //ここからどいった動きをしているかわかりません。
206
+
206
-
207
+     //ここからどいった動きをしているかわかりません。
207
-
208
208
 
209
209
 
210
210
 
@@ -216,7 +216,7 @@
216
216
 
217
217
 
218
218
 
219
- // インデックス分のスペースを表示
219
+
220
220
 
221
221
  for (i = 0; i < 12; i++)
222
222
 
@@ -228,7 +228,7 @@
228
228
 
229
229
 
230
230
 
231
- // バイナリ表示のヘッダー
231
+
232
232
 
233
233
  for (i = 0; i <= 15; i++)
234
234
 
@@ -248,7 +248,7 @@
248
248
 
249
249
 
250
250
 
251
- // 文字表示のヘッダー
251
+
252
252
 
253
253
  for (i = 0; i <= 15; i++)
254
254
 
@@ -274,7 +274,7 @@
274
274
 
275
275
  {
276
276
 
277
- // x:インデックス表示のカウント用、y:バイナリ表示のカウント用、z:文字表示のカウント用、n:バイト調整に使用
277
+
278
278
 
279
279
  int x = 0;
280
280
 
@@ -288,23 +288,23 @@
288
288
 
289
289
  FileStream fs = new FileStream(openFileDialog1.FileName, FileMode.Open);
290
290
 
291
- byte[] data = new byte[fs.Length]; // byteに変換
291
+ byte[] data = new byte[fs.Length];
292
292
 
293
293
  fs.Read(data, 0, data.Length);
294
294
 
295
- string[] str = new string[fs.Length]; // 全角文字等の「.」格納用
295
+ string[] str = new string[fs.Length];
296
296
 
297
297
  fs.Close();
298
298
 
299
299
 
300
300
 
301
- // ファイル内容のループ
301
+
302
302
 
303
303
  while (y < data.Length)
304
304
 
305
305
  {
306
306
 
307
- // インデックス
307
+
308
308
 
309
309
  DataBox.Text += string.Format("{0:X7}" + '0', x);
310
310
 
@@ -314,8 +314,6 @@
314
314
 
315
315
 
316
316
 
317
- // バイナリ表示のループ
318
-
319
317
  while (y < data.Length)
320
318
 
321
319
  {
@@ -324,7 +322,7 @@
324
322
 
325
323
  y++;
326
324
 
327
- // 15バイト目が終わったらスペースを表示し、バイナリループを抜ける
325
+
328
326
 
329
327
  if (y % ST == 0)
330
328
 
@@ -340,7 +338,7 @@
340
338
 
341
339
 
342
340
 
343
- // 最終行のバイナリ表示が15バイトより前で終わった場合---不足している分をスペースで調整
341
+
344
342
 
345
343
  if ((y == data.Length) && (y % ST != 0))
346
344
 
@@ -358,13 +356,13 @@
358
356
 
359
357
 
360
358
 
361
- // 文字表示のループ
359
+
362
360
 
363
361
  while (z < data.Length)
364
362
 
365
363
  {
366
364
 
367
- if (data[z] >= 0x20 && data[z] <= 0x7e) // 半角文字
365
+ if (data[z] >= 0x20 && data[z] <= 0x7e)
368
366
 
369
367
  {
370
368
 
@@ -374,7 +372,7 @@
374
372
 
375
373
  }
376
374
 
377
- else // 全角文字、不可視文字は「.」に変換
375
+ else
378
376
 
379
377
  {
380
378
 
@@ -386,7 +384,7 @@
386
384
 
387
385
  }
388
386
 
389
- // 15バイト目が終わったら改行して文字ループを抜ける---全体のループ先頭へ
387
+
390
388
 
391
389
  if (z % ST == 0)
392
390
 

1

コードを見やすく。質問を修正。

2019/02/19 06:57

投稿

niconicodryyy
niconicodryyy

スコア36

test CHANGED
File without changes
test CHANGED
@@ -10,6 +10,16 @@
10
10
 
11
11
 
12
12
 
13
+ 先生の作った模範回答なら先生に聞きなさいとの事ですが、この模範解答は今、授業をしていただいてる先生と違う先生が作ったみたいで。詳しい事を聞きに言っても「私のコードじゃないと」逃げられてしまいます。実際に作った先生は誰かわかりません。
14
+
15
+ また、コード全部とは言わないので「ここからここはどういう意味なんですか?」と言った文法に関する限定的な質問も、現在この課題と違う課題がメインとなっているので、今すべきことをしてくださいと取り合ってくれず、友達も理解できていないので聞ける人がいないのが現状です。
16
+
17
+ ルール違反をしている場違いな質問かも知れませんが、どなたかお答えいただけると幸いです。
18
+
19
+
20
+
21
+
22
+
13
23
  詳しいルールと致しまして。以下の内容となっております。
14
24
 
15
25
  1、ボタンをクリックしたらファイル参照ダイアログを開いてファイルを選択させる。
@@ -54,7 +64,9 @@
54
64
 
55
65
  ### 該当のソースコード
56
66
 
57
- //C#言語
67
+ ```C#
68
+
69
+ コード
58
70
 
59
71
 
60
72
 
@@ -106,7 +118,7 @@
106
118
 
107
119
 
108
120
 
109
- // ファイル選択画面へ
121
+ // ファイル選択画面へ
110
122
 
111
123
  private void FileSelect_Click(object sender, EventArgs e)
112
124
 
@@ -132,33 +144,33 @@
132
144
 
133
145
  {
134
146
 
135
- // エラメッセージ---ファイル未選択
147
+ // ファイルエラーー---ファイル未選択
136
148
 
137
149
  if(String.IsNullOrEmpty(Fname.Text))
138
150
 
139
151
  {
140
152
 
141
- DialogResult dr = MessageBox.Show("ファイルが選択されていません。参照ボタンより、ファイルを選択してください。",
153
+ DialogResult dr = MessageBox.Show("ファイルが選択されていません。参照ボタンより、ファイルを選択してください。",
142
-
154
+
143
- "メッセージ", MessageBoxButtons.OK, MessageBoxIcon.Exclamation);
155
+ "メッセージ", MessageBoxButtons.OK, MessageBoxIcon.Exclamation);
144
156
 
145
157
  } else
146
158
 
147
159
  {
148
160
 
149
- // 再度ファイルバイナリ表示をするときはテキストボックスリセット
161
+ // 再度ファイルバイナリ表示をするときはテキストボックスリセット
150
162
 
151
163
  DataBox.ResetText();
152
164
 
153
165
 
154
166
 
155
- // ヘッダークラスの呼び出し
167
+ // ヘッダークラスの呼び出し
156
168
 
157
169
  header();
158
170
 
159
171
 
160
172
 
161
- // 表示内容クラスの呼び出し
173
+ // 表示内容クラスの呼び出し
162
174
 
163
175
  binaryviewer();
164
176
 
@@ -168,7 +180,7 @@
168
180
 
169
181
 
170
182
 
171
- // OKをクリックで閉じる
183
+ // OKをクリックで閉じる
172
184
 
173
185
  private void Close_Click(object sender, EventArgs e)
174
186
 
@@ -204,7 +216,7 @@
204
216
 
205
217
 
206
218
 
207
- // インデックス分のスペースを表示
219
+ // インデックス分のスペースを表示
208
220
 
209
221
  for (i = 0; i < 12; i++)
210
222
 
@@ -216,7 +228,7 @@
216
228
 
217
229
 
218
230
 
219
- // バイナリ表示のヘッダー
231
+ // バイナリ表示のヘッダー
220
232
 
221
233
  for (i = 0; i <= 15; i++)
222
234
 
@@ -236,7 +248,7 @@
236
248
 
237
249
 
238
250
 
239
- // 文字表示のヘッダー
251
+ // 文字表示のヘッダー
240
252
 
241
253
  for (i = 0; i <= 15; i++)
242
254
 
@@ -262,7 +274,7 @@
262
274
 
263
275
  {
264
276
 
265
- // x:インデックス表示のカウント用、y:バイナリ表示のカウント用、z:文字表示のカウント用、n:バイト調整に使用
277
+ // x:インデックス表示のカウント用、y:バイナリ表示のカウント用、z:文字表示のカウント用、n:バイト調整に使用
266
278
 
267
279
  int x = 0;
268
280
 
@@ -286,13 +298,13 @@
286
298
 
287
299
 
288
300
 
289
- // ファイル内容のループ
301
+ // ファイル内容のループ
290
302
 
291
303
  while (y < data.Length)
292
304
 
293
305
  {
294
306
 
295
- // インデックス
307
+ // インデックス
296
308
 
297
309
  DataBox.Text += string.Format("{0:X7}" + '0', x);
298
310
 
@@ -302,7 +314,7 @@
302
314
 
303
315
 
304
316
 
305
- // バイナリ表示のループ
317
+ // バイナリ表示のループ
306
318
 
307
319
  while (y < data.Length)
308
320
 
@@ -312,7 +324,7 @@
312
324
 
313
325
  y++;
314
326
 
315
- // 15バイト目が終わったらスペースを表示し、バイナリループを抜ける
327
+ // 15バイト目が終わったらスペースを表示し、バイナリループを抜ける
316
328
 
317
329
  if (y % ST == 0)
318
330
 
@@ -328,7 +340,7 @@
328
340
 
329
341
 
330
342
 
331
- // 最終行のバイナリ表示が15バイトより前で終わった場合---不足している分をスペースで調整
343
+ // 最終行のバイナリ表示が15バイトより前で終わった場合---不足している分をスペースで調整
332
344
 
333
345
  if ((y == data.Length) && (y % ST != 0))
334
346
 
@@ -346,7 +358,7 @@
346
358
 
347
359
 
348
360
 
349
- // 文字表示のループ
361
+ // 文字表示のループ
350
362
 
351
363
  while (z < data.Length)
352
364
 
@@ -374,7 +386,7 @@
374
386
 
375
387
  }
376
388
 
377
- // 15バイト目が終わったら改行して文字ループを抜ける---全体のループ先頭へ
389
+ // 15バイト目が終わったら改行して文字ループを抜ける---全体のループ先頭へ
378
390
 
379
391
  if (z % ST == 0)
380
392
 
@@ -405,3 +417,5 @@
405
417
  }
406
418
 
407
419
  }
420
+
421
+ ```