編集履歴

回答編集履歴

追記

2016/04/06 04:40

投稿

スコア4666

test CHANGED Viewed

@@ -136,7 +136,7 @@
 ---
-###**追記**
+###**追記１**
 > MemberクラスのmemberRankプロパティに、各会員の状態を持たせる場合、どのような実装があり得ますか？
@@ -345,3 +345,251 @@
 ```
+###**追記２**
+> getPriceのようなメソッドを他にもいくつか追加したい場合、どういったクラス構成が望ましいですか？
+会員ランク毎に必要な振る舞いであれば、MemberRankで全て実装すべきです。
+例えば以下は、購入金額から付与されるポイントを計算するメソッドと
+特別会員かどうかを判断するメソッドを追加してみました。
+これは、特別会員だけが見れる項目や、特別会員専用ページなどの制御で使うことを想定しています。
+```java
+/**
+ * 会員ランクを表す列挙
+ */
+enum MemberRank {
+	/** 一般会員 */
+	GENERAL(1) {
+		@Override
+		public BigDecimal getPrice(BigDecimal catalogPrice) {
+			// 一般会員は定価のまま
+			return catalogPrice;
+		}
+		@Override
+		public BigDecimal getPoint(BigDecimal totalPrice) {
+			// 一般会員は購入金額の1％をポイント付与
+			BigDecimal percentage = new BigDecimal("0.01");
+			return totalPrice.multiply(percentage).setScale(0, BigDecimal.ROUND_DOWN);
+		}
+		@Override
+		public boolean isSpecialMember() {
+			// 一般会員は特別会員でない
+			return false;
+		}
+	},
+	/** シルバー会員 */
+	SILVER(2) {
+		@Override
+		public BigDecimal getPrice(BigDecimal catalogPrice) {
+			// 5%引き
+			BigDecimal discountPrice = catalogPrice.multiply(new BigDecimal("0.05"));
+			return catalogPrice.subtract(discountPrice);
+		}
+		@Override
+		public BigDecimal getPoint(BigDecimal totalPrice) {
+			// シルバー会員は購入金額の2％をポイント付与
+			BigDecimal percentage = new BigDecimal("0.02");
+			return totalPrice.multiply(percentage).setScale(0, BigDecimal.ROUND_DOWN);
+		}
+		@Override
+		public boolean isSpecialMember() {
+			// シルバー会員は特別会員でない
+			return false;
+		}
+	},
+	/** ゴールド会員 */
+	GOLD(3) {
+		@Override
+		public BigDecimal getPrice(BigDecimal catalogPrice) {
+			// 10%引き
+			BigDecimal discountPrice = catalogPrice.multiply(new BigDecimal("0.1"));
+			return catalogPrice.subtract(discountPrice);
+		}
+		@Override
+		public BigDecimal getPoint(BigDecimal totalPrice) {
+			// ゴールド会員は購入金額の3％をポイント付与
+			BigDecimal percentage = new BigDecimal("0.03");
+			return totalPrice.multiply(percentage).setScale(0, BigDecimal.ROUND_DOWN);
+		}
+		@Override
+		public boolean isSpecialMember() {
+			// ゴールド会員は特別会員
+			return true;
+		}
+	};
+	/**
+	 * @param catalogPrice 定価
+	 * @return ランクに応じた金額
+	 */
+	public abstract BigDecimal getPrice(BigDecimal catalogPrice);
+	/**
+	 * @param totalPrice 購入合計金額
+	 * @return ランクに応じたポイント数
+	 */
+	public abstract BigDecimal getPoint(BigDecimal totalPrice);
+	/**
+	 * 特別会員かどうかを返す。
+	 * <pre>
+	 * 特別会員とはゴールド会員以上の会員のこと。
+	 * （現状、ゴールド会員が最高ランクだが、来年にはプラチナ会員を導入予定）
+	 * </pre>
+	 *
+	 * @return true..特別会員
+	 */
+	public abstract boolean isSpecialMember();
+	/** 各列挙が持つ値 */
+	private int value;
+	/**
+	 * コンストラクタ
+	 */
+	private MemberRank(int value) {
+		this.value = value;
+	}
+	/**
+	 * ファクトリー
+	 */
+	public static MemberRank of(int value) {
+		return Stream.of(MemberRank.values()).filter(r -> r.value == value)
+				.findFirst().orElseThrow(IllegalArgumentException::new);
+	}
+}
+```

追記

2016/04/06 04:40

投稿

root_jp

スコア4666

test CHANGED Viewed

@@ -131,3 +131,217 @@
 BigDecimal price = getMember().getMemberRank().getPrice(catalogPrice);
 ```
+---
+###**追記**
+> MemberクラスのmemberRankプロパティに、各会員の状態を持たせる場合、どのような実装があり得ますか？
+問題はこれです。
+通常よくあるのは、データベースの会員情報から MemberRank を作成したいわけですから、
+O/Rマッパーが Member取得時に自動で入れてくれたりもします。
+データベースの値を MemberRank Enum に変換してくれます。
+そういうのを省いて考えた場合どうなるかというと
+例えば、以下のような感じだと台無しなわけです。
+```java
+if (rank.equals("general")) {
+	member.setMemberRank(MemberRank.GENERAL);
+} else if (rank.equals("silver")) {
+	member.setMemberRank(MemberRank.SILVER);
+} else if (rank.equals("gold")) {
+	member.setMemberRank(MemberRank.GOLD);
+}
+```
+こういった場合は、MemberRank に以下のようなファクトリーメソッドを用意するといいです。
+```java
+public static MemberRank of(String name) {
+	return MemberRank.valueOf(name.toUpperCase());
+}
+```
+そうすると、以下のように設定できます。
+```java
+// rank は、"general", "silver", "gold" の文字列
+member.setMemberRank(MemberRank.of(rank));
+```
+データに "general" などの文字列ではなく、
+一般会員 = 1, シルバー会員 = 2, ゴールド会員 = 3 などのように
+数値として持ちたいという事も多いでしょう。
+その時は、MemberRankを以下のようにします。
+```java
+/**
+ * 会員ランクを表す列挙
+ */
+enum MemberRank {
+	/** 一般会員 */
+	GENERAL(1) {
+		@Override
+		public BigDecimal getPrice(BigDecimal catalogPrice) {
+			// 一般会員は定価のまま
+			return catalogPrice;
+		}
+	},
+	/** シルバー会員 */
+	SILVER(2) {
+		@Override
+		public BigDecimal getPrice(BigDecimal catalogPrice) {
+			// 5%引き
+			BigDecimal discountPrice = catalogPrice.multiply(new BigDecimal("0.05"));
+			return catalogPrice.subtract(discountPrice);
+		}
+	},
+	/** ゴールド会員 */
+	GOLD(3) {
+		@Override
+		public BigDecimal getPrice(BigDecimal catalogPrice) {
+			// 10%引き
+			BigDecimal discountPrice = catalogPrice.multiply(new BigDecimal("0.1"));
+			return catalogPrice.subtract(discountPrice);
+		}
+	};
+	/** 各列挙が持つ値 */
+	private int value;
+	/**
+	 * コンストラクタ
+	 */
+	private MemberRank(int value) {
+		this.value = value;
+	}
+	/**
+	 * @param catalogPrice 定価
+	 * @return ランクに応じた金額
+	 */
+	public abstract BigDecimal getPrice(BigDecimal catalogPrice);
+	/**
+	 * ファクトリー
+	 */
+	public static MemberRank of(int value) {
+		return Stream.of(MemberRank.values()).filter(r -> r.value == value)
+				.findFirst().orElseThrow(IllegalArgumentException::new);
+	}
+}
+```
+そうすると、以下のように設定できます。
+```java
+// rank は、1, 2, 3 の数値
+member.setMemberRank(MemberRank.of(rank));
+```