From 729124a1349bb8bb704786012fbd92a96b05769f Mon Sep 17 00:00:00 2001
From: Habu <habu1010+github@gmail.com>
Date: Sun, 30 Jun 2024 00:30:38 +0900
Subject: [PATCH] [Refactor] PlayerRealm::get_book
MIME-Version: 1.0
Content-Type: text/plain; charset=utf8
Content-Transfer-Encoding: 8bit

魔法領域に対応する魔法書のItemKindTypeを返す関数 get_book は、魔法領域の
列挙値と魔法書の列挙値の並びが同じになっている事を前提としており
気持ちが悪いので、対応表を定数として定義してそこから算出するように
変更する。
また、元の魔法領域なしの場合に ItemKindType::LIFE_BOOK - 1 を返すという
仕様もおかしいので、魔法領域なしの場合は ItemKindType::NONE を返す
ようにする。
---
 src/player/player-realm.cpp | 27 ++++++++++++++++++++++++++-
 1 file changed, 26 insertions(+), 1 deletion(-)

diff --git a/src/player/player-realm.cpp b/src/player/player-realm.cpp
index 0bb16db09..dd2dadc43 100644
--- a/src/player/player-realm.cpp
+++ b/src/player/player-realm.cpp
@@ -6,6 +6,27 @@
 #include "system/player-type-definition.h"
 #include "util/enum-converter.h"
 
+namespace {
+
+const std::map<magic_realm_type, ItemKindType> realm_books = {
+    { REALM_NONE, ItemKindType::NONE },
+    { REALM_LIFE, ItemKindType::LIFE_BOOK },
+    { REALM_SORCERY, ItemKindType::SORCERY_BOOK },
+    { REALM_NATURE, ItemKindType::NATURE_BOOK },
+    { REALM_CHAOS, ItemKindType::CHAOS_BOOK },
+    { REALM_DEATH, ItemKindType::DEATH_BOOK },
+    { REALM_TRUMP, ItemKindType::TRUMP_BOOK },
+    { REALM_ARCANE, ItemKindType::ARCANE_BOOK },
+    { REALM_CRAFT, ItemKindType::CRAFT_BOOK },
+    { REALM_DAEMON, ItemKindType::DEMON_BOOK },
+    { REALM_CRUSADE, ItemKindType::CRUSADE_BOOK },
+    { REALM_MUSIC, ItemKindType::MUSIC_BOOK },
+    { REALM_HISSATSU, ItemKindType::HISSATSU_BOOK },
+    { REALM_HEX, ItemKindType::HEX_BOOK },
+};
+
+}
+
 /*!
  * 職業毎に選択可能な第一領域魔法テーブル
  */
@@ -101,7 +122,11 @@ const magic_type &PlayerRealm::get_spell_info(int realm, int spell_idx)
 
 ItemKindType PlayerRealm::get_book(int realm)
 {
-    return ItemKindType::LIFE_BOOK + realm - 1;
+    const auto it = realm_books.find(i2enum<magic_realm_type>(realm));
+    if (it == realm_books.end()) {
+        THROW_EXCEPTION(std::invalid_argument, format("Invalid realm: %d", realm));
+    }
+    return it->second;
 }
 
 const magic_type &PlayerRealm::get_realm1_spell_info(int num) const
-- 
2.11.0