String型の変数から母音だけを抜き出してList型の変数に代入したい。

Question

前提・実現したいこと

String型の変数　String　resule="ajisai"から母音のa,i,a,iを抜き出してList型listに追加したい。

まず①String型のajisaiをStringのsplitメソッドで分割したのちにString型配列String[]に｛a,j,i,s,a,i｝として格納し、②その後asListメソッドで新しく作ったList型のコレクションに代入しました。③その次にlistの要素をfor文で取り出し、if文の条件式中でlist.get(i)と"a","i","u","e","o"を比較して条件に合致するとき（あるいはしないとき）に、あらかじめ用意していた別のlistにlist.addで追加（もしくはもとのlistからlist.removeで要素を削除）して母音だけが入ったlistを得ようとしました。
ですが①、②はうまくいくものの③がうまくいかず、listの中身が空になってしまいます。
System.out.printlnで③の段階でうまく動いていないのは特定できたのですが、いろいろ方法を試したものの
独力では解決できかったので非常に基本的なことでつまずいているのかもしれませんがどうかご助力いただければ嬉しいです。

発生している問題・エラーメッセージ

[a, j, i, s, a, i]
[a, j, i, s, a, i]
a
i
a
i
[a, j, i, s, a, i]
[a, j, i, s, a, i]
[]

該当のソースコード

import java.util.ArrayList;
import java.util.Arrays;
import java.util.Iterator;
import java.util.List;

public class test2 {
	
	public static void main(String[]args) {
		
		String result="ajisai";
		henkan(result);
		
		
	}

	public static List<String> henkan(String result) {
		
		

		String str = result;

		String[] ar = str.split("");//String型から配列に変換

		List<String> list = Arrays.asList(ar);//配列からリストに変換
		List<String>list1=new ArrayList<String>();//asListのデメリットを回避
		list1=list;
		System.out.println(list);
		System.out.println(list1);;
		System.out.println(list.get(0));
		System.out.println(list.get(2));
		System.out.println(list1.get(0));
		System.out.println(list1.get(2));


		ArrayList<String> list2 = new ArrayList<>();

		Iterator<String> iterator = list2.iterator();
		while (iterator.hasNext()) {
		    String str1 = iterator.next();
		    if (str1 != "a"&&str1 != "i"&&str1 != "u"&&str1 != "e"&&str1 != "o") {

		        iterator.remove();
		        System.out.println(list2);
		    }
		    System.out.println(list2);
		}

		/*for (int i = 0; i < list1.size(); i++) {

			if (list1.get(i) == "a") {

				list2.add(list1.get(i));//リストに要素を追加

			} else if (list1.get(i) == "i") {

				list2.add(list1.get(i));//リストに要素を追加

			} else if (list1.get(i) == "u") {

				list2.add(list1.get(i));//リストに要素を追加

			} else if (list1.get(i) == "e") {
				list2.add(list1.get(i));//リストに要素を追加

			} else if (list1.get(i) == "o") {
				list2.add(list1.get(i));//リストに要素を追加

			}

		}*/



		/* // b,c,d を削除したいので、この３つの要素を持つCollectionを生成させる
		 List<String> set = new ArrayList<>();
		 Collections.addAll(set, "b", "c", "d");

		 // list1をlist2に退避する
		 List<String> list2 = new ArrayList<>(list1);*/

		// 削除前の要素をすべて表示する
		//System.out.println(list2.get(0));
		//System.out.println(list2.get(2));

		System.out.println(list);
		System.out.println(list1);
		System.out.println(list2);

		/*// b,c,dをlist2から削除する
		list2.removeAll(set);

		// 削除後の要素をすべて表示する
		System.out.println(list2);*/
		return list2;
	}
}

試したこと

for文のほかに拡張for文、イテレーターを試してみたり、if文の条件式を変えたり
してみましたが改善しませんでした。

補足情報（FW/ツールのバージョンなど）

開発環境はeclipseで実行環境はjava13です。

Comments

[退会済みユーザー]

Arrays.asList で取得できるものは

java.util.ArrayList ではなく java.util.Arrays.ArrayList という unmodifiedList ですが理解してますか?

Answer 1

Stream。使ってみようか。

import java.util.Arrays;
import java.util.List;
import java.util.stream.Collectors;

public class xxStreamSample {
  public static void main(String[] main) {
    final String text = "Ajisai";
    final List<String> list =
        Arrays.stream(text.split(""))
            .filter(it -> "aiueo".contains(it.toLowerCase()))
            .collect(Collectors.toList());

    System.out.println(
        list.stream()
            .collect(Collectors.joining(", "))
    );
    // A, i, a, i
  }
}

Answer 2

上記のコメントアウトしたfor文中if文の条件式の比較演算子を==からequalsにしたところ以下の
コードで思った通りの動きができました。

for (int i = 0; i < list1.size(); i++) {

			if (list1.get(i).equals("a")) {

				list2.add(list1.get(i));//リストに要素を追加

			} else if (list1.get(i).equals("i")) {

				list2.add(list1.get(i));//リストに要素を追加

			} else if (list1.get(i).equals("u")) {

				list2.add(list1.get(i));//リストに要素を追加

			} else if (list1.get(i).equals("e")) {
				list2.add(list1.get(i));//リストに要素を追加

			} else if (list1.get(i).equals("o")) {
				list2.add(list1.get(i));//リストに要素を追加

			}

しかしイテレーターを使って書いた方は!(…equals(""))と書き直しても動きませんでした。

Iterator<String> iterator = list2.iterator();
		while (iterator.hasNext()) {
		    String str1 = iterator.next();
		    if (!(str1.equals("a"))&&!(str1.equals("i"))&&!(str1.equals("u"))&&!(str1.equals("e"))&&!(str1.equals("o"))){

		        iterator.remove();
		        System.out.println(list2);
		    }
		    System.out.println(list2);

Answer 3

まずlist2をArrayListクラスでnewした直後にiterator変換してますが、
それだとリストに何も要素が入ってないのでループに入ることもなく処理が終わってしまいます。

また要素の入っているlist、list1をiteratorでループさせたとしても、
母音をチェックするif分クラスで複数の条件を&&で繋げています。
&&はAnd条件になるので、これだと一文字が"a"、"i"、"u"、"e"、"o"全てであるというありえない条件になってしまいます。
Or条件になる||を使いましょう。

後、Arrays.asList()メソッドで配列を変換して作ったListクラスは正確にはArrayListではなく、Arrays.ArrayListというArraysクラスの内部クラスでaddやremoveによる要素変更ができない特殊なものなので注意してください。

この場合、list、list1をループさせて、空のlist2クラスにaddで追加して別なリストを作っていくのが無難だと思います。

Comments

[退会済みユーザー]

どなたの回答も非常に参考になったのですが、自分の煮詰まっているところを分かりやすく説明してくださったので選ばせていただきました。とくにArrays.asList()メソッドが要素変更できないというのは見落としていたところで非常に助かりました。

[yureighost]

補足ですが、And条件のところは否定条件を繋いでいるので正しかったですね。
これは私が読み間違えていました。
間違った情報が書いてあるとまずいので打ち消しで修正させていただきました。

Answer 4

全然違う書き方だけど、参考になれば・・・

import java.util.List;
import java.util.Arrays;

public class Main {

    public static void main(String[] main) {
        var text = "Ajisai";
        var result = henkan(text);
        result.forEach(System.out::println);
    }

    public static List<String> henkan(String text) {
        return Arrays.asList(
                   Arrays.stream(text.split(""))
                         .filter(s -> "aiueo".contains(s.toLowerCase()))
                         .toArray(String[]::new));
    }
}

実行結果

A
i
a
i

Comments

[退会済みユーザー]

まだjavaを学習し始めたばかりでうまくコードをかけないのですが、使いこなせばこんなにスマートに書けるんですね。ソースコードの部分で初めて見るような書き方もあるのでぜひ調べて今後の参考にさせてもらいたいと思います。ありがとうございました。

Answer 5

javaでstringを比較するとき== (!=)を使っても思ったような結果にはなりません
equalsを使いましょう

Comments

[退会済みユーザー]

回答ありがとうございます。追記にあるように==をequalsに変更したところ、一部動作が確認できました。
初歩的なところで恥ずかしい限りです。数時間ハマっていたので本当に助かりました。