【2/9】レイドイベント6-1期 #593

【2/9】レイド1期 #584
通常更新とチーム対抗レイドの戦闘処理を修正しました

event_no = 12
Raid_battle_model.php にまとめたかった…

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

実装の方向性は問題なくて、できれば継承を使用して欲しいです。

以下の内容は、対応してもしなくても良いです。
あくまで保守性をあげるだけなので、実際の処理には関係ありません。

• 定数以外をconstで設定しない

max_lvなど
変数（今後変更が予想される値）は、configに設定する

• 継承について

レイド戦闘に関わるクラスは全て継承を想定して、public,protectedにしているので、可能であれば継承が望ましい

• team_raid_monsterクラスを継承する

vsクラスとほぼ同じなので継承して変更に強く
daoが変わる問題は、クラスにdao名を変数で持たせて同一処理で別daoにアクセスできるようにする

#daoを変数に格納
protected $dao = 'dao/team_raid_monster_vs';
$this->dao = 'dao/team_raid_monster_vs';
$res = S_LoadModelClass($this->dao)->get_by_team_no($team_no);

• attack_bonusクラスを継承する

team_raid_monsterクラスと同様
attack_bonusにteam_no追加しても良い

• battle_user内でteam_raid_monsterクラスを参照する

vsクラスがteam_raid_monsterを継承することで、team_raid_monsterのuserだけで問題ない？
クイックアタックは現在使用していないので対応しなくて良い
update_battleとupdate_battle_vsの共通箇所はメソッド化する（別クラスにする場合）

#update_battleを別クラスとする場合
public function update_battle(Team_raid_monster $raid) {
    #個別処理
    $this->{共通処理}();
}
public function update_battle_vs(Team_raid_monster_vs $raid) {
    #個別処理
    $this->{共通処理}();
}

#update_battleを統一する場合
public function update_battle(Team_raid_monster $raid) {
    if ($raid->is_normal()) {
        #個別処理
    } else if ($raid->is_rare()) {
        #個別処理
    } else if ($raid->is_vs()) {
        #個別処理
    }
    #共通処理
}

これは仕様にないので、削除で良いです。