OSDN Git Service

Replace sprintf() simply. Does part of the work to resolve https://github.com/hengba...
[hengbandforosx/hengbandosx.git] / src / lore / lore-calculator.cpp
index 9430e08..91d04c2 100644 (file)
@@ -5,8 +5,9 @@
 #include "monster-race/race-ability-flags.h"
 #include "monster-race/race-flags1.h"
 #include "mspell/mspell-damage-calculator.h"
-#include "system/monster-race-definition.h"
+#include "system/monster-race-info.h"
 #include "system/player-type-definition.h"
+#include "util/string-processor.h"
 
 /*!
  * @brief ダイス目を文字列に変換する
@@ -26,19 +27,22 @@ void dice_to_string(int base_damage, int dice_num, int dice_side, int dice_mult,
         return;
     }
 
-    if (base_damage != 0)
-        sprintf(base, "%d+", base_damage);
+    if (base_damage != 0) {
+        strnfmt(base, sizeof(base), "%d+", base_damage);
+    }
 
-    if (dice_num == 1)
-        sprintf(dice, "d%d", dice_side);
-    else
-        sprintf(dice, "%dd%d", dice_num, dice_side);
+    if (dice_num == 1) {
+        strnfmt(dice, sizeof(dice), "d%d", dice_side);
+    } else {
+        strnfmt(dice, sizeof(dice), "%dd%d", dice_num, dice_side);
+    }
 
     if (dice_mult != 1 || dice_div != 1) {
-        if (dice_div == 1)
-            sprintf(mult, "*%d", dice_mult);
-        else
-            sprintf(mult, "*(%d/%d)", dice_mult, dice_div);
+        if (dice_div == 1) {
+            strnfmt(mult, sizeof(mult), "*%d", dice_mult);
+        } else {
+            strnfmt(mult, sizeof(mult), "*(%d/%d)", dice_mult, dice_div);
+        }
     }
 
     sprintf(msg, "%s%s%s", base, dice, mult);
@@ -52,22 +56,26 @@ void dice_to_string(int base_damage, int dice_num, int dice_side, int dice_mult,
  * @details
  * The higher the level, the fewer kills needed.
  */
-bool know_armour(MONRACE_IDX r_idx, const bool know_everything)
+bool know_armour(MonsterRaceId r_idx, const bool know_everything)
 {
-    monster_race *r_ptr = &r_info[r_idx];
+    auto *r_ptr = &monraces_info[r_idx];
     DEPTH level = r_ptr->level;
     MONSTER_NUMBER kills = r_ptr->r_tkills;
 
-    bool known = (r_ptr->r_cast_spell == MAX_UCHAR) ? true : false;
+    bool known = r_ptr->r_cast_spell == MAX_UCHAR;
 
-    if (know_everything || known)
+    if (know_everything || known) {
         return true;
-    if (kills > 304 / (4 + level))
+    }
+    if (kills > 304 / (4 + level)) {
         return true;
-    if (!(r_ptr->flags1 & RF1_UNIQUE))
+    }
+    if (r_ptr->kind_flags.has_not(MonsterKindType::UNIQUE)) {
         return false;
-    if (kills > 304 / (38 + (5 * level) / 4))
+    }
+    if (kills > 304 / (38 + (5 * level) / 4)) {
         return true;
+    }
     return false;
 }
 
@@ -83,24 +91,28 @@ bool know_armour(MONRACE_IDX r_idx, const bool know_everything)
  * the more damage an attack does, the more attacks you need
  * </pre>
  */
-bool know_damage(MONRACE_IDX r_idx, int i)
+bool know_damage(MonsterRaceId r_idx, int i)
 {
-    monster_race *r_ptr = &r_info[r_idx];
+    auto *r_ptr = &monraces_info[r_idx];
     DEPTH level = r_ptr->level;
-    s32b a = r_ptr->r_blows[i];
+    int32_t a = r_ptr->r_blows[i];
 
-    s32b d1 = r_ptr->blow[i].d_dice;
-    s32b d2 = r_ptr->blow[i].d_side;
-    s32b d = d1 * d2;
+    int32_t d1 = r_ptr->blow[i].d_dice;
+    int32_t d2 = r_ptr->blow[i].d_side;
+    int32_t d = d1 * d2;
 
-    if (d >= ((4 + level) * MAX_UCHAR) / 80)
+    if (d >= ((4 + level) * MAX_UCHAR) / 80) {
         d = ((4 + level) * MAX_UCHAR - 1) / 80;
-    if ((4 + level) * a > 80 * d)
+    }
+    if ((4 + level) * a > 80 * d) {
         return true;
-    if (!(r_ptr->flags1 & RF1_UNIQUE))
+    }
+    if (r_ptr->kind_flags.has_not(MonsterKindType::UNIQUE)) {
         return false;
-    if ((4 + level) * (2 * a) > 80 * d)
+    }
+    if ((4 + level) * (2 * a) > 80 * d) {
         return true;
+    }
 
     return false;
 }
@@ -111,43 +123,50 @@ bool know_damage(MONRACE_IDX r_idx, int i)
  * @param SPELL_NUM 呪文番号
  * @param msg 表示する文字列
  */
-void set_damage(player_type *player_ptr, lore_type *lore_ptr, RF_ABILITY ms_type, concptr msg)
+void set_damage(PlayerType *player_ptr, lore_type *lore_ptr, MonsterAbilityType ms_type, concptr msg)
 {
-    MONRACE_IDX r_idx = lore_ptr->r_idx;
+    MonsterRaceId r_idx = lore_ptr->r_idx;
+    char *tmp = lore_ptr->tmp_msg[lore_ptr->vn];
+    size_t tmpsz = sizeof(lore_ptr->tmp_msg[lore_ptr->vn]);
+
+    if (!know_armour(r_idx, lore_ptr->know_everything)) {
+        strnfmt(tmp, tmpsz, msg, "");
+        return;
+    }
+
     int base_damage = monspell_race_damage(player_ptr, ms_type, r_idx, BASE_DAM);
     int dice_num = monspell_race_damage(player_ptr, ms_type, r_idx, DICE_NUM);
     int dice_side = monspell_race_damage(player_ptr, ms_type, r_idx, DICE_SIDE);
     int dice_mult = monspell_race_damage(player_ptr, ms_type, r_idx, DICE_MULT);
     int dice_div = monspell_race_damage(player_ptr, ms_type, r_idx, DICE_DIV);
-    char dmg_str[80], dice_str[sizeof(dmg_str) + 10];
-    char *tmp = lore_ptr->tmp_msg[lore_ptr->vn];
-    dice_to_string(base_damage, dice_num, dice_side, dice_mult, dice_div, dmg_str);
-    sprintf(dice_str, "(%s)", dmg_str);
-
-    if (know_armour(r_idx, lore_ptr->know_everything))
-        sprintf(tmp, msg, dice_str);
-    else
-        sprintf(tmp, msg, "");
+    char dmg_str[80] = "(";
+    dice_to_string(base_damage, dice_num, dice_side, dice_mult, dice_div, dmg_str + 1);
+    angband_strcat(dmg_str, ")", sizeof(dmg_str));
+    strnfmt(tmp, tmpsz, msg, dmg_str);
 }
 
 void set_drop_flags(lore_type *lore_ptr)
 {
-    if (!lore_ptr->know_everything)
+    if (!lore_ptr->know_everything) {
         return;
+    }
 
-    lore_ptr->drop_gold = lore_ptr->drop_item = (((lore_ptr->r_ptr->flags1 & RF1_DROP_4D2) ? 8 : 0) + ((lore_ptr->r_ptr->flags1 & RF1_DROP_3D2) ? 6 : 0)
-        + ((lore_ptr->r_ptr->flags1 & RF1_DROP_2D2) ? 4 : 0) + ((lore_ptr->r_ptr->flags1 & RF1_DROP_1D2) ? 2 : 0)
-        + ((lore_ptr->r_ptr->flags1 & RF1_DROP_90) ? 1 : 0) + ((lore_ptr->r_ptr->flags1 & RF1_DROP_60) ? 1 : 0));
+    lore_ptr->drop_gold = lore_ptr->drop_item = ((lore_ptr->r_ptr->drop_flags.has(MonsterDropType::DROP_4D2) ? 8 : 0) + (lore_ptr->r_ptr->drop_flags.has(MonsterDropType::DROP_3D2) ? 6 : 0) + (lore_ptr->r_ptr->drop_flags.has(MonsterDropType::DROP_2D2) ? 4 : 0) + (lore_ptr->r_ptr->drop_flags.has(MonsterDropType::DROP_1D2) ? 2 : 0) + (lore_ptr->r_ptr->drop_flags.has(MonsterDropType::DROP_90) ? 1 : 0) + (lore_ptr->r_ptr->drop_flags.has(MonsterDropType::DROP_60) ? 1 : 0));
 
-    if (lore_ptr->r_ptr->flags1 & RF1_ONLY_GOLD)
+    if (lore_ptr->r_ptr->drop_flags.has(MonsterDropType::ONLY_GOLD)) {
         lore_ptr->drop_item = 0;
+    }
 
-    if (lore_ptr->r_ptr->flags1 & RF1_ONLY_ITEM)
+    if (lore_ptr->r_ptr->drop_flags.has(MonsterDropType::ONLY_ITEM)) {
         lore_ptr->drop_gold = 0;
+    }
 
     lore_ptr->flags1 = lore_ptr->r_ptr->flags1;
     lore_ptr->flags2 = lore_ptr->r_ptr->flags2;
     lore_ptr->flags3 = lore_ptr->r_ptr->flags3;
     lore_ptr->ability_flags = lore_ptr->r_ptr->ability_flags;
-    lore_ptr->flagsr = lore_ptr->r_ptr->flagsr;
+    lore_ptr->aura_flags = lore_ptr->r_ptr->aura_flags;
+    lore_ptr->behavior_flags = lore_ptr->r_ptr->behavior_flags;
+    lore_ptr->resistance_flags = lore_ptr->r_ptr->resistance_flags;
+    lore_ptr->feature_flags = lore_ptr->r_ptr->feature_flags;
 }