Java で引数の null チェックで迷った話

yusuiked@gmail.com (yusuiked) — Tue, 15 Dec 2015 17:00:00 +0900

これは Java Advent Calendar 2015 の 15 日目の記事です。

昨日は @opengl_8080 さんの Byteman 使い方メモ＋α でした。明日は @irof さんです。

前置き

ついこないだチームでちょっとだけ話題に上って、みんなある程度指針は持っているものの、割と悩みつつ明確に答えを出せなかったので、もっと良い意見があればと思って晒してみます。まぁよくある話だし、Java 8 で Optional が使えるようになって null について語られるケースが増えたと思うので、再考するちょうどよい機会になればいいなーと思います。初心者向けです。

どう処す？処す？

こんな状況の時にあなたならどうしますか？

// Generics なのは例です。String でもなんでもいいです
public T doSomething(T input) {
    // input が null の時にどう処す？処す？
}

もちろん、呼び出し側のコンテキストとか、ライブラリを何使うかとか、Java 8 or それ以前とか、そういった前提によっていろいろ対処は変わってくるとは思うのですが、いくつか選択肢があると思います。良い悪いを置いといて、よく見るのは、

null を返す
何らかのデフォルト値を返す
java.lang.IllegalArgumentException などの例外を投げる

辺りかなぁと思います。null を返すパターンの時は、もし返り値が Collection や配列なのに、null 返しちゃうようなのを見かけたら、Effective Java をそっと差し出してあげてください。

おぬし気が効いてるのう

個人的に、以下のような実装を見たら、おお、おぬしホスピタリティを心得ておるのぅ、って思います。

Null Object パターンで何もしないオブジェクトを返す
- これも一種のデフォルト値を返すパターンと言える

public interface Command {
    void execute();
}
public class ABCCommand implements Command {
    public void execute() {
        System.out.println("ABC");
    }
}
// こいつが Null Object
public class NullCommand implements Command {
    public void execute() {
        // do nothing
    }
}
public class Main {
    public static void main(String... args) {
        Command missing = createCommand(null); // 普通こんなことやらないけど
        missing.execute();  // createCommand の戻り値が null だったら〜という null チェックが要らない
    }
    public static Command createCommand(String name) {
        if ("abc".equals(name)) {
            return new ABCCommand();
        } else {
            return new NullCommand();
        }
    }
}

ガード節を表現する実装をする

Tips on yusuiked's blog

Java で引数の null チェックで迷った話

前置き

どう処す？処す？

おぬし気が効いてるのう