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

回答編集履歴

3

追記2

2016/06/20 00:27

投稿

ttyp03
ttyp03

スコア17002

answer CHANGED
@@ -29,4 +29,42 @@
29
29
  ```
30
30
 
31
31
  追記
32
- methodというメソッド名は、処理の内容がわからないのでよろしくないです。
32
+ methodというメソッド名は、処理の内容がわからないのでよろしくないです。
33
+
34
+ 追記2
35
+ FieldChkクラスのサンプルです(イメージレベル)
36
+ ```Java
37
+ class FieldChk {
38
+ private ファイル名;
39
+ private 読み込みバッファ;
40
+ コンストラクタ(引数){
41
+ 引数をファイル名に保持
42
+ }
43
+ public read(){
44
+ FReadクラスでファイルを読み込んで、読み込みバッファに格納
45
+ }
46
+ public boolean check1(){
47
+ 読み込みバッファを使ってチェック1処理
48
+ 結果を返す(true/false)
49
+ }
50
+ public boolean check2(){
51
+ 読み込みバッファを使ってチェック2処理
52
+ 結果を返す(true/false)
53
+ }
54
+ public boolean check3(){
55
+ 読み込みバッファを使ってチェック3処理
56
+ 結果を返す(true/false)
57
+ }
58
+ }
59
+ ```
60
+ check1~3メソッドの戻り値の型をbooleanに変更しました。
61
+ ok/ngを判定するメソッドなので、そちらの方がふさわしいかと思います。
62
+ なのでmainの方では、次のように判定できます。
63
+
64
+ ```Java
65
+ if(Chk.check1() == true && Chk.check2() == true && Chk.check3() == true) {
66
+
67
+ もしくは
68
+
69
+ if(Chk.check1() && Chk.check2() && Chk.check3()) {
70
+ ```

2

追記

2016/06/20 00:27

投稿

ttyp03
ttyp03

スコア17002

answer CHANGED
@@ -26,4 +26,7 @@
26
26
 
27
27
  }
28
28
  }
29
- ```
29
+ ```
30
+
31
+ 追記
32
+ methodというメソッド名は、処理の内容がわからないのでよろしくないです。

1

質問内容が更新されていたので訂正

2016/06/17 08:21

投稿

ttyp03
ttyp03

スコア17002

answer CHANGED
@@ -1,9 +1,9 @@
1
- 全部のクラスが掲載されていないので違うかもしれませんが、FieldChk01クラス、FieldChk02クラス、FReadDistinctクラス、それぞれでファイルの読み込み処理を行っているように見えます。
1
+ FieldChk01クラス、FieldChk02クラス、FReadDistinctクラス、それぞれでファイルの読み込み処理を行っているのが無駄かと思います。
2
2
  クラスは1つにして、コンストラクタでファイル名を渡す。
3
3
  コンストラクタか読み込み用のメソッドを作って、ファイルを読み込む。
4
4
  チェック用のメソッドを3つ用意する。
5
5
  各メソッドでは読み込み済みのデータでチェック処理をして、結果を返却する。
6
- このような感じの方がスマートな作りになると思います。
6
+ このような感じの方がスマートな作り且つオブジェクト指向的な感じになると思います。
7
7
 
8
8
  メインの方の流れだけ書いてみました。
9
9
  メソッド名とかは適当です。
@@ -26,5 +26,4 @@
26
26
 
27
27
  }
28
28
  }
29
-
30
29
  ```