回答編集履歴
2
コードを見て追記
test
CHANGED
@@ -8,11 +8,17 @@
|
|
8
8
|
|
9
9
|
せっかくなので気になるところを指摘。
|
10
10
|
|
11
|
+
(指摘はしますが質問内容と外れるところが多いため、挙げたことを質問されてもここでは答えません。
|
12
|
+
|
13
|
+
復習するなり別途1つずつ質問を立てるなりしてください。)
|
14
|
+
|
15
|
+
|
16
|
+
|
11
17
|
|
12
18
|
|
13
19
|
### 質問文の内容から
|
14
20
|
|
15
|
-
- コンストラクタクラス
|
21
|
+
- コンストラクタクラスという用語がわかりません。
|
16
22
|
|
17
23
|
- 今現在appletに拘る理由がない気がしますが、なぜappletでやりたかったのでしょう?
|
18
24
|
|
@@ -28,7 +34,7 @@
|
|
28
34
|
|
29
35
|
|
30
36
|
|
31
|
-
### コードから
|
37
|
+
### 全体的なコードから
|
32
38
|
|
33
39
|
- インデントを付けましょう。
|
34
40
|
|
@@ -37,3 +43,23 @@
|
|
37
43
|
- 英単語はなるべく正しく書きましょう。(speace→space)
|
38
44
|
|
39
45
|
- 大文字小文字の規則は統一させましょう。
|
46
|
+
|
47
|
+
|
48
|
+
|
49
|
+
### Mineクラスから
|
50
|
+
|
51
|
+
- インポートしているものはJFrame以外ないので、`import javax.swing.*`以外不要です。
|
52
|
+
|
53
|
+
- mainメソッドしかないならJFrameを継承している意味がないです。
|
54
|
+
|
55
|
+
- whileループでキー判定をしていますが、MyFrameクラスのキーイベントで良いのでは?
|
56
|
+
|
57
|
+
|
58
|
+
|
59
|
+
### MyFrameクラスから
|
60
|
+
|
61
|
+
- Mineクラスをインスタンス化している理由が不明です。
|
62
|
+
|
63
|
+
- クラス名からこちらがJFrameクラスを継承すべきな気がします。
|
64
|
+
|
65
|
+
- `key_inport`メソッド(これも誤字)はどこから呼び出されるのですか?
|
1
いろいろ指摘追加
test
CHANGED
@@ -1 +1,39 @@
|
|
1
1
|
パッとしか見てませんが、どのキーイベントも例外を最初に投げてるから動作しないのではないですか?
|
2
|
+
|
3
|
+
|
4
|
+
|
5
|
+
---
|
6
|
+
|
7
|
+
|
8
|
+
|
9
|
+
せっかくなので気になるところを指摘。
|
10
|
+
|
11
|
+
|
12
|
+
|
13
|
+
### 質問文の内容から
|
14
|
+
|
15
|
+
- コンストラクタクラスってなんですか?
|
16
|
+
|
17
|
+
- 今現在appletに拘る理由がない気がしますが、なぜappletでやりたかったのでしょう?
|
18
|
+
|
19
|
+
|
20
|
+
|
21
|
+
### 質問文の書き方から
|
22
|
+
|
23
|
+
- 「書き方がわからない」のと「動作しない」のでは意味が異なります。
|
24
|
+
|
25
|
+
「動作しない」のであれば理想の出力と現状の出力、もしくはエラー文を提示しましょう。
|
26
|
+
|
27
|
+
- 1質問1要件が理想です。ついでにーという質問が増えるのは回答しにくくなるのでおすすめしません。
|
28
|
+
|
29
|
+
|
30
|
+
|
31
|
+
### コードから
|
32
|
+
|
33
|
+
- インデントを付けましょう。
|
34
|
+
|
35
|
+
- 不必要な空行はなくしましょう。
|
36
|
+
|
37
|
+
- 英単語はなるべく正しく書きましょう。(speace→space)
|
38
|
+
|
39
|
+
- 大文字小文字の規則は統一させましょう。
|