【10/7】チームレイド攻撃処理修正 #1185

概要

チームレイドの攻撃処理を通常レイドから継承を行い保守性を高める対応です。
岩本さん実装の チームレイドの攻撃処理修正 #673のPRを作り直しました。

テスト方法、確認した内容

• 通常レイド、レアレイド、チームレイドそれぞれで攻撃できるか
• アタックブースト、スピードブーストが有効か
• 蒼玉5個消費攻撃時にも最強海賊の一撃が更新されるか

TODO

次回レイド開催時(10/12から)反映予定

@wada @iwamoto
お手隙で確認よろしくお願いいたします。

@kouho
lib/models/raid_vs/Team_raid_monster_vs.php の対応が抜けているような気がするんですが、コミット漏れなどないですか？

@wada
すみません、確かにlib/models/raid_vs/Team_raid_monster_vs.php の修正のコミットできていませんでしたのでコミットしました。また、Raid_battle_vs_modelの単発の方にも $this->save_damage();を入れています。

@kouho
コミットありがとうございます。
Team_raid_monster_vs の format_to_card_data も削除して良いと思います。

@wada
Team_raid_monster_vs の format_to_card_dataも削除しました。

@kouho
Team_raid_monster_vs のメンバ変数について、
Team_raid_monster の定義と重複してしまうので、動作上問題ありませんが、間違いを避けるため削除して欲しいです。
必要であればそのままで大丈夫です。

	const RAID_TYPE_NORMAL = 1;
	const RAID_TYPE_RARE = 2;
	const RAID_TYPE_TEAM = 3;
	const RAID_CARD_POS = 29;

	protected $damage_log = null;
	protected $card_data = null;
	public $initial_hp = null;

@wada
Team_raid_monster_vsの不要なメンバー変数も削除しました

@kouho
ありがとうございます。確認OKです。

@kouho
確認しました。下記指摘以外は問題ないと思います。

lib/models/Raid_battle_model.php にて追加されているこちらについて

//リザルト画面のランキング用に
$return_data['new_record'] = S_SetUserRaidBestDamage($this->user->user_no, $this->raid->raid_monster_no, $this->raid->lv, $return_data['raid_dmg']);

下記2点確認をお願い致します。

1. lib/models/Raid_battle_model.php側で処理させるなら、lib/public/Raid-Battle.inc記載の同一処理の削除をお願いします
2. lib/models/Raid_battle_model.phpのget_multi_result()(5回+1回攻撃)に追加されていますが、1回攻撃時のみのだったと思います

@iwamoto
確認ありがとうございます。
lib/public/Raid-Battle.inc記載の同一処理の削除をしました。

また5回+1回攻撃にも最強海賊の一撃を更新する方針となりましたので通常レイドとレアレイドの5回+1回攻撃時にも最強海賊の一撃を更新されていたらOKとしています。

@kouho
承知しましたー、であれば大丈夫かと思います
修正も確認しました。

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

@kouho
単発の方にも $this->save_damage(); が必要です。