PR

【7/6】ラッシュ発動時に攻撃するように修正 #2060

kouho opened this at 2022-07-06T17:29:13; closed

Description

opened by kouho at 2022-07-06T17:29:13; updated 2022-07-07T17:03:04

概要

https://git.dev.d-es.co.jp/gitbucket/jolly/jr-cs-issue/issues/47

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

ラッシュ+1と攻撃スキルを持ったカードをバトルデッキに
ラッシュが30%で発動した際に通常攻撃後他の攻撃スキルが発動することを確認

TODO

確認でき次第本番に修正を反映予定

Comment

#20718 by kouho at 2022-07-06T17:29:21; updated 2022-07-06T17:29:21

@wada
確認お願いいたします

Comment

#20721 by wada at 2022-07-06T18:12:55; updated 2022-07-06T18:12:55

@kouho
ローカル環境のバトルシミュレータで試したところ、
以下のような結果になりましたが、これで良いでしょうか?

<img src="https://git.dev.d-es.co.jp/gitbucket/jolly/jr-web/_attached/165709864583118tbUIl04H" width="300">

  • ラッシュ未発動

通常攻撃 228
2乱射 228 x 2
合計3回攻撃

  • ラッシュ発動

ラッシュ+1 攻撃UP
2乱射 328 x 2
合計2回攻撃

Comment

#20722 by kouho at 2022-07-06T18:28:16; updated 2022-07-06T18:28:16

@wada
ラッシュ発動時に通常攻撃をしているものと見間違えました。
通常攻撃をするように修正しましたので改めて確認お願いいたします。

Comment

#20723 by wada at 2022-07-06T18:56:53; updated 2022-07-06T18:56:53

@kouho
ありがとうございます。
3回攻撃確認できましたが、「ラッシュ+1」の表示がなくなっていました。

Comment

#20724 by kouho at 2022-07-07T10:28:16; updated 2022-07-07T10:28:16

@wada
通常攻撃処理を入れる中でスキル名をリセットしてしまっていましたので修正しました

Comment

#20735 by wada at 2022-07-07T11:25:27; updated 2022-07-07T11:25:27

@kouho
こちらで確認した範囲だと、以下のような対応が必要そうでした。
142: クリティカルを参考にしました。

//func
#攻撃アップ
#0ターン目の行動として登録
$attack_data[$id][0]['def'] = 0;
$attack_data[$id][0]['atk'] = 0;
$attack_data[$id][0]['skill_no'] = $value;
$attack_data[$id][0]['def_flg'] = 0;

#通常攻撃
#次ターンの行動として登録
#通常攻撃キーは、他スキルで$key+1として設定されていることが多いが、1でも良さそう
$def_array = S_SkillDefencePosition(7,$num);
foreach($def_array as $key => $def){
	$defense_flg = S_SkillDefenseEffectiveness( $card_data[$num] , $card_data[$def] , $num , $def);//防御スキル判定
	$attack_data[$id][{通常攻撃キー}] = S_GetAttackDamage($defense_flg , $num , $def , 0 , 0 , $raid_monster_flg);//ダメージ計算
	$attack_data[$id][{通常攻撃キー}]['skill_no'] = $value;
}

//js
#演出
格納するskill_list_arrayを変更

Comment

#20740 by kouho at 2022-07-07T14:48:25; updated 2022-07-07T14:48:25

@wada
確認ありがとうございます。本家も修正したようなので修正を持ってきました。

昨日共有していただいたラッシュ+1、ラッシュ+2のスキルにつきまして、
発動した際に攻撃補助スキルのような挙動となっていた部分を修正いたしました。
【今日ぐらい】ラッシュ+1と+2の修正 #9374
https://github.com/rzstaffgit/jrce2_github/pull/9374/files
演出についても、攻撃補助スキルの処理になっていたため変更しています。

Comment

#20747 by wada at 2022-07-07T15:56:13; updated 2022-07-07T15:56:13

@kouho
ありがとうございます。ラッシュ発動時の3回攻撃を確認できました。
ラッシュ発動時の2乱射ダメージが+100されなくなったんですが、これは挙動として正しいんでしょうか?

Comment

#20748 by kouho at 2022-07-07T16:38:24; updated 2022-07-07T16:38:24

@wada
確かに元の挙動と変わってしまいますが、こちらが正しい挙動です。
攻撃スキルが攻撃補助スキルのような挙動となっていたのが修正されています。

Comment

#20751 by wada at 2022-07-07T16:58:53; updated 2022-07-07T16:58:53

@kouho
了解しました。確認OKです。

wada add_label at 2022-07-07T16:58:55: 和田 LGTM
kouho merge at 2022-07-07T17:03:04: 【7/6】ラッシュ発動時に攻撃するように修正
kouho close at 2022-07-07T17:03:04: Close

Commit range

basead0db2cde82180aeffd16e01783c6e762f53ca25
head6297e9da62000d849d180798c61ebaa6ed8e0f4d
base branchjolly/jr-web:develop
head branchjolly/jr-web:fix_skill_rush

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/SkillFunc.inc
modifiedpublic_html/js/JOLLYROGER_battle.js
Patch body is stored as a separate file to keep this page lightweight.