Listデータを重復せずにSetに格納したい

質問

お世話になっています。

以下のクラスがあるとき

public class UserFormBean_2 {

private int userNum;
private String userId;
private String password;

public void setUserNum(int userNum) {
this.userNum = userNum;

}

public int getUserNum() {
return this.userNum;

}

public void setUserId(String userId) {
this.userId = userId;

}

public String getUserId() {
return this.userId;

}

public void setPassowd(String password) {
this.password = password;

}

public String getPassword() {
return this.password;

}

public String toString() {

return this.userId;

}

}

Listにこのクラスが格納されている場合、
userIdをキーに、userIdを重複せずに格納することは可能なのでしょうか？
実行ソースは以下です。
public class Run{
public static void main(String[] args){

Set ss = new HashSet();

List array2 = new ArrayList();
(データ1)
UserFormBean_2 userFr = new UserFormBean_2();
userFr.setUserId("maki");
userFr.setPassowd("なつのおとしご");
array2.add(userFr);

//仮に、先にSetに値を格納する
ss.add(userFr);

(データ2)
userFr = new UserFormBean_2();
userFr.setUserId("sayaka");
userFr.setPassowd("motomiti");
array2.add(userFr);

(データ3)
userFr = new UserFormBean_2();
userFr.setUserId("neneko");
userFr.setPassowd("ゆずは!");
array2.add(userFr);

(データ4)
userFr = new UserFormBean_2();
userFr.setUserId("森の山手");
userFr.setPassowd("なつのおとしご");
array2.add(userFr);
//仮にSetに値を格納する
ss.add(userFr);

(データ5)
userFr = new UserFormBean_2();
userFr.setUserId("maki");
userFr.setPassowd("なつのおとしご");
array2.add(userFr);

System.out.println("size:"+array2.size());

System.out.println("set_size:"+ss.size());

UserFormBean_2 uB = new UserFormBean_2();
UserFormBean_2 uf = new UserFormBean_2();

System.out.println(" -- set -- ");
}
}

よろしくお願いします。

for( Iterator u=array2.iterator();u.hasNext(); ) {

uB = (UserFormBean_2)u.next();
System.out.println(uB.getUserId());
for(Iterator s=ss.iterator();s.hasNext(); ) {

uf = (UserFormBean_2)s.next();
System.out.println(uf.getUserId());

ss.add(uB);

}

回答No.4

root139 · ベストアンサー

> 一般的にSetはあまり使わない方がいいのでしょうか？

ケース・バイ・ケースだと思います。
String や Integer などの様に等価性が明確なオブジェクトを格納するので有れば、Setを使うことは問題有りません。

もし、UserFormBean_2 で userId が一意性を表すフィールドであれば、先の equals と hashCode の実装に問題は無く、したがってSetを使うことに問題は有りません。
言い換えれば、 userId が同じ UserFormBean_2 オブジェクトは他のフィールドも同じになるべきなのであれば、Setを使うことに問題は無いでしょう。

逆に、userId が同じでuserNumかpasswordの異なるオブジェクトがいくつか有り、その中から最初のものを取り出すのであれば、先の equals と hashCode の実装は間違いであり、 Mapを使うべきでしょう。

なお、 Map を使う場合でも、キーとするオブジェクトは (Setの要素と同様に) 等価性が明確なものを選ぶ必要が有ります。
JDKのソースを見れば分かりますが、実際に HashSet は HashMap を使って実装されています。

お礼

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

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

回答No.3

mononoke99

UserFormBean_2#userIdをキーとするjava.util.HashMapを使いましょう。
キーの評価はHashMap#containsKey(Object)を使用すればよいです。

UserFormBean_2 bean1 = new UserFormBean_2();
bean1.setUserId("user1");
UserFormBean_2 bean2 = new UserFormBean_2();
bean2.setUserId("user2");
UserFormBean_2 beanAdd = new UserFormBean_2();
beanAdd.setUserId("user1");

Map map = new HashMap();
map.put(bean1.getUserId(), bean1);
map.put(bean2.getUserId(), bean2);

if (!map.containsKey(beanAdd.getUserId())) {
map.put(beanAdd.getUserId(), beanAdd);
}

System.out.println(map);

格納コンテナ側のアクセサに評価実装を追加するならまだしも
Entity側に依存した実装を追加するのはお世辞にもよいデザインとは言えません。

もしSetオブジェクトを最終的に生成したいのであれば
Set.addAll(Collection)を使用し

Set set = new HashSet();
set.addAll(map.values());

とすればよいです。

補足

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

試してみました。
その際、
mapにuserFormBeanクラスデータを格納時、
すでにkeyが同じなら、その時に格納されないようになっているようなのですが、
一般的にSetはあまり使わない方がいいのでしょうか？

今回だと、Mapでいけると思ったので。。

よろしくお願いします。

回答No.2

root139

> boolean isExists = uf.equals( uB.getUserId());

は、UserFormBean_2 と String を比べているので、必ず false になりますよ。

boolean isExists = uf.equals( uB );

か、

boolean isExists = uf.getUserId().equals( uB.getUserId() );

では？

お礼

解答ありがとうございます。
無事解决できました。

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

回答No.1

root139

端的に言えば、UserFormBean_2 で equals() と hashCode() をオーバーライドして、userId のみによってそれらの戻り値を導出すれば、Set に格納したものは userId が一意となります。

ただし、 equals() と hashCode() をオーバーライドすると言うことは、そのクラスの等価性を定義することになります。
つまり
「userId の値が同じであれば、UserFormBean_2オブジェクト自体も等しいと見なす」
と決めたことになります。

例) ---------------------------------------------------------------------
public class UserFormBean_2 {
　・
　・
　・
　public int hashCode() {
　　return userId != null ? userId.hashCode() : 0;
　}

　public boolean equals(Object obj) {
　　if (this == obj) return true;
　　if (!(obj instanceof UserFormBean_2)) return false;
　　UserFormBean_2 other = (UserFormBean_2) obj;
　　if (userId == null) return other.userId == null;
　　return userId.equals(other.userId);
　}
}
-------------------------------------------------------------------------

参考URL：

http://www.ibm.com/developerworks/jp/java/library/j-jtp05273/

補足

さっそくの解答ありがとうございます。
参考URL確認してみました。
それで試してみたのですが、
List内のデータとSet内のデータが等しい時に「true」を返ってきません。

作成手順は以下です。
//UserFormBean_2クラスのインスタンス
UserFormBean_2 uB = new UserFormBean_2();
UserFormBean_2 uf = new UserFormBean_2();

//List内のループ
for(Iterator a=array2.iterator();a.hasNext(); ) {
uB = (UserFormBean_2)a.next();
System.out.println("list_Value:"+uB.getUserId());
//Set内のループ
for(Iterator s=ss.iterator();s.hasNext(); ) {

uf = (UserFormBean_2)s.next();
System.out.println("set_Value:" + uf.getUserId());
//データのSet内での有無
boolean isExists = uf.equals( uB.getUserId()));
}

}

UserFormBean_2クラスには、equals()メソッドとhashCode()メソッドを提示してもらった通りでオーバーライドしています。

よろしくお願いします。