Jump to content

Recommended Posts

Posted (edited)

Hi,

 

I'm wondering how I can work around an NPE caused by tile.hasCapability(CapabilityEnergy.ENERGY, (EnumFacing) null)

The code there should actually check if the TileEntity has the capability at all without taking care of the sides. (thus all sides are checked first and then the side null at last)

 

Reasons for having the null check there:

On 4/19/2017 at 10:50 PM, diesieben07 said:
  • You might want to call getCapability with side null, too.
On 4/19/2017 at 11:22 PM, diesieben07 said:

Because the capability might be exposed with side null and not any specific side. In that case your pipe would ignore that capability.

 

 

Thx in advance.

Bektor

Edited by Bektor

Developer of Primeval Forest.

Posted

*Looks for code*

*Fails to find code*

*Hits "back" on browser*

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

Posted
3 minutes ago, Draco18s said:

*Looks for code*

*Fails to find code*

*Hits "back" on browser*

I just copied the one line into the post, but here is the complete method. Arguments used: the tilenentity, null

    public static boolean hasCapability(TileEntity tile, EnumFacing orientation) {
    	EnumFacing side = orientation != null ? orientation.getOpposite() : (EnumFacing) null;
    	
    	if(tile.hasCapability(CapabilityEnergy.ENERGY, orientation)) return true;
    	else if(TeslaHelper.isEnergyHolder(tile, side)) return true;
    	else if(TeslaHelper.isEnergyProducer(tile, side)) return true;
    	else if(TeslaHelper.isEnergyReceiver(tile, side)) return true;
    	
    	return false;
    }

Developer of Primeval Forest.

Posted
14 minutes ago, Bektor said:

I just copied the one line into the post, but here is the complete method. Arguments used: the tilenentity, null


    public static boolean hasCapability(TileEntity tile, EnumFacing orientation) {
    	EnumFacing side = orientation != null ? orientation.getOpposite() : (EnumFacing) null;
    	
    	if(tile.hasCapability(CapabilityEnergy.ENERGY, orientation)) return true;
    	else if(TeslaHelper.isEnergyHolder(tile, side)) return true;
    	else if(TeslaHelper.isEnergyProducer(tile, side)) return true;
    	else if(TeslaHelper.isEnergyReceiver(tile, side)) return true;
    	
    	return false;
    }

 

Uh...

if(tile.hasCapability(CapabilityEnergy.ENERGY, orientation)) return true;

Let me dive into that...

orientation.getOpposite()

if(tile.hasCapability(CapabilityEnergy.ENERGY, orientation)) return true;

Let me dive into that...

orientation.getOpposite()

if(tile.hasCapability(CapabilityEnergy.ENERGY, orientation)) return true;

Let me dive into that...

orientation.getOpposite()

if(tile.hasCapability(CapabilityEnergy.ENERGY, orientation)) return true;

Let me dive into that...

orientation.getOpposite()

if(tile.hasCapability(CapabilityEnergy.ENERGY, orientation)) return true;

Let me dive into that...

orientation.getOpposite()

if(tile.hasCapability(CapabilityEnergy.ENERGY, orientation)) return true;

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

Posted

Where is the exception being thrown?

 

Is it being thrown on the line that calls TileEntity#hasCapability or is it being thrown inside TileEntity#hasCapability?

 

Post the stacktrace.

Please don't PM me to ask for help. Asking your question in a public thread preserves it for people who are having the same problem in the future.

Posted
6 minutes ago, Draco18s said:

 

Uh...

if(tile.hasCapability(CapabilityEnergy.ENERGY, orientation)) return true;

Let me dive into that...

orientation.getOpposite()

if(tile.hasCapability(CapabilityEnergy.ENERGY, orientation)) return true;

Let me dive into that...

orientation.getOpposite()

if(tile.hasCapability(CapabilityEnergy.ENERGY, orientation)) return true;

Let me dive into that...

orientation.getOpposite()

if(tile.hasCapability(CapabilityEnergy.ENERGY, orientation)) return true;

Let me dive into that...

orientation.getOpposite()

if(tile.hasCapability(CapabilityEnergy.ENERGY, orientation)) return true;

Let me dive into that...

orientation.getOpposite()

if(tile.hasCapability(CapabilityEnergy.ENERGY, orientation)) return true;

Ok.... wtf have I done there.... Must have been one of these midnight programming nights. ^^

Developer of Primeval Forest.

Posted

You need to post the full code context, and the exceptions.

We dont know whats erroing if you dont give us the context.

As for @Draco18s Its not recursive, thats a static function.

But yes we dont have enough information, so post your logs, post your code, post everything even remotely related.

I do Forge for free, however the servers to run it arn't free, so anything is appreciated.
Consider supporting the team on Patreon

Posted
1 minute ago, LexManos said:

As for @Draco18s Its not recursive, thats a static function.

But yes we dont have enough information, so post your logs, post your code, post everything even remotely related.

You're right, I assumed it was the tile's own hasCap function.

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

Posted (edited)
16 hours ago, Draco18s said:

You're right, I assumed it was the tile's own hasCap function.

It wasn't the tiles own capability function. It is a method which should check if a given tile entity has the capability at the given orientation.

The method is used when for example a cable or a machine or whatever wants the send energy, but beforehand make sure that the block

to which the energy should be send even got the capability. 

Thought in your second answer I noticed that some variable moved to some place it was never supposed to be in the first place.

The variable side should be used in the if-statement like in all other statements, but it was not which resulted into the crash

 as the variable orientation got never casted to EnumFacing if it was null. Thought in my opinion null is still null, but without the cast it seems

like it is not quite null (I don't know the internal processing there).

So using the side variable which should have been used (and probalby was before some internal code iterations) solves the problem.

Edited by Bektor

Developer of Primeval Forest.

Posted
25 minutes ago, Bektor said:

Thought in my opinion null is still null, but without the cast it seems like it is not quite null (I don't know the internal processing there).

The casting is there for the compiler. Null is still null, but because null can "be" anything, you have to cast it to a type.

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

Posted
1 minute ago, Draco18s said:

The casting is there for the compiler. Null is still null, but because null can "be" anything, you have to cast it to a type.

What do you mean by null is there for the compiler? I am wondering why the compiler just doesn't cast it then automatically to the correct type

(I know C++ can do this, but don't ask me in which cases C++ will do this automatically ^^)

Also if null can be anything, how does it work to check if an object is null or not if it can be anything. Thought I'm wondering what this in memory actually

is if it can be anything. Will the PC just say, ok there you go, take this bit of RAM with whatever might be left in there and let it be null.

Developer of Primeval Forest.

Posted
13 minutes ago, diesieben07 said:

If you store an object in a variable, you are not storing that object, you are storing a reference to that object

Which sounds like C++ and not Java. ^^

13 minutes ago, diesieben07 said:

null is just a fancy name for "the address 0".

meaning nullptr

 

I somehow have the feeling that all this Java pointers do not exists there and all of that stuff just makes it more

complicated to understand. xD

In my brain its just somehow programmed: Java "no" real pointers, C++ pointers

Developer of Primeval Forest.

Posted
Just now, diesieben07 said:

Java is not magic. It does the same thing under the hood, it just hides it in a layer of abstraction and type-safety checks.

 

Yeah, I know. But if feels somehow as if this makes it more complicated to understand the null stuff.

I mean, I know that null has to be casted in this case. Thought I never really thought about C++ nullptr.

It's there and that's all I've ever thought about. What it is and all that stuff is the same with Java null, I've

just used it like null means nothing assigned to the value. What this nothing is, that's something I've never learned

in school nor is it written in any of my Java books and C++ nullptr, might be that it is better described when I

get to the point where it is explained. ^^

 

6 minutes ago, diesieben07 said:

Java stuff under the hood uses the same pointers C or C++ does.

Which makes sense as if I recall correctly the Java compiler is written in C++ and code of the java API itself references through JNI to C++ stuff?

Thought I know for sure that LWJGL does this.

I guess when I've got more time I'll look into VS memory display to see what nullptr is there exactly.

Developer of Primeval Forest.

Posted (edited)
18 minutes ago, diesieben07 said:

Java variables that store objects are references

Yeah, but all of this null stuff. I've got the feeling you know more about it then the ones who wrote the books I read a long time ago. (Starting with some Java for kids books and later books for adults)

I guess it is better explained in books like "Java ist auch eine Insel", but I've never read those books as most of the Java features those books covered I already learned

and I got interested in programming and thus started learning Java even before I could understand English or played Minecraft.

Edited by Bektor

Developer of Primeval Forest.

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

    • Version 1.19 - Forge 41.0.63 I want to create a wolf entity that I can ride, so far it seems to be working, but the problem is that when I get on the wolf, I can’t control it. I then discovered that the issue is that the server doesn’t detect that I’m riding the wolf, so I’m struggling with synchronization. However, it seems to not be working properly. As I understand it, the server receives the packet but doesn’t register it correctly. I’m a bit new to Java, and I’ll try to provide all the relevant code and prints *The comments and prints are translated by chatgpt since they were originally in Spanish* Thank you very much in advance No player is mounted, or the passenger is not a player. No player is mounted, or the passenger is not a player. No player is mounted, or the passenger is not a player. No player is mounted, or the passenger is not a player. No player is mounted, or the passenger is not a player. MountableWolfEntity package com.vals.valscraft.entity; import com.vals.valscraft.network.MountSyncPacket; import com.vals.valscraft.network.NetworkHandler; import net.minecraft.client.Minecraft; import net.minecraft.network.syncher.EntityDataAccessor; import net.minecraft.network.syncher.EntityDataSerializers; import net.minecraft.network.syncher.SynchedEntityData; import net.minecraft.server.MinecraftServer; import net.minecraft.server.level.ServerPlayer; import net.minecraft.world.entity.EntityType; import net.minecraft.world.entity.Mob; import net.minecraft.world.entity.ai.attributes.AttributeSupplier; import net.minecraft.world.entity.ai.attributes.Attributes; import net.minecraft.world.entity.animal.Wolf; import net.minecraft.world.entity.player.Player; import net.minecraft.world.entity.Entity; import net.minecraft.world.InteractionHand; import net.minecraft.world.InteractionResult; import net.minecraft.world.item.ItemStack; import net.minecraft.world.item.Items; import net.minecraft.world.level.Level; import net.minecraft.world.phys.Vec3; import net.minecraftforge.event.TickEvent; import net.minecraftforge.eventbus.api.SubscribeEvent; import net.minecraftforge.network.PacketDistributor; public class MountableWolfEntity extends Wolf { private boolean hasSaddle; private static final EntityDataAccessor<Byte> DATA_ID_FLAGS = SynchedEntityData.defineId(MountableWolfEntity.class, EntityDataSerializers.BYTE); public MountableWolfEntity(EntityType<? extends Wolf> type, Level level) { super(type, level); this.hasSaddle = false; } @Override protected void defineSynchedData() { super.defineSynchedData(); this.entityData.define(DATA_ID_FLAGS, (byte)0); } public static AttributeSupplier.Builder createAttributes() { return Wolf.createAttributes() .add(Attributes.MAX_HEALTH, 20.0) .add(Attributes.MOVEMENT_SPEED, 0.3); } @Override public InteractionResult mobInteract(Player player, InteractionHand hand) { ItemStack itemstack = player.getItemInHand(hand); if (itemstack.getItem() == Items.SADDLE && !this.hasSaddle()) { if (!player.isCreative()) { itemstack.shrink(1); } this.setSaddle(true); return InteractionResult.SUCCESS; } else if (!level.isClientSide && this.hasSaddle()) { player.startRiding(this); MountSyncPacket packet = new MountSyncPacket(true); // 'true' means the player is mounted NetworkHandler.CHANNEL.sendToServer(packet); // Ensure the server handles the packet return InteractionResult.SUCCESS; } return InteractionResult.PASS; } @Override public void travel(Vec3 travelVector) { if (this.isVehicle() && this.getControllingPassenger() instanceof Player) { System.out.println("The wolf has a passenger."); System.out.println("The passenger is a player."); Player player = (Player) this.getControllingPassenger(); // Ensure the player is the controller this.setYRot(player.getYRot()); this.yRotO = this.getYRot(); this.setXRot(player.getXRot() * 0.5F); this.setRot(this.getYRot(), this.getXRot()); this.yBodyRot = this.getYRot(); this.yHeadRot = this.yBodyRot; float forward = player.zza; float strafe = player.xxa; if (forward <= 0.0F) { forward *= 0.25F; } this.flyingSpeed = this.getSpeed() * 0.1F; this.setSpeed((float) this.getAttributeValue(Attributes.MOVEMENT_SPEED) * 1.5F); this.setDeltaMovement(new Vec3(strafe, travelVector.y, forward).scale(this.getSpeed())); this.calculateEntityAnimation(this, false); } else { // The wolf does not have a passenger or the passenger is not a player System.out.println("No player is mounted, or the passenger is not a player."); super.travel(travelVector); } } public boolean hasSaddle() { return this.hasSaddle; } public void setSaddle(boolean hasSaddle) { this.hasSaddle = hasSaddle; } @Override protected void dropEquipment() { super.dropEquipment(); if (this.hasSaddle()) { this.spawnAtLocation(Items.SADDLE); this.setSaddle(false); } } @SubscribeEvent public static void onServerTick(TickEvent.ServerTickEvent event) { if (event.phase == TickEvent.Phase.START) { MinecraftServer server = net.minecraftforge.server.ServerLifecycleHooks.getCurrentServer(); if (server != null) { for (ServerPlayer player : server.getPlayerList().getPlayers()) { if (player.isPassenger() && player.getVehicle() instanceof MountableWolfEntity) { MountableWolfEntity wolf = (MountableWolfEntity) player.getVehicle(); System.out.println("Tick: " + player.getName().getString() + " is correctly mounted on " + wolf); } } } } } private boolean lastMountedState = false; @Override public void tick() { super.tick(); if (!this.level.isClientSide) { // Only on the server boolean isMounted = this.isVehicle() && this.getControllingPassenger() instanceof Player; // Only print if the state changed if (isMounted != lastMountedState) { if (isMounted) { Player player = (Player) this.getControllingPassenger(); // Verify the passenger is a player System.out.println("Server: Player " + player.getName().getString() + " is now mounted."); } else { System.out.println("Server: The wolf no longer has a passenger."); } lastMountedState = isMounted; } } } @Override public void addPassenger(Entity passenger) { super.addPassenger(passenger); if (passenger instanceof Player) { Player player = (Player) passenger; if (!this.level.isClientSide && player instanceof ServerPlayer) { // Send the packet to the server to indicate the player is mounted NetworkHandler.CHANNEL.send(PacketDistributor.PLAYER.with(() -> (ServerPlayer) player), new MountSyncPacket(true)); } } } @Override public void removePassenger(Entity passenger) { super.removePassenger(passenger); if (passenger instanceof Player) { Player player = (Player) passenger; if (!this.level.isClientSide && player instanceof ServerPlayer) { // Send the packet to the server to indicate the player is no longer mounted NetworkHandler.CHANNEL.send(PacketDistributor.PLAYER.with(() -> (ServerPlayer) player), new MountSyncPacket(false)); } } } @Override public boolean isControlledByLocalInstance() { Entity entity = this.getControllingPassenger(); return entity instanceof Player; } @Override public void positionRider(Entity passenger) { if (this.hasPassenger(passenger)) { double xOffset = Math.cos(Math.toRadians(this.getYRot() + 90)) * 0.4; double zOffset = Math.sin(Math.toRadians(this.getYRot() + 90)) * 0.4; passenger.setPos(this.getX() + xOffset, this.getY() + this.getPassengersRidingOffset() + passenger.getMyRidingOffset(), this.getZ() + zOffset); } } } MountSyncPacket package com.vals.valscraft.network; import com.vals.valscraft.entity.MountableWolfEntity; import net.minecraft.network.FriendlyByteBuf; import net.minecraft.server.level.ServerLevel; import net.minecraft.server.level.ServerPlayer; import net.minecraft.world.entity.Entity; import net.minecraft.world.entity.player.Player; import net.minecraftforge.network.NetworkEvent; import java.util.function.Supplier; public class MountSyncPacket { private final boolean isMounted; public MountSyncPacket(boolean isMounted) { this.isMounted = isMounted; } public void encode(FriendlyByteBuf buffer) { buffer.writeBoolean(isMounted); } public static MountSyncPacket decode(FriendlyByteBuf buffer) { return new MountSyncPacket(buffer.readBoolean()); } public void handle(NetworkEvent.Context context) { context.enqueueWork(() -> { ServerPlayer player = context.getSender(); // Get the player from the context if (player != null) { // Verifies if the player has dismounted if (!isMounted) { Entity vehicle = player.getVehicle(); if (vehicle instanceof MountableWolfEntity wolf) { // Logic to remove the player as a passenger wolf.removePassenger(player); System.out.println("Server: Player " + player.getName().getString() + " is no longer mounted."); } } } }); context.setPacketHandled(true); // Marks the packet as handled } } networkHandler package com.vals.valscraft.network; import com.vals.valscraft.valscraft; import net.minecraft.resources.ResourceLocation; import net.minecraftforge.network.NetworkRegistry; import net.minecraftforge.network.simple.SimpleChannel; import net.minecraftforge.network.NetworkEvent; import java.util.function.Supplier; public class NetworkHandler { private static final String PROTOCOL_VERSION = "1"; public static final SimpleChannel CHANNEL = NetworkRegistry.newSimpleChannel( new ResourceLocation(valscraft.MODID, "main"), () -> PROTOCOL_VERSION, PROTOCOL_VERSION::equals, PROTOCOL_VERSION::equals ); public static void init() { int packetId = 0; // Register the mount synchronization packet CHANNEL.registerMessage( packetId++, MountSyncPacket.class, MountSyncPacket::encode, MountSyncPacket::decode, (msg, context) -> msg.handle(context.get()) // Get the context with context.get() ); } }  
    • Do you use features of inventory profiles next (ipnext) or is there a change without it?
    • Remove rubidium - you are already using embeddium, which is a fork of rubidium
  • Topics

×
×
  • Create New...

Important Information

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