Javaのメソッド、戻り値

Question

質問内容
このcountOneメソッドに関してif分のあとに何を入力すればいいのかわかりません、//Fixedには何を入力すればいいのでしょうか

mainメソッドでSystem.out.println(nOne)を表示することは指定されています

問題文：　int型配列の各要素に0または1が格納されているとき、配列中に含まれる1の数を返すメソッドcountOne()を定義せよ。配列の初期値は、int[] bits = { 0, 0, 0, 1, 0, 1, 1, 0 };とし、mainからメソッドcountOne()を呼び出し、返り値を表示すること。

   	public static void countOne(int[] n) {
		for(int i=0;i<n.length;i++) {
		if(n[i]==1) {
		//Fixed
			}
		}return nOne;
		
	  }

	public static void main(String[] args){
    int[] bits = { 0, 0, 0, 1, 0, 1, 1, 0 };
    int nOne = countOne(bits);
    System.out.println(nOne);
  }

}


コード

エラー
Exception in thread "main" java.lang.Error: Unresolved compilation problem:
Type mismatch: cannot convert from void to int

at countOne.main(countOne.java:15)

Comments

[ozwk]

前の質問が放置されています
https://teratail.com/questions/303451

Answer 1

教えていただいたものでできました

細かい解説などをしていただき大変助かりました

Answer 2

countOneという名前から推察して、入力であるn[]の'1'であるものの個数を求めていると思われます。

変更
0. countOneの戻り値をintに変更します
0. //Ficedではn[i]が1であるものの個数が数えます。
0. 数えた個数を返します。

public static int countOne(int[] n) {
        int nOne = 0;     // n[]で0である個数が入る

        for(int i=0;i<n.length;i++) {
           if(n[i]==1) {
           　　　//Fixed
　　　　　　　　　// nOneを1増やす
           }
        }
        return nOne;        // 数えた個数を返す
}