Merge pull request #1783 from habu1010/feature/refactor-rd-integer

[Refactor] セーブファイルから整数値を読み込む関数のインターフェース

Merge pull request #1786 from Hourier/Remove-MIN-Macro-Function

MIN() のマクロ関数をstd::min() に差し替えた

[Refactor] #1434 GCCのコンパイルエラーに対応した (short かつ 3文字以内はバッファオーバーフローする可能性がある。-101等)

[Refactor] #1434 Resolved all compilation warnings

[Refactor] #1432 Reshaped dump-util.cpp automatically

[Refactor] #1432 MIN() のマクロ関数をstd::min() に差し替えた / Replaced the macro function MIN() to std::min()

[Refactor] 重複していたプリプロを整理・統合した

Merge pull request #1785 from Hourier/Remove-ABS-Macro-Function

ABS() のマクロ関数をstd::abs() に差し替えた

[Fix] #1433 MAX() の置換漏れを修正した

[Refactor] #1434 ABS() のマクロ関数をstd::abs() に差し替えた / Replaced the macro function ABS() to std::abs()

Merge pull request #1784 from Hourier/Remove-MAX-Macro-Function

MAX() のマクロ関数をstd::max() に差し替えた

[Refactor] #1433 MAX() のマクロ関数をstd::max() に差し替えた / Replaced the macro function MAX() to std::max()

[Refactor] セーブファイルから整数値を読み込む関数のインターフェース

tmp8u などの一時変数を頻繁に使用する事になっている主な原因として、セーブ
ファイルから整数を読み込む関数群 rd_byte/s16b/u16b/s32b/u32b のインター
フェースが引数にポインタを受け取りそのポインタが指す領域にセーブファイル
から読み込んだ値を書き込むようになっている事が挙げられる。
そもそも読み込んだデータを戻り値として返すのが自然であるし、コードの冗長
性が大幅に減らせるのでそのようにインターフェースを変更する。

Merge pull request #1774 from habu1010/feature/refactor-weapon-skill-table

[Refactor] 武器熟練度のテーブルを std::map にする

[Refactor] 武器熟練度のテーブルを std::map にする

既存の武器熟練度のテーブルは弓のtvalをベースにした武器のtvalのオフセットを
要素にした配列で持っているためアクセスするのに余計な引き算や列挙型と基底型
の変換が必要になってしまう。
列挙型で直接アクセスでき、データの意味合い的にも連想コンテナのほうがふさわ
しいので std::map を使用するようにする。

Merge pull request #1777 from Hourier/Fix-Building-Class-Race-Error

店主のクラス・種族リストが未初期化のまま追加されようとしてソフトウェアが落ちていた不具合を解消した

[Fix] #1776 店主のクラス・種族リストが未初期化のまま追加されようとしてソフトウェアが落ちていた不具合を解消した

Merge pull request #1754 from Hourier/Change-Player-Class-Type-to-Enum-Class

player_class_type を enum → enum class に変換した

[Refactor] #1754 Habu氏の指摘に対応した
- PlayerClassType::MAX をshortとして扱っている箇所が多いので別途定義した
- world.h のFlagGroupをEnumClassFlagGroupに差し替えた

[Refactor] #1752 GCCのコンパイルエラーを解消した

[Refactor] #1752 PlayerClassTypeをenumからenum classへ変更した

[Refactor] #1752 Replaced arrays in building-type-definition.h to vector

[Refactor] #1752 クラス称号が日英で分かれていたのを統一し、更にstring_viewに差し替えた

[Refactor] #1752 Replaced arrays in class-info.h to vector

[Refactor] #1752 Replaced arrays in initial-equipments-table.h to vector

[Refactor] #1752 Replaced arrays in birth-explanations-table.h to vector

[Refactor] #1752 Changed the name of player_class_type to PlayerClassType

Merge pull request #1771 from habu1010/feature/enum-class-calculation

[Feature] 列挙型の値の加減算

Merge pull request #1773 from Hourier/Fix-Wounds

元素使いで切り傷を直すと却って傷が深くなる不具合を修正した

[Refactor] tval2realm の呼び出しから enum2i の変換を削除

tval2realm の呼び出しは引数に直接 ItemKindType を渡せるので enum2i を削除
する。
また現在使用している箇所の他に tval2realm を使用できる箇所がいくつかあった
ので使用するようにする。

[Refactor] 列挙値間の差の計算部からenum2iを取り除く

先のコミットで列挙値間の差を計算する operator-() を実装したので、不要に
なった enum2i を削除する。

[Fix] #1772 元素使いで切り傷を直すと却って傷が深くなる不具合を修正した

Merge pull request #1522 from Slimebreath6078/feature/Rename_spell_msg

敵の呪文詠唱のメッセージを表示する関数の文字列変数を改名・構造体化する

[Feature] 列挙型の値の加減算

列挙型の基底型の整数値が並んでいるのを前提とした加減算処理が多数あるので、
列挙型に対する operator+() operator-() を実装して加減算を行いやすくする。

以下の3種の加減算を行える。
- 列挙型の値 + 整数値n : 基底型の整数値をnプラスした列挙型を返す
- 列挙型の値 - 整数値n : 基底型の整数値をnマイナスした列挙型を返す
- 列挙型の値 - 列挙型の値 : それぞれの列挙型の基底型の整数値の差を返す

Merge pull request #1767 from Hourier/Change-Tval-Type-To-Enum-Class

enum tval_type を enum class ItemKindType に改名した

[Refactor] #1766 Habu氏の指摘に従いItemKindTypeに改名し、接頭辞の「TV_」を消した

Merge pull request #1768 from habu1010/feature/english-text-append-spacing

[Feature] 英語版のフレーバーテキストを良い感じに連結する

Merge pull request #1770 from habu1010/feature/enum-range-class

[Feature] 列挙値の範囲を扱うクラス

simpler_monspell_message()のメッセージを構造体化

spell_badstatus_message_to_***のメッセージ群を構造体化

spell_badstatus_message()が対プレイヤーと対モンスターで内容が違うので分離

盲目、非盲目時対プレイヤー、非盲目時対モンスターのメッセージを構造体にする

[Feature] 英語版のフレーバーテキストを良い感じに連結する

現在*_info.txtに記述される英語版のフレーバーテキストは、2行以上にわたる
場合スペースを挟んで連結されるように行末か行頭にスペースを入れている。
しかし、行末にスペースを入れると視認しづらいし、どちらに入れるにしても
メンテナンスが面倒になるので、パーサの処理時に自動的にスペースを挟んで
連結するようにする。
既存のフレーバーテキストのスペースは削除しなくて済むように、パーサ側で
両端のスペースは削除しながら連結する。

[Refactor] StoreSaleType の範囲に対する処理

汎用性に欠けるので StoreSaleType のイテレーション専用の関数群を削除し、
EnumRange クラスを代わりに使用する。

[Feature] 列挙値の範囲を扱うクラス

Imprement #1769.
enum および enum class の列挙値の範囲を扱うクラス EnumRange を追加する。

[Refactor] #1766 Changed ItemPrimaryType from enum to enum class

Merge pull request #1760 from sikabane-works/feature/refactor-StoreSaleType

[Refactor] 店舗の種類を enum class StoreSaleType 型に変更しリファクタリング。

[Refactor] 店舗の種類を enum class StoreSaleType 型に変更しリファクタリング。

Merge pull request #1764 from Hourier/Move-Arena-Type

arena-type構造体をbuilding-type-definition.h から arena-info-table.h に移した

[Refactor] Moved the struct arena-type from building-type-definition.h to arena-info-table.cpp/h and changed the type of arena_info from array to std::vector

Merge pull request #1762 from Hourier/Reshape-Patron-List

Patronクラスのコンストラクタからp_ptrを外した

[Refactor] Patronクラスのコンストラクタからp_ptrを外した

Merge pull request #1759 from habu1010/feature/refactor-rng

乱数生成に STL を使用する

[Refactor] STL の正規分布アルゴリズムを使用する

従来のテーブルによる擬似正規分布生成から、STL の std::normal_distribution
を使用した生成に変更する。

[Refactor] next_bool() を削除

one_in_(2) で十分なので不要。わざわざ別途用意する意義を感じない。
そもそも next_bool() が半々の確率で true / false が返される関数だとは
受け取りにくい。

[Refactor] 一様分布乱数の生成を STL で行う

STL の <random> ヘッダで提供されるアルゴリズムを使用できるようにするため、
<random> ヘッダを使用するための Uniform Random Bit Generator の要件を満た
した Xoshiro128StarStar(xoshiro128** 乱数生成アルゴリズム) クラスを実装
する。

これまで一様分布乱数を取得するのに Rand_div() を使用していたが、代わりに
上記のクラスと std::uniform_int_distribution を使用するように変更する。

Merge pull request #1757 from sikabane-works/feature/delete-rd_unique_info

[Fix] 機能的に無意味と判断されたexe_reading_savefile()中のrd_unique_info()の呼び出しと関数そのものを削除

Merge pull request #1745 from Hourier/Make-SaveDataFlags-Enum-Class

セーブデータフラグ群をenum class として定義した

[Fix] 機能的に無意味と判断されたexe_reading_savefile()中のrd_unique_info()の呼び出しと関数そのものを削除

[Refactor] #1744 重複しているenum列挙子を削除した

[Refactor] #1744 savedata_item_flag_type とsavedata_monster_flag_type をenum classとして定義した

Merge pull request #1751 from Hourier/Change-Player-Race-Type-to-Pasacl-Case

player_race_type を PlayerRaceType に置換した

Merge pull request #1742 from habu1010/feature/show-smith-activation-flag-in-character-info

[Fix] 発動を付与した鍛冶アイテムのフラグをキャラクタ情報に表示する

[Refactor] Changed player_race_type to Pascal case: PlayerRaceType

Merge pull request #1749 from Hourier/Improve-Boolean-Judgement

'? true : false' 形式の判定を適切に差し替えた

[Refactor] #1523 '? true : false' 形式の判定を適切に差し替えた

Merge pull request #1741 from Hourier/Improve-Random-Quest-Reward-Design

make_object() の引数rq_level をデフォルト引数にし、型もintからstd::optional<int>に変更した

[Refactor] #1737 Habu氏の指摘に対応した＋微修正

[Fix] 発動を付与した鍛冶アイテムのフラグをキャラクタ情報に表示する

Resolve #1740.
元々 object_flags_known では発動フラグの付加がわざわざ除かれていたので
それに倣っていたが、キャラクタ情報の発動フラグに表示されなくなるので
object_flags と同様に発動を付与した鍛冶アイテムには TR_ACTIVATE フラグを
付加するようにする。

[Refactor] #1737 make_object() の引数rq_level をデフォルト引数にし、型もintからstd::optional<int>に変更した

Merge pull request #1738 from sikabane-works/release/3.0.0Alpha40

[Release] 3.0.0Alpha40

[Release] 3.0.0Alpha40

Merge pull request #1736 from Hourier/Improve-Random-Quest-Reward-Quality

ランダムクエスト報酬を改善した

[Refactor] #1413 Added check_quality() into make_reward()

[Feature] #1413 Separated set_ammo_quantity() from make_object()

[Feature] #1413 Added get_base_floor() into make_object()

Merge pull request #1725 from Hourier/Divide-QuestCompletionChecker

QuestCompletionCheckerクラスの分割

[Refactor] #1717 Divided switch_completion()

[Refactor] #1717 Reshaped switch_completion()

[Refactor] #1717 Separated make_reward() from complete()

[Refactor] #1717 Separated make_stairs() from complete()

[Refactor] #1717 Separated switch_quest_process() from complete()

[Refactor] #1717 Defined q_ptr as private variable

[Refactor] #1717 Separated set_quest_idx() from complete()

Merge pull request #1735 from Hourier/Fix-Tower-Untaken

塔のクエストがクリア状態にならない不具合を解消した

[Fix] 塔のクエストがクリア状態にならない不具合を解消した

Merge pull request #1734 from Hourier/Fix-Ninja-Kawarimi

忍者の変わり身が意図せず発動する事象を修正した

Merge pull request #1732 from habu1010/feature/fix-spell-hex-bug

呪術のおかしな挙動の修正

Merge pull request #1733 from sikabane-works/feature/change-icon2021

[Modify] Windows版アイコンを斑多氏案に変更。

[Fix] #1727 忍者の変わり身が意図せず発動する事象を修正した / Resolved the issue that the ninja's kawarimi was unintentionally triggered

Merge pull request #1726 from habu1010/feature/ninja-class-specific-data

[Refactor] 忍者のデータを職業固有データに移動する

[Modify] Windows版アイコンを斑多氏案に変更。

[Fix] 呪術の中断時に不要なメッセージが出る

詠唱中の呪術がなくなった時、「呪文の詠唱を中断しました。」の後に「呪文を
詠唱していません。」と不要なメッセージが出る。
これは本来呪文を詠唱していないときに詠唱をやめるレイシャルを使用した時の
メッセージだが、リファクタリングでコールチェーンが変わった結果表示される
ようになってしまっている。
呪文の詠唱を中断しました。の後には呪文を詠唱していません。は表示されない
ようにする。

[Fix] 呪術の全復活で詠唱中断後に詠唱中の表示が残る

Fix #1708.
他に詠唱している呪文があるかどうかの判定が反転している。
正しい判定に修正して他に詠唱中の呪文がなければ ACTION_NONE をセットする
ようにする。

Merge pull request #1730 from habu1010/feature/smith-act-stone-mud-essence-consumption

[Feature] 鍛冶で岩石溶解の発動付加のエッセンス消費量を増やす

Merge pull request #1728 from habu1010/feature/fix-cheat-option-cursor-dont-move

[Fix] 詐欺オプションでカーソルが動かない

[Fix] 詐欺オプションでカーソルが動かない

Fix #1702.
カーソルの位置を持つ変数の宣言位置がまずく毎回0になってしまっている。
ループの前に宣言するように修正する。
ついでに変化する事がない定数もループの外に出しておく。

[Feature] 鍛冶で岩石溶解の発動付加のエッセンス消費量を増やす

適当に25にしていたが、消費量が少なすぎるという意見があった。
採掘+1あたり10を消費する事を考えると採掘+2.5相当で3ターン毎に岩石溶解が
発動できるのは確かにバランスが悪いと思われるので、エッセンス消費量を100
とする。

[Refactor] 忍者のデータを職業固有データに移動する

忍者特有の超隠密・変わり身の発動中かどうかのフラグを忍者の職業固有データ
ninja_data_type を定義して移動させる。
早駆けについは player_type::action だけで制御できるようなのでそのままに
しておく。
また、超隠密状態を解除する処理 set_superstealth(player_ptr, false) を呼ぶ
時に事前に超隠密状態かどうかをチェックしているが、set_superstealthの内部
でも超隠密状態かどうかチェックされており、超隠密状態でなければ何も起きない
ので、呼ぶ前のチェックをすべて削除する。

Merge pull request #1712 from Hourier/Make-Quest-Completion-Checker-Class

quest-completion-checker.cpp/h をクラス化した