PR

【なる早】レイド6連攻撃実行時にステータスUP系スキルが反映されない問題への対応 #2324

kouho opened this at 2022-10-03T18:26:17; closed

Description

opened by kouho at 2022-10-03T18:26:17; updated 2022-10-04T10:40:04

概要

問い合わせで確認されたレイドでの6連攻撃時のダメージ計算がおかしい問題への対応

実際に6連攻撃時に1回しか乙姫のスキル恵みの雨が通っていない
本来3回予測計算で3回実際にバトル実行されるのでスキルの処理には3回通らないとおかしいが1度しか通らなかった。


バトル開始時発動系スキルがレイドでの6連攻撃で2回目の攻撃以降計算に含まれていない?

  • Raid_battle_model->reset_global_data関数の中身を一式リセットするように修正
  • Raid_battle_model->prepare_battle関数の中身のうち $this->user->prepare_battle を毎回行うように修正
  • Battle_user->prepare_battle関数に $this->attack_count = 0; を含めず Battle_userのコンストラクタに $this->attack_count = 0; を移動
  • Raid_battle_model->get_battle_card_data関数の$card_dataの扱いをグローバル変数に変更

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

1. ユーザーのデッキをコピー
問い合わせユーザーyama-p
user_no 3242

(!2!,@135@,150,2,2387438,150,a545a,b957b,c673c,0,118,10004,-1,-1,0,NOW),(!3!,@232@,155,4,1411696,155,a497a,b1233b,c589c,0,113,212,0,-1,0,NOW),(!1!,@629@,44,4,1680,160,a1391a,b457b,c797c,8,17,236,601,-1,0,NOW),(!7!,@654@,160,6,694719,160,a1138a,b1132b,c547c,0,11,1415,1416,601,0,NOW),(!5!,@496@,140,1,109203,140,a1735a,b398b,c373c,0,1615,1206,1417,601,0,NOW),(!4!,@330@,43,5,565,95,a130a,b782b,c130c,2,1207,484,10545,0,0,NOW),(!8!,@720@,70,5,80869,70,a176a,b604b,c170c,0,44,1319,-1,-1,0,NOW),(!6!,@483@,30,1,41479,30,a0a,b0b,c0c,0,43,1210,506,10542,0,NOW),(!9!,@10040@,1,1,0,1,a0a,b0b,c0c,0,-1,-1,-1,-1,0,NOW)

2. レイドを出現させて6回攻撃を実行

修正前
2回目以降のダメージが極端に減る

修正後
2回目以降のダメージが減らないことを確認

3. スキルの処理を通っているかの判断

SkillFunc.incで処理を通っているかログを仕込んで6連攻撃を実行
3回該当スキルの処理を通れば修正されたと判断

TODO

kouho change_title at 2022-10-03T18:43:36: [WIP]【なる早】レイド6連攻撃実行時にステータスUP系スキルが反映されない問題への対応 【なる早】レイド6連攻撃実行時にステータスUP系スキルが反映されない問題への対応

Comment

#22747 by kouho at 2022-10-03T18:43:44; updated 2022-10-03T18:43:44

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

Comment

#22748 by wada at 2022-10-04T10:37:51; updated 2022-10-04T10:37:51

@kouho
確認OKです。

wada add_label at 2022-10-04T10:38:00: 和田 LGTM
kouho merge at 2022-10-04T10:40:04: 【なる早】レイド6連攻撃実行時にステータスUP系スキルが反映されない問題への対応
kouho close at 2022-10-04T10:40:04: Close

Commit range

base00e6e2962d4e954b4941db0c4000669089dbaf11
head759f467a3afab765a4c666e472bf2dc4b1a4e551
base branchjolly/jr-web:master
head branchjolly/jr-web:fix_raid_6_attack

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed2 filesView patch
statusfile
modifiedlib/models/Raid_battle_model.php
modifiedlib/models/raid/Battle_user.php
Patch body is stored as a separate file to keep this page lightweight.