From 6bc04b98083216a9ddf78be10454aa5cb98a5620 Mon Sep 17 00:00:00 2001 From: Hourier <66951241+Hourier@users.noreply.github.com> Date: Sat, 1 Jun 2024 16:21:42 +0900 Subject: [PATCH] =?UTF-8?q?[Refactor]=20#4127=20MonsterRace::is=5Fvalid()?= =?UTF-8?q?=20=E3=82=92=E5=BB=83=E6=AD=A2=E3=81=97=E3=81=9F?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- src/alliance/alliance.cpp | 2 +- src/knowledge/knowledge-alliance.cpp | 2 +- src/monster-race/monster-race.cpp | 17 ----------------- src/monster-race/monster-race.h | 2 -- src/system/monster-race-info.cpp | 4 ++++ 5 files changed, 6 insertions(+), 21 deletions(-) diff --git a/src/alliance/alliance.cpp b/src/alliance/alliance.cpp index 0f451d6ddb..bb551280ff 100644 --- a/src/alliance/alliance.cpp +++ b/src/alliance/alliance.cpp @@ -96,7 +96,7 @@ int64_t Alliance::calcCurrentPower() for (auto &[r_idx, r_ref] : monraces_info) { if (r_ref.alliance_idx == this->id) { if (r_ref.mob_num > 0) { - res += MonsterRace(r_idx).calc_eval() * r_ref.mob_num; + res += MonsterRace(r_idx).calc_power() * r_ref.mob_num; } } } diff --git a/src/knowledge/knowledge-alliance.cpp b/src/knowledge/knowledge-alliance.cpp index c54323daad..b73236c089 100644 --- a/src/knowledge/knowledge-alliance.cpp +++ b/src/knowledge/knowledge-alliance.cpp @@ -46,7 +46,7 @@ void do_cmd_knowledge_alliance(PlayerType *player_ptr, bool detail) for (auto &[r_idx, r_ref] : monraces_info) { if (r_ref.alliance_idx == a.second->id) { - fprintf(fff, _("%s %-40s レベル %3d 評価値 %9d", "%s %-40s LEVEL %3d POW %9d"), r_ref.kind_flags.has(MonsterKindType::UNIQUE) ? "[U]" : "---", r_ref.name.c_str(), r_ref.level, MonsterRace(r_idx).calc_eval()); + fprintf(fff, _("%s %-40s レベル %3d 評価値 %9d", "%s %-40s LEVEL %3d POW %9d"), r_ref.kind_flags.has(MonsterKindType::UNIQUE) ? "[U]" : "---", r_ref.name.c_str(), r_ref.level, MonsterRace(r_idx).calc_power()); if (r_ref.kind_flags.has_not(MonsterKindType::UNIQUE)) { if (r_ref.mob_num > 0) { fprintf(fff, "x %d\n", r_ref.mob_num); diff --git a/src/monster-race/monster-race.cpp b/src/monster-race/monster-race.cpp index 1acb538f54..1feb492e5f 100644 --- a/src/monster-race/monster-race.cpp +++ b/src/monster-race/monster-race.cpp @@ -49,23 +49,6 @@ MonsterRaceId MonsterRace::pick_one_at_random() return table.pick_one_at_random(); } -/*! - * @brief コンストラクタに渡された MonsterRaceId が正当なもの（実際に存在するモンスター種族IDである）かどうかを調べる - * @details モンスター種族IDが MonsterRaceDefinitions に実在するもの(MonsterRaceId::PLAYERは除く)であるかどうかの用途の他、 - * m_list 上の要素などの r_idx にMonsterRaceId::PLAYER を入れることで死亡扱いとして使われるのでその判定に使用する事もある - * @return 正当なものであれば true、そうでなければ false - */ -bool MonsterRace::is_valid() const -{ - return this->r_idx != MonsterRaceId::PLAYER; -} - -int MonsterRace::calc_eval() const -{ - const auto *r_ptr = &monraces_info[this->r_idx]; - return this->calc_power() * r_ptr->level; -} - /*! * @brief モンスター種族が賞金首の対象かどうかを調べる。日替わり賞金首は対象外。 * diff --git a/src/monster-race/monster-race.h b/src/monster-race/monster-race.h index d91bbeb945..7cb7e8b114 100644 --- a/src/monster-race/monster-race.h +++ b/src/monster-race/monster-race.h @@ -15,8 +15,6 @@ class MonsterRace { static MonsterRaceId empty_id(); static MonsterRaceId pick_one_at_random(); - bool is_valid() const; - int calc_eval() const; bool is_bounty(bool unachieved_only) const; int calc_power() const; diff --git a/src/system/monster-race-info.cpp b/src/system/monster-race-info.cpp index 75b0d7f9fe..a6a664ef24 100644 --- a/src/system/monster-race-info.cpp +++ b/src/system/monster-race-info.cpp @@ -10,6 +10,10 @@ MonsterRaceInfo::MonsterRaceInfo() } /*! + * @brief 正当なモンスター (実際に存在するモンスター種族IDである)かどうかを調べる + * @details モンスター種族IDが MonsterRaceDefinitions に実在するもの(MonsterRaceId::PLAYERは除く)であるかどうかの用途の他、 + * m_list 上の要素などの r_idx にMonsterRaceId::PLAYER を入れることで死亡扱いとして使われるのでその判定に使用する事もある + * @return 正当なものであれば true、そうでなければ false * @todo 将来的に定義側のIDが廃止されたら有効フラグのフィールド変数を代わりに作る. */ bool MonsterRaceInfo::is_valid() const