java 最高得点の科目を表示するメソッドがうまく機能しない

Question

前提・実現したいこと

javaで最高得点の科目名を表示のメソッドを作っています。
最高得点を求めたメソッドを利用して作っています。

発生している問題・エラーメッセージ

なぜか返り値が初期化されたままで、for文とif文の処理がうまくいかず、先頭の国語としか表示されません。

該当のソースコード

public class Score {
    String name;
    int[] subjects = new int[5];

    public int getJapanese(){
        return subjects[0];
    }

    public int getMathmatics(){
        return subjects[1];
    }

    public int getScience(){
        return subjects[2];
    }

    public int getSocialStudies(){
        return subjects[3];
    }

    public int getEnglish(){
        return subjects[4];
    }

    public void setJapanese(int value){
        subjects[0] = value;
    }

    public void setMaMathmatics(int value){
        subjects[1] = value;
    }

    public void setScience(int value){
        subjects[2] = value;
    }

    public void setSocialStudies(int value){
        subjects[3] = value;
    }

    public void setEnglish(int value){
        subjects[4] = value;
    }

    public int getHighScore(){       //最高得点を求める
        int max=subjects[0];
        for(int high : subjects){
            if(max < high){
                max = high;
            }
        }
        return max;
    }

    String getHighScoreSubject(){         //最も点数の高い科目を表示
        String[] maxsub = {"国語","数学","理科","社会","英語"};
        String st = maxsub[0];
        for(int j=0; j==subjects.length; j++) {
            if (getHighScore() == subjects[j]) {
                st = String.join(",",maxsub[j]);
            }
        }
        return st;
    }


    void getprintInfo(){         //結果の出力
        System.out.println("氏名:" + name);
        System.out.println("国語:" + subjects[0] + "点");
        System.out.println("数学:" + subjects[1] + "点");
        System.out.println("理科:" + subjects[2] + "点");
        System.out.println("社会:" + subjects[3] + "点");
        System.out.println("英語:" + subjects[4] + "点");
        System.out.println("最も点数が高い科目:" + getMaxSubject() + getHighScore() + "点");
    }

試したこと

どこが間違っているのかわかりません。

補足情報（FW/ツールのバージョンなど）

氏名や点数の入力は別のクラスで行っています。
最高得点の科目を表示する部分以外はちゃんと実行できます。

Comments

[K_3578]

とりあえずteratailではソースコードはMarkDown方式でコードブロック化してご提示ください。

ヘルプに書いてありますが、書き方が分からない場合は下記質問の回答も参考にしてください。
https://teratail.com/questions/238564

[K_3578]

>編集を受けて
質問者さんが書いている「,,,」はカンマです。
コードブロック化する際に必要なこれ「```」はバッククォートです。
フツーのキーボードならShift+「＠」のキーで打てます。

めんどかったらコードを全選択して<code>って書いてるところを押せば早いっす。

[gknjnjtnj]

すいません直りました。
手間をかけて申し訳ありません。

[K_3578]

別に暇なときに指摘してるだけなのでお気になさらず。

多分コード部分的に切り取られてるので極力全部コピペで貼った方が良いっすよ。

Answer 1

for(int i=0; i==subjects.length; i++) {

ループの条件式が間違っているように思えます。
i==subjects.lengthではなく、i<subjects.lengthではないでしょうか。

Comments

[gknjnjtnj]

ありがとうございます！うまく動作しました！
しかし、最高得点が同じだった場合(国語と数学が同率など)、数学しか表示されません。
二つとも表示する方法を教えていただいてもよろしいでしょうか？

[gknjnjtnj]

自己解決しました

Answer 2

if (getHighScore() == subjects[j]) {

jじゃなくてiでは

Comments

[gknjnjtnj]

すいません。そこは他のメソッドと混同したときの名残です。