【5/15】育成パックマリー #4055

https://docs.google.com/spreadsheets/d/1uec1H0v3_SsFYNJYDPz4v5wnsLlDt9CXePqePnxcho8/edit?gid=1416308177#gid=1416308177

本家2023年スキルを含むため、以下のプルリクよりフレイムダンプと共鳴バーンズウィプのみ抜粋
https://github.com/rzstaffgit/jrce2_github/pull/10392

@wada
レビューお願いいたします。

@kouho
①master_training_pack.card_noは転生前と後どちらのカードNoを設定するのですか？

@kouho
②今回初めて強化アイテムIDに50000番台を使用しています。
各強化アイテム処理が50000番台に対応してそうですか？

@wada
確認ありがとうございます。

> ①master_training_pack.card_noは転生前と後どちらのカードNoを設定するのですか？

転生後を設定しておりましたので修正しました

> ②今回初めて強化アイテムIDに50000番台を使用しています。各強化アイテム処理が50000番台に対応してそうですか？

対応しておりませんでしたので、修正しました。

@wada
確認ありがとうございます。修正いたしました。
> 条件式は問題ないですが、メソッド内での記述形式は統一してほしいです。
・ifの後、カッコ内のスペース
・条件の等号不等号

@kouho
すいません、伝え方が悪かったです。
追加した行の上に1000番台のif文があります。
既存・新規ルールどちらでも良いので、メソッド内（1000番台と50000番台）の記述方法を合わせてほしいです。

#既存のルールに合わせる場合（元の方法に合わせる
#1: ifの後にスペース、閉じ括弧の後にスペース
#2: xx1 <= $item_no && $item_no <= xx99
if (1001 <= $item_no && $item_no <= 1999) {
...
if (50001 <= $item_no && $item_no <= 59999) {

#新たなルールにする場合（今回追加した更新に合わせる
#1: カッコの内側にスペース
#2: xx0 <= $item_no && $item_no < xx0
if( 1000 <= $item_no && $item_no < 2000 ) {
...
if( 50000 <= $item_no && $item_no < 60000 ) {

@wada
すみません、if文後の半角スペースが足りないということでしょうか？既存ルールに合わせました。

@kouho
以下でお願いします。

if (50001 <= $item_no && $item_no <= 59999) {

@wada
修正いたしました。
> 以下でお願いします。
if (50001 <= $item_no && $item_no <= 59999) {

@kouho
確認OKです。

@kouho
条件式は問題ないですが、メソッド内での記述形式は統一してほしいです。
・ifの後、カッコ内のスペース
・条件の等号不等号