Merge pull request #4190 from habu1010/feature/check-too-new-savefile-version

[Feature] 新しいバージョンのセーブデータを読み込まないようにする

[Feature] 新しいバージョンのセーブデータを読み込まないようにする

おかしなデータでゲームが再開できてしまうのを防ぐため、セーブデータの
バージョンが現在のプログラムで想定されるものより新しい場合は読み込みを
中断しゲームを終了する。

Merge pull request #4178 from Hourier/Change-RandomArtBiasType-EnumClass

random_art_bias_type をenum からenum class に変えた

[Refactor] monster-group-table.h からlanguage-switcher.h のインクルードをcpp に変えた

言語切替マクロが_() の時はcpp からのインクルードでも良いが、'ifdef JP/ifndef JP'で切り替える時は言語のマクロ定義が見えないと強制的に英語モードになるという基本設計のため

Merge pull request #4182 from habu1010/feature/fix-hex-spell-revenge

[Fix] 呪術魔法の我慢が正常に機能していない

[Refactor] #4177 random_art_bias_type をenum class RandomArtifactBias に変えた

[Refactor] #4177 artifact_bias_name をARTIFACT_BIAS_NAMES に変えた

[Refactor] #4177 artifact_bias_name をconcptr[] からmap に変えた

[Refactor] モンスターの攻撃後のテレポート処理

モンスターが金を盗むなどの攻撃をプレイヤーに行った後テレポートする処理は
どう考えてもSpellHexクラスのメソッドでやるべき内容ではないので、
MonsterAttackPlayerクラスに処理を移動する。

[Fix] 呪術魔法の我慢が正常に機能していない

SpellHexクラスのMonsterAttackPlayer*を第2引数に受け取るコンストラクタで
メンバ変数spell_hex_dataを正しく初期化していないため、このコンストラクタで
生成したオブジェクト経由では正しくダメージが蓄積されない。
これが原因で、モンスターの直接攻撃によるダメージが我慢の反撃ダメージに
蓄積されていなかった。
また、魔法やブレスによる攻撃はこのコンストラクタを使用していないため
正しく蓄積が機能していた。
そもそもMonsterAttackPlayer*をメンバに持つほどの必要性が感じられないので、
このコンストラクタを削除し、現在MonsterAttackPlayer*を使用している
メソッドには直接必要な引数を渡すようにする。

Merge pull request #4179 from habu1010/feature/fix-target-compare-function

[Fix] ターゲット比較関数がSTLのソート用比較関数の要件を満たしていない

[Fix] ターゲット比較関数がSTLのソート用比較関数の要件を満たしていない

TargetSorter::compare_importance がSTLのソートで使用する比較関数に要求
される `a < a == false` を満たしていないため、場合によってはデバッグ用
ビルドのアサーションにひっかかることがある。
最初に2つの引数が同じかどうかをチェックし同じなら false を返すように
して、要件を満たすようにする。
TargetSorter::compare_distance に関しては要件を満たしていたが、同じ位置
であれば距離を計算する必要もないのでこちらも先に引数が同じかをチェック
する。

Merge pull request #4176 from Hourier/Change-DisplayGroupList-Signature

display_group_list() の引数をconcptr[] からvector<string> に変えた

[Fix] #4053 地形だけ英語表記しかなかったので日本語にも対応させた

[Refactor] #4053 object-group-table.cpp/h をitem-group-table.cpp/h に変え、定数変数を大文字に変えた

[Refactor] #4053 display_group_list() の引数をconcptr[] からvector<string> に変えた

Merge pull request #4175 from habu1010/feature/fix-learn-reflection

[Fix] モンスターが使用する一部の魔法にプレイヤーの耐性の学習が反映されていない

[Fix] 一部ボルト魔法で学習する属性の誤り

ボルト魔法で学習する属性に以下の誤りがあるので修正する。

- 極寒の矢でなぜか地獄耐性を学習するようになっている
- スターライトアローで閃光耐性を学習しない

[Fix] スターライトアローに閃光耐性の学習が反映されていない

前コミットと同様の理由により、閃光耐性を学習してもスターライトアローは
無条件で選択肢として残ってしまっている。
閃光耐性を学習した時に除外する魔法として追加する。（除外率は50%/25%）

[Fix] 一部のボルト魔法に反射の学習が反映されていない

ボルト魔法にメテオストライクとスターライトアローを追加したが、この時に
反射を学習した場合に除外する魔法として追加していなかったため、学習後も
この2つのボルト魔法を選択してしまう。
反射を学習した場合の除外魔法に追加する。

Merge pull request #4172 from Hourier/Replace-SharedPtr-NormalClass-2

TimedEffects クラスのフィールド変数をshared_ptr から普通のクラスに変えた その2

[Refactor] #4171 不要になったplayer-cut.h のインクルードを外した

[Refactor] #4171 TimedEffects にしか依存しない関数からPlayerType への依存を外した

[Refactor] #4171 TimedEffects::PlayerPoison をshared_ptr から普通のクラスに差し替えた

[Refactor] #4171 TimedEffects::PlayerDeceleration をshared_ptr から普通のクラスに差し替えた

[Refactor] #4171 TimedEffects::PlayerAcceleration をshared_ptr から普通のクラスに差し替えた

[Refactor] #4171 MindPowerGetter のコンパイル警告を解消した

[Refactor] #4171 TimedEffects::PlayerStun をshared_ptr から普通のクラスに差し替えた

[Refactor] #4171 TimedEffects::PlayerParalysis をshared_ptr から普通のクラスに差し替えた

Merge pull request #4170 from Hourier/Replace-SharedPtr-NormalClass

TimedEffects クラスのフィールド変数をshared_ptr から普通のクラスに変えた その1

[Refactor] #4169 monster-damage.cpp/h のコンパイル警告を解消した

[Refactor] #4169 TimedEffects::PlayerHallucination をshared_ptr から普通のクラスに差し替えた

[Refactor] #4169 TimedEffects::PlayerFear をshared_ptr から普通のクラスに差し替えた

[Refactor] #4169 TimedEffects::PlayerCut をshared_ptr から普通のクラスに差し替えた

[Refactor] #4169 TimedEffects::PlayerConfusion をshared_ptr から普通のクラスに差し替えた

[Refactor] #4169 self-info.cpp のコンパイル警告を解消した

[Refactor] #4169 TimedEffects::PlayerBlindness をshared_ptr から普通のクラスに差し替えた

Merge pull request #4168 from Hourier/Rename-ColoredChar-DisplaySymbol

[Refactor] #4166 colored-char.h をクラス名に合わせてdisplay-symbol.h に変えた

[Refactor] #4166 colored-char.h をクラス名に合わせてdisplay-symbol.h に変えた

Merge pull request #4167 from Hourier/Unify-ItemEntity-GetSymbol

ItemEntity::get_symbol() の戻り値をDisplaySymbol に変え、色と文字を取得するメソッドはprivate に変えた

[Refactor] #4165 c_roff_wrap() の戻り値の型をDisplaySymbol に変えた

[Refactor] #4165 change_display_by_mutation() の引数と戻り値の型を調整してポインタ引数をなくした

[Refactor] #4165 compensate_stat_by_weapon() の引数と戻り値の型を調整してポインタ引数をなくした

[Refactor] #4165 bolt_pict() の返り値をDisplaySymbol に変えた

[Refactor] #4165 term_putch() とterm_addch() の引数をDisplaySymbol に変えた

[Refactor] #4165 print_rel() の引数をDisplaySymbol に変えた

[Refactor] #4165 ItemEntity::get_symbol() の戻り値をDisplaySymbol に変え、色と文字を取得するメソッドはprivate に変えた

シンボル変更コマンドでシンボルが変わってしまってもいいように、get_symbol() はベースアイテム情報から都度再取得する

Merge pull request #4163 from Hourier/Replace-ColoredChar-DisplaySymbol

ColoredChar をDisplaySymbol に差し替えた

[Refactor] #4133 BaseitemInfo、MonsterRaceInfo、TerrainType について、表示シンボルの定義値を表すフィールド変数をdefinition/definitions に正確化した

元はdef/defs だったので省略可し過ぎており少し分かりにくい部分があった

[Refactor] #4133 ColoredChar をDisplaySymbol に置き換えた

機械的な単純置換だが行数は爆発的に多いので注意

[Refactor] #4133 monster-pain-describer.cpp のsymbol をcharacters に改名した

Merge pull request #4162 from habu1010/feature/use-stl-for-target-sorting

[Refactor] ターゲットのソートにSTLを使用する

Merge pull request #4161 from Hourier/Replace-SaveFileBase-Path

savefile_base をstring からpath に変えた

[Refactor] ターゲットのソートにSTLを使用する

[Refactor] savefile_base をstring からpath に変えた

変更当初はstring として扱うべきかpath として扱うべきか不明瞭だったので一旦string にしていた
改めて影響範囲を確認したところ、path で問題なさそうだと判断したのでpath に変えた

Merge pull request #4159 from Hourier/Replace-AngSort-TargetSorter

util/sort.cpp/h をtarget/target-sorter.cpp/h へ移設し、クラス化した

[Refactor] #4158 Double Distance の計算処理を別メソッドに分離した

[Refactor] #4158 compare_distance() をPos2D とPos2Vec を使った形にまとめた

[Refactor] #4158 ang_sort_comp_distance() をTargetSorter のオブジェクトメソッドに繰り込んだ

[Refactor] #4158 ang_sort_comp_importance() をTargetSorter のオブジェクトメソッドに繰り込んだ

[Refactor] #4158 exe_sort()、ang_sort_comp_distance()、ang_sort_comp_importance() の引数名を統一した

[Refactor] #4158 TargetSorter のコンストラクタを調整した

[Refactor] #4158 exe_ang_sort() をTargetSorter::exe_sort() に繰り込んだ

[Refactor] #4158 ang_sort() をTargetSorter::sort() にクラス化した

[Refactor] #4158 sort.cpp/h をtarget-sorter.cpp/h にリネームした

Merge pull request #4157 from Hourier/Prepare-Replace-AngSort-TargetSorter

ang_sort_comp_distance() / ang_sort_comp_importance() / ang_sort_swap_position() を廃止する準備

Merge pull request #4160 from Hourier/Remove-DebugSave-Delete

デバッグセーブに関するグローバル変数と、古いデバッグセーブを削除するルーチンを除去した

[Refactor] デバッグセーブに関するグローバル変数と、古いデバッグセーブを削除するルーチンを除去した

古いデバッグデーブは残ってしまうが、悪さはしないのとリリースノートにこの旨記載することで各自の環境でデバッグセーブを削除して頂く運用とする

[Refactor] #4155 ang_sort() の引数からPlayerType を除去した

[Refactor] #4155 sort.cpp の中を必要なフィールドだけ引数にするよう変更した

[Refactor] #4155 enum class SortKind を作り、関数ポインタをなくした

[Refactor] #4155 ang_sort_comp_distance() の中身を整形した

[Refactor] #4155 ang_sort_comp_importance() の中身を整形した

[Refactor] #4155 ang_sort_swap_position() を廃止した

[Refactor] #4155 ang_sort_swap_position() をstatic関数に変えた

Merge pull request #4156 from Hourier/Replace-AngbandSortCompareNestMonsterInfo

nest_mon_info_type のソート関数の自身のオブジェクトメソッドに繰り込んだ

[Refactor] #4154 nest_mon_info_type 構造体をNestMonsterInfo クラスに変えた

[Refactor] #4154 MonsterRaceList::get_monrace() を実装し、nest_mon_info_type でカプセル化した

[Refactor] #4154 nest_mon_info_type::order_nest() を実装し、スワップ関数を廃止した

Merge pull request #4153 from Hourier/Add-GridTemplate-Sort

grid_template_type のソートをSTLのソートに変えた

[Refactor] #4147 GridTemplate::matches() を実装して、generate_sorted_grid_templates() とwr_saved_floor() を軽量化した

[Refactor] #4147 sort.h を必要としないcpp ファイルからインクルードをなくした

[Refactor] #4147 wr_saved_floor() からgenerate_sorted_grid_templates() を分離した

[Refactor] #4147 ang_sort_comp_cave_temp()、ang_sort_swap_cave_temp() を廃止した

[Refactor] #4147 wr_monster() の引数をMonsterEntity* からconst MonsterEntity& に変えた

[Refactor] #4147 wr_item() の引数をItemEntity* からconst ItemEntity& に変えた

[Refactor] #4147 grid_template_type 構造体をGridTemplate クラスに変えた

Merge pull request #4152 from Hourier/Add-Quest-Sort

ang_sort_comp_quest_num() / ang_sort_swap_quest_num() を廃止した

[Fix] ソートメソッド群の比較演算を正しく設定し直した

comp(a, a) == false でなければならないが、trueになってしまう条件式 (<=) だったので等号を含まない形に修正した

[Refactor] #4140 get_sorted_quest_ids() を実装し、ang_sort_comp_quest_num() とang_sort_swap_quest_num() を廃止した

TODO vector<QuestId> ではなくvector<shared_ptr<const QuestType>> を直接返してしまった方が受け取った側で扱いやすそうである
しかしそのために必要な修正量はかなり多いので後回しにする

[Refactor] #4140 QuestList::order_completed() を実装した

Merge pull request #4151 from Hourier/Reshape-QuestList-2

QuestType/QuestList の整形作業 その2

[Refactor] #4146 QuestList::operator[] をget_quest() に差し替えた

[Refactor] #4146 fmt はformat() への引数、mes は同戻り値とした方がシグネチャと合わせやすいので揃えた

[Refactor] #4146 quest_list をquests に改名した (フォルダの頭文字p～w)

[Refactor] #4146 quest_list をquests に改名した (フォルダの頭文字i～m)

[Refactor] #4146 quest_list をquests に改名した (フォルダの頭文字b～f)

Merge pull request #4150 from Hourier/Reshape-QuestList-1

QuestType/QuestList の整形作業 その1

[Refactor] #4139 concptr[] find_quest_map をvector<string> に変えた

[Refactor] #4139 QuestType → QuestList の順番でヘッダが定義されているのでQuestTypeのオブジェクトメソッドはcppファイルの上に並べた