Javaのプログラム分割について

Question

前提・実現したいこと

都道府県リストを作り、全データを出力するところまではできました。
ここに追加の機能として、
Scanner sc = new Scanner(System.in);
String line = sc.next();
で入力された県名を都道府県リストに格納されている県名と比較して、一致した県名のデータ(面積、人口、人口密度)を出力したいです。
また、プログラムは繰り返され、入力された県名が存在しなかった場合終了としたいです。
そこで、ここからMainクラスを作成し、計3つのクラスにした上で上記の処理を実装したいのですが、どうしたらいいか分からず詰まってしまいました。
実装例と解説を頂けると幸いです。
ご教授のほどよろしくお願いします。

該当のソースコード

// Prefクラス
import java.nio.file.Path;
import java.nio.file.Paths;
import java.nio.file.Files;
import java.util.List;
import java.util.ArrayList;
import java.io.IOException;
import java.util.Scanner;

public class Pref {
	public static void main(String[] args) {
		
		List<Prefecture> prefectures = new ArrayList<Prefecture>();
		
		try{
			// ファイル読み込み
			Path readPath = Paths.get("revenge.txt");
			Scanner file = new Scanner(readPath);
			//Scanner sc = new Scanner(System.in);
			//String line = sc.next();
			
			while(file.hasNext()) {
				final String str = file.nextLine();
				final String[] array = str.split(",");
				
				if (array == null && array.length != 3) {
					//エラー処理
					System.out.println("データが存在しません");
				}
				
				final String name = array[0];
				final double area = Double.parseDouble(array[1]);
				final double population = Double.parseDouble(array[2]);
				final Prefecture p = new Prefecture(name, area, population);
				prefectures.add(p);
			}
		}
		catch(IOException e) {
			e.printStackTrace();
		}
		
		// 出力する
		for(Prefecture p:prefectures) {
			System.out.println(p);
		}
	}
}

// Prefectureクラス
public class Prefecture {
	private String name;
	private double area;
	private double population;
	
	public Prefecture(String name, double area, double population) {
		this.name = name;
		this.area = area;
		this.population = population;
	}
	
	public final String getName() {
		return name;
	}
	
	public final double getArea() {
		return area;
	}
	
	public final double getPopulation() {
		return population;
	}
	
	public final double getPopulationDensity() {
		return area == 0 ? null:(double)population/area;
	}
	
	@Override
	public String toString(){
		return getName() + ":"+ String.format("%.0f", getArea()) + ":"+ 
			String.format("%.0f", getPopulation()) + ":"+ String.format("%.0f", getPopulationDensity());
	}
}

Comments

[Zuishin]

このコードは誰が書いたんですか？
これは設問の一部で、このコード以外を自分で書けという課題ではありませんか？

[retantaro]

まだ初心者なもので段階的に作成していて、PrefクラスはMainクラスとして作成していたんですが、機能を追加するにはそれをPrefクラスに実装するのではなく新たにMainクラスを作って実装するというヒントをもらったのですが、そこでコードをどう改変するかで詰まってしまいました。

課題の設問は、
ファイル(revenge.txt)から県名、面積、人口、人口密度を読み込み、標準入力から県名を入力する。入力された県の面積、人口、人口密度を出力するプログラムを作成せよ。ただし、県名の入力と面積等の出力は、県名として存在しない県名が入力されるまで繰り返されるものとする。
です。

[Zuishin]

> prefectures.add(p);

これの代わりに、あるいはこれの前にその処理を入れれば良いと思います。
p.getName() で得られる県名が既存の県名リストに存在しなければループを中止すれば良いでしょう。

存在するならば toString() の書式を出力として指定されたフォーマットに合わせ、それを出力すればいいんじゃないですか？

[retantaro]

ありがとうございます。

比較の処理はPrefクラスに実装するということでしょうか？

[Zuishin]

それでいいと思います。

Answer 1

クラスの分け方は言ってしまえば人それぞれですので、ヒントをくれた方がどういう意図でそのようなヒントにされたのかは、ご本人で無ければ分かりません。

java に限らず、プログラムを作る際は「塊」を意識して分けることになると思います。
ご質問の内容の場合、大きくは

1. 入力に応じて出力するループ
2. ファイル
3. ファイルの個々のデータ

になると思います。

1 はまず 2 を作る。ループを開始し、ユーザからの入力を待ち、入力されたら 2 からデータを探して表示してループを続け、無ければループ終了(=プログラムも終了)。

2 は生成時にファイルを読んで 3 を複数作り内部にリストもしくはマップとして保存しておく。県名から 3 を探すメソッドを持ち、 それは該当データがあれば 3 を返し、無ければ null を返す。

3 は県名・面積・人口を保持し、人口密度も計算のうえで全データを表示するメソッドを持つ。

Comments

[retantaro]

以後プログラムを組むときは意識するようにします。ありがとうございました。