Javaで人数を編集したい

Question

現在Javaで開発をしています。

ある項目の人数の編集をしたいのですがどのようなコードを書けばいいか困っています。

仕様は以下です。

１. checkがtrueの場合
a.人数下限のみの場合、「１人〜」
b.人数下限と上限がある場合「1人〜１０人」
c.人数上限のみの場合「〜１０人」
２. checkがfalseの場合、空文字

以下のようなコードを書きました。
もっといい書き方があれば教えてください。お願い致します。

public class Test {
	public static void main(String[] args) {
		Test t = new Test();
		System.out.println(t.getNumberOfPeople(false, 1, 10)); //
		System.out.println(t.getNumberOfPeople(true, 1, null)); // 1人~
		System.out.println(t.getNumberOfPeople(true, null, 10)); // ~10人
		System.out.println(t.getNumberOfPeople(true, 1, 10)); // 1人~10人

	}

	public String getNumberOfPeople(boolean check, Integer from, Integer to) {
		StringBuilder sb = new StringBuilder("");
		if (check == false)
			return sb.toString();
		if (check == true) {
			if (from != null) {
				sb.append(from.toString());
				sb.append("人");
			}
			if (from != null || to != null) {
				sb.append("~");
			}
			if (to != null) {
				sb.append(to.toString());
				sb.append("人");
			}
		}
		return sb.toString();
	}
}

Answer 1

public String getNumberOfPeople(boolean check, Integer from, Integer to) {
    if (!check) return "";
    if (from == null && to == null) throw new IllegalArgumentException();
    StringBuilder builder = new StringBuilder("~");
    if (from != null) builder.insert(0, "人").insert(0, from);
    if (to != null) builder.append(to).append("人");
    return builder.toString();
}

最初に"~"を確定させておき、fromがあれば先頭に足す形にしてみた
後、StringBuilderのコンストラクタに空文字を渡すのは無駄です。

追記
せっかく両方nullの可能性を最初に排除したのにそれを生かすのを忘れてた

public String getNumberOfPeople(boolean check, Integer from, Integer to) {
    if (!check) return "";
    if (from == null && to == null) throw new IllegalArgumentException();
    StringBuilder builder = new StringBuilder();
    if (from != null) builder.append(from).appemd("人");
    builder.append("~");
    if (to != null) builder.append(to).append("人");
    return builder.toString();
}

Comments

[k499778]

回答ありがとうございます。
シンプルでコード量も少なく非常に参考になるコードでした。

ちなみに「 throw new IllegalArgumentException();」をしている意図や理由はなんでしょうか？

[swordone]

コードの目的からして、fromとtoの両方がnullという状況はおかしいので、例外を発してそれを伝達する目的で入れました。

[k499778]

返答ありがとうございます。
なるほど！

このデータが例えばweb APIで取得しており、値がないときは項目自体が配列から消えているとします。
そのときはNULLの可能性があるのかな。それを踏まえてIntegerにしておく必要があるのかなと思ったのですが、その考え方に関してはどうでしょうか？
詳細設計部分の作りについて詳しくなりたかったので質問させていただきました。お忙しかったら申し訳ないです。

[k499778]

連投申し訳ないです。

御回答頂いていたソースを修正されましたが、
if (from != null) builder.insert(0, from).append("人");
↓
if (from != null) builder.insert(0, "人").insert(0, from);

に変わったのはなぜでしょうか？
元のコードでも正しく実行されるように思いましたが。

[swordone]

Web APIのほうは明るくないため答えられません。すみません。

コード変更の理由:
append("人")の場合、最初にbuilderに入れた"~"のあとに"人"が入ってしまい、"3~人"のようになってしまうからです。

[k499778]

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

返答いただけただけでありがたいです。

あ、ほんとですね。勘違いしてました。

追記頂きありがとうございます。
fromもtoも同じロジックになっているのでこちらの方がわかりやすいですかね。お付き合い頂きありがとうございました。