【4/7】パイレーツパスシーズン14 #4004

概要

https://docs.google.com/spreadsheets/d/14HIucczU2rZwwi9mB-p-O9h-LsDQ5ZbwgfD7ynrvTr8/edit?gid=893381404#gid=893381404&range=A1

• 新規スキルあり
• ヴァルキリーの強襲をヴァルキリーの騎行ベースではなく、王者の邁進ベースで作成

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

@kouho
以下の処理でcheck_atkの確認は必要ないですか？

		//ヴァルキリーの強襲による攻撃力上昇
		if($_BattleGlobal['attack_flg'][$atk_team][105607] == 1) {
			if($num != 19 && $num != 29) {
				$atk_up = 0;
				foreach($_BattleGlobal_2['attack_flg100211'][$atk_team] as $key2 => $value2) {
					//元のスキルを発動したカードが生きているとき
					if(S_SkillLifeCheck($atk_team*10+$value2)) {
						$tmp_atk_up = (int)($card_data[$atk_team*10+$value2]['atk']);
						//上昇量の多いほうを優先
						if($tmp_atk_up > $atk_up) {
							$atk_up = $tmp_atk_up;
						}
					}
				}

@kouho
> ヴァルキリーの強襲をヴァルキリーの騎行ベースではなく、王者の邁進ベースで作成

こちらの理由はなんでしょうか？

@wada
確認ありがとうございます。
> 以下の処理でcheck_atkの確認は必要ないですか？

check_atkの確認は追加しました。

> ヴァルキリーの強襲をヴァルキリーの騎行ベースではなく、王者の邁進ベースで作成
こちらの理由はなんでしょうか？

ヴァルキリーの騎行をベースにして、例えばヴァルキリーの騎行の記載箇所の下にコピーしてスキルIDを書き換えた場合、スキルが発動しませんでした。

王者の邁進の場合、かなり共通点が多かったのとヴァルキリーの騎行よりソースがまとまっているように見えましたので王者の邁進をベースに作成しました。

• ヤマダ版でスキルNOを独自作成
• 回復・支援系スキル
• 対象が〜の場合効果アップ
• [バトル開始時発動]
• [倒れるまで有効]

@wada
> 証の数が仕様書では60個になっています。

証の数を修正しました

@kouho
> 王者の邁進の場合、かなり共通点が多かったのとヴァルキリーの騎行よりソースがまとまっているように見えましたので王者の邁進をベースに作成しました。

すでに正しい処理があるので、ヴァルキリーの騎行をベースで実装してください。
ソースコード確認のコスト、メンテナンス（スキルマージや本家で該当箇所に更新が入った際など）のコストが高くなります。

もし、実装してスキルが発動しない場合は教えて下さい。
実装の工数が足りない場合も教えて下さい。

@wada
すみません、ヴァルキリーの強襲をヴァルキリーの騎行をベースの修正にしましたので確認をお願いいたします。

@kouho
スキル「天翼のフリート」と「煌翼のレギオン」は倍率が変わるだけ？ですが、タグと説明文が少し違います。
<img src="https://git.dev.d-es.co.jp/gitbucket/jolly/jr-web/_attached/17440028737448rfQiEjYaS" width="500">

@wada
確認ありがとうございます。以下修正いたしました。
> スキル「天翼のフリート」と「煌翼のレギオン」は倍率が変わるだけ？ですが、タグと説明文が少し違います。

@kouho
JOLLYROGER_battle.js 3122行目のif文条件に含めなくても良いですか？

//例外スキルアイコン対応
//5605:船撃2倍(ダンプシップ)
//1704:コールドアーマーの演出用
//478:ガンパウダーの付与効果
//5651:船撃+lv(シップアタッカー)
//3083:グラントショック（ウィングブーツ）
//5848:スピードロック
//「攻+」系スキル
if( jQuery.inArray(skill_type, [ 410,497,444,5601,5605,5608,5641,1704,5611,5624,478,483,243,5621,5623,5622,5626,5653,5615,105603,105605,105607,

@kouho
SkillFunc.incでヴァルキリーの強襲の攻撃力＋の処理が、ヴァルキリーの騎行の攻撃力＋と離れた箇所かつ実装方法が違いますが、なにか問題がありましたか？

管理のしやすさから処理はまとめて実装されているのが望ましいです。

@wada
47114行目と58429行目に定義しても現状の判定だと入らないので、離れた場所にはなりますが今の場所に入れていました。

@kouho
> 47114行目と58429行目に定義しても現状の判定だと入らないので、離れた場所にはなりますが今の場所に入れていました。

58429行目はそうですね。
47114行目は同じネスト内なので入ると思います。

@wada
すみません、確かに47114行目の方は入りましたので修正しました。
> 58429行目はそうですね。
47114行目は同じネスト内なので入ると思います。

@kouho
確認OKです。

@kouho
証の数が仕様書では60個になっています。