銃アドオンをscriptapi化したいのですが簡単な動作テストコードでさえ動かないのでわかりやすく教えてほしいです
#scriptapiを教えてほしい
desert schooner
いまは[scripting][error]-plugin[puchineko guns 2 -1.0.0] - does not contain main file [script/main.js].というエラーが出ています
vestal merlin
scriptsファイル内にmain.jsを作成してください
long path
sが抜けてるだけな気が
desert schooner
すでにmain.jsは存在しています（練習として銃を撃つと銃の名前をチャットに表示するようになっています）
long path
afterEventsのitemUseです
desert schooner
この鯖の過去ログやscriptapiについて書いてあるサイトなどあちこち見て見様見真似で組んだので怪しい部分もあります・・・（というか１００％あってるのか怪しい）
vestal merlin
銃こだわっててすごいな
desert schooner
scriptapiを使って
右クリックで発射 スニーク左クリックでタクティカルリロード
左クリックでモード切替 って感じで作りたいです・・・
desert schooner
空振り検知は不可・・・って感じですか？
vestal merlin
そうですね
desert schooner
じゃああの鯖のアドオンはplayer.json使ったアドオン（操作系を参考にしようと思ってた）・・って感じですかね・・（ここでほかの鯖名・アドオン名を出すのはやめておきます）
desert schooner
銃弾が当たっちゃいますね・・・
vestal merlin
そしたら攻撃通らなくなっちゃいますよね
crisp sigil
空振りはアニコン使うしかないと思う
long path
ブロックかエンティティかどっちか殴れば動くようにはできるんだけどね
desert schooner
ブロックに殴りだとどんな感じになります？
long path
entityHitBlockで取れる
desert schooner
ありがとうございます 試してみます
desert schooner
source.runCommand
ここを変更して銃弾（プロジェクタイル）を飛ばしたいのですがどうすればいいでしょうか？
long path
アイテムのイベントを呼んでshootを起動するとかじゃないですかね
long path
データが多すぎて翻訳に時間がかかることがある
long path
triggerEventメソッドがあります
long path
ItemStackのtriggerEventを使ってますか？
desert schooner
あ・・・
sourceになってました・・・
[Scripting][warning]-[Watchdog] 129.349 ms script spike detected in behavior pack 'puchineko guns 2'
このエラーが出ます・・・
long path
watchdogなので程々に気にすればいいです
desert schooner
import { world } from "@minecraft/server";
world.afterEvents.itemUse.subscribe( m16a1=> {
const { source, itemStack} = m16a1;
if(itemStack.typeId==='ar:m16'){
const ze = world.getPlayers();
source.runCommand(me @a m16a1)
itemStack.triggerEvent('m16shoot')
}
});
コードはこんな感じです
desert schooner
watchdogなので程々に気にすればいいです
わかりました
long path
runCommandが割とステート食ってる可能性は高いです
desert schooner
[Scripting][warning]-[Watchdog] 129.349 ms script spike detected in behavior pac...
今はこのエラーが出て弾が出てないですね・・・
long path
そのエラーと弾が出ないのは一切関係性はないですね
desert schooner
ええ・・・
long path
ちなみにですがgetPlayersはなぜ書いてあるのですか
あとイベントを呼んでも動かないのはconditionを通過できていないからな気がします
一度shootだけのイベントを用意してみて使ってみてください
desert schooner
まだ動作していません・・・・
いくつか改良試したんですけどどれもダメでした・・
"testshoot":{
"sequence":[
{
"shoot":{
"projectile":"ar:m16bullet",
"launch_power":3
}
}
]
}
呼んでるイベントはこれです
パックの機能 'editorExtension' はサポートされていません。
エラーはパック画面でこれが出ています 実行時はエラー表示なしです
long path
manifestを見せてください
long path
capabilitiesからeditorExtensionをお外し下さい
desert schooner
editorextansionのエラーが消えて現在はエラーなしで弾が出ない状態です
long path
ちなみにアイテムのイベントの方でsayコマンドとか実行した場合も動かないんでしたっけ？
desert schooner
動きませんね・・・
"testshoot":{
"sequence":[
{
"run_command":{
"command":[
"say @p"
],
"target":"holder"
}
}
]
}
eventの中身はこんな感じになってます
desert schooner
editorextansionのエラーが消えて現在はエラーなしで弾が出ない状態です
やっぱりitemStack.が必要だったんでしょうか？
crisp sigil
triggerEventはItemStackに生えているメソッドなので必要ですね
desert schooner
そうすると今度はエラーないけど何も動かなくなっちゃうんですよね・・・
アイテムのjsonはこれですね・・・（m16shootイベントはscriptapi化前の残骸です）
zenith orchid
ItemStackのtriggerEventメソッドは存在はしていますが、アイテムイベントは実行されず使えなかったはずです。
なのでアイテムイベントでやりたいこともスクリプト側で書いた方が良いかと
desert schooner
なるほどです・・・・
crisp sigil
まだ治ってなかったんだ
desert schooner
"shoot":{
"projectile":"ar:m16bullet",
"launch_power":3
}
これとまだ未実装ですがlore管理の銃の残弾を減らす処理ですね・・・
プロジェクタイルを飛ばす処理ってありますかね・・・（なんとなくだけどない気がする・・・）
zenith orchid
プロジェクタイルもエンティティなのでスポーンさせられます。<Entity>.applyImpulseなどで好きな方向に飛ばせます。
尚、この方法ではプロジェクタイルに誰が発射したかというデータがないため、キルログが「(プレイヤー名)に倒された」ではなく「(プロジェクタイル名)に倒された」になるなどの差はあります
ScriptAPIの開発者がプロジェクタイル関係でどんな機能があったら便利かを質問していたので、今後アイテムイベントの代わりにScriptでもプロジェクタイルを扱いやすくなるはずです
long path
Entityにshootメソッド生やしてくれ
desert schooner
尚、この方法ではプロジェクタイルに誰が発射したかというデータがないため、キルログが「(プレイヤー名)に倒された」ではなく「(プロジェクタイル名)に倒された」にな...
銃ごとに弾エンティティ分けてるから（撃たれたプレイヤー）は（銃の名前）で撃たれた
ってlangで設定すれば・・・
long path
まあそれで良ければいいですが案件ですね（
他の方法があるとすれば発射したprojectileにタグ等を与えておきヒット時にダメージ処理をscriptAPI側で行うようにする とかですかね
scriptAPI側でダメージが自由にいじれる ダメージを与える際にタグを元に発射者を特定し発射者に攻撃されたことにできる等メリットもあります
アイテムのタグを使ってアイテムを追加するだけで弾の動作を設定する なんてこともできますね
弾丸自体の状態を変えようと思うといくつかのパターンを用意して…等にはなりますが
desert schooner
あー・・・・
実際突き詰めると銃弾は3種になりますね・・・・
１．普通の弾
２．グレネード弾
３．ロケット弾頭
（２と３は落下の仕方の違い）
long path
まあscriptAPIから制御出来ない可変要素と言うとサイズくらいですけどね
ちなみに処理を完全にscriptAPIにするメリットのひとつとしてアイテム自体に依存した動作設計をするのでアイテムを増やすだけで10の種類がふやせる = 追加コンテンツを簡単に増やせる
というものもあります
もちろん用意した範囲内で ではありますがダメージ量やら発射間隔やら弾数やらは自由に設定出来ますし
desert schooner
アイテムのタグを使ってアイテムを追加するだけで弾の動作を設定する なんてこともできますね
なるほど・・・ 追加のアイテム用意しとけば特殊弾のマガジンのデータ＋特殊弾用の処理用意すれば徹甲弾とか曳光弾とかの特殊弾薬を簡単に実装できる・・・
って認識でおkですか？
long path
そんな感じにも可能ですね
当たったら貫通しろ とかprojectile自体に影響するものはprojectileの情報をいくつか用意するかあるいは別のprojectileにしておくなどが必要ではありますがタグを元に出すエンティティを変える なんて容易ですので
銃を作る気はないけどそんな感じの設計がしたくなってきた
desert schooner
現在のイメージとしては
銃ごとに残弾とかのデータ持たせる＝＞右クリック検知＝＞プロジェクタイルスポーン＝＞残弾データー１
残弾データ＝０or地面をスニーク左クリック＝＞インベントリにマガジンあること検知＝＞リロード（残弾を初期値or初期値＋１に）
って感じですね
desert schooner
他の方法があるとすれば発射したprojectileにタグ等を与えておきヒット時にダメージ処理をscriptAPI側で行うようにする とかですかね
このタグってコマンドでつけるほうのタグですか？
desert schooner
タグ≠データ値 で合ってますかね？・・・・
long path
？
desert schooner
アイテムに付けるタグとtagコマンドでこんがらがってますね・・・・
long path
projectileに与えるのはコマンドの方のタグで武器の動作分岐はアイテムにつけるタグですね
desert schooner
コマンドのほうのタグは発射したプレイヤー名を持たせる
アイテムのタグは銃の状態を持たせる
って感じで合ってます？
desert schooner
例えば僕が撃った銃弾はDisloyalSquid93ってタグが付くってことですかね？
long path
例えばそんな感じですね
desert schooner
タグの内容を取得＝＞当たったプレイヤーに僕から相手へ飛び道具で与えたとしてダメージ処理
って感じですかね？
long path
そんな感じですね
desert schooner
例えば僕が撃った銃弾はDisloyalSquid93ってタグが付くってことですかね？
これだったら （撃たれたプレイヤー）はDisloyalSquid93に倒された って感じかな？
long path
applyDamageで発射者を指定する感じですね
desert schooner
銃の種類も取得して （撃たれたプレイヤー）は（撃ったプレイヤー）の（銃の種類）で撃たれた ってのも可能ですか？
（langもいじらないといけないし多分実装するのかなり先）
long path
キルログを消す必要がありますが消してしまえばあとは出したいように出すだけではありますね
desert schooner
||僕の場合はとりあえず発射とか残弾とかの基礎部分作るのが先ですね・・・||
desert schooner
scriptapiでエンティティをsummonごり押し以外で出す方法ってありますか？
（現在summonごり押し実装ですｗ）
crisp sigil
Dimension.spawnEntity()が使えます
crisp sigil
Dimensionにはエンティティを出したいディメンションを入れてください
desert schooner
全ディメンションで出したい場合は全ディメンション分やる感じですか？
crisp sigil
イベントのsourceに入っているPlayerから source.dimension のようにして使った人のディメンションを取得できるので
そこからspawnEntityに繋げてみてください
desert schooner
world.afterEvents.itemUse.subscribe( ak47=> {
const { itemStack} = ak47;
const { source} = source.dimension;
if(itemStack.typeId==='ar:ak47'){
source.spawnEntity('ar:ak47bullet')
}
});
こんな感じですか？
crisp sigil
sourceはイベント変数のak47から取得します
desert schooner
const { source} = ak47 で合ってますか？
crisp sigil
⭕
desert schooner
エラー出ますね・・・
world.afterEvents.itemUse.subscribe( ak47=> {
const { itemStack} = ak47;
const { source} = ak47;
if(itemStack.typeId==='ar:ak47'){
const ze = world.getPlayers(source.dimension);
source.Dimension.spawnEntity('ar:ak47bullet')
}
});
コードこんな感じです
crisp sigil
dimensionのdは小文字です
desert schooner
この手のポカミス初めからめっちゃやってるな・・・
crisp sigil
エラー出ますね・・・
world.afterEvents.itemUse.subscribe( ak47=> {
const { ite...
const { itemStack, source } = ak47;のようにまとめて書くこともできます
desert schooner
またエラー出ました・・
world.afterEvents.itemUse.subscribe( ak47=> {
const { source,itemStack} = ak47;
if(itemStack.typeId==='ar:ak47'){
const ze = world.getPlayers(source.dimension);
source.dimension.spawnEntity('ar:ak47bullet')
}
});
今のコードです
crisp sigil
sourceがプレイヤーを指しているのでgetPlayersは不要です
desert schooner
const ze = world.getPlayers(source.dimension);
この行自体が不要ってことでおkですか？
crisp sigil
はい
desert schooner
そうするとこのエラーが出ますね・・・
world.afterEvents.itemUse.subscribe( ak47=> {
const { source,itemStack} = ak47;
if(itemStack.typeId==='ar:ak47'){
source.dimension.spawnEntity('ar:ak47bullet')
}
});
crisp sigil
spawnEntityの2つ目の引数に座標を渡さないとでした
desert schooner
自分の視点の先に出したいときはどうすればいいですか？
desert schooner
自分の座標にZ+１くらいのほうがいいかもですね・・・
desert schooner
こうなりました
crisp sigil
あーだめですね()
desert schooner
そっこーで銃弾に視点の方向に加速度かければ行けますかね？・・・
crisp sigil
先に飛ばす処理書いてみますか
crisp sigil
ですね
crisp sigil
spawnEntity()の返り値が出したエンティティになるので
このように変数に入れてあげましょう
const bullet = ...spawnEntity(...)
(...は略)
そうするとbullet.applyImpluse()のように書けます
desert schooner
world.afterEvents.itemUse.subscribe( ak47=> {
const { source,itemStack} = ak47;
if(itemStack.typeId==='ar:ak47'){
source.dimension.spawnEntity('ar:ak47bullet',source.getHeadLocation());
const bullet = source.dimension.spawnEntity('ar:ak47bullet',source.getHeadLocation());
bullet.applyImpluse(15)
}
});
こんな感じですね・・・
crisp sigil
16行目はどこですか？
tawny granite
impulse
long path
プレイヤー使えないからね しょうがないね
crisp sigil
こんな感じで見ている方向に速度を与えられます
const speed = 3;
const impulse = Vector.multiply(player.getViewDirection(), speed);
bullet.applyImpulse(impulse);
speedの値はいじってみてください
desert schooner
const { source,itemStack} = ak47;
if(itemStack.typeId==='ar:ak47'){
source.dimension.spawnEntity('ar:ak47bullet',source.getHeadLocation());
const bullet = source.dimension.spawnEntity('ar:ak47bullet',source.getHeadLocation());
const speed = 3;
const impulse = Vector.multiply(player.getViewDirection(), speed);
bullet.applyImpulse(impulse);
}
});```
エラー出てしまいますね・・・
desert schooner
const { source,itemStack} = ak47;
if(itemStack.typeId==='ar:ak47'){
const bullet = source.dimension.spawnEntity('ar:ak47bullet', source.getHeadLocation());
const speed = 5;
const impulse = Vector.multiply(source.getViewDirection(), speed);
bullet.applyImpulse(impulse);
}
});```
こんな感じで合ってますか？そんな感じですかね
desert schooner
このエラーが出ていますね・・・
long path
importにVectorを追加してください
long path
行けると思います
desert schooner
フルオートはifでアイテムに記録した射撃モードをチェック＝＞右クリック押しっぱなし検知＝＞発射コードをループ
切り替えは左クリック検知＝＞アイテムに書き込み
ってできますか？
long path
実装方法は様々ですが私であれば発射処理部分を同一にするためイベントからプレイヤー等の情報を配列上に放り込み runIntervalからそのデータを参照 単発なら発射後削除 連射なら維持 離した際のイベントから削除 という形にしますね
まあやりやすいやり方でいいかと
desert schooner
この場合って持ってるすべての銃で射撃モードって共有になりますか？
（現状の計画だと６パターンくらい射撃モードのパターンあるので・・・）
・セミオートオンリー
・フルオートオンリー
・セミオート フルオート切り替え
・セミオート フルオート グレネードランチャー切り替え
・セミオート 3点バースト フルオート切り替え
・単発銃
long path
どうとでもなるかと
desert schooner
マルチで動作テストしたら自分に当たっちゃいますね・・・
（ソロだとちゃんと撃てます）
crisp sigil
特にその必要はないです
スポーンさせる位置を視点方向に少しずらしてみましょう
const spawnLocation = Vector.add(source.getHeadLocation(), source.getViewDirection());
const bullet = source.dimension.spawnEntity('ar:ak47bullet', spawnLocation);
これでちょうど1ブロック先に出てくるはずです
desert schooner
マルチでもしっかり動作しました ありがとうございます！
desert schooner
今度はフルオート射撃オンリーにしてみたんですが・・・ また動作しなくなりました・・・
desert schooner
実装方法は様々ですが私であれば発射処理部分を同一にするためイベントからプレイヤー等の情報を配列上に放り込み runIntervalからそのデータを参照 単発なら...
このやり方でセミ・フル切り替え機能を付ける前にまずはフルオートで撃てるようにしておこうかな＝＞色々ググった結果で組んでみる＝＞現状って感じですね・・・
crisp sigil
setDynamicPropertyはこのように使います
<Player>.setDynamicProperty("キー", 値);
world.afterEvents.itemStopUse.subscribe
iは小文字です
runInterval内で未定義の変数が使われています
desert schooner
こんな感じですか？
crisp sigil
消す時はundefined(=何も書かない)を入れます
<Player>.setDynamicProperty("キー");
<Player>の場所にはただplayerを入れるのではなく適切な変数名を入れる必要があります
itemUseイベント内では使ったプレイヤーをsourceプロパティから取得し変数に定義しているので、ここでは source.setDynamicProperty() のように書くのが適切です
crisp sigil
runInterval内でsourceが未定義なのが原因です
プレイヤーが渡されるイベント以外の場所でプレイヤーに処理をする場合は
world.getPlayers()やworld.getAllPlayers()などを使って全プレイヤーを取得し、for ofなどを使用してそれぞれに対して処理を行います
EX:```js
system.runInterval(() => {
for (const player of world.getPlayers()) { // 全プレイヤーを取得し、それぞれを"player"として定義
player.sendMessage('hi'); // 全プレイヤーにメッセージが送られる
}
});
今のところdynamicplopaty以外には特に何も射撃時に書き込まれません
エラーのほうは自己解決しましたが・・・ 射撃止まんなくなりました・・・
itemStopUseでダイナミックプロパティが変わって射撃止まるはずなんですが・・・
なぜか止まりません・・・
crisp sigil
それは謎🤔
desert schooner
uuid変えて入れなおしたら直りますかね？
crisp sigil
変えても同じことが起きる気はしますが...
long path
そもそもitemStopUseは発生しているのですか
long path
stopUseが発生していないならそれが原因でしょう
long path
アイテムに使用する要素がなければストップもないと思います
desert schooner
アイテムにon use があったんですけど・・・ scriptapiに改造時に消しましたね・・・
アイテムのほうにfoodとかの右クリックで何か起きるコンポーネントつけとけばいい感じですか？
desert schooner
アプデ後色々ぶっ壊れたみたいで全く動きません・・・
akが表示されない＋アイコンが全部出てないですね・・・ あとは右クリックのコンポーネントが反応してないです・・・
desert schooner
1.20.41ですね・・・ （前のテストしてたバージョンは1.20.40ですね・・・）
desert schooner
結構エラー吐いたりしてますし・・・（エラー出すぎて問題の部分が見えないレベル）
long path
アドオンに関する変更点皆無ですね チェンジログ上は ですが
41の変更点は
・realms関連のクラッシュバグ修正
・マケプレ関連のバグ修正
・PlayStationでデモからフルバージョンにアップグレードできない問題を修正
だけですね
desert schooner
manifestを作り直したら直りました・・・（たまにこれで治るけど普通に謎）
desert schooner
itemStopUseは発生してませんね・・
自己解決しました
desert schooner
https://qiita.com/JUN-SUZU/items/bd8c13b4ebf5c600bc9b
これをベースに残弾システムを作ろうと思っているのですがプレイヤーの今持っているアイテムにloreを付ける部分と
scoreboardに今持ってるアイテムのloreに保存されてる残弾を代入する方法をおしえてもらえないでしょうか？
zenith orchid
18行目で undefined に getDynamicProperty は存在しない、というエラーです。
（entityHitBlockイベントでの source.getDynamicProperty の source が undefined）
entityHitBlockイベントにはsourceプロパティはありません、あるのは blockFace、damagingEntity、hitBlockだけです。
https://learn.microsoft.com/ja-jp/minecraft/creator/scriptapi/minecraft/server/entityhitblockafterevent?view=minecraft-bedrock-stable
desert schooner
プレイヤーを取っていないのでエラー・・って感じですか？
desert schooner
今度はエラーなしで弾が出ない感じですね・・・
long path
sourceはないですがEntity自体はdamagingEntityにありますよ
crisp sigil
ここは値を持っていないことを条件にするなら!で反転させるかundefinedと比較ですね
source.getDynamicProperty("m16a1mode")===''
long path
damagingEntityとかが無くなってるような
desert schooner
if(damagingEntity.getDynamicProperty("m16a1mode")===undefined){
source.setDynamicProperty("m16a1mode")
}
for(const source of world.getPlayers())
if(source.getDynamicProperty("m16a1mode")===''){
source.setDynamicProperty("m16a1mode",'auto')
}```
ここですか？これtrueたりうるのか？
''を設定してれば話は別だが
desert schooner
こんな感じですか？
long path
2つ目のifをelse ifにしておかないとautoをセットした後にautoを比較してしまって消されてしまいますね
それと最初のifのせいでautoだった場合最初のifを通過しないのでその後のautoとの比較まで行かないかと
desert schooner
const { damagingEntity} = m16;
if(damagingEntity.getDynamicProperty("m16a1mode")===undefined){
damagingEntity.setDynamicProperty("m16a1mode",'auto')
}
else if(damagingEntity.getDynamicProperty("m16a1mode")==='auto'){
damagingEntity.setDynamicProperty("m16a1mode")
}
});}```
こんな感じですか？最後のカッコが気になりますがそんな感じですね
desert schooner
現状はこんな感じになっていて
（残弾をアクションバーに表示する用の）スコアボードにloreの一部の行に保存した残弾を代入して残弾＝０になるorスニーク右クリックで初期値に残弾を変える・・・って処理を考えたのですが・・・
addScore(participant: Entity | ScoreboardIdentity | string, scoreToAdd: number): number
これを使う・・・くらいしか・・・
教えてもらえると嬉しいです
long path
代入ならsetScoreでいいかと
desert schooner
https://qiita.com/JUN-SUZU/items/bd8c13b4ebf5c600bc9b
loreの部分はこれを参考にしようと思ってます
long path
import { world, system, Vector } from "@minecraft/server";
world.afterEvents.itemUse.subscribe(m16 => {//射撃開始
const { source, itemStack } = m16;
if(itemStack.typeId === 'ar:m16') {
source.setDynamicProperty("m16a1", 'fire');
}
});
world.afterEvents.itemUse.subscribe(ak47=> {//後でセミ・フル切り替えに修正
const { source, itemStack } = ak47;
if(itemStack.typeId==='ar:ak47') {
const spawnLocation = Vector.add(source.getHeadLocation(), source.getViewDirection());
const bullet = source.dimension.spawnEntity('ar:ak47bullet', spawnLocation);
const speed = 5;
const impulse = Vector.multiply(source.getViewDirection(), speed);
bullet.applyImpulse(impulse);
}
});
world.afterEvents.itemUse.subscribe( m1911a1=> {//モード切替がないセミオートオンリー銃
const { source, itemStack } = m1911a1;
if(itemStack.typeId==='hg:m1911') {
const spawnLocation = Vector.add(source.getHeadLocation(), source.getViewDirection());
const bullet = source.dimension.spawnEntity('hg:m1911bullet', spawnLocation);
const speed = 5;
const impulse = Vector.multiply(source.getViewDirection(), speed);
bullet.applyImpulse(impulse);
}
});
world.afterEvents.itemStopUse.subscribe(m16 => {//射撃終了
const { source, itemStack } = m16;
if(itemStack.typeId === 'ar:m16') {
source.setDynamicProperty("m16a1");
}
});
world.afterEvents.entityHitBlock.subscribe(m16 => {//モード切替
const { damagingEntity } = m16;
switch(damagingEntity.getDynamicProperty("m16a1mode")) {
case 'semi':
damagingEntity.setDynamicProperty("m16a1mode", 'auto');
break;
case 'auto':
damagingEntity.setDynamicProperty("m16a1mode", 'semi');
break;
}
});
system.runInterval(function() {
for (const player of world.getPlayers()) {
if(player.getDynamicProperty("m16a1") !== 'fire') continue;
const spawnLocation = Vector.add(player.getHeadLocation(), player.getViewDirection());
player.runCommand(`playanimation @s animation.item.m16a1.shoot`)
const bullet = player.dimension.spawnEntity('ar:m16bullet', spawnLocation);
const speed = 5;
const impulse = Vector.multiply(player.getViewDirection(), speed);
bullet.applyImpulse(impulse);
if(player.getDynamicProperty("m16a1mode") === 'semi') {//セミオート
player.setDynamicProperty("m16a1");
}
}
}, 5);
最低限の修正のみしました
long path
ちなみにですがLoreをスコアに入れたい理由は何ですか？
desert schooner
アクションバーに残弾を表示するためです
titlerawでスコアボードを表示する・・くらいしか思いつかなかったので・・・
long path
onScreenDisplayから直接アクションバーに書き込みができるのであとは残弾管理処理も混じえて表示でもいいとは思います
スコアで弾数管理したい とかならそれもありだとは思います
desert schooner
最低限の修正のみしました
修正ありがとうございます
long path
処理の統一もしようかと思ったんですがまあどのような処理になるかとか今後どのような処理が増えるのかとかが不明なのでひとまずはこのくらいにしました
crisp sigil
質問とは関係ないですがitemUseイベントやgetPlayersの処理は一つにまとめた方がいいと思います
desert schooner
射撃モードの種類が同じ銃は同じ処理にまとめたほうがいい感じですか？
long path
銃器の処理を大雑把に捉えると基本的には
弾丸Aを速度BでC方向に飛ばす
といったふうに可変的な部分を持つ同一の表し方ができるものだと思います
↑射出の処理はこうまとめれると思います
あとは使用の仕方が違う 等がなければ銃nを使用しているなら使用しているフラグを立てる
使用を辞めたなら使用しているフラグを落とす といった部分は同一の処理でいいと思います
desert schooner
なるほど・・・
desert schooner
https://tn-docs-test.vercel.app/articles/1.20.60-changelog
これのitemstackdynamicplopatyって実験機能がオンのワールドなら使える感じですか？
（もしアプデ待たないと使えないならアプデ待つつもりです）
long path
previewなら使えると思うよ
crisp sigil
少なくとも来るのは1月下旬らへんかなぁ
desert schooner
アプデ来たので速攻で
EntityProjectileComponent
を用いたコード改良をしてみましたが・・・
ItemStack Dynamic Propertyがよくわからない感じですね・・・
desert schooner
itemStack.setDynamicProperty(" ");
みたいに書けばいい感じですか？
long path
そんな感じです
desert schooner
なるほどです
すでに入ってる数値から-1したい場合は
itemStack.setDynamicProperty("m16a1ammo",'m16a1ammo'-1)
こんな感じで合っていますか？
desert schooner
system.runInterval(function() {
for (const player of world.getPlayers()) {
if(player.getDynamicProperty("m16a1") !== 'fire') continue;
const spawnLocation = Vector.add(player.getHeadLocation(), player.getViewDirection());
player.runCommand(`playanimation @s animation.item.m16a1.shoot`)
const m16a1bullet = player.dimension.spawnEntity("ar:m16bullet", spawnLocation);
const projectile = m16a1bullet.getComponent("minecraft:projectile");
const velocity = Vector.multiply(player.getViewDirection(), 5);
projectile.shoot(velocity);
const ammo =itemStack.getDynamicProperty("ammo")
ammo = Number(ammo);
itemStack.setDynamicProperty("ammo", ammo-1);
if(player.getDynamicProperty("m16a1mode") === 'semi') {//セミオート
player.setDynamicProperty("m16a1");
}
}
}, 5);
こんな感じですか？
（||僕の中途半端な||知識総動員して書きました）
desert schooner
メインハンドののアイテムを取るにはどうすればいいですか？
long path
インベントリからselectedSlotのindexで取得するかequippableからMainhandを取得するかのどちらかです
desert schooner
system.runInterval(function() {
for (const player of world.getPlayers()) {
if(player.getDynamicProperty("m16a1") !== 'fire') continue;
const spawnLocation = Vector.add(player.getHeadLocation(), player.getViewDirection());
player.runCommand(`playanimation @s animation.item.m16a1.shoot`)
const m16a1bullet = player.dimension.spawnEntity("ar:m16bullet", spawnLocation);
const projectile = m16a1bullet.getComponent("minecraft:projectile");
const velocity = Vector.multiply(player.getViewDirection(), 5);
projectile.shoot(velocity);
const itemStack = equipmentCompPlayer.getEquipment(EquipmentSlot.Mainhand, new ItemStack(MinecraftItemTypes.ar:m16));
itemStack.setDynamicProperty("ammo", ammo-1);
if(player.getDynamicProperty("m16a1mode") === 'semi') {//セミオート
player.setDynamicProperty("m16a1");
itemStack.setDynamicProperty("ammo", ammo-1);
}
}
}, 5);
これだとコードエディターの時点でエラー吐きますね・・
（銃のアイテムIDはar:m16です）
crisp sigil
既に手に持っているアイテムを取得するので新しくItemStackを作る必要はありません
...getEquipment(EquipmentSlot.Mainhand);でok
desert schooner
わかりました
crisp sigil
アイテムごとに処理を分岐させるのであればitemStack.typeIdを条件に分岐させてください
desert schooner
発射部分を共通化してアイテムidで分岐させるのと発射モードカテゴリ別にそれぞれに処理するのどっちがいいんでしょうか・・・
（銃としては大きくカテゴリ分けすると射撃モード別で５種あります セミ セミ・フル セミ・フル・バースト フル セミ・フル・ランチャー）
crisp sigil
射撃モードによって何が変わるんですか？
crisp sigil
切り替えのパターン がよく分かりませんがオブジェクトで一括管理すると良さそうです
例えばこのように定義しておくと```js
const gunData = {
"ar:m16": {
animation: "animation.item.m16a1.shoot",
bullet: "ar:m16bullet",
propertyId: "m16a1mode",
modes: {
"auto": { speed: 5 },
"semi": { speed: 3 }
}
},
"アイテムID": { ... }
}
このように1つのコードで複数の銃に対応できます
```js
const data = gunData[item.typeId]; // IDに対応する銃のデータを取得
if (data) {
player.runCommand(`playanimation @s ${data.animation}`);
...
const bullet = player.dimension.spawnEntity(data.bullet, spawnLocation);
...
const mode = player.getDynamicProperty(data.propertyId);
const speed = data.modes[mode].speed;
const velocity = Vector.multiply(..., speed);
projectile.shoot(velocity);
}
semiモードが単発射撃 autoモードが連射ですね・・・
crisp sigil
dataとmodeが未定義です
long path
これ前から思ってたけどfireをDPに保存する必要ってないよな
desert schooner
銃のデータを別ファイルに置くことってできますか？
main.jsを処理 別ファイルをデータ置き場にしたいのですが・・・
zenith orchid
別のjsファイルを作りexportすることで、main.jsでimportして扱うことができます
https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Statements/export
desert schooner
こんな感じですか？
現状こんな感じですね・・・ 弾の代わりにエラーが出ます・・・
動作しないですね・・・
[Scripting][error]-ReferenceError: 'item' is not defined at <anonymous> (main.js:49)
エラーはこんな感じです・・・
desert schooner
現状こんな感じですね・・・ 弾の代わりにエラーが出ます・・・
こっちは自力で解決しました
desert schooner
[Scripting][error]-ReferenceError: 'item' is not defined at <anonymous> (main...
今はこっちのエラーだけ出てる感じです
long path
このエラーはitemが未定義 というエラーです
使用している場所を見てきちんと定義されているか確認してみてください
desert schooner
切り替えのパターン がよく分かりませんがオブジェクトで一括管理すると良さそうです
例えばこのように定義しておくと```js
const gunData = {
...
const data = gunData[itemStack.typeId]; // IDに対応する銃のデータを取得
これを参考に||（というよりそのまま利用）||した感じですね・・・
desert schooner
マイクラからitemをインポートすればいい感じですか？
long path
このitemはおそらく使用したアイテムのことだと思います
desert schooner
itemStackに書き換えたほうがいいですか？
long path
itemStackで定義されているならそういうことです
desert schooner
今度はエラーも弾も出なくなりました・・
long path
使用しているのは上のファイルですか
desert schooner
エラー見たら単純に書き間違いでした・・・
desert schooner
エラー見たら単純に書き間違いでした・・・
エラーは直しましたがまだ弾は出ないですね・・・
desert schooner
使用しているのは上のファイルですか
使用しているのはこれです
desert schooner
DPのammoの初期値は9行目で登録しているはずなんですが・・・
long path
52行目を見てください
tawny granite
detaとdataのタイポ
desert schooner
定義しなおしましたがまだですね・・・
tawny granite
const ammo = itemStack.setDynamicProperty("ammo")//現在の装弾数を取得
setしてますね
long path
ただ、import文で相対パス指定をしないと参照する場所がズレた気がするので相対パス指定にした方がいい気がします
desert schooner
同じ回送なのでファイル名になってますが・・・ ../scripts/deta.jsにしたほうがいい感じですか？
long path
エラーは
desert schooner
なしです
long path
ならどこまで動いているかを確認してください
desert schooner
一切動いてない感じですね・・・
射撃処理で弾が入ってない場合はno ammo
弾がある場合は”残弾”/最大装弾数
とコマンドで出るはずなので・・・
long path
なのでどこまで動いているのかを確認してください
desert schooner
テスト用に各工程でno ammo表示するようにしたら・・・
常時no ammo表示になりました・・・
long path
メッセージを同じにしたらどこが出してるか分からないです…
long path
これで動いていないのはrunIntervalのどこかであることがわかったはずです
もっと細分化しどこまで動いているかを確認してください
ついでに使用している値も出力するといいでしょう
desert schooner
player.runcommand('say ${ammo}') がundifinedを返してますね・・・
desert schooner
setitemのやり方ってあってますか？
long path
sourceにsetItemはありません
desert schooner
どうすればいいですか？
long path
個人的にはsetItemするよりContainerSlotを使った方がいい気がします
desert schooner
world.afterEvents.itemUse.subscribe(m16 => {//射撃開始
const { itemStack,source } = m16;
if(itemStack.typeId === 'ar:m16') {
source.runCommand(`say start`)
itemStack.setDynamicProperty("gun", 'fire');
itemStack.setDynamicProperty("ammo", 30);
ContainerSlot.setItem(itemStack?:m16);
}
});
こんな感じですか？
long path
ContainerSlotはいったいどこから来たのですか
crisp sigil
<Container>.getSlot(index: number): ContainerSlotです
desert schooner
メインハンドのアイテムだとどうなりますか？
long path
<Player>.selectedSlotを使用してください
desert schooner
world.afterEvents.itemUse.subscribe(m16 => {//射撃開始
const { itemStack,source } = m16;
if(itemStack.typeId === 'ar:m16') {
source.runCommand(`say start`)
itemStack.setDynamicProperty("gun", 'fire');
itemStack.setDynamicProperty("ammo", 30);
source.setItem(index,m16:source.selectedSlot);
}
});
これはコードエラー・・・
crisp sigil
ちなみにgetSlotならsetItemが不要になります
desert schooner
こんな感じですか？
crisp sigil
イベントから取ってるのはItemStackなのでDPをセットしたあとsetItemが必要です
desert schooner
source.setItem(source.selectedSlot,itemStack);
これをDPいじった後すべてに入れる感じですか？
desert schooner
こんな感じですか？
crisp sigil
DynamicPropertyの操作は全てContainerSlotから可能なので書き換えてあげましょう
そうすればgetEquipmentもsetItemも要らなくなります
desert schooner
ContainerSlot.setitem(slot.selectedSlot,itemStack);
みたいな感じですか？
long path
setしなくていいです
crisp sigil
ContainerSlotは書き換えるとすぐに値が反映されるものです
desert schooner
なるほどです
desert schooner
こんな感じですか？
long path
sourceにsetItemはありません
crisp sigil
ContainerSlot(slot.selectedSlot,itemStack);はいりません
runInterval内でしているItemStackの操作は全てContainerSlotで可能です
つまりitemStack.〇〇と書いているものはslot.〇〇と書き換えられます
crisp sigil
そうすればgetEquipmentもsetItemも要らなくなります
↑
イベント内ではitemStackに対してDynamicPropertyの操作などをしているので、それらを保存するために<Container>.setItemが必要です
desert schooner
world.afterEvents.itemUse.subscribe(m16 => {//射撃開始
const { itemStack,source,ContainerSlot,slot } = m16;
if(itemStack.typeId === 'ar:m16') {
source.runCommand(say start)
itemStack.setDynamicProperty("gun", 'fire');
itemStack.setDynamicProperty("ammo", 30);
ContainerSlot.setItem(slot.selectedSlot,itemStack);
}
});
こっちはこんな感じですか？
crisp sigil
<Container>.setItemです
selectedSlotはPlayerのプロパティ(=source)です
Containerはインベントリコンポーネントを取得し、<InventoryComponent>.containerから取得します
desert schooner
world.afterEvents.itemUse.subscribe(m16 => {//射撃開始
const { itemStack,source,ContainerSlot } = m16;
if(itemStack.typeId === 'ar:m16') {
source.runCommand(say start)
itemStack.setDynamicProperty("gun", 'fire');
itemStack.setDynamicProperty("ammo", 30);
const inventory = player.getComponent(EntityInventoryComponent.componentId);
inventory.setItem(source.selectedSlot,itemStack);
}
});
こんな感じですか？
crisp sigil
inventory.container.setItemですね
desert schooner
world.afterEvents.itemUse.subscribe(m16 => {//射撃開始
const { itemStack,source,player } = m16;
if(itemStack.typeId === 'ar:m16') {
source.runCommand(say start)
itemStack.setDynamicProperty("gun", 'fire');
itemStack.setDynamicProperty("ammo", 30);
const inventory = player.getComponent(EntityInventoryComponent.componentId);
inventory.container.setItem(source.selectedSlot,itemStack);
}
});
こんなエラーが出てますね・・・
[Scripting][error]-TypeError: cannot read property 'getComponent' of undefined at <anonymous> (main.js:11)
desert schooner
world.afterEvents.itemUse.subscribe(m16 => {//射撃開始
const { itemStack,source } = m16;
if(itemStack.typeId === 'ar:m16') {
source.runCommand(say start)
itemStack.setDynamicProperty("gun", 'fire');
itemStack.setDynamicProperty("ammo", 30);
const inventory = source.getComponent(EquipmentSlot.Mainhand);
inventory.container.setItem(source.selectedSlot,itemStack);
}
});
現在はこのエラーです・・・
[Scripting][error]-TypeError: cannot read property 'container' of undefined at <anonymous> (main.js:12)
desert schooner
world.afterEvents.itemUse.subscribe(m16 => {//射撃開始
const { itemStack,source } = m16;
if(itemStack.typeId === 'ar:m16') {
source.runCommand(say start)
itemStack.setDynamicProperty("gun", 'fire');
itemStack.setDynamicProperty("ammo", 30);
const inventory = source.getComponent(EntityInventoryComponent.componentId);
inventory.container.setItem(source.selectedSlot,itemStack);
}
});
現状こんな感じですね・・・
crisp sigil
EntityInventoryComponentはインポートしてください
desert schooner
これまで教えてもらったこと＋自分で調べたことで修正しましたが…まだエラー出ます・・
desert schooner
crisp sigil
https://jaylydev.github.io/scriptapi-docs/1.20.60/classes/_minecraft_server_1_7_0.EntityHitBlockAfterEvent.html
そういえばentityHitBlockにsourceとitemStackはありませんでしたね(
PlayerはdamagingEntityから取ってください
アイテムについてはrunInterval内と同じくContainerSlotを取得してsetDynamicPropertyすると扱いやすいと思います
desert schooner
エラー出なくなりました！
crisp sigil
runInterval内でしているItemStackの操作は全て**ContainerSlot**で可能です
つまりitemStack.〇〇と書いているものはs...
あとはこちらですね
desert schooner
world.afterEvents.entityHitBlock.subscribe(m16 => {//モード切替・リロード
const { damagingEntity } = m16;
const equipmentCompPlayer = damagingEntity.getComponent(EntityComponentTypes.Equippable);
const itemStack = equipmentCompPlayer.getEquipment(EquipmentSlot.Mainhand);
const inventory = damagingEntity.getComponent(EntityInventoryComponent.componentId);
const maxammo =gunData[itemStack.typeId.maxammo];
const isSneaking = damagingEntity.isSneaking; // スニークしているか(true/false)
if(isSneaking==true){
if(itemStack.getDynamicProperty("ammo") >=1){
itemStack.setDynamicProperty("ammo", maxammo+1);
}else{
itemStack.setDynamicProperty("ammo", maxammo);//スニーク左クリックでリロード
}
inventory.container.setItem(damagingEntity.selectedSlot,itemStack);
}else{
switch(itemStack.getDynamicProperty("mode")) {
case 'semi':
itemStack.setDynamicProperty("mode", 'auto');
inventory.container.setItem(damagingEntity.selectedSlot,itemStack);
break;
case 'auto':
itemStack.setDynamicProperty("mode", 'semi');
inventory.container.setItem(damagingEntity.selectedSlot,itemStack);
break;
case 'undifined':
itemStack.setDynamicProperty("mode", 'semi');
const mode = itemStack.getDynamicProperty("mode")
player.runCommand(`say ${mode}`)
inventory.container.setItem(damagingEntity.selectedSlot,itemStack);
break;
}
}
});
現在これと残弾減らす処理が動いてない感じですね・・・（しかもエラーなしで）
crisp sigil
ContainerSlotよりもItemStack使いたい感じですかね...?
long path
そういえばテストした時にアイテムが更新される関係かアニメーションに乱れがあったので気をつけてください
desert schooner
ContainerSlotよりもItemStack使いたい感じですかね...?
そうですね・・・
僕的にはそっちのほうがわかりやすかったので・・・
現状発射されたりされなかったり残弾減ったり減らなかったりですね・・・
long path
ItemStackとContainerSlotにそこまで大きな違いは無いと思いますけどね
crisp sigil
getItemがgetSlotになって後からsetItemする必要がなくなるだけではある()
ここはconst maxammo = gunData[itemStack.typeId].maxammo;です
crisp sigil
case 'undifined':のところ
undefinedは文字ではないので case undefined: に変えてみてください
desert schooner
case 'undifined':のところ
undefinedは文字ではないので `case undefined:` に変えてみてください
改良しました
残弾の保存を射撃終了時に置き換えようと思うのですがどうすればいいですか？
desert schooner
う～ん・・・
runinterval内でconstに書き込んでitemstopuseでrunintervalで設定したconstをsetdynamicpropertyでやる・・・っての思いついたんですが・・・
desert schooner
runinterval内でconstに書き込んでitemstopuseでrunintervalで設定したconstをsetdynamicpropertyでやる・...
処理イメージ
最後の円柱はconst ammoをDPammoに書き込みの間違いです
desert schooner
runinterval内でconstに書き込んでitemstopuseでrunintervalで設定したconstをsetdynamicpropertyでやる・...
こんな感じで残弾が減らずに発射されたり超連射始まって荒ぶったりするのでこの処理思いついたのですが・・・・
desert schooner
これの対処法ってありますか？
crisp sigil
runIntervalの処理が上手くいってない感じ...?
desert schooner
そうですかね・・・・・
desert schooner
とりあえず動作してないという事実だけが残ってますね・・・
desert schooner
射撃処理中はプレイヤーに残弾のデータ書き込む形で修正してみましたが・・・
なぜか動作しません・・
desert schooner
射撃処理中はプレイヤーに残弾のデータ書き込む形で修正してみましたが・・・
なぜか動作しません・・
数か所タイプミスあったので修正しましたが・・・
やっぱり動いてません・・・（弾は出ます）
動作的には
射撃開始時にアイテムのDP ammoをプレイヤーのDP nowammoに書き込み
1発撃つごとにプレイヤーのDP nowammoを-1
射撃終了時にプレイヤーのDP nowammoをアイテムのDP ammoに書き込みです
long path
今ちょっと荷解き中なのであれですが後で確認してみます
desert schooner
射撃処理は直せたんですが・・・ 今度はリロードアニメーションが動かないですね・・・
tawny granite
ar:m16以外のデータにはreloadプロパティが存在していないですね。読んでないか。m16でもアニメーションが動きませんか？またエラーは出ていますか
tawny granite
animation.item.m16a1.reloadは素で発動して動作しますか
desert schooner
あれ・・・ 動作しませんね・・・（scriptapi化する前の銃アドオンからモデルとかアニメーション周り全部移植したんですが・・・）
tawny granite
それを治したら動いてくれるんじゃないかなあと思います…アニメーションは私には分かりません
desert schooner
なぜかanimation.item.m16a1.reloadだけ再生されないんですよね・・・
desert schooner
コード修正しましたがセミオート射撃が動作しません・・・
desert schooner
どう修正すればいい感じですか？
tawny granite
せめてエラーがあるのかと、どの段階の動作が機能していないのかくらいの情報を書いた方が良いと思います
desert schooner
一切エラーないですね・・・
おそらく if(itemStack.getDynamicProperty("mode") === 'semi') {//セミオート
player.setDynamicProperty("gun");
}
がどう出してない感じかと・・・
tawny granite
modeのセットが出来てないのかな
tawny granite
そのときgunはfireですか？
tawny granite
19にも同じのがあるようですね
desert schooner
ええ・・・
long path
gunDataが不明なので憶測ですが
desert schooner
gunDataは銃のIDとか装弾数とかが記録されてるデータファイルです
data.jsに保存されています
long path
まあどちらにせよgunDataの中身が例えば
{
"gun:gun_1": "gun:gun_1"
}
とでもなってない限りこのifは通れないです
tawny granite
射撃処理は直せたんですが・・・ 今度はリロードアニメーションが動かないですね・・・
この辺の
desert schooner
まあどちらにせよgunDataの中身が例えば
```json
{
"gun:gun_1": "gun:gun_1"
}
```
とでもなってない限りこのif...
こんな感じにすればいいですか？
long path
条件が無くなったのでスタックできるアイテムを持っていた場合エラーになりますね
desert schooner
コード修正しましたがセミオート射撃が動作しません・・・
前のコードに戻してdata.jsに追加すればいい感じですか？
desert schooner
これで合ってますか？
long path
問題ないです
desert schooner
あと修正する場所ってありますか？
desert schooner
不要コードのコメントアウトをしましたが・・・（コメントアウトなのは発射用の最低限のコードになっているため）
マルチで動作してません・・・
状況としてはは残弾表示が消えて銃弾も出なくなります・・・
zenith orchid
エラーは出ていませんか
残弾表示はrunInterval内の処理のようなので、全プレイヤーのループ処理中にエラーが出て止まっているのではないでしょうか→マルチだと動かない
system.runInterval(function(shoot) {//射撃処理
for (const player of world.getPlayers()) {
const equipmentCompPlayer = player.getComponent(EntityComponentTypes.Equippable);
const itemStack = equipmentCompPlayer.getEquipment(EquipmentSlot.Mainhand);
const maxammo = gunData[itemStack.typeId].maxammo;````gunData[itemStack.typeId]`が`undefined`の可能性があります、その状態で`gunData[itemStack.typeId].maxammo`とするとエラーになるでしょう。例えば銃を持っていないプレイヤーがいるとその人以降の処理がエラーで止まってしまいます。
`function (`~~shoot~~`) {` ~~shoot不要~~なるほどです
プレイヤーごとに処理すればいい感じですか？
zenith orchid
そもそもエラーを出さないようにしてください
tawny granite
何も変わっていないですね
desert schooner
こっち修正後のファイルです・・・ ミスってごめんなさい・・・
tawny granite
```js
system.runInterval(function(shoot) {//射撃処理
for (const player of world.ge...
ここで指摘されたmaxammoの後に分岐しているので意味がないですね
desert schooner
なるほど・・・
constより前にif持ってくる感じですか？（マルチプレイの検証が相方の都合でできてないので・・・）
tawny granite
itemStackを得なければtypeIdも得られないのでそれより後ではあります。具体的にはdataを取得した直後にdataがあるかを確認するのが無難な気がしますが、maxammoを少し動かす必要がありますね
そういえば引数にshootが残ったままですね
desert schooner
なるほどです
修正してみます
desert schooner
こんな感じですか？
zenith orchid
system.runInterval(function() {//射撃処理
for (const player of world.getPlayers()) {
const equipmentCompPlayer = player.getComponent(EntityComponentTypes.Equippable);
const itemStack = equipmentCompPlayer.getEquipment(EquipmentSlot.Mainhand);
if(gunData[itemStack.typeId]!=undefined){
const maxammo = gunData[itemStack.typeId].maxammo;
const data = gunData[itemStack.typeId]; // IDに対応する銃のデータを取得 ←ココ````const data`で`gunData[itemStack.typeId]`を代入しているなら
```js
system.runInterval(function() {//射撃処理
for (const player of world.getPlayers()) {
const equipmentCompPlayer = player.getComponent(EntityComponentTypes.Equippable);
const itemStack = equipmentCompPlayer.getEquipment(EquipmentSlot.Mainhand);
const data = gunData[itemStack.typeId]; // IDに対応する銃のデータを取得
if(data!=undefined){
const maxammo = data.maxammo; // gunData[itemStack.typeId]をdataと短く書ける```こうやって利用すればより良く書けるのではと思わなくもないですが、一応そのコードでその場所のエラーは無くなるのではないでしょうか
試してご確認してくださいリロード時にconst guntypeにリロードしている銃のデータを入れてリロード処理用の銃に置き換えた後にconst guntypeに記録したデータで元の銃に戻すって処理を書きたいのですが最後の部分が認識されません（constの表示部分が薄いままになります）
tawny granite
スコープの問題ですね。if内で宣言したGuntypeを外から参照はできません
desert schooner
スコープの問題ですね。if内で宣言したGuntypeを外から参照はできません
わかりました 修正してみます
tawny granite
49行目closeが未定義といっています
desert schooner
gundetaにbolt_typeという指定があり
そこで銃のタイプごとに処理分けてる感じになります（クローズドボルトの銃は途中リロードで＋１発の再現のためです
tawny granite
おそらく"close"と比較したいということですかね
desert schooner
gundetaのbolt_typeがcloseか比較したい感じです
tawny granite
bolt_typeは文字列ですから"close"と比較してください
desert schooner
わかりました
このような感じでいいでしょうか？
tawny granite
=1つは代入です。2つか3つにしてください
desert schooner
わかりました
~~このような感じでいいでしょうか？~~
次はこちらのエラーが出ました・・・
tawny granite
またsetDynamicPropertyに<ItemStack>を渡すことはできません
desert schooner
リロード処理用から普通のに戻す際はelse ifの繰り返しになるのでしょうか？
tawny granite
リロード処理用から普通のに戻す...というのがどこのどういうものなのか知らないので...
好きに実装してください
desert schooner
一旦アイテムを置き換えちゃうのでどこかに元のアイテムを保存しておかないといけないんですよね・・・
const itemstackを取った後プレイヤーのDPに保存してDPからデータを改修して・・・ってやろうとしたんですが・・・
tawny granite
DPにアイテムを直接保存することはできません。元のアイテムを完全に保存する必要があるのか、データさえ保存できればよいのか、などによっても対応は変わるかと思います
desert schooner
できれば元のアイテムを保存したいですね・・・
（カスタム機能実装予定のため）
すいません この後アルバイトがあるので７時まで戻れません… 回答ありがとうございました・・・
desert schooner
すいません この後アルバイトがあるので７時まで戻れません… 回答ありがとうございました・・・
元のアイテムを完全に保存する方法ってありますか？（ご飯も食べて戻りました）
tawny granite
変数のまま保つか、チェストなどに保存する、ですかね
tawny granite
とかですね
パラメータという形で分離できるのであればitemstackとして保存する必要はないとは思います
desert schooner
変数に入れる場合はカスタム機能のパラメーターをリロード中プレイヤーに移動させる・・・とかですかね？
パラメーターはitemstackdynamicplopatyですね・・・
tawny granite
であれば分離して保存して復元することも可能ですね
desert schooner
次はこちらのエラーが出ました・・・
修正しましたが同じエラーですね・・・
tawny granite
reloadfinishというのはDPに格納できないオブジェクトですね
desert schooner
手に持ってるアイテムのIDをDPに保存したい場合はどうすればいいですかね・・・・
tawny granite
手に持ってるアイテムのIDをsetDynamicPropertyに渡してください
desert schooner
メインハンドのアイテムID取得は過去ログにあった気がするので見て確認してきます
tawny granite
既にアイテム自体はitemStackという変数に取得していそうですね
desert schooner
はい
desert schooner
メインハンドのアイテムID取得は過去ログにあった気がするので見て確認してきます
const typeId = itemStack.typeId;
アイテムのIDの取得はこれでいいですか？
一応確認です
long path
ItemStackクラスを使用して下さい
desert schooner
~~現状こんな感じです~~
エラーのほうは修正しました
tawny granite
修正後はどうなりましたか
desert schooner
リロード処理用の銃に置き換わらないだけですね・・・
確認したところタイプミスあったのでそれだけ直しましたが・・・
今試してる最中です
エラーなし 動作なしですね・・
現在のコードです
tawny granite
ItemStackクラスを使用して下さい
setItemには<ItemStack>の形にしてから渡してください。そのためにnew ItemStack()を使用できます
desert schooner
なるほどです
new ItemStack(reloadgun)
って感じですか？
tawny granite
はい
tawny granite
ItemStackをimportしてください
desert schooner
new ItemStack(reloadgun)
の後にですか？ それとも一番上にですか？
tawny granite
1番上です
desert schooner
プレイヤーからconstしてるitemstackは消す感じですか？
tawny granite
消しません
ItemStackの先頭は大文字です。itemStackと区別してください
またnew ItemStack()は生成した<ItemStack>を返すので返り値を変数に保持するか直接引数に渡すなどしてください。ただ書くだけでは無意味です
tawny granite
その時のselectedSlotをどこかに保持すればよいかと
desert schooner
const slot = player.selectedSlot;
みたいな感じですか？
tawny granite
はい。それをrunTimeout内で参照すれば良いと思いますいえまあやりたいように実装してください
desert schooner
inventory.container.setItem(damagingEntity.slot,itemStack);
こんな感じでいいでしょうか？
tawny granite
const slot = player.selectedSlot;
みたいな感じですか？
こうしたならslotは変数です。damagingEntityには生えていません
desert schooner
const reloadslot = damagingEntity.selectedSlot; でスロットを取って
inventory.container.setItem(reloadslot,itemStack); で置き換えればいい感じですか？
できました！
回答ありがとうございました！
またエラーとかわからないところ出たら質問すると思います・・・
desert schooner
こちらの機体の武装である機銃が自機に当たってしまうのですがどうすればよろしいでしょうか？（アイテム発射方式です）
desert schooner
こちらの機体の武装である機銃が自機に当たってしまうのですがどうすればよろしいでしょうか？（アイテム発射方式です）
自己解決しました
desert schooner
アップデートによりこちらのコードが動かなくなってしまいました
エラーには
[Scripting][error]-Plugin [puchineko guns 2 - 1.0.0] - [main.js] ran with error: [SyntaxError: Could not find export 'Vector' in module '@minecraft/server']
と出ています
tawny granite
#script-api message
Vectorは削除されました
tawny granite
mathを使う場合math自体をアドオン内に複製しないといけないですね
long path
正規実装されていない が正しいかと
desert schooner
？？？
long path
mathモジュールはアドオン自体に入れるためimport文だけで大丈夫です
desert schooner
こんな感じですか？
long path
配置の仕方次第ですが基本的には'./math/index.js'になります
desert schooner
import { Vector }from "./math/index.js"
こうですか？
[Scripting][error]-Plugin [puchineko guns 2 - 1.0.0] - [main.js] ran with error: [ReferenceError: Module [math/index.js] not found. Native module error or file not found.]
このエラーが出ますね・・・
desert schooner
mathを使う場合math自体をアドオン内に複製しないといけないですね
どのようにアドオン内に複製すればいいですか？
desert schooner
なるほど(元となるファイルがどこにあるのかわかってない)
desert schooner
何とか自力で複製してvector3インポートできましたがこちらのエラーが出てしまいますね・・・
minecraft-math.jsはmain.jsと同階層に複製してあってコードはこんな感じです
long path
minecraft-math.jsのdefault exportは何になっていますか
まあそもそもdefault exportされているものはわざわざfromで名前を変える必要は無いんですけどね
desert schooner
コレ・・・ですか？
long path
なんだこれ
long path
リンクを貰えれば
long path
元コードがcommonだから把握しずらいな
zenith orchid
import mcmath from './minecraft-math.js'; // mcmathは好きな名前に```のようにインポートして、```js
mcmath.Vector3Utils.add(a, b)```のように使えます
desert schooner
わかりました
tawny granite
52行のif文で、setItemがelseの方の60行目にしかないせいかと
desert schooner
マルチプレイ時にほかプレイヤーが素手の時のみ[Scripting][error]-TypeError: cannot read property 'typeId' of undefined at <anonymous> (main.js:100)と出て銃のシステムが動作しません・・・
現在のコードです
desert schooner
if(gunData[itemStack.typeId]!=undefined){
になります
long path
単純に装備していない時の処理がないからでは？
desert schooner
なるほど！
desert schooner
if(itemStack.typeId!=air){
こんな感じですか？
long path
itemStackとundefinedを比較してください
desert schooner
.typeIdは不要ですか？
long path
存在しない可能性があるので不要です
desert schooner
なるほどです
desert schooner
if(itemStack?.typeId!=undifined){
こういう場合なら問題ないという認識で大丈夫ですか？
long path
はい
desert schooner
なるほどです
desert schooner
専用アイテムをオフハンドに持つと専用UIを開いて銃のカスタムをできるようにしたいのですが可能でしょうか？
イメージとしてはhttps://i.ytimg.com/vi/JXKN7Fg13EA/maxresdefault.jpg
のようなUIを開いてアイテムのDPに書き込んでカスタムをアイテムモデルに適用する感じです
（画像は拾い物です）
desert schooner
アプデ対応したのですがこのようなエラーをはいてしまいます
原因を教えて頂けないでしょうか？
自己解決しました
追加で質問なのですがインベントリ内に特定のアイテムを持っているときだけリロード処理を実行するというコードはどのようにすればいいのでしょうか？
やり方としては再起処理ですべてのスロット探査しかないのでしょうか？
zenith orchid
再帰処理というよりは普通にforループだと思いますが、それはさておきインベントリ全てを調べてください
pulsar sable
追加で質問なのですがインベントリ内に特定のアイテムを持っているときだけリロード処理を実行するというコードはどのようにすればいいのでしょうか？
function hasItemInInventory(player, itemId) {
const { container } = player.getComponent('inventory');
for (let i = 0; i < container.size; i++) {
const item = container.getItem(i);
if (!item) continue;
if (item.typeId === itemId) return true;
}
return false;
}
プレイヤーのインベントリ内に指定したアイテムと同じものがあればtrueを返し、無かったらfalseを返すコードです
pulsar sable
このような形で大丈夫ですか？
なにかしらエラーが出ましたらまた教えてください
zenith orchid
hasItemInInventory関数を他で使わないのなら定義(function has...(...) { ... })を書く場所はそこでいいと思いますが、問題なのはすぐ下のifに書いてある関数の呼び出しです
関数を呼び出すときはfunction は書きません
呼び出し方はhasItemInInventory(damagingEntity, 'アイテムID')のような感じになるはずです
pulsar sable
専用アイテムをオフハンドに持つと専用UIを開いて銃のカスタムをできるようにしたいのですが可能でしょうか？
イメージとしてはhttps://i.ytimg.com...
const OffhandEquipment = player.getComponent(mc.EntityComponentTypes.Equippable).getEquipment(mc.EquipmentSlot.Offhand);
オフハンドのアイテムは確かこれで取得できたはず
tawny granite
(mc.はないことに注意
desert schooner
なるほどです
desert schooner
多忙なためなかなか試せなくて結果だせず申し訳ありません
desert schooner
現在こちらのコードでエラーなし動作せずとなっております・・・・
tawny granite
何をする部分が動作していないのですか
desert schooner
モード切替・リロードの部分のfunction以下ですね・・
tawny granite
function hasItemInInventory(reloaditem) {この様に宣言してhasItemInInventory(damagingEntity, reloaditem)この様に呼び出すのはおかしいですね
tawny granite
というよりそもそも関数の実装が正しくない...どうしよう
tawny granite
```js
function hasItemInInventory(player, itemId) {
const { container } = pl...
まず関数の宣言自体はこれをそのまま使って良いです
これはplayerがitemIdの指すアイテムをインベントリに所持しているかを返す関数で、例えば以下のように使います
hasItemInInventory(damagingEntity, "minecraft:diamond")```
これはdamagingEntity(の指すプレイヤー)がダイヤモンドを持っているかをtrue/falseで返しますで、今行いたいことはdamagingEntityがリロードアイテム(const reloadgun = gunData[itemStack.typeId].reloadgun;の指すIDのアイテム)を持っているかを確認したいということでよいですかね
(data.js見てないからわからん
tawny granite
あ、ちなみに関数が何かわからない場合は調べてください
desert schooner
一応調べて見よう見まねでかいてみたのですが・・・
tawny granite
で、今行いたいことはdamagingEntityがリロードアイテム(`const reloadgun = gunData[itemStack.typeId].r...
後はこのやりたいことをコードに落とすだけといえばだけです
desert schooner
変数いれるとこうなっちゃいますね・・・
desert schooner
凡ミスですね・・・
desert schooner
動作しました！
ありがとうございました！
zenith orchid
projectile.shoot(velocity, { uncertainty: 10 });のようにuncertaintyの値を増やすとブレます、0だと全くブレずに飛んでいきます。
desert schooner
ありがとうございます
今夜試してみます(予定があるので今日夜まで試せません)
desert schooner
動作しました ありがとうございました！
desert schooner
dynamicplopatyを使ってアイテムのモデルを追加で表示することってできますか？
イメージとしては銃のモデルに追加でサイトなどのカスタムパーツ（別ファイル）を表示する感じです
イメージとしては旧ゲリラアドオンのカスタム機能をスプリクト化した感じです
long path
※propertyです
attachablesから検出可能なものに変換さえしてやれば可能かとは思います
desert schooner
アニコン...とかでしょうか？
desert schooner
蓮根か
desert schooner
カスタムパーツのモデルと銃ごとの位置指定のアニメーションを用意してdynamicpropertyを検出してスプリクトでアニメーションを実行する・・・とかですかね？
desert schooner
残弾がない時でも発射アニメーションが実行されるバグ対策として残弾ゼロの際に発射アニメーションに移行しない銃に持ってるアイテムを変更するようにしたのですがモードや残弾等がundefinedになってしまいます
どのようにすればよろしいでしょうか？
（置き換え自体は動作しています）
tawny granite
分かっていたら何行目らへんの処理の何がundefinedなのかも書いてほしいです
desert schooner
リロード処理の残弾書き換え部分ですね
修正試みた結果がこれ出す
リロード演出用のアイテムに置き換えがうまくってないのと
残弾がundefinedになる感じですね
tawny granite
言葉で言われても探すのが大変なので具体的なコードや行数も教えてくださいという意味です
desert schooner
７１から７８行目です
inventory.container.setItem(damagingEntity.selectedSlotIndex,reload);//リロード用銃に置き換え
system.runTimeout(() => {
inventory.container.setItem(reloadslot,finishgun);
const itemStack = equipmentCompPlayer.getEquipment(reloadslot.Mainhand);
itemStack.setDynamicProperty("ammo", maxammo);
},reloadtime);
inventory.container.setItem(reloadslot,itemStack);
この辺が怪しいです
本来は演出用の銃に置き換えた後
アニメーション終了のタイミングで最大装弾数書き込んだ銃に置き換えてます
tawny granite
おそらくrunTimeout内で変更されたItemStackをsetしていないせいかと思われます
元のほうだとfinishgun、今書かれたものだとitemStackですかね
desert schooner
78…？
多分constに再代入しようとして怒られてるのかと思います
すいません７８はタイプミスです
itemStack = new equipmentCompPlayer.getEquipment(EquipmentSlot.Mainhand);
こんな感じで大丈夫でしょうか？
tawny granite
というかnew…？
んーと
そこにおいてitemStackは42らへんで宣言されたconstのitemStackを参照します
constは再代入できないのでエラーになります
そしてnewというのはクラスやコンストラクタにつけて新たなオブジェクトを生成する演算子です
なんであるのか分かりませんが要りません
desert schooner
constを消すだけで大丈夫な感じですか？
tawny granite
変数宣言を理解してください
constを消したら宣言がなくなってしまいます
42のitemStackを上書きしてもいいならletを使ってもいいし、75で新たなitemStackの宣言をしても良いです
被りが嫌なら変数名を変えてください
desert schooner
なるほどです
desert schooner
このような形で別宣言を用意するのもいい感じですか？
また８０から８８行目のコードで本来ならこのような形でリロードアニメーション処理用の銃に置き換えているのですがこちらでは動作しません
どのようにすればいいですか？
desert schooner
自己解決しました
desert schooner
ありがとうございました
desert schooner
３９行目のコードの書き方について教えてもらいたいです
やりたいことは射撃時に撃ったプレイヤーをソースにダメージを与えることと銃弾でプレイヤーをキルしたときに
キルしたプレイヤー名はキルされたプレイヤーを射殺した
とキルログを変更したいです
どのようにすればよろしいでしょうか？
reef willow
projectileHitEntityの各種プロパティについて
sourceで投擲物を当てた本人であるエンティティを返します。
projectileには発射した投擲物エンティティ(銃弾のEntityID)等を返します
getEntityHitを使うと当てられたエンティティを返します。
イメージ
world.afterEvents.projectileHitEntity.subscribe((ev) => {
const { source, projectile } = ev;
const hitEntity = ev.getEntityHit.entity; //ヒットされたエンティティオブジェクトを返した分を参照するように宣言する。
const HP = hitEntity.getComponent('minecraft:health).currentValue;
if(HP < 0 && hitEntity.typeId == 'minecraft:player' && source.typeId === 'minecraft:player') {
world.sendMessage(`${hitEntity.name} は ${source.name} に射殺された。`);
//ヒットした時体力0になった、尚且つプレイヤーにやられたら発火するようにする。
}
});
私はこうしてます。
イメージはこんな感じなので、他の方だと多分全く違うかもしれません。
スマホで手打ちしてるので構文エラー出るかもしれません。
あとはキルログはバニラのものと独自のものが同時に出ますのでゲームルールで消すのもありです。
または、バニラのテキストを変えるならリソパの言語ファイル辺りになります。
desert schooner
回答ありがとうございます
バニラのキルログを銃のダメージの場合のみ消したいときはどうすればいいですか？
beforeeventから ev.cancel = true
というのは何とかわかったのですがキャンセルするために死亡時にログを出すイベントがわからないので質問させていただきました
desert schooner
projectileHitEntityの各種プロパティについて
`source`で投擲物を当てた本人であるエンティティを返します。
`projectile`には...
すいません コード記述したところこのようなエラーが出るようになりました
どうすればいいでしょうか？
zenith orchid
43行目のgetEntityHitはメソッドなのでev.getEntityHit().entityのようにする必要があります
https://learn.microsoft.com/ja-jp/minecraft/creator/scriptapi/minecraft/server/projectilehitentityafterevent?view=minecraft-bedrock-experimental#getentityhit
desert schooner
回答ありがとうございます！
zenith orchid
32行目当たりのsource.getEquipmentですが、sourceはPlayerです。PlayerにはgetEquipmentというメソッドはありません。
他のところと同じようにEquippableコンポーネントを使ってください
これ以外にもミスがあるかまでは調べてないですが誤字などは丁寧に見直してみてください
reef willow
projectileHitEntityの各種プロパティについて
`source`で投擲物を当てた本人であるエンティティを返します。
`projectile`には...
()入れるの忘れてた、教えていたのにごめんなさい…
desert schooner
わかりました
long path
プレイヤーのviewDirectionを任意の値増やしてapplyImpluseで与えてやれば良いかと
desert schooner
乗っているエンティティの取得はどうすればいいでしょうか？
desert schooner
回答ありがとうございます
しばらくの間多忙なので少し解答空くかと思いますがこれからもよろしくお願いします・・・
zenith orchid
モジュールのタイプが"data"になっていますが、ScriptAPIを使用したいなら"script"である必要がありますよ
tawny granite
エラーは出ていますか
desert schooner
特に出ていないですね
DMで実際のアドオン送りましょうか？
（確かDMで送るのはよかったはず）
不要コード掃除したものになります
コード書きなおしたほうが早いですかね？
desert schooner
https://103.gigafile.nu/0511-437a635466102f3b76449d3498cb4c35
銃アドオンのバグ収めた映像です
このような形で連射が起きてしまいどのような改良を行ってもうまく作動しません
原因や修正方法等教えていだ焚けないでしょうか？
エラーログです
long path
エラーを見て順番に潰していきましょう
desert schooner
エラーを見て順番に潰していきましょう
アイテム関連のエラーを直したのですがまだ異常動作してますね・・・
アイテムへのDPの書き込みがうまく動いてないみたいです・・・
desert schooner
アイテムの使用をやめたって処理もうまく動いてないみたいです・・・
desert schooner
＝マイクソ案件説出てきた
undone nova
コードが謎いのでマイクソ案件ではないと思います...
desert schooner
もともと1.19.x系で書いたコードで1.20.x系になってから動かなくなってるんですよね・・・
undone nova
まだ同じエラーが出てるんですか？
desert schooner
https://103.gigafile.nu/0511-437a635466102f3b76449d3498cb4c35
銃アドオンのバグ収めた映像です
スプリクトのエラーは全解消したのですが1.20系アプデ前と動作順序が変わったのかこの動画のようになってしまいます
undone nova
動画見れないですね...
undone nova
動作順序が不安定なものに依存していた場合もあるので書き直した方が速いかもしれないです
(コードの全貌が見えづらいので何とも言えないです)
desert schooner
一応こちらが全貌になります
順番に実行したい場合ってどうすればいいですか？