Jump to content

storing skills per player?


Atijaf

Recommended Posts

Currently, I am testing how to store specific skill levels and xp in classes.  I know this won't be saved to a file and that it will be lost on restart.(Learning this will be later)

 

What I'm currently trying to figure out is how to store levels for each player.  Right now, if I level up, everyone else does too.

Would I create an instance of a class when a player joins? (This is the only thing I can think of)

 

Feel free to ask me questions if you need clarification on what I'm trying to figure out.

Any help will be greatly appreciated.  Thanks

Link to comment
Share on other sites

Thanks for such a quick reply.

 

I Implemented IExtendedEntityProperties, but am unsure how to differentiate between each instance each player has.

 

 

public class WoodCutting implements IExtendedEntityProperties{

public int level;

public int xp;

 

public WoodCutting(){

this.level = 1;

this.xp = 0;

}

 

private int getLevel(){

return this.level;

}

 

private int getXp(){

return this.xp;

}

 

private void addXp(int amount){

this.xp += amount;

}

 

 

public void giveXp(int amount, EntityPlayer player){

addXp(amount);

while(Experience.RequiredXp(this.level, this.xp)){

this.level++;

CustomMessage.SendPlayerLevelUpMessage(level, "WoodCutting", player);

}

}

 

@Override

public void saveNBTData(NBTTagCompound compound) {

// TODO Auto-generated method stub

 

}

@Override

public void loadNBTData(NBTTagCompound compound) {

// TODO Auto-generated method stub

 

}

@Override

public void init(Entity entity, World world) {

// TODO Auto-generated method stub

 

}

}

 

Link to comment
Share on other sites

The tricky part is how to associate it to a player so it persists after you die or leave game. A lot of people don't realize it, but I'm pretty sure the EntityPlayer instance is made new every time you join the server. Its association to a username is temporary for the session. The server does save the extended properties, but from a programming point of view that information isn't loaded into same instance, but rather a fresh instance which doesn't check username before loading.

 

So in CoolAlias' tutorial, he creates the extended properties such that it contains a hashmap of extended properties keyed to the username. In other words, the extended properties will first contain extended properties for the first username that plays, then it will add if/when other players play.

 

Anyway, that's the part to pay attention to, and also the reason why just making extended entity properties for players is a bit complicated.

 

Another note: you don't have to worry about the persistence of the extended properties if the properties are supposed to be used for temporary purposes within game. For example, if you had an extended property for "mana", you might decide it is easier to allow user to start every session with full mana bar instead of remembering it depending on your mod's gameplay concept.

Check out my tutorials here: http://jabelarminecraft.blogspot.com/

Link to comment
Share on other sites

@jabelar Actually, that part of the tutorial is deprecated. There is a much better way to handle it, and that is PlayerEvent.Clone. This is fired each time a new player instance is created and provides you with both the old instance and the new one, giving you the perfect opportunity to copy whatever data you had in your IEEP class to the new instance.

 

The only thing to keep in mind at that point is that if the client needs to be aware of that data, you still need to send a packet after cloning, and the best place (imo) to do so is in EntityJoinWorldEvent.

Link to comment
Share on other sites

@jabelar Actually, that part of the tutorial is deprecated. There is a much better way to handle it, and that is PlayerEvent.Clone. This is fired each time a new player instance is created and provides you with both the old instance and the new one, giving you the perfect opportunity to copy whatever data you had in your IEEP class to the new instance.

 

Cool. But that still supports my point -- that it is a new instance that needs to get stuff copied over from previous. I've seen a lot of posts in past couple days (like first response on this thread) where it is clear that some people don't realize that programmatically it is a new/clone instance.

 

 

Check out my tutorials here: http://jabelarminecraft.blogspot.com/

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Unfortunately, your content contains terms that we do not allow. Please edit your content to remove the highlighted words below.
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Announcements



  • Recently Browsing

    • No registered users viewing this page.
  • Posts

    • Hello everyone, I'm making this post to seek help for my modded block, It's a special block called FrozenBlock supposed to take the place of an old block, then after a set amount of ticks, it's supposed to revert its Block State, Entity, data... to the old block like this :  The problem I have is that the system breaks when handling multi blocks (I tried some fix but none of them worked) :  The bug I have identified is that the function "setOldBlockFields" in the item's "setFrozenBlock" function gets called once for the 1st block of multiblock getting frozen (as it should), but gets called a second time BEFORE creating the first FrozenBlock with the data of the 1st block, hence giving the same data to the two FrozenBlock :   Old Block Fields set BlockState : Block{minecraft:black_bed}[facing=east,occupied=false,part=head] BlockEntity : net.minecraft.world.level.block.entity.BedBlockEntity@73681674 BlockEntityData : id:"minecraft:bed",x:3,y:-60,z:-6} Old Block Fields set BlockState : Block{minecraft:black_bed}[facing=east,occupied=false,part=foot] BlockEntity : net.minecraft.world.level.block.entity.BedBlockEntity@6d1aa3da BlockEntityData : {id:"minecraft:bed",x:2,y:-60,z:-6} Frozen Block Entity set BlockState : Block{minecraft:black_bed}[facing=east,occupied=false,part=foot] BlockPos{x=3, y=-60, z=-6} BlockEntity : net.minecraft.world.level.block.entity.BedBlockEntity@6d1aa3da BlockEntityData : {id:"minecraft:bed",x:2,y:-60,z:-6} Frozen Block Entity set BlockState : Block{minecraft:black_bed}[facing=east,occupied=false,part=foot] BlockPos{x=2, y=-60, z=-6} BlockEntity : net.minecraft.world.level.block.entity.BedBlockEntity@6d1aa3da BlockEntityData : {id:"minecraft:bed",x:2,y:-60,z:-6} here is the code inside my custom "freeze" item :    @Override     public @NotNull InteractionResult useOn(@NotNull UseOnContext pContext) {         if (!pContext.getLevel().isClientSide() && pContext.getHand() == InteractionHand.MAIN_HAND) {             BlockPos blockPos = pContext.getClickedPos();             BlockPos secondBlockPos = getMultiblockPos(blockPos, pContext.getLevel().getBlockState(blockPos));             if (secondBlockPos != null) {                 createFrozenBlock(pContext, secondBlockPos);             }             createFrozenBlock(pContext, blockPos);             return InteractionResult.SUCCESS;         }         return super.useOn(pContext);     }     public static void createFrozenBlock(UseOnContext pContext, BlockPos blockPos) {         BlockState oldState = pContext.getLevel().getBlockState(blockPos);         BlockEntity oldBlockEntity = oldState.hasBlockEntity() ? pContext.getLevel().getBlockEntity(blockPos) : null;         CompoundTag oldBlockEntityData = oldState.hasBlockEntity() ? oldBlockEntity.serializeNBT() : null;         if (oldBlockEntity != null) {             pContext.getLevel().removeBlockEntity(blockPos);         }         BlockState FrozenBlock = setFrozenBlock(oldState, oldBlockEntity, oldBlockEntityData);         pContext.getLevel().setBlockAndUpdate(blockPos, FrozenBlock);     }     public static BlockState setFrozenBlock(BlockState blockState, @Nullable BlockEntity blockEntity, @Nullable CompoundTag blockEntityData) {         BlockState FrozenBlock = BlockRegister.FROZEN_BLOCK.get().defaultBlockState();         ((FrozenBlock) FrozenBlock.getBlock()).setOldBlockFields(blockState, blockEntity, blockEntityData);         return FrozenBlock;     }  
    • It is an issue with quark - update it to this build: https://www.curseforge.com/minecraft/mc-mods/quark/files/3642325
    • Remove Instant Massive Structures Mod from your server     Add new crash-reports with sites like https://paste.ee/  
    • Update your drivers: https://www.amd.com/en/support/graphics/amd-radeon-r9-series/amd-radeon-r9-200-series/amd-radeon-r9-280x
  • Topics

×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use.