回答編集履歴

3

追記2

2016/06/20 00:27

投稿

ttyp03
ttyp03

スコア16998

test CHANGED
@@ -61,3 +61,79 @@
61
61
  追記
62
62
 
63
63
  methodというメソッド名は、処理の内容がわからないのでよろしくないです。
64
+
65
+
66
+
67
+ 追記2
68
+
69
+ FieldChkクラスのサンプルです(イメージレベル)
70
+
71
+ ```Java
72
+
73
+ class FieldChk {
74
+
75
+ private ファイル名;
76
+
77
+ private 読み込みバッファ;
78
+
79
+ コンストラクタ(引数){
80
+
81
+ 引数をファイル名に保持
82
+
83
+ }
84
+
85
+ public read(){
86
+
87
+ FReadクラスでファイルを読み込んで、読み込みバッファに格納
88
+
89
+ }
90
+
91
+ public boolean check1(){
92
+
93
+ 読み込みバッファを使ってチェック1処理
94
+
95
+ 結果を返す(true/false)
96
+
97
+ }
98
+
99
+ public boolean check2(){
100
+
101
+ 読み込みバッファを使ってチェック2処理
102
+
103
+ 結果を返す(true/false)
104
+
105
+ }
106
+
107
+ public boolean check3(){
108
+
109
+ 読み込みバッファを使ってチェック3処理
110
+
111
+ 結果を返す(true/false)
112
+
113
+ }
114
+
115
+ }
116
+
117
+ ```
118
+
119
+ check1~3メソッドの戻り値の型をbooleanに変更しました。
120
+
121
+ ok/ngを判定するメソッドなので、そちらの方がふさわしいかと思います。
122
+
123
+ なのでmainの方では、次のように判定できます。
124
+
125
+
126
+
127
+ ```Java
128
+
129
+ if(Chk.check1() == true && Chk.check2() == true && Chk.check3() == true) {
130
+
131
+
132
+
133
+ もしくは
134
+
135
+
136
+
137
+ if(Chk.check1() && Chk.check2() && Chk.check3()) {
138
+
139
+ ```

2

追記

2016/06/20 00:27

投稿

ttyp03
ttyp03

スコア16998

test CHANGED
@@ -55,3 +55,9 @@
55
55
  }
56
56
 
57
57
  ```
58
+
59
+
60
+
61
+ 追記
62
+
63
+ methodというメソッド名は、処理の内容がわからないのでよろしくないです。

1

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

2016/06/17 08:21

投稿

ttyp03
ttyp03

スコア16998

test CHANGED
@@ -1,4 +1,4 @@
1
- 全部のクラスが掲載されていないので違うかもしれませんが、FieldChk01クラス、FieldChk02クラス、FReadDistinctクラス、それぞれでファイルの読み込み処理を行っているように見えます。
1
+ FieldChk01クラス、FieldChk02クラス、FReadDistinctクラス、それぞれでファイルの読み込み処理を行っているのが無駄かと思います。
2
2
 
3
3
  クラスは1つにして、コンストラクタでファイル名を渡す。
4
4
 
@@ -8,7 +8,7 @@
8
8
 
9
9
  各メソッドでは読み込み済みのデータでチェック処理をして、結果を返却する。
10
10
 
11
- このような感じの方がスマートな作りになると思います。
11
+ このような感じの方がスマートな作り且つオブジェクト指向的な感じになると思います。
12
12
 
13
13
 
14
14
 
@@ -54,6 +54,4 @@
54
54
 
55
55
  }
56
56
 
57
-
58
-
59
57
  ```