Перенос NBT умершего на нового Player

[Antony_wes]

Приветствую, ищу способ как перенести nbt умершего player на нового(для того, что бы при смерти nbt переносилось). Я думаю, что ServerPlayerEvents.AfterRespawn поможет мне с этим, прописал эвент, который не работает:

public class PlayerSpawnHandler implements ServerPlayerEvents.AfterRespawn {
@Override
    public void afterRespawn(ServerPlayerEntity oldPlayer, ServerPlayerEntity newPlayer, boolean alive) {

PacketByteBuf buffer = PacketByteBufs.create();

if (oldPlayer instanceof IEntityDataSaver DatSaverPlayer){

buffer.writeInt(DatSaverPlayer.getPersistentData().getInt("maxmana"));

        }
if (newPlayer instanceof IEntityDataSaver DatSaverPlayer){
MAXMANAData.addMaxMana(DatSaverPlayer, buffer.readInt());}
ServerPlayNetworking.send(newPlayer, ModMessages.MAXMANA_SYNC_ID, buffer);

newPlayer.sendMessage(Text.literal("Maxmana sync succ"));


buffer = PacketByteBufs.create();

if (oldPlayer instanceof IEntityDataSaver DatSaverPlayer){

buffer.writeInt(DatSaverPlayer.getPersistentData().getInt("mana"));

        }
if (newPlayer instanceof IEntityDataSaver DatSaverPlayer){
ManaData.addMana(DatSaverPlayer, buffer.readInt());
        }
ServerPlayNetworking.send(newPlayer, ModMessages.MANA_SYNC_ID, buffer);

newPlayer.sendMessage(Text.literal("Mana sync Succ"));

    }
}

P.S:не ругайтесь на instanceof, я знаю что надо было бы изначально 2 переменные сделать, а потом вызывать их. Это просто прототип

public class MaxmanaSyncS2CPacket {
    public static void receive(MinecraftClient client, ClientPlayNetworkHandler handler,
                               PacketByteBuf buf, PacketSender responseSender) {
        if (client.player instanceof IEntityDataSaver playerDataSaver) {
            playerDataSaver.getPersistentData().putInt("maxmana", buf.readInt());
        }

    }
}

Хотелось бы узнать, есть ли способ покрасивее, и если нет, то помочь с этим кодом либо предложить свой.

 

[HomaPlus]

Подобные ошибки связаны с

if (oldPlayer instanceof IEntityDataSaver DatSaverPlayer){ buffer.writeInt(DatSaverPlayer.getPersistentData().getInt("mana")); } if (newPlayer instanceof IEntityDataSaver DatSaverPlayer){ ManaData.addMana(DatSaverPlayer, buffer.readInt()); } ServerPlayNetworking.send(newPlayer, ModMessages.MANA_SYNC_ID, buffer); newPlayer.sendMessage(Text.literal("Mana sync Succ")); } }

Так делать нельзя.

Если в пакет записано ровно 2 числа - то должно быть прочитано так же ровно 2 числа. Ни больше, ни меньше.
Если ты не знаешь сколько чисел считывать - то как вариант записать 1-е число (количество), и после числа, и при чтении считать количество и уже потом содержимое

 

[Antony_wes]

Если в пакет записано ровно 2 числа - то должно быть прочитано так же ровно 2 числа. Ни больше, ни меньше.

Если ты про buffer, то я в 19 строке его очищаю, видел метод clear, но либо я что то не так делаю, либо clear очищает что то другое. Если ты про те пакеты, которые я отправляю, то тут тоже самое, я считываю с старого player и отправляю к новому player, потом очищаю buffer, потом снова считываю с старого player и отправляю к новому player, попутно синхронизируя. buffer это временная переменная, в которую удобно сложить одно число и позже взять его.
Возможно я тебя не понял, такое тоже может быть, но я не понимаю, где я мог сложить в пакет 2 числа.

 

[HomaPlus]

Это не меняет суть ошибки

Так делать нельзя.

Если в пакет записано ровно 2 числа - то должно быть прочитано так же ровно 2 числа. Ни больше, ни меньше.

Гарантированно записывай в буфер ровно 1 число или ровно 2 числа, и считывай столько-же

У меня сложилось мнение, что условия 7, 12 (или 21, 26) не выполнились - в буфер ничего не записалось, ты отправил пустой буфер и пытаешься что-то читать

 

[Antony_wes]

У меня сложилось мнение, что условия 7, 12 (или 21, 26) не выполнились - в буфер ничего не записалось, ты отправил пустой буфер и пытаешься что-то читать

Возможно ты прав, тем более, что если лог читать, то не похоже, что 12 выполнился, сейчас перепишу с использованием alive и скажу работает или нет

 

[Antony_wes]

переписал таким образом:

@Override
    public void afterRespawn(ServerPlayerEntity oldPlayer, ServerPlayerEntity newPlayer, boolean alive) {

        PacketByteBuf buffer1 = PacketByteBufs.create();
        PacketByteBuf buffer2 = PacketByteBufs.create();

        if (oldPlayer instanceof IEntityDataSaver DatSaverPlayer){

            buffer1.writeInt(DatSaverPlayer.getPersistentData().getInt("maxmana"));
            buffer2.writeInt(DatSaverPlayer.getPersistentData().getInt("mana"));

        }
        if (newPlayer instanceof IEntityDataSaver DatSaverPlayer && alive){
            MAXMANAData.addMaxMana(DatSaverPlayer, buffer1.readInt());
            ManaData.addMana(DatSaverPlayer, buffer2.readInt());
        }
        ServerPlayNetworking.send(newPlayer, ModMessages.MAXMANA_SYNC_ID, buffer1);
        newPlayer.sendMessage(Text.literal("Maxmana sync succ"));
        ServerPlayNetworking.send(newPlayer, ModMessages.MANA_SYNC_ID, buffer2);
        newPlayer.sendMessage(Text.literal("Mana sync Succ"));

    }

Он не крашит майн, но и не работает как надо, mana и maxmana не переносятся

 

[Antony_wes]

Я забыл, что в addMana/addMaxMana синхронизация уже прописана, так что 17, 19 строки не нужны. Но проблемы это не решает

 

[HomaPlus]

Так делать нельзя.

Если в пакет записано ровно 2 числа - то должно быть прочитано так же ровно 2 числа. Ни больше, ни меньше.

Я имел в виду нечто подобное.
Если по каким-то причинам у тебя будет -1 при чтении - то это уже проблема на уровне бизнес-логики, а не на сетевом. Твой код ошибки - конкретно сетевой уровень

void sendSomething(Object... obj){
    PacketByteBuf buf = PacketByteBufs.create();
    int data = -1;
    
    //do something

    if(someLogick){
        data = 10;
    }
    //ровно 1 раз записывай. БЕЗ УСЛОВИЙ
    buf.writeInt(data);
}


void read(PacketByteBuf buf){
    //ровно 1 раз читай. БЕЗ УСЛОВИЙ
    int data = buf.readInt();
    
    if(data == -1){
        //do something
    } else {
        //do something
    }
}

 

[Antony_wes]

Я забыл, что в addMana/addMaxMana синхронизация уже прописана, так что 17, 19 строки не нужны. Но проблемы это не решает

Прошу прощения, я имел ввиду, что так как синхронизации не нужны, то и использование buffer тоже, так что я вполне могу использовать переменные. Но опять же, проблему с переносом nbt из умершего в нового Player это не решает. Если можешь помочь с этим, то буду очень благодарен