メソッドを使って配列を使った計算を行いたい

Question

java初心者です。メソッドを使って計算を行いたいのですが、自分にはわからないエラーが出てしまうので解決策を教えていただきたいです。

import java.util.Scanner;
import static java.lang.System.in;
import static java.lang.System.out;

class iroiro{
  static int max,min,i;
  
  public static void main(String []args){
    Scanner stdIn = new Scanner(in);
    out.println("最大値と最小値を求めます。");
    out.println("要素数:");
    int n = stdIn.nextInt();
    
    int a[] = new int[n];//箱用意
    
    for(i = 0; i < n; i++){
      out.println("要素番号[" + (i+1) +"]");
      a[i] = stdIn.nextInt();
    }
    
    out.println("");
    
    
    for(i = 0; i < n; i++){
      maxCul(a,n);
    }
    out.println("最大値は" + max + "です。\n");
    
    min = a[0];
    for(i = 0; i < n; i++){
      if(min > a[i]){
        min = a[i];
      }
    }
    out.println("最小値は" + min + "です。");
  }
  
  
  public static int maxCul(int a[i],int n){
    max = a[0];
    for(i = 0; i < n; i++){
      if(max < a[i]){
        max = a[i];
      }
    }
    return max;
  }
  
  
}

エラー

iroiro.java:39: エラー: ']'がありません
  public static int maxCul(int a[i],int n){
                                 ^
iroiro.java:39: エラー: ';'がありません
  public static int maxCul(int a[i],int n){
                                         ^
エラー2個

どうかよろしくお願いします。

Answer 1

public static int maxCul(int a[i],int n){

↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓

public static int maxCul(int[] a){ 

要素数の情報は配列オブジェクト自体が持っているので不要です。
nの代わりに a.length を利用出来ますし、また拡張for文を使えばさらに簡潔になります。

参考：
ループを使わないで配列要素の最大値を求める方法については、次のページで議論されています。
StackOverflow - Finding the max/min value in an array of primitives using Java

---

max、min、i は全てローカル変数で簡単に代替できるはずです。
staticフィールドとして用意する必要はありません。

Comments

[rink]

なるほど…。
また良いお勉強になりました。。
ありがとうございました！

[rink]

わざわざ追加の解説どうもありがとうございます！！
参考にさせていただきます！

Answer 2

エラーを解消するだけなら、次のように変更すれば良さそうです。

	for(i = 0; i < n; i++){
      maxCul(a,n);
    }
    out.println("最大値は" + max + "です。\n");
 ...
  public static int maxCul(int a[i],int n){
   ...
  }

==>

  max = maxCul(a, a.length);
  out.println("最大値は" + max + "です。\n");
  ...
  public static int maxCul(int[] a,int n){
   ...
  }

追記
エラー解消だけでなく、全体を少し書き直してみました。

import java.util.Scanner;

public class iroiro {
    public static void main(String[] args) {
        Scanner stdIn = new Scanner(System.in);
        System.out.println("最大値と最小値を求めます。");
        System.out.println("要素数:");
        int n = stdIn.nextInt();
        int a[] = new int[n]; // 箱用意
        for (int i = 0; i < n; i++) {
            System.out.println("要素番号[" + (i + 1) + "]");
            a[i] = stdIn.nextInt();
        }

        System.out.println("最大値は" + maxCul(a) + "です。");
        System.out.println("最小値は" + minCul(a) + "です。");
    }

    public static int maxCul(int[] a) {
        if (a.length == 0) {
            throw new IllegalArgumentException("a.length == 0");
        }
        int max = a[0];
        for (int v : a) {
            if (max < v) {
                max = v;
            }
        }
        return max;
    }

    public static int minCul(int[] a) {
        if (a.length == 0) {
            throw new IllegalArgumentException("a.length == 0");
        }
        int min = a[0];
        for (int v : a) {
            if (v < min) {
                min = v;
            }
        }
        return min;
    }
}

Comments

[rink]

ご回答ありがとうございます！
とても分かりやすかったです！！！
またよろしくお願いします！

[rink]

追記読ませていただきました！
これとても便利ですね！！参考にさせていただきます！
いつもありがとうございます！