正規表現とArrayListを使ったコードについて

Question

実現したいこと

userIdArrayにあるID(ASD1234,asdfgh1,aSd1234,asd1223,mmm1２34,1234ASD)を正規表現でチェックし、OKリストとNGリストに振り分けたいです。
【チェック内容】
・１～３文字目は大文字半角英字→OKリストへ
・４～７文字目は半角数字→OKリストへ
・７文字以上→OKリストへ
・その他→NGリストへ

↓のような出力内容にしたいです。（IDの出力順序は問いません）
OKリスト---------------
・ASD1234
NGリスト---------------
・asdfgh1
・aSd1234
・asd1223
・mmm1２34
・1234ASD

発生している問題・分からないこと

【for文とif文を使ったver】と【配列をArrayListに変換し、最後にリストを取り出す形のver】を書いてみたのですが、どちらも引数に適用できないという下記のエラーメッセージが出てきてしまいます。

エラーメッセージ

【for文とif文を使ったver】
Exception in thread "main" java.lang.Error: Unresolved compilation problems: 
	型 Pattern のメソッド matcher(CharSequence) は引数 (int) に適用できません
	型 Pattern のメソッド matcher(CharSequence) は引数 (int) に適用できません
	型 List<String> のメソッド add(int, String) は引数 (int) に適用できません
	型 List<String> のメソッド add(int, String) は引数 (int) に適用できません

	at Main.main(Main.java:14)

【配列をArrayListに変換し、最後にリストを取り出す形のver】
Exception in thread "main" java.lang.Error: Unresolved compilation problems: 
	型 Pattern のメソッド matcher(CharSequence) は引数 (int) に適用できません
	型 Pattern のメソッド matcher(CharSequence) は引数 (int) に適用できません
	型 List<String> のメソッド add(int, String) は引数 (int) に適用できません
	型 List<String> のメソッド add(int, String) は引数 (int) に適用できません

	at Main.main(Main.java:14)

該当のソースコード

//for文とif文を使ったver

import java.util.ArrayList;
import java.util.List;
import java.util.regex.Matcher;
import java.util.regex.Pattern;

public class Main {

	public static void main(String[] args) {
		String[] userIdArray = {"ASD1234","asdfgh1","aSd1234","asd1223","mmm1２34","1234ASD"};
		List<String> okList= new ArrayList<>();
		List<String> ngList = new ArrayList<>();
		for(int i = 0; i<userIdArray.length; i++){ //配列繰り返し
			Pattern p1 = Pattern.compile("^[A-Z]{3}"); //1~3文字目は大文字半角英字を設定する
			Matcher m1 = p1.matcher(i);
			Pattern p2 = Pattern.compile("[0-9]{4-7}"); //4~7文字目は半角数字を設定する
			Matcher m2 = p2.matcher(i);
			if(m1.find() && m2.find() && userIdArray.length >=7){ //7文字以上はOK
				okList.add(i);
			}else{
				ngList.add(i);
			}
		System.out.println("OKリスト---------");
		for(int i1 = 0; i1<okList.size(); i1++){
			System.out.println(okList.get(i1));
		}
		System.out.println("NGリスト---------");
		for(int i1 = 0; i1<ngList.size(); i1++){
			System.out.println(ngList.get(i1));
		}
		}
	}
}

//配列をArrayListに変換し、最後にリストを取り出す形のver

import java.util.ArrayList;
import java.util.Arrays;
import java.util.List;
import java.util.stream.Collectors;
public class Main{
	public static void main(String[] args) {
		String[] userIdArray = {"ASD1234","asdfgh1","aSd1234","asd1223","mmm1２34","1234ASD"};
		ArrayList<String> idList = new ArrayList<String>(Arrays.asList(userIdArray)); //配列からリストに変換
		System.out.println("チェックOKリスト-------------");
		List<String> okList = idList.stream() //リストとして取り出す
			.filter("^[A-Z]{3}","[0-9]{4-7}")
			.collect(Collectors.toList());
		System.out.println("チェックNGリスト-------------");
		List<String> ngList = idList.stream()
			.filter(idList.size()<=7)
			.filter("^[?!.*A-Z]{3}.*","[?!.*0-9]{4-7}.*")
			.collect(Collectors.toList());
	}
}

試したこと・調べたこと

• teratailやGoogle等で検索した
• ソースコードを自分なりに変更した
• 知人に聞いた
• その他

上記の詳細・結果

【for文とif文を使ったver】で、最初はif()の中に直接正規表現を書いていたのですが、patternとmatchでチェックするという記載が出てきたので、使いました。

補足

【for文とif文を使ったver】【配列をArrayListに変換し、最後にリストを取り出す形のver】片方だけの回答でも助かります。よろしくお願いいたします。

Comments

[jimbe]

>・１～３文字目は大文字半角英字→OKリストへ
>・４～７文字目は半角数字→OKリストへ
>・７文字以上→OKリストへ

この書き方だとどれか1つを満たせば OK のように見えますが、実際は3つとも満たさなければならないということであっていますか?

[h__]

申し訳ございません、説明不足でした。3つとも満たすということで合っております。

Answer 1

型 Pattern のメソッド matcher(CharSequence) は引数 (int) に適用できません
Matcher m1 = p1.matcher(i);

メッセージをきちんと読まれていないようです。
matcher には対象となる文字列(CharSequence)を指定しなければならないのに i を渡しているのでダメだと言われています。

public Matcher matcher(CharSequence input)

たとえば、

Matcher m1 = p1.matcher(userIdArray[i]);

のように渡さなければなりません。

また、 Pattern.compile はループの中で何度もしなければならないようなことではありませんし、 Pattern を二つに分ける必要も無さそうですし、ok/ngListの表示も振り分けてからでよいのではありませんか?

import java.util.*;
import java.util.regex.*;

public class Main {
	public static void main(String[] args) {
		String[] userIdArray = { "ASD1234", "asdfgh1", "aSd1234", "asd1223", "mmm1２34", "1234ASD" };

		List<String> okList = new ArrayList<>();
		List<String> ngList = new ArrayList<>();

		Pattern p = Pattern.compile("[A-Z]{3}[0-9]{4}.*");

		for(String userId : userIdArray) {
			if(p.matcher(userId).matches()) {
				okList.add(userId);
			} else {
				ngList.add(userId);
			}
		}

		System.out.println("OKリスト---------");
		System.out.println(okList);

		System.out.println("NGリスト---------");
		System.out.println(ngList);
	}
}

OKリスト---------
[ASD1234]
NGリスト---------
[asdfgh1, aSd1234, asd1223, mmm1２34, 1234ASD]

---

【配列をArrayListに変換… ver】改め stream 版では、 ok/ng を true/false としてキーにする Map に集めるようにすることで、stream から二度集めたり二つの正規表現を扱う必要を無くします。

import java.util.*;
import java.util.regex.*;
import java.util.stream.*;

public class Main {
    public static void main(String[] args) {
        String[] userIdArray = { "ASD1234", "asdfgh1", "aSd1234", "asd1223", "mmm1２34", "1234ASD" };

        Pattern p = Pattern.compile("[A-Z]{3}[0-9]{4}.*");

        Map<Boolean,List<String>> map =
            Arrays.stream(userIdArray).collect(Collectors.groupingBy(
                userId -> p.matcher(userId).matches() // ok なら true, ng なら false
            ));

        System.out.println("OKリスト---------");
        System.out.println(map.get(true));

        System.out.println("NGリスト---------");
        System.out.println(map.get(false));
    }
}

Comments

[h__]

ご回答ありがとうございます。問題点も説明してくださり、また試したところ解決いたしました。
ベストアンサーに選ばせていただきました。

[jimbe]

お疲れさまでした。