Stringクラスメソッドで詰まっている

Question

文字列（"JANUARY"~"DESEMBER")の配列を作成した後、
’R’が含まれる文字列と
各文字列中に最初に’R’がくるのは何文字目か？というのをやっているのですが、うまくいきません。

ちなみに実行例は↓
JANUARY6文字目
FEBURUARY4文字目
～
DSCEMBER8文字目
です。

自分が書いたソース↓
package practice;

public class practice21 {

public static void main(String[] args) {
	int i;
	String[] str= {"JANUARY","FEBRUARY","MARCH","APRIL","MAY","JUNE",
					"JULY","AUGUST","SEPTEMBER","OCTOBER","NOVEMBER","DECEMBER"};
	
	for(i=0; i<12;i++) {
		
		int s=str[i].indexOf("R");
		
	  if(str[i].equals("R")) {
		System.out.println(str[i]+s+"文字目");
	  }
	  
	}
}

}
としても何も表示されず詰まっています。
またint s=str[i].indexOf("R");で配列0から数えてしまうので、JANUARY6文字目がJANUARY5文字目となってしまいます。
解決法分かりましたらお願いします。

Answer 1

＞Stars1024さんへ
コメント欄では書ききれなかったので。
配列のr_positionを使うなら。こうでしょうか。

public class A119800 {

	public static void main(String[] args) {
		String[] year = { "JANUARY", "FEBRUARY", "MARCH", "APRIL", "MAY", "JUNE", "JULY", "AUGUST", "SEPTEMBER",
				"OCTOBER", "NOVEMBER", "DECEMBER" };
		int[] r_position = new int[year.length];
		for (int i = 0; i < year.length; i++) {
			r_position[i] = year[i].indexOf("R");
		}
		System.out.println("Rを含む月 および　Rが最初に現れるのはそれぞれ");
		for (int i = 0; i < year.length; i++) {
			if (r_position[i] != -1) {
				System.out.print(i + 1 + "月:" + year[i] + " ");
				System.out.println(r_position[i] + 1 + "文字目");
			}
		}
	}
}

使わないなら以下のようにも。

public class A119800 {

	public static void main(String[] args) {
		String[] year = { "JANUARY", "FEBRUARY", "MARCH", "APRIL", "MAY", "JUNE", "JULY", "AUGUST", "SEPTEMBER",
				"OCTOBER", "NOVEMBER", "DECEMBER" };
		System.out.println("Rを含む月 および　Rが最初に現れるのはそれぞれ");
		for (int i = 0; i < year.length; i++) {
			int pos = year[i].indexOf("R");
			if (pos == -1) {
				continue;
			}
			System.out.print(i + 1 + "月:" + year[i] + " ");
			System.out.println(pos + 1 + "文字目");
		}
	}
}

Comments

[退会済みユーザー]

ありがとうございます。こっちのほうがコードが少なくてわかりやすいですね。

Answer 2

if(str[i].equals("R")) {
    System.out.println(str[i]+s+"文字目");
}

「"R"に等しいとき」に出力しようとするので、"R"に等しい文字列が1つもないこの配列では何も出力できません。
やりたいことの意味としてはこういうこと？

if(s >= 0) {
    System.out.println(str[i]+s+"文字目");
}

Comments

[s_ayana]

そうです！
ありがとうございます。
後はint s=str[i].indexOf("R");で配列0から数えてしまうので、JANUARY6文字目がJANUARY5文字目となってしまいまうまんだいだけです。

[y_waiwai]

1足そう

[退会済みユーザー]

つ JAVA 仕様

Answer 3

こんにちは、

if(str[i].equals("R")) {
 System.out.println(str[i]+s+"文字目");
 }

これだと例えばi = 0の時　比較するのは"R"と"JANUARY"になってしまいます。
私はそれぞれのstr[i]をsubstringで一文字ずつ分けて比較しました。

public class  Main{
     public static void main(String[] args) {
        
            String[] year = {"JANUARY","FEBRUARY","MARCH","APRIL","MAY","JUNE",
                            "JULY","AUGUST","SEPTEMBER","OCTOBER","NOVEMBER","DECEMBER"}; 
            int []flag = new int[year.length];
            int []r_position = new int[year.length];
            for(int i = 0; i < year.length; i++){
                flag[i] = 0; /*これは初期化です。*/
                r_position[i] = 0;
               }
            for(int i = 0; i < year.length ;i++) { 
                for(int j = 0; j < year[i].length(); j++){
                    if('R' == year[i].charAt(j)){
                        flag[i] = 1;
                        r_position[i] = year[i].indexOf("R");
                        break;
                    }
                }
             }
            System.out.println("Rを含む月 および　Rが最初に現れるのはそれぞれ");
            for(int i = 0; i < year.length; i++){
                if(flag[i] == 1){
                    System.out.print(i + 1 + "月:" + year[i] + " ");
                    System.out.println(r_position[i] + 1 + "文字目");
                }
            }
        }
 }

Comments

[umyu]

◇String#indexOfがありまして。
https://docs.oracle.com/javase/jp/8/docs/api/java/lang/String.html#indexOf-int-

[退会済みユーザー]

すみません　問題には「Rが最初に出るのは何文字目か」なのに
「Rは何文字目にあるか全部出力する」と勘違いしました。コードを修正します。

[LouiS0616]

あと String#substring(j, j+1) は書くならば String#charAt(j) の方が分かりやすいでしょうね。
https://docs.oracle.com/javase/jp/8/docs/api/java/lang/String.html#charAt-int-

[退会済みユーザー]

いろいろありがとうございます。そこも修正しておきます。

[s_ayana]

参考になりました！

[退会済みユーザー]

こちらこそ　いろいろ間違えてすみません