【Java】最も年齢の小さいオブジェクトの参照を返したい

Question

最近Javaの勉強を始めた初学者です。

前提・実現したいこと

配列の中のPersonオブジェクトの中で、
最も年齢の小さいオブジェクトの参照を返したいです。

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

Exception in thread "main" java.lang.NullPointerException
at package.Age.main(Age.java:27)

//結果：18とコメントアウトしている部分の中にmainクラスで宣言した配列[0]の年齢を入れて
for文とif文で「18よりも小さい年齢が存在していた場合、ageにその値を入れる」という処理をしているのですが、
用意した配列p1 ~ p3の中に18よりも小さい数字がないためぬるぽが返されてしまいます。
（”こうせい”, 18）以下の年齢を18より若くすれば正常終了します。

iのカウントをint i = 0;にしてみたり比較が違う？など考えましたが上手くいかず、
どう修正すればよいのか解決に至りませんでしたのでご教示いただきたいです。

該当のソースコード

public class Age {
Person getYoungest(Person[] persons) {

	int age = persons[0].getAge(); //結果：18

	for (int i = 1; i < persons.length; i++) {
		if (age >= persons[i].getAge()) { //ここが違う？
			age = persons[i].getAge();
			return persons[i];
		}
		
	}
	return null;
}

public static void main(String[] args) {
	Age age = new Age();
	Person p0 = new Person("こうせい", 18);
	Person p1 = new Person("まさこ", 23);
	Person p2 = new Person("のぶ代", 35);
	Person p3 = new Person("わさび", 48);

	Person[] person = {p0, p1, p2, p3};
	Person p = age.getYoungest(person);
	System.out.println(p.getName() + "," + p.getAge());
}
}

public class Person {
	private String name;
	private int age;
 
	Person(String name, int age) {
		this.name = name;
		this.age = age;
	}
 
	String getName() {
		return name;
	}
 
	int getAge() {
		return age;
	}
}

よろしくお願いいたします。

Answer 1

nullを返す時って、どんな時か？を先に考えれば実装は簡単になるかと

ちなみに、現状の実装だと一番若い人じゃなくて、配列の最初の人(p0)より若い人が出現した時点で
Person返しちゃいますけど意図した動きでないのでは？

Person getYoungest(Person[] persons) {
    if(persons == null || persons.length == 0) {
        // return null;
        throw new IllegalArgumentException("person does not exist");
    }

    Person youngest = persons[0];
    int age = youngest.getAge();

    // なんやかやの比較

    // return null; ← 問題なのは、ここ
    return youngest;
}

Comments

[neroS2eng]

できました！！

nullを参照型ならどんな型にも変換できるメタモン的存在と曖昧な覚え方をしているので要復習します。
意図した動きになっていないのも自分では気が付いていませんでした。

もういちど復習します。
大変助かりました、ありがとうございました！

[momon-ga]

nulllはメタモン的存在というより、何でもないので、いろんなところで出てきますが
実際に使おうとするとNullPointerExceptionが発生するので、
できるだけソースから減らした方が良い習慣だと思います（まぁ、楽だから使っちゃうよねぇ）

Answer 2

コードで書く前に、日本語で考えてみましょう。

こうせい(18), まさこ(23), のぶ代(35), わさび(48)
の4人の中で、最も若い人を選ぶのですから、「こうせい(18)」を返さないといけませんよね。

Comments

[neroS2eng]

回答ありがとうございます！

①ageには初期値としてこうせい(18)の(18)が入っている
②もしこうせいより年下がいれば上書きする
③存在しない場合こうせいのまま返す
までは出ているのですが、③の方法がどうしても思いつかないままでいます。。。
ifをネストさせる方法も考えたのですが18を18のまま返すうまい書き方でも悩んでいる状況です(；；）

[Daregada]

ageだけを保持する必要はないんですよ。
「最小の年齢を持つPersonクラスのオブジェクト」を保持する変数を作ればよろしい。

それから、元のコードだと、「ageより小さな年齢のオブジェクトが見つかったら即return」していますが、「さらに小さな年齢の人が後ろにいる可能性」を考えないと。

Answer 3

フィギュアスケートの得点のように、順番に数字を見て、最大もしくは最小を決定していく過程は現実にもあります。ああいうのはどのようにして優勝者を決めているでしょうか？あるいは我々はどうやって優勝する可能性のある人物を追っているでしょうか？

Answer 4

もしもヌルポを回避したいだけでしたら、以下で十分です。

public static void main(String[] args) {
    Age age = new Age();
    Person p0 = new Person("こうせい", 18);
    Person p1 = new Person("まさこ", 23);
    Person p2 = new Person("のぶ代", 35);
    Person p3 = new Person("わさび", 48);

    Person[] person = {p0, p1, p2, p3};

    // if追加
    if (age.getYoungest(person) != null) {
        Person p = age.getYoungest(person);
        System.out.println(p.getName() + "," + p.getAge());
    }
}

追記

URL添付します。
配列内の最小値を求める

Comments

[退会済みユーザー]

ただし、質問者様は
「配列の中のPersonオブジェクトの中で、
最も年齢の小さいオブジェクトの参照を返したいです。」

と仰っているので、もっと改善点はあります。

[退会済みユーザー]

回答にURL添付しましたので、確認してください。

[neroS2eng]

とても助かります！
確認してみます、ありがとうございます。

[退会済みユーザー]

ちなみにnullについては、まったく何もないという意味です。

String s = "";
というのは、sに空の文字が入っておりますが、

String s = null;
というのは、sはまったくの無です。(なんにでも入るので勘違いしやすいですが)