文字列の中から特定の文字を数える

Question

	public void moji(String a){

		int count = 0;

		//これを繰り返していきたい
		int num = a.indexOf("a");

		System.out.println(num);

		for(int i = 0; i < num+1; i++){
			num = a.indexOf("a", num+1);
			count++;
		}
		System.out.println(count);

クラスとかは端折ってます。

引数で渡した文字列は"aavvvava"です。
適当です。
この中からaの個数が何個存在するかを出力したいです。
このやり方個数は数えられるのですが、無駄なことをしているので効率的なやり方を知りたいです。

すみません。
学習のためにindexOfを使って回答お願いいたします。

Comments

[m.ts10806]

「正規表現」と聞いてピンときますか？

[m.ts10806]

「正しいやり方」というのはありません。
「こうしたい」に対してその通りの結果になっていればどのようなロジックを辿っていても「正しい」です。正解はないのです。
「効率的なやり方」という表現にとどめておいては？

Answer 1

indexOfが２つあるのと、iが無駄だと思ったので消してみました。

int index = 0;
int count = 0;
while(true) {
    index = a.indexOf("a", index) + 1;
    if (index == 0) break;
    count++;
}

Comments

[fuzzball]

コードが短くなっただけで、効率は変わっていないと思います。

Answer 2

4年前の記事ですが色々なやり方を紹介している記事がありましたので参考にしてください。

• Javaで文字の出現回数を数えたい

学習のためにindexOfを使って回答お願いいたします。

んーだとしたら他にないと思います。
indexOf()を使う以上は「指定した文字が出現する位置」を返すわけですし、
「何回出現したか」となると１文字ずつ回すのが適当なやり方と思います。

Answer 3

学習のためにindexOfを使って回答お願いいたします。

第三者にとって価値ある問答にする為、敢えてこの条件は無視して回答します。

---

次の方法が最も簡潔であるように思います。

var count = a.chars().filter(ch -> ch == 'a').count();

Comments

[LouiS0616]

indexOf縛りは何だったのか。

Answer 4

String s = "aavvvava"; 
int count = 0;
int p = -1;
while(true) {
  p = s.indexOf("a", p + 1);
  if (p < 0) {
    break;
  }
  count++;
}
System.out.println(count);

効率は上がっていません。
でも for でなく while でループさせるほうがわかりやすくなると思います、

Answer 5

ご質問のコードのどの辺りが「無駄なことをしている」という認識なのかが問題でしょうか.
以下は「index(i)をループ外に出さない風」です.
fuzzball さんと同様(コードは…効率は…)ですね.

int count = 0;
for(int i=0; (i=a.indexOf("a",i)+1) > 0; count++);

Answer 6

対象の文字列を空文字に置換し、最初の文字数との差で個数を求めるというのはどうでしょうか。

a.length() - a.replace("a", "").length();