条件判定処理をインターフェースで与える

Question

###前提・実現したいこと

多くのクラスで使われてかつ組み合わせて使いたい条件判定をインターフェースで定義して継承し利用しています。
そのため、下記のコードのようなIs**というインターフェースがたくさんあります。
この状況はよくない状況でしょうか？
###該当のソースコード

public interface IsMoreThanTheProbability {
	default public  boolean isMoreThanTheProbability(int probability) {
		if((Math.random()*100)+1 <= probability)return true;
		return false;
	}
}

Answer 1

インタフェースのdefaultメソッドを利用したロジックの共有化は非常に有効です。

Java7まではインタフェースを実装しなければならなかったものをabstractで実装してから継承する、など面倒なものでしたが、必要な機能だけ差し込む処理がインタフェースの宣言だけで利用できるため便利です。

例えばWebアプリケーションの入力チェック機構をあげると

import javax.servlet.http.HttpServletRequest;

public interface AdminRoleAware {

	public static final String  ROLE_ADMIN = "ADMIN";

	public default boolean isAdmin(HttpServletRequest request) {
		if ( request == null ) {
			return false;
		}

		return request.isUserInRole(ROLE_ADMIN);
	}
}

入力チェックする機能は他にもあるでしょうから、これらのインタフェースを複数実装して、必要なだけメソッドを呼び出せば実現できます。

DIコンテナを使ったアノテーション利用に近いものがJava8標準で実装できるので、実装方針としては特に問題はないでしょう。

Answer 2

インターフェースの使い方として誤っているように感じます。

インターフェース(checkメソッドを持つ)をimplementsして各判定を実装すべきかなーと思います。