【4/14】アタックブースト改善 #1732

概要

#1719 の対応を入れるPRです。
対応を入れた後もユーザーからの問い合わせが引き続き来て問題が発生しているため優先度を上げて対応しました。

boost_countとbooster_arrayの形式を分けるメリットがなかったためユーザーごとの回数を入れる形に修正しています。
テンプレート側のboost_countが紛らわしい変数だったので名称を変更しています。

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

Raid-Default.incとRaid-Vs.inc以下の変数の中身を見ながらアタックブーストに入れて表示とmemcacheの値がおかしくないか確認。
var_dump($raid_atk_bonus);
var_dump($_temp['boost']);

TODO

開発で反映後本番に反映予定。

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

@wada
確認ありがとうございます。指摘箇所を修正しました。Raid-Default.incの分表示を秒数表示については実装ミスでした。
元々秒まで表示されてたいたところが通常・レアレイドを分表示にしてしまっていました。(チームレイドは秒まで表示させたまま)

@kouho
Raid-Default.incの修正について、以下のような修正内容が必要だと思います。

#修正前
if( isset($_temp['boost']['bonus_time']) && _temp['boost']['bonus_time'] > $time ){
	$_temp['current_boost_count'] = ( int )$_temp['boost']['current_boost_count'][$user_info['user_no']];
}else{
	$_temp['current_boost_count'] = ( int )$_temp['boost']['booster_array'][$user_info['user_no']];
}

#修正後
//$_temp['boost']の中にbonus_timeは格納されていないので、$raid_atk_bonusを参照
if( $raid_atk_bonus['bonus_time'] > $time ){
	//$_temp['boost']の中にcurrent_boost_countは格納されていないので、bonus_countを使用
	$_temp['current_boost_count'] = ( int )$_temp['boost']['boost_count'][$user_info['user_no']];
	var_dump($_temp['boost']['boost_count'][$user_info['user_no']]);
}else{
	$_temp['current_boost_count'] = ( int )$_temp['boost']['booster_array'][$user_info['user_no']];
}

@wada
ありがとうございます。Raid-Default.incとRaid-Vs.incの修正をしました。

@kouho
修正確認しました。
ひととおり確認して問題ありませんでしたが、複数人で同時に攻撃した際など確認できなかったので、開発での確認を丁寧に行いたいと思います。

@kouho
boost_countの加算がbooster_arrayに置き換わったので、ここのif文の内容をif ($this->bonus_meter < 20)の前に記載してください。
if文の中だと20回目の攻撃がボーナスの攻撃回数に加算されません。

@kouho
ボーナスに入った時にリセットされているので、ここでのリセットは大丈夫です。
同時に攻撃してボーナスの更新がずれた時に、攻撃回数がリセットされるかもしれません。

@kouho
Attack_bonus.phpと同じ理由でリセット不要です。

@kouho
ここの処理は、上のif文に入れたら良いと思います。
bonus_timeの確認もtemplate側で行っているので無くて大丈夫です。
bonus_timeを使用する場合は、$raid_atk_bonus['booster_limit']を使用してください。

@kouho
Raid-Default.incと同じ対応をお願いします。