PR

【5/17】ヤマトコラボ:カード系設定 #797

wada opened this at 2021-05-12T20:58:11; closed

Description

opened by wada at 2021-05-12T20:58:11; updated 2024-07-14T14:27:34

仕様書

https://docs.google.com/spreadsheets/d/1tVnOuuFznw-M26KrIDse-178_wEFbazvOcSidHorbuA/edit#gid=1881284159

Branch

web: yamato_skill
db: yamato_skill

更新内容

  • 追加スキルの処理、演出実装
  • 海賊カード画像の追加
  • 船のアイコン画像追加
  • コラボ船の戦争デスペナルティ設定
  • コラボ船の強化費用設定

追加スキルは、本家仕様を調整して実装しています。
調整内容は仕様書を確認してください。

# カードID
1492: 古代進
1493: 森 雪
1494: 山本玲
1495: アベルト・デスラー
10063: 宇宙戦艦ヤマト(画像未設定)
10064: コスモタイガーⅡ(画像未設定)

# スキルID
1289 砲撃命令
1376: エースパイロット
1468: 愛の記憶【古代進】
1545: スカイハイ
2384: 波動砲
2385: 艦砲射撃
3467: 抑圧
9026: ヒーリングクルー

Comment

#10866 by wada at 2021-05-13T10:48:20; updated 2021-05-13T10:48:20

@kouho @iwamoto
以下の内容を確認お願いします。

  • カード、船のパラメータは正しいか
  • スキルの挙動が説明通りか
  • コラボ船が強化できるか

カード以外のマスタデータはdbブランチから作成できます。
カードデータは、開発環境のカード検索からエクスポートしてください。
コラボ船の設計図による造船は、まだ実装していないので、手持ちの船のIDをコラボ船に変更して確認お願いします。

wada change_title at 2021-05-13T10:49:32: [WIP]【5/24】ヤマトコラボカード設定 【5/24】ヤマトコラボカード設定
wada add_label at 2021-05-13T10:49:33: レビュー待ち
wada change_priority at 2021-05-13T10:50:00: No priority:highest
wada change_title at 2021-05-13T10:54:46: 【5/24】ヤマトコラボカード設定 【5/17】ヤマトコラボ:カード系設定

Comment

#10894 by kouho at 2021-05-13T17:55:25; updated 2021-05-13T19:16:02

@wada

カードデータについて

10063: 宇宙戦艦ヤマト(画像未設定)
10064: コスモタイガーⅡ(画像未設定)
これらの説明文が空っぽでした。

スキルについて

何点か気になることがありましたのでご確認ください。

1376: エースパイロット

山本玲の攻撃力がコスモタイガー10064に反映されない。
試しにコスモタイガーの攻撃力を100 山本玲の攻撃力を10000にしたところ
コスモタイガーが乱射で与えるダメージが120でした

攻撃力がコスモタイガーⅡの方が大きい場合、山本玲の攻撃力ではなくコスモタイガーⅡの攻撃力が加算されるがOKか?
(SubFunc 45008行目のif文)

1545: スカイハイ
レベルが他のカードの方が山本玲より大きい場合、山本玲のレベルではなく他のカードのレベル分加算されるがOKか?
(SubFunc 58472目のif文)

1376と1545については他のスキルを見たところ基本的に
同様の仕様っぽいですが一応確認です。

2384: 波動砲 演出が再生されない。攻撃時の演出が再生されませんでした。
2385: 艦砲射撃 skill_nameの設定はいらないか 参考47353行目。表示上は問題なさそうでした。
3467: 抑圧 SkillFunc.inc 42680行以降にskill_nameの追加いるか。表示上は問題なさそうでした。
9026: ヒーリングクルー SkillFunc.inc 42680行以降にskill_nameの追加いるか。表示上は問題なさそうでした。

1289 砲撃命令 OK
1468: 愛の記憶【古代進】 OK
10104:【戦】根性 OK
10103:【戦】蘇生 OK

船の強化と費用・アイコンについて

確認OKです

覇戦デスペナルティ

確認OKです

Comment

#10896 by wada at 2021-05-13T20:22:43; updated 2021-05-13T20:22:43

@kouho
確認ありがとうございます。
> (SubFunc 45008行目のif文)
> (SubFunc 58472目のif文)

ここの比較に使われる配列にはスキル発動者のデータだけが入っているので、他のカードの数値に置き換わらないです。

skill_nameは一括で設定しているので、skill_nameの追加はなくて大丈夫です。

if($result['skill_no'] > 0 && $global_master_skill[$result['skill_no']]['skill_name'] != ""){
	$skill_name = $global_master_skill[$result['skill_no']]['skill_name'];
}else{
	$skill_name = "攻撃!";
}

Comment

#10917 by kouho at 2021-05-14T13:30:45; updated 2021-05-14T13:30:45

@wada
返信ありがとうございます。skill_nameと比較について了解です。

エースパイロットの山本玲の攻撃力がコスモタイガー10064に反映されない件についてはどうでしょうか?

Comment

#10925 by iwamoto at 2021-05-14T15:09:54; updated 2021-05-14T15:09:54

@wada
こちらの環境の問題かもしれませんが、一応連絡です
船に宇宙戦艦ヤマトを設定し際、他カードに「古代進」がない場合下記エラーが発生しました。

Invalid argument supplied for foreach()
SkillFunc.inc (37391)

Comment

#10926 by wada at 2021-05-14T15:19:03; updated 2021-05-14T15:20:07

@kouho
エースパイロットについては、それぞれのダメージを確認して欲しいです。
・スキル発動時、海賊へのダメージ
・スキル発動時、船へのダメージ
・スキル未発動時、海賊へのダメージ
・スキル未発動時、船へのダメージ

こちらで同じ条件で確認したところ攻撃力の増加は確認できました。

Comment

#10941 by wada at 2021-05-14T16:43:28; updated 2021-05-14T16:43:28

@iwamoto
スキル対応入れました。
根本的な解決は難しいので、とりあえずの対応です。

Comment

#10986 by iwamoto at 2021-05-17T10:58:49; updated 2021-05-17T11:39:14

@wada
確認しました。動作的には大丈夫かと思います…

iwamoto add_label at 2021-05-17T10:58:54: 岩本 LGTM

Comment

#10990 by kouho at 2021-05-17T18:03:12; updated 2021-05-17T18:03:12

@wada
エースパイロットについても確認しました。問題なしでした。
動かしてみた時に感じた仕様の違和感で
エースパイロット:対象キャラが倒されても発動し続ける
砲撃命令:対象キャラが倒されるまで効果が発動する
とエースパイロットは感覚的に対象者キャラが倒されるまで効果が発動の方がイメージ通りだと思いました
(変更は色々面倒&だと思いますが一応書いておきます)。

kouho add_label at 2021-05-17T18:04:23: 皇甫 LGTM
wada close_comment at 2021-05-18T12:56:54: ローカルでマージ済
wada refer at 2021-05-21T19:43:18: 824:【5/24】ヤマトコラボリリース

Commit range

base773889cf85d47d56841e25ec049023392e416864
head2ebadc9fc0ac3e43b5ba8e2b4a4ddf6ff5f65258
base branchjolly/jr-web:develop
head branchjolly/jr-web:yamato_skill

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed105 filesView patch
statusfile
modifiedfiles/battle/battle.swf
modifiedfiles/battle/battle_light.swf
modifiedfiles/battle/battle_raid.swf
modifiedfiles/teamWar/teamWar.swf
modifiedfla/チーム戦争/teamWar.fla
modifiedfla/バトル/as/func.as
modifiedfla/バトル/battle.swf
modifiedfla/バトル/battle_light.swf
modifiedfla/レイドボスバトル/as/func.as
modifiedfla/レイドボスバトル/battle_raid.swf
modifiedlib/SkillFunc.inc
modifiedlib/SubFunc.inc
addedlib/conf/class/SkillFriendConfig.class.php
modifiedlib/conf/class/TeamWarConfig.class.php
modifiedlib/conf/define.inc
modifiedlib/mobile/Battle__Player.inc
modifiedlib/mobile/Help__SkillSearch.inc
modifiedlib/mobile/TeamWar__swf.inc
modifiedlib/public/Help-SkillSearch.inc
addedpublic_html/img/battle/skill_no2384_ef/effect01.png
addedpublic_html/img/battle/skill_no2384_ef/effect02.png
addedpublic_html/img/battle/skill_no2384_ef/effect03.png
addedpublic_html/img/battle/skill_no2384_ef/effect04.png
addedpublic_html/img/battle/skill_no2384_ef/effect05.png
addedpublic_html/img/battle/skill_no2384_ef/effect06.png
addedpublic_html/img/battle/skill_no2384_ef/effect07.png
addedpublic_html/img/battle/skill_no2384_ef/spark_1.png
addedpublic_html/img/battle/skill_no2384_ef/spark_2.png
addedpublic_html/img/battle/skill_no2384_ef/text_1.png
addedpublic_html/img/battle/skill_no2384_ef/text_2.png
addedpublic_html/img/battle/skill_no2384_ef/text_3.png
addedpublic_html/img/battle/skill_no2384_ef/thunder01.png
addedpublic_html/img/battle/skill_no2384_ef/thunder02.png
addedpublic_html/img/battle/skill_no2384_ef/thunder03.png
addedpublic_html/img/battle/skill_no2384_ef/thunder04.png
addedpublic_html/img/card/120/10063.jpg
addedpublic_html/img/card/120/10064.jpg
addedpublic_html/img/card/120/1492.jpg
addedpublic_html/img/card/120/1493.jpg
addedpublic_html/img/card/120/1494.jpg
addedpublic_html/img/card/120/1495.jpg
addedpublic_html/img/card/240/10063.jpg
addedpublic_html/img/card/240/10064.jpg
addedpublic_html/img/card/240/1492.jpg
addedpublic_html/img/card/240/1493.jpg
addedpublic_html/img/card/240/1494.jpg
addedpublic_html/img/card/240/1495.jpg
addedpublic_html/img/card/26/10063.jpg
addedpublic_html/img/card/26/10064.jpg
addedpublic_html/img/card/26/1492.jpg
addedpublic_html/img/card/26/1493.jpg
addedpublic_html/img/card/26/1494.jpg
addedpublic_html/img/card/26/1495.jpg
addedpublic_html/img/card/320/10063.jpg
addedpublic_html/img/card/320/10064.jpg
addedpublic_html/img/card/320/1492.jpg
addedpublic_html/img/card/320/1493.jpg
addedpublic_html/img/card/320/1494.jpg
addedpublic_html/img/card/320/1495.jpg
addedpublic_html/img/card/640/10063.jpg
addedpublic_html/img/card/640/10064.jpg
addedpublic_html/img/card/640/1492.jpg
addedpublic_html/img/card/640/1493.jpg
addedpublic_html/img/card/640/1494.jpg
addedpublic_html/img/card/640/1495.jpg
addedpublic_html/img/card/70/10063.jpg
addedpublic_html/img/card/70/10064.jpg
addedpublic_html/img/card/70/1492.jpg
addedpublic_html/img/card/70/1493.jpg
addedpublic_html/img/card/70/1494.jpg
addedpublic_html/img/card/70/1495.jpg
addedpublic_html/img/card/70main/10063.jpg
addedpublic_html/img/card/70main/10064.jpg
addedpublic_html/img/card/70main/1492.jpg
addedpublic_html/img/card/70main/1493.jpg
addedpublic_html/img/card/70main/1494.jpg
addedpublic_html/img/card/70main/1495.jpg
addedpublic_html/img/card/battle/154/10063.jpg
addedpublic_html/img/card/battle/154/10064.jpg
addedpublic_html/img/card/battle/154/1492.jpg
addedpublic_html/img/card/battle/154/1493.jpg
addedpublic_html/img/card/battle/154/1494.jpg
addedpublic_html/img/card/battle/154/1495.jpg
addedpublic_html/img/card/battle/58/10063.jpg
addedpublic_html/img/card/battle/58/10064.jpg
addedpublic_html/img/card/battle/58/1492.jpg
addedpublic_html/img/card/battle/58/1493.jpg
addedpublic_html/img/card/battle/58/1494.jpg
addedpublic_html/img/card/battle/58/1495.jpg
addedpublic_html/img/card/battle_result/ship_10063.png
addedpublic_html/img/card/battle_result/ship_10064.png
addedpublic_html/img/mission/card/1492.png
addedpublic_html/img/mission/card/1493.png
addedpublic_html/img/mission/card/1494.png
addedpublic_html/img/mission/card/1495.png
addedpublic_html/img/team/tw_enemyship_10063_left.png
addedpublic_html/img/team/tw_enemyship_10063_right.png
addedpublic_html/img/team/tw_enemyship_10064_left.png
addedpublic_html/img/team/tw_enemyship_10064_right.png
addedpublic_html/img/team/tw_ship_10063_left.png
addedpublic_html/img/team/tw_ship_10063_right.png
addedpublic_html/img/team/tw_ship_10064_left.png
addedpublic_html/img/team/tw_ship_10064_right.png
modifiedpublic_html/js/JOLLYROGER_battle.js
modifiedpublic_html/template/Battle.tpl
Patch body is stored as a separate file to keep this page lightweight.