[Artifact 1453] 霜華の弓の作成 by ogachu0331 · Pull Request #2056 · ProjectTSB/Asset

ogachu0331 · 2026-06-10T01:17:45Z

霜華の弓とそのobjectの作成

EllaCoat

まぁ大雑把に見ただけだから、まだありそうだけど、取り敢えず軽く見た感じの指摘ねー
[NITS] とか [MUST] とかの意味は調べたら出てくるので、調べて（）

EllaCoat · 2026-06-10T10:39:11Z

+    scoreboard objectives add Ffb.Charge dummy
+    scoreboard objectives add Ffb.LatestChargeTick dummy


[NITS]
神器用スコアに関してだけど、可能であれば「神器 ID」を冒頭につけてもらえると良いかも。
神器 ID : 1453 (36進数の場合 14D) なので……

Suggested change

scoreboard objectives add Ffb.Charge dummy

scoreboard objectives add Ffb.LatestChargeTick dummy

scoreboard objectives add 14D.Charge dummy

scoreboard objectives add 14D.LatestChargeTick dummy

こうだと、この神器のスコアであることがひと目で分かるよー。

修正しました！

EllaCoat · 2026-06-10T10:41:22Z

+# 基本的な使用時の処理(MP消費や使用回数の処理など)を行う
+    function asset:artifact/common/use/mainhand


[MUST]
コピペミスだとは思うけど、
function asset:artifact/common/use/mainhand が 2 回起動してしまっているから、なにかしら理由がないなら消して欲しいー
（なにか理由があるなら、それはそれで返信お願いー）

完全にミスですね
修正しました

EllaCoat · 2026-06-10T10:44:02Z

@@ -0,0 +1,10 @@
+#> asset:artifact/1453.frost_flake_bow/using_item/0.load


[MUST]
同じスコアを 2 重登録する必要はなさそう。
多分 Linter のエラーを消すためだと思うけど、`0.load' はテンプレート経由で作成しない限り、load 時に呼ばれないから気をつけて。

削除しました

EllaCoat · 2026-06-10T10:45:53Z

+# @within tag/function asset:artifact/load
+
+#> 定義類はここに
+# @within function asset:artifact/1453.frost_flake_bow/trigger/**


[IMO]
ここをこうすると、Linter がエラーを消してくれるよー

Suggested change

# @within function asset:artifact/1453.frost_flake_bow/trigger/**

# @within function asset:artifact/1453.frost_flake_bow/**

元々は trigger 以下のファイル全部でスコアを使って良いよってルールなんだけど、
今回は using_item でも使いたいだろうから、こうすると良いと思う。

やり方わからなかったので助かります
修正しました！

EllaCoat · 2026-06-10T10:48:47Z

+# チャージ段階をFieldOverrideへ
+    data modify storage api: Argument.FieldOverride.Charge set from storage api: Return.Effect.Stack


[WANT]
多分、FieldOverride.Charge は使われていない……気がするんだけど、どうだろう？
使わなさそうなら、消しちゃっても良いかな。

使わないので消しました

EllaCoat · 2026-06-10T10:52:36Z

+    execute if score @s Ffb.Charge matches 20 anchored eyes positioned ^ ^ ^1 run function asset:artifact/1453.frost_flake_bow/using_item/vfx/charge1
+
+    execute if score @s Ffb.Charge matches 30 run playsound ui.toast.in player @s ~ ~ ~ 1 1.3
+    execute if score @s Ffb.Charge matches 34 run playsound entity.experience_orb.pickup player @s ~ ~ ~ 0.1 2


[Q]
うちのラビリア（Claude）が気になったらしいので質問。
Charge のスコアが 30 で二段階目チャージが終了しているという認識なんだけど
それ以降の演出として用意している感じで OK ？

だいたいそんな感じですね
（チャージした後も引いてるプレイヤーにチャージ終わったよ感のある音を鳴らしたかった）
再生を4tick遅らせてるのはこうすると前の音(ui.toast.in)といい感じに合うからで、演出以上の
理由は特にないです。

EllaCoat · 2026-06-10T10:54:21Z

+# 神器の発動条件 (TextComponentString) (オプション)
+    data modify storage asset:artifact Condition set value '{"text":"最大まで引き絞る"}'
+# 攻撃に関する情報 -Damage量 (literal[]/literal) Wikiを参照 (オプション)
+    data modify storage asset:artifact AttackInfo.Damage set value "800-1120 / 500-700"


[Q]
これはどっちが 1 段目でどっちが 2 段目かな？
仮に 1 段目が右だとしたら、既存神器の慣習として、左に変えたほうが良いかもしれない
（そもそもとして、こういう場合の表記が難しいのも分かる）

1段目が右だったので変えました！

…SB/Asset into dev/ogachu_artifact1453

EllaCoat

もう一度レビューして洗い出し直したので、再度直してー

EllaCoat · 2026-06-12T08:11:34Z

+# 5. 計算結果をdata(ストレージ)に戻す
+# ※取得時に100倍、倍率計算で100倍のスケールになっているため、戻す時に 0.0001 (1万分の1) を掛けて元のスケールに戻します
+    execute store result storage asset:context this.Damage double 0.0001 run scoreboard players get $value Temporary
+    scoreboard players reset * Temporary 


[MUST]
これは絶対にやらないで！（強い断言）
この世に存在するスコアを持ちうる存在の Temporary スコアを全てリセットする
=> この世に存在するエンティティ全員を対象として実行するから非常に重い！

あと、コアデータパックの方に Temporary を tick を跨いで使っているものもあるから、普通にシステムが破壊されちゃいますわ～

なのでリセットする場合は、以下のような明示式にしましょう。

scoreboard players reset $multiplier Temporary scoreboard players reset $value Temporary

（余談：私は下の方が好きだけど、同じスコアホルダーが何処かで使われる可能性もあるので、上の方が安全ではある。よく分からないなら、上で OK）

scoreboard players reset $multiplier scoreboard players reset $value # 接頭語として ID を使ったりすると、スコアホルダー被りが発生しづらくて個人的には好き # 以下は例、AU はいわゆる 36 進数表記 scoreboard players reset $AU.Temp

EllaCoat · 2026-06-12T08:16:05Z

+    scoreboard players set $4 Temporary 4
+    scoreboard players set $100 Temporary 100
+


[WANT]
コアデータパックの方に既に Const スコアで定数スコアが用意されているので、それを使ってー
https://github.com/ProjectTSB/TheSkyBlessing/blob/master/TheSkyBlessing/data/core/functions/define_const.mcfunction

後、面倒かもしれないけど $multiplier とかのスコアホルダーは基本的に、定数スコアでもない限り declare しておくと良いよー

#> Private # @private #declare score_holder $multiplier #declare score_holder $value

EllaCoat · 2026-06-12T08:24:36Z

+# 5. 計算結果をdata(ストレージ)に戻す
+# ※取得時に100倍、倍率計算で100倍のスケールになっているため、戻す時に 0.0001 (1万分の1) を掛けて元のスケールに戻します
+    execute store result storage asset:context this.BloomDamage double 0.0001 run scoreboard players get $value Temporary
+    scoreboard players reset * Temporary 


[MUST]
別箇所で指摘したのと同様の理由

EllaCoat · 2026-06-12T08:26:52Z

+    execute store result score $UserID Temporary run data get storage asset:context this.UserID
+    execute as @a if score @s UserID = $UserID Temporary run function api:damage/modifier


[WANT]
想定 $UserID Temporary のリセット忘れ
（$UserID Temporary 自体はそこまで使われていないけど、スコアホルダー被りによるバグが若干怖い）

ogachu0331 added 3 commits June 9, 2026 01:12

Artifact1453霜華の弓の作成

19c7437

ダメバフなどの追加

f19bfbc

Merge branch 'master' into dev/ogachu_artifact1453

d0914dd

ogachu0331 requested review from ChenCMD and Lapis-LJA and removed request for ChenCMD June 10, 2026 01:39

ogachu0331 changed the title ~~[Artifact 1453] 霜華の弓~~ [Artifact 1453] 霜華の弓の作成 Jun 10, 2026

EllaCoat requested changes Jun 10, 2026

View reviewed changes

ogachu0331 added 3 commits June 10, 2026 21:57

指摘点の修正

b8d26d5

Merge branch 'dev/ogachu_artifact1453' of https://github.com/ProjectT…

cf9c1de

…SB/Asset into dev/ogachu_artifact1453

二段チャージしなくても矢が炸裂する問題の修正

99823ea

EllaCoat requested changes Jun 12, 2026

View reviewed changes

		scoreboard objectives add Ffb.Charge dummy
		scoreboard objectives add Ffb.LatestChargeTick dummy

		# 基本的な使用時の処理(MP消費や使用回数の処理など)を行う
		function asset:artifact/common/use/mainhand

		@@ -0,0 +1,10 @@
		#> asset:artifact/1453.frost_flake_bow/using_item/0.load

	# @within function asset:artifact/1453.frost_flake_bow/trigger/**
	# @within function asset:artifact/1453.frost_flake_bow/**

		# チャージ段階をFieldOverrideへ
		data modify storage api: Argument.FieldOverride.Charge set from storage api: Return.Effect.Stack

		scoreboard players set $4 Temporary 4
		scoreboard players set $100 Temporary 100

		execute store result score $UserID Temporary run data get storage asset:context this.UserID
		execute as @a if score @s UserID = $UserID Temporary run function api:damage/modifier

Conversation

ogachu0331 commented Jun 10, 2026

Uh oh!

EllaCoat left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

EllaCoat left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants