JAVA Listから重複項目の表示について

Question

JAVAでオブジェクトを定義してEメールアドレスなどの重複を表示するプログラムを作成しています。

public class User {
	/** メールアドレス */
	private String email;

	public String getEmail() {
		return email;
	}

	public void setEmail(String email) {
		this.email = email;
	}

}

import java.util.HashSet;
import java.util.List;
import java.util.Set;

public class Test {

	public static void main(String[] args) {
		// ここにコードを挿入
		// メールアドレスset
		Set<String> set = new HashSet<String>();

		
		List<User> userList = asList('ABC@', 'testa@', 'ABC@', 'testb@');

		for(User user : userList) {
			String bufEmail = user.getEmail();
			if(set.contains(bufEmail)) {
				// 存在する場合
				System.out.println(bufEmail + "が重複しています。");
			} else {
				// 存在しない場合、setに追加
				set.add(bufEmail);
			}
		}

	}

}

この処理を実施して、「ABC@」が重複しています。
と表示させたいです。
Testクラスの「List<User> userList = asList('ABC@', 'testa@', 'ABC@', 'testb@');」
の記述が間違っているみたいですが、どこが間違いなのか教えていただきたく思います。
宜しくお願いします。

Comments

[ozwk]

質問者様のスキルの確認です。1.Userクラスのインスタンスを1つ生成して、emailを設定するコードを書けますか? 2.Listを使ったことがありますか?

Answer 1

Set.addメソッドは、実行した結果Setが変更された場合(=Setに引数の要素がなかった場合)trueを返し、変更されなかった場合(=Setに引数の要素があった場合)falseを返します。これを利用して判別できます。
あと、先の人も回答していますが、asListはArrays.asListでしょうか？

public class Test {

    public static void main(String[] args) {
        // ここにコードを挿入
        // メールアドレスset
        Set<String> set = new HashSet<String>();

        // java.util.Arraysをインポート
        List<String> userList = Arrays.asList("ABC@", "testa@", "ABC@", "testb@");

        for(String user : userList) {
            //setに追加してみて、追加できなかった(重複していた)場合
            if(!set.add(user)){
                System.out.println(user + "が重複しています。");
            }
        }

    }

}

Comments

[kumakumatan]

ありがとうございます。
とても素晴らしいです。

Answer 2

List<String> userList = Arrays.asList("ABC@", "testa@", "ABC@", "testb@");

これで、連続した文字列(String)をList型のクラスへ格納できます。一気に別の型(Userなど)へ変換する場合には、文字列からUserを作るための変換処理が必要です。

Userへの変換処理はとりあえず置いて、まずはそれぞれの文字列の重複がないかを調べる、で良いかと思いますよ。

Answer 3

class User {

	private String email;

	public User(String email) {
		this.email = email;
	}

	public String getEmail() {
		return email;
	}

	public void setEmail(String email) {
		this.email = email;
	}
}

public class Main {
	public static void main(String[] args) throws Exception {
		List<User> userList = Arrays.asList(
				new User("ABC@"),
				new User("testa@"),
				new User("ABC@"),
				new User("testb@"));

		userList.stream()
				.collect(Collectors.groupingBy(User::getEmail, Collectors.counting()))
				.forEach((k, v) -> {
					if (v > 1) {
						System.out.println(k + " が重複しています。");
					}
				});
	}
}

Comments

[kumakumatan]

ご回答ありがとございます。
こちらの処理は今は直接、値を設定していますが。例えば下記のようなCSVファイルを読み込んで、処理する場合はどうなりますでしょうか？
===CSVファイル======
ABC123,テスト　Ａ,123456,060-5938-1234,
ABC456,テスト　Ｂ,1234567,060-7886-567,
ABC789,テスト　Ｃ,1234568,060-3981-8910,
ABCD123,テスト　Ｄ,123459,060-6487-8911,
ABCD456,テスト　Ｅ,123459,060-4697-8912,テスト
ABCD789,テスト　Ｆ,1234581,060-4577-8913,
ABCD12,テスト　Ｇ,1234582,060-1266-8914,
ABCD34,テスト　Ｈ,12345834,060-7412-8915,
========================================
3列目のチェックをして「123459が重複しています。」とメッセージを表示させていです。
宜しくお願いします。

Answer 4

突っ込みどころが多すぎて…

まず第一にasListというメソッドはあなたのコード上存在していません。
おそらく以下のような処理を期待していると思います。

public class Test {
    public static void main(String[] args) {
        //省略
    }
    private static List<User> asList(String... emails) {
        List<User> list = new ArrayList();
        for(int i=0;i<emails.length;i++) {
            User user = new User();
            user.setEmail(emails[i]);
            list.add(user);
        }
        return list;
    }
}

次にasListに渡す引数がシングルクォーテーションで書かれていることが問題です。
Javaでのシングルクォーテーションは「文字」であり、「文字列」ではありません。2文字以上の長さの文字列はダブルクォーテーションで囲む必要があります。

余談ですが、そもそもJavaのSetクラスは重複を自動で判定してくれます。
とりあえず全メールアドレスを突っ込めば、変数setの中はダブリのないメールアドレス一覧になっています。
重複である、というメッセージを出すことが目的であれば、判定部分はいまのままのコードで大丈夫です。

#追記
追加したasListはstaticメソッドというものです。メソッドを定義すべき場所、staticの意味を理解できていないのでしたら、そのへんは必ず勉強したほうが今後のためになるでしょう。

Comments

[kumakumatan]

ご回答ありがとうございます。
記載していただいたコードをどの部分に埋め込めばいいでしょうか？
Testクラスの中に記載したらよろしいでしょうか？
宜しくお願いします。

[masaya_ohashi]

ちょっと訂正しました。
Testクラスのmainから呼び出すのであれば、Testクラスの中で、かつstaticをつけてメソッド定義すればよいです。

[kumakumatan]

ありがとうございます。
頭の中がぐちゃぐちゃになりました。
具体的にどのように記載したらよいかご教授いただけますでしょうか？
お願いします。

[kumakumatan]

ありがとうございます。
やりたい事ができました。
「static」はプログラム実行開始直前に1つだけ実体化され、プログラム終了まで存在し続けるとの認識です。
「asList(String... emails)」の「...」は何を意味するのでしょうか。
宜しくお願いします。

[masaya_ohashi]

staticの認識はそれで正しいです。
...は可変長引数と呼ばれるもので、可変長引数とは長さを自由に持つことができる引数です。
asListには今4つの引数が渡されていますが、そこにいくらでも書き足すことで一気に10個だろうが100個だろうが渡せます。そして、asListで受け取ったemailsは配列としてアクセスすることが可能です。
渡された可変長引数は必ず引数の最後に書かなければいけないという制約がありますが、使い方を知っていれば便利なものです。詳しくは自分で調べてみてください。

Answer 5

asListはどのクラスのメソッドですか？見たことありません。
また、シングルクォーテーションで文字を囲っていますが、ダブルクォーテーションの誤りじゃないでしょうか。

リストを作るのであればArrayListをインスタンス化し、ArrayList#addで要素を追加しましょう。