Iteratorパターンの亊で、質問します。

質問

Iteratorパターンの亊で、質問します。

いつもお世話になっています。
http://www.techscore.com/tech/DesignPattern/Iterator.html#dp1-2,
http://www.techscore.com/tech/DesignPattern/Iterator2.htmlを参考にしたのですが、
このとき、Mainクラスを実行してもなにも表示されません。

それで、直してみました。
おそらく、原因は、StudentListクラスのaddメソッドだと思ったのですが、

変更した、StudentListクラス、、、
public class StudentList {

protected Student[] students;
private int last = 0;

public StudentList(int studentCount) {
this.students = new Student[studentCount]; }

public void add(Student student) {
System.out.println("student --");
System.out.println(student.getName() );
//ここに原因がある。。。
students[last] = student;
//追加...
last++;
}

public Student getStudentAt(int index) {
return students[index];　}

public int getLastNum() {
return last; }

実行にあたり、使用したクラスなどは、
Student,StudentList,Teacher,Main、です。

また、他にも、方法があるというのですが、
どんなのがあるかよくわかりません。
なんらかのヒントをもらえないでしょうか？

http://okwave.jp/qa/q5886678.htmlの問い、と被るのですが、
DBの接続、データ取得の汎用的な処理をDB接続用のクラスにもたせて、
具体的な取得方法など別に用意して、そのクラスに渡すというやりかたです。
この場合の考えかたを、Iteratorで考えるのはどうなのでしょうか？

このやり方だと、仮に、Teacherインターフェースに追加されると、
MyTeacherクラスにも追加が要求され、手間がかかる。

この場合、もっとも汎用性があるソースとはどんなのなのでしょうか？

よろしくお願いします。

回答No.5

hoge_piyo · ベストアンサー

nullチェックは、場合によるので難しいです。
そこら中全部にnullチェックをいれてもコードが読みづらくなりますし。

クラスを使う側がある程度限定されていてわかってるなら、クラスを使う側を含めてテストをちゃんと実施すれば、実際のコードにnullチェックを残す必要はないと思いますし。

nullチェックというか、範囲チェックも含めてStudentクラスのコンストラクタとかは、必要だと思います。nullや、不正な値を渡しても、その時にはエラーが起こらない場合です。（オブジェクトが本来なってはいけない不正な状態にもかかわらず、その時点ではエラーが起きない場合）
後で取り出す場合になってエラーになるとわかりづらいバグになるので。

お礼

解答ありがとうございます。

でしたら、nullに関して、ソースをまた作り変えてから新たに質問させてもらいます。

また、よろしくお願いします。

ながなが、とありがとうございました。

回答No.4

hoge_piyo

すいません、前の回答で一箇所間違えました。

public static int MAN = 0;
public static int WOMAN = 1;

ではなく

public static final int MAN = 0;
public static final int WOMAN = 1;

です。

回答No.3

hoge_piyo

＞この場合、性別は最終的にString型にした方がいいのでしょうか？

場合によりますが、メソッドから返すときも、特に理由がなければString型で返す必要はないと思います。もしも、文字としてわかりやすい形で取り出す必要がある場合は、enumの中でtoStringをオーバーライドしておくか、Studentクラスを使う側で変換すればいいですから。

あと、enumでSexを宣言したなら、引数の型をSexにしないとまったく意味が無いです。enumを使用した場合以下のようになります↓

public class Student{
private final String name;
private final Sex sex;

public Student(String name,Sex sex){
this.name = name;
this.sex = sex;
}
public String getName(){
return name;
}
public Sex getSex(){
return sex;
}
}

public enum Sex{MAN,WOMAN}

あと、intで定数にする場合は、private static finalではなく、public static finalです。privateで
は引数で渡せないので意味がありません。たとえばStudentクラスの中で

public static int MAN = 0;
public static int WOMAN = 1;

としておいて、Studentクラスを使う側は

new Student("たろう",Student.MAN);

というようにして使います。

キャストしてる箇所

((Aggregate)studentLists).iterator();
とか
System.out.println(((Student)itr.next()).getName());
とか

補足

解答ありがとうございます。
大変、助かりました。

エラーチェックを行う場合ですが、
この時のチェックが抜けているというのは、
if(studentLists !=null){}　とかといった処理なのでしょうか？

よろしくお願いします。

回答No.2

hoge_piyo

省略している部分

一番ひどいのは性別がintで、1か2でそのまま数値で渡しているところでしょうか。明らかにenumにするべきです。
(enumがない古いVersionの時に書かれたものだとしても、せめてpublic static finalで定数としてMANとWOMANを定義しておいて、それを使用するようにするべき。また1か2以外も渡せるようになっていて、チェックしてない)
あとはsetterメソッドのnullチェックとか、ジェネリックとか(ジェネリック使ってないからキャストしなければならなくなってる。そして、キャストに失敗した場合のエラー処理も無い)
配列のindexの範囲外にアクセスしてしまう可能性がある。

などでしょうか。

ただ、あくまでも実用的な観点から見た場合ですので、デザインパターンの説明のサンプルとしては、悪くはないとは思います。こういう短いサンプルではそういった(エラー処理などの)省略などはよくあることなので、そういった点を一応気に留めておいて読んだほうがいいです。

補足

解答ありがとうございます。

＞明らかにenumにするべきです。。。。
この場合、性別は最終的にString型にした方がいいのでしょうか？

直してみました。

public class Student {

private String name;
private String sex;

private static final int MAN = 1;
private static final int WOMAN = 2;

public Student(String name, int sex) {
this.name = name;
//性別
//enumを使う場合
if(sex == 1 ) {
this.sex = Sex.MAN.name();
}
else if(sex == 2) {
this.sex = Sex.WOMAN.name();
}

//final定数を使う場合
if(sex == MAN) {
this.sex = "MAN";
}
else if(sex == WOMAN) {
this.sex = "WOMAN";
}
}

public String getName() {
return this.name;
}

public String getSex() {
return this.sex;
}

}

public enum Sex {

MAN,
WOMAN

}

＞
あとはsetterメソッドのnullチェックとか、ジェネリックとか(ジェネリック使ってないからキャストしなければならなくなってる。
そして、キャストに失敗した場合のエラー処理も無い)。。。。

これは、どのクラスの亊なのでしょうか？

よろしくお願いします。

回答No.1

hoge_piyo

それで、直してみました。
おそらく、原因は、StudentListクラスのaddメソッドだと思ったのですが、

直したところはあってます。他にもスペルミスがあるし、これを書いた人は実際に実行してないみたいですね・・・

この場合、もっとも汎用性があるソースとはどんなのなのでしょうか？

普通はStudentクラスだけ作って、あとはJavaのクラスを利用するだけで、他のクラスはつくりません。この例だと、それ以外のクラスをつくっても(少なくとも今の機能では)汎用性がなく、殆ど無駄です。

Iteratorパターン自体が、もっと綺麗に実用的に(ジェネリックなども使って)すでにJavaで実装されているので。
あくまでも、Iteratorパターン自体を説明するために、必要最低限のクラスとメソッドを実装したサンプルと思った方がいいです。実用の観点から見ると色々やらなければいけないことを省略している部分が多いです。

補足

解答ありがとうございます。

＞実用の観点から見ると色々やらなければいけないことを省略している部分が多いです。
例えば、どういった部分なのでしょうか？

よろしくお願いします。