質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

88.60%

平均と最大のメソッド、配列についてです。

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 2,230

frdhy

score 11

市町村のデータをソートし、その平均と最大、最小を求めたいのですが、
平均と最大がうまくできません。
平均は"二項演算子'+'のオペランド型が不正です"と表示され、intからTownに直せないことはわかるのですが、全くどうしたらいいのかわかりません。
最大はもはやエラーも出ないし、実行した時に表示もされません。
どうしたら解決することができるでしょうか?
よろしくお願いします。

import java.util.Scanner;
class Town {   
    String pref;
    String name;
    int population;
}

public class Population {
    public static void main(String args[]) {
    Scanner kbd = new Scanner(System.in);
    String p=null;
    String n=null;
    int num=0;
    Town[] table = new Town[2000];
    Town temp;
    int count=0;

    for(int i=0; kbd.hasNext() && i<2000; i++) {
        p = kbd.next();
        System.out.println(p);
        n = kbd.next();
        System.out.println(n);
        num = kbd.nextInt();
        System.out.println(num);
        temp = new Town();
        temp.pref = p;
        temp.name = n;
        temp.population = num;
        table[i] = temp;
        count++;
    }

    sort(table,count);

    }


    public static void sort(Town[] t,int num) {  
    Town temp;

    for(int i=0; i<num; i++) {   
        for(int j=i+1; j<num; j++) {
        if(t[i].population<t[j].population) {
            temp = t[i];
            t[i] = t[j];
            t[j] = temp;
        }
        }
    }

    Middle(t,num);

    }

    public static void Middle(Town[] t, int num) {   

    System.out.println("県 : "+t[(num-1)/2].pref+": 市町村 :"
               +t[(num-1)/2].name+" : 人口 : "+t[(num-1)/2].population);
    Average(t,num);
    }

    public static void Average(Town[] t, int num) {
//このメソッドと
    int total=0;
    double ave=0;
    for(int i=0; i<t.length; i++) {
        total += t[i];
    }
    ave = (double)total/(double)(num-1);
    System.out.println("市町村人口の平均は"+ave);

    Max(t,num);

    }

    public static void Max(Town[] t, int num) {   //ここです。
    for(int i=0; i>t.length; i++) {
    System.out.println("人口が最大の市町村は  県 : "+t[num-1].pref+": 市町村 :"+t[num-1].name+" : 人口 : "+t[num-1].population);
    break;
    }

    Min(t,num);

    }

    public static void Min(Town[] t, int num) {
    for(int i=0; i<1; i++)
    System.out.println("人口が最小の市町村は  県 : "+t[num-1].pref+": 市町村 :"+t[num-1].name+" : 人口 : "+t[num-1].population);
    }

}    
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 2

checkベストアンサー

+5

total += t[i];


ここでt[i]はTownインスタンスなので、このインスタンスの人口を取る必要があります。
他ではやっていますが、ここだけ抜けているようです。

total += t[i].population;


最大に関しては

for(int i=0; i>t.length; i++) {
    System.out.println("人口が最大の市町村は  県 : "+t[num-1].pref+": 市町村 :"+t[num-1].name+" : 人口 : "+t[num-1].population);
break;


iが0から始まって、t.lengthは2000(それ以外でも0以上であることは確実)なので、for文の継続条件を最初から満たさず、for文の中に入ることができません。それ以前にfor文の存在が無意味です。

コード修正

Townクラス

まずTownはコンストラクタを用意して、必要なパラメータをまとめて設定しましょう。
また、Townのデータを出力する箇所がいくつかありますが、全部同じ形式です。こういうものこそTownのメソッドとして用意しましょう。ObjectクラスのtoString()メソッドをオーバーライドすれば、出力が簡単になります。

class Town {   
    String pref;
    String name;
    int population;

    public Town(String p, String n, int pop) {
        pref = p;
        name = n;
        population = pop;
    }

    @Override
    public String toString() {
        return "県 : " + pref + ": 市町村 :" + name + " : 人口 : " + population;
    }
}

// 例えばMiddleの出力はこうできる
System.out.println(t[(num-1)/2]);
要素数が決まっていないなら配列よりList

要素数が決まっていないなら配列を使用するには向いていません。Listを使えばnumという変数は不要になります。課題などで配列限定なら仕方ないですが。

メソッドについて

メソッドはその役割だけを担うようにするべきです。sortはソートのみやるべきで、平均の出力をそのあとにやるということをこのメソッドが担当するべきではありません。
また、メソッド名は小文字始まりにしましょう。

public static void main(String args[]) {
    Scanner kbd = new Scanner(System.in);
    Town[] table = new Town[2000];
    int count;

    for(count = 0; kbd.hasNext() && count < 2000; count++) {
        String p = kbd.next();
        String n = kbd.next();
        int num = kbd.nextInt();
        table[count] = new Town(p, n, num);
    }

    sort(table, count);
    middle(table, count);
    average(table, count);


}

参考コード(動作未検証)

class Town {   
    String pref;
    String name;
    int population;

    public Town(String p, String n, int pop) {
        pref = p;
        name = n;
        population = pop;
    }

    @Override
    public String toString() {
        return "県 : " + pref + ": 市町村 :" + name + " : 人口 : " + population;
    }
}

public class Population {
    public static void main(String args[]) {
        Scanner kbd = new Scanner(System.in);
        List<Town> table = new ArrayList<>();

        while(kbd.hasNext()) {
            String p = kbd.next();
            String n = kbd.next();
            int num = kbd.nextInt();
            table.add(new Town(p, n, num));
        }

        Comparator<Town> comparator = Comparator.comparingInt(t -> t.population))

        Collections.sort(table, comparator);
        System.out.println("中央: " + table.get((table.size() - 1) / 2));
        System.out.println("人口平均値: " + table.stream().mapToInt(t -> t.population).average().orElse(0.0));
        table.stream().max(comparator).ifPresent(t -> System.out.println("人口最大: " + t));
        table.stream().min(comparator).ifPresent(t -> System.out.println("人口最小: " + t));
    }


}

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/10/08 19:57

    とてもわかりやすかったです!!
    ありがとうございました!!!

    キャンセル

+1

import java.util.Scanner;

class Town {
    private final String pref;
    private final String name;
    private int population;

    public Town(String pref, String name, int population) {
        this.pref = pref;
        this.name = name;
        this.population = population;
    }

    public String getPref() {
        return pref;
    }

    public String getName() {
        return name;
    }

    public int getPopulation() {
        return population;
    }
}

public class Population {
    static final int MAX_TABLE_NUM = 2000;

    public static void main(String args[]) {
        @SuppressWarnings("resource")

        Scanner kbd = new Scanner(System.in);
        Town[] table = new Town[MAX_TABLE_NUM];
        int count = 0;

        for (int i = 0; kbd.hasNext() && i < MAX_TABLE_NUM; i++) {
            String pref = kbd.next();
            System.out.println(pref);
            String name = kbd.next();
            System.out.println(name);
            int population = kbd.nextInt();
            System.out.println(population);

            table[count] = new Town(pref, name, population);
            count++;
            System.out.println("" + count + "番目を追加しました。" + pref + ", " + name + ", " + population);
        }

        sort(table, count);
        showResult(table, count);
    }

    public static void sort(Town[] t, int num) {
        for (int i = 0; i < num; i++) {
            for (int j = i + 1; j < num; j++) {
                if (t[i].getPopulation() < t[j].getPopulation()) {
                    Town temp = t[i];
                    t[i] = t[j];
                    t[j] = temp;
                }
            }
        }
    }

    public static void showResult(Town[] table, int num) {
        if (num <= 0) {
            System.out.println("データが空です。");
            return;
        }

        showMiddle(table, num);
        showAverage(table, num);
        showMin(table, num);
        showMax(table, num);
    }

    public static void showMiddle(Town[] t, int num) {
        System.out.println("人口の中央値は  県 : " + t[(num - 1) / 2].getPref() + ": 市町村 :" + t[(num - 1) / 2].getName() + " : 人口 : "
                + t[(num - 1) / 2].getPopulation());
    }

    public static void showAverage(Town[] t, int num) {
        double total = 0;
        for (int i = 0; i < num; i++) {
            total += t[i].getPopulation();
        }
        double ave = total / num;
        System.out.println("市町村人口の平均は" + ave);
    }

    public static void showMax(Town[] t, int num) {
        System.out.println("人口が最大の市町村は  県 : " + t[num - 1].getPref() + ": 市町村 :" + t[num - 1].getName() + " : 人口 : "
                + t[num - 1].getPopulation());
    }

    public static void showMin(Town[] t, int num) {
        System.out.println(
                "人口が最小の市町村は  県 : " + t[0].getPref() + ": 市町村 :" + t[0].getName() + " : 人口 : " + t[0].getPopulation());
    }


実行例:
イメージ説明

Town に コンストラクタ、getter を追加しました。
結果の表示は main から個別に呼び出すようにしました。

これは、いちおうは動作してますが、
main() で table[] を生成しているのをメソッドに切り出すようにする、 Town[] でなく List<Town> でデータを扱うようにする、town クラスに sort や、平均を求めメソッドを定義するようにする などの変更をしていくと良いと思います。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/10/08 19:56

    とても詳しくありがとうございます!!
    出来ました!!!

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 88.60%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る