正しい実行結果が得られません。

Question

以下はテキストファイルから最大値を読み込むプログラムです。

import java.util.Scanner;
class MaxArray{ 
public static void main(String[] args){ 
Scanner kbd = new Scanner(System.in); 
int[] data = new int[1000];
int n=1;
int max=data[n];
System.out.printf("max = %d\n", max);
}
static int  Max(int[] data,int n,int max){
        if(n==0) System.out.println("データがありません。");
	else {
	      max = data[0];
	    for(n=1; n<data.length;n++) {
		if(data[n]>max) max=data[n];
	    }
	}
	    return max;
}
}
```実行結果はmax=0となりました。何かがおかしいはずなんですが、どこを直したらよいでしょうか？

Comments

[Zuishin]

インデントがガタガタなので直してください。
前回の質問は解決したはずなので、解決済みにしてください。

Answer 1

何かがおかしいはずなんですが、どこを直したらよいでしょうか？

たくさんありますよw

1. テキストファイルを読み込んでいない

Scanner kbd = new Scanner(System.in);

は、テキストファイルではなく標準入力を読み込もうとしています（実際には、標準入力も読んでいませんが）。

2. メソッドMaxを呼び出していない

3. for (n = 1; n < data.length; n++) {

Javaでは、配列のインデックスは 0 から始まります。
なので、

for (n = 0; n < data.length; n++) {

としないと、1行目が無視されてしまいます。

---

1. についてどうすればよいかは、まずはご自分で調べてみてください。

「Java ファイル 読み込み」
などで検索すると、いろいろ情報が見つかると思います。

Comments

[prog1717nm]

パイプでプログラムの入力と出力を行うので大丈夫です

Answer 2

テキストファイルではなく標準入力から読み取っているように見えますが、その辺は問題無いですか？

Scanner kbd = new Scanner(System.in); 

そして、このkbdという値はその後何も使っていないようですが、確認しましたか？

何かがおかしいと思うなら、どこがどうなるべきだと思うが、こうなってしまうということを書いたほうがい
いと思いますよ。

Comments

[prog1717nm]

パイプでプログラムの入力と出力を行うのでそれは大丈夫です

Answer 3

Max メソッドを呼び出してないので。