【10/13】バトルイベント 4 #332

【10/13】バトルイベント #317

@wada
こちらレビュー等お願い致します。

達成報酬の12000が快速に、90000が抜けてます。

@wada
すみません、こちら修正対応を行いました

達成報酬の12000を特急に
達成報酬の90000を限定カードを追加

船ボーナスが今回はキャラックスタートみたいです。

@wada
船ボーナス修正しました

Battle_event_model.phpで存在しないメソッドを呼び出してます。

$user_event_pt_dao->create_user_event_pt($user_info['user_no'], $user_info['team_no'], 0);

@iwamoto
処理上は問題ないところですが、とりあえず以下の2点を確認して欲しいです。

絶対に使用しないであろうコメントアウトは削除して欲しい

//require_once dirname(__FILE__).'/raid/Team_raid_monster.php';
//require_once dirname(__FILE__).'/raid/Battle_user.php';
//require_once dirname(__FILE__).'/raid/Attack_bonus.php';
//
//use lib\models\raid\Team_raid_monster;
//use lib\models\raid\Battle_user;
//use lib\models\raid\Attack_bonus;

ある程度、現状のコーディングルールに従って欲しい

# メソッド名の後にスペースを入れない
public function pirates_num_of_defeated_rest (array $user_info)

# アロー演算子の前後にスペースを入れない
$battle_up_rate = $battle_event_class -> battle_rate_adjustment($user_mission['m_event_limit']);

# 返り値がnullでない場合、型宣言でnullable指定しない
public function battle_rate_adjustment(int $m_event_limit): ?int // int

@wada
すみません、上記指摘のコード修正を行いました。

@iwamoto
確認しました。