PR

【確認出来次第】Razestさんから指摘のスキル周りの処理修正 #3283

kouho opened this at 2023-12-15T17:34:40; closed

Description

opened by kouho at 2023-12-15T17:34:40; updated 2023-12-22T19:22:30

概要

一騎討ちで不具合が発生したためRazestさんに相談したところ以下の指摘を受けたため対応。

不具合内容
https://scry.slack.com/archives/C0164SMGG8P/p1702023325585779

指摘内容

S_BattleBeforeSkillEffectSetにある変数で初期化していないものがありますので、
そちら初期化したほうがいいかもしれません。

一騎打ちで複数回バトルを回したときに、もしかしたら前の戦闘のデータが残った状態で始まるかもしれません。
今のところそのくらいしかわかりませんでした。
体力が「1」になる件とは別件なのですが、
SkillFunc.incを確認していて気になる箇所がありましたので、念のためご確認お願いいたします。

「100104」のスキルの処理で二か所、下記のようにif文で代入している箇所があるようでした。
if($value == 2358 || $value = 100104){
いただいたSubFuncやバトルデータを拝見して、恐らく今回の不具合とは関係無さそうですが2点気になる点がございました。
・10万番台のスキルの扱い
スキルNoが10万番台のスキルにつきまして、防御関連に当たらないスキルも$card_data[$pos]['defense_skill']配列に格納されているようです。
防御系スキルの処理に今後影響を及ぼす可能性があるため、早めに振り分け処理を見直した方が良いかと存じます。

・処理のみ存在するスキルの動作修正
まだ実装されていないスキルかと存じますが、下記スキルの動作修正の共有が漏れていたようなのでお手すきの際にご確認ください。
【即日】レジェンドアクトの修正
https://github.com/rzstaffgit/jrce2_github/pull/9065
【4/21】潜在能力の発動回数設定をバトル開始時に持ってくる
https://github.com/rzstaffgit/jrce2_github/pull/10495

対応

1. S_BattleBeforeSkillEffectSet関数内の初期化していない変数を初期化
2. 「100104」のスキルの処理で二か所、下記のようにif文で代入している箇所
3. 10万番台のスキルの扱いの修正

処理のみ存在するスキルの動作修正についてはスキルマージ時に修正するものとして対応していません。
こちらでのスキル追加時のスキルIDの桁数がこれ以上増えない想定で実装しています。

確認したこと

スキル滅界の動作確認
一騎討ちのローカル環境での実行でエラーが起きないか
バトルでエラーが起きないか

kouho change_title at 2023-12-15T17:54:33: [WIP]Razestさんから指摘のスキル周りの処理修正 【確認出来次第】Razestさんから指摘のスキル周りの処理修正

Comment

#32465 by kouho at 2023-12-15T17:55:42; updated 2023-12-15T17:55:42

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

Comment

#32655 by wada at 2023-12-22T19:01:22; updated 2023-12-22T19:01:22

@kouho
確認OKです。

wada add_label at 2023-12-22T19:01:27: 和田 LGTM
kouho merge at 2023-12-22T19:22:30: 【確認出来次第】Razestさんから指摘のスキル周りの処理修正
kouho close at 2023-12-22T19:22:30: Close

Commit range

base5e042a4202948c2e7b89291a8d0def57e92cae66
head1efe517d89093aa6d82cb856a19e905333ff2d42
base branchjolly/jr-web:master
head branchjolly/jr-web:fix_skill_various

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