Jump to content

Recommended Posts

Posted

I've implemented an energy-cable-system in my mod, and I'm having trouble with the communication between the two machines on either side of a cable.
I've made a generator, "SolarPanel", and a consumer, "Pulverizer". They both work perfectly, transferring generated energy from the solar panel to the pulverizer through the cable.
(The cables forward what the machine on the opposite end of the cables is, to the the machine on the first end)

However, I'm now implementing a second consumer, "ChargingStation", which is a battery-bank of sorts. It has an internal storage of energy.
When my SolarPanel tries to use the ChargingStations increaseCharge()-method, the method executes, but the result; a higher energy-value in the storage, does not get recorded.
In other words: if I'm looking at the GUI of the ChargingStation, I can see the energy in the storage increase, but as soon as I close (and reopen) the GUI, the value resets to 0.

I assumed this was because of client/server mismatch, so I made the call of increaseCharge() from the SolarPanel check for !isRemote(), which solved the problem, but only if the ChargingStation is "initialized" by having it use increaseCharge() itself (which is done when the ChargingStation tries to charge using a battery in its inventory.) Otherwise, it never starts charging to begin with.

Any help on how to fix this would be awesome. Thank you.

Code:

ChargingStation: https://github.com/44tim44/BetterThanElectricity_1.12.2/blob/master/src/main/java/se/sst_55t/betterthanelectricity/block/chargingstation/TileEntityChargingStation.java

SolarPanel: https://github.com/44tim44/BetterThanElectricity_1.12.2/blob/master/src/main/java/se/sst_55t/betterthanelectricity/block/solarpanel/TileEntitySolarPanel.java

Cable: https://github.com/44tim44/BetterThanElectricity_1.12.2/blob/master/src/main/java/se/sst_55t/betterthanelectricity/block/cable/TileEntityCable.java

Posted

My immediate guess is that you are synchronizing the value to the client.

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

Ok, I misread at first. I thought you said you couldn't see the value updating.

One minute...

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

Update:
If I use

if(!world.isRemote){
	chargingStation.increaseCharge();
}

then everything works, except it won't start unless the SOLARPANEL is initialized, by placing a battery in the solarpanels charging-slot

Posted

Thing I noticed as I was perusing your code to see what it does:

1) Why are you asking an item stack what the charge time is when sending energy to a TE?

https://github.com/44tim44/BetterThanElectricity_1.12.2/blob/master/src/main/java/se/sst_55t/betterthanelectricity/block/solarpanel/TileEntitySolarPanel.java#L62

2) Why are you allowing for a null item stack? Item stacks are never null in 1.12

https://github.com/44tim44/BetterThanElectricity_1.12.2/blob/master/src/main/java/se/sst_55t/betterthanelectricity/block/solarpanel/TileEntitySolarPanel.java#L85

3) Not that getItemChargeTime gives a shit about the item passed to it anyway...

4) Why is IBattery not an IChargable?

https://github.com/44tim44/BetterThanElectricity_1.12.2/blob/master/src/main/java/se/sst_55t/betterthanelectricity/block/solarpanel/TileEntitySolarPanel.java#L36

These two interfaces even supply the same method:

https://github.com/44tim44/BetterThanElectricity_1.12.2/blob/master/src/main/java/se/sst_55t/betterthanelectricity/block/solarpanel/TileEntitySolarPanel.java#L45-L51

5) Why are you using Interfaces at all? This is why capabilities exist. Forge has even supplied a capability definition for Energy.

6) This else return false is extraneous.

https://github.com/44tim44/BetterThanElectricity_1.12.2/blob/master/src/main/java/se/sst_55t/betterthanelectricity/block/solarpanel/TileEntitySolarPanel.java#L121

7) This boolean does nothing

https://github.com/44tim44/BetterThanElectricity_1.12.2/blob/master/src/main/java/se/sst_55t/betterthanelectricity/block/solarpanel/TileEntitySolarPanel.java#L42

 

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:

Thing I noticed as I was perusing your code to see what it does:

1) Why are you asking an item stack what the charge time is when sending energy to a TE?

https://github.com/44tim44/BetterThanElectricity_1.12.2/blob/master/src/main/java/se/sst_55t/betterthanelectricity/block/solarpanel/TileEntitySolarPanel.java#L62

2) Why are you allowing for a null item stack? Item stacks are never null in 1.12

https://github.com/44tim44/BetterThanElectricity_1.12.2/blob/master/src/main/java/se/sst_55t/betterthanelectricity/block/solarpanel/TileEntitySolarPanel.java#L85

3) Not that getItemChargeTime gives a shit about the item passed to it anyway...

4) Why is IBattery not an IChargable?

https://github.com/44tim44/BetterThanElectricity_1.12.2/blob/master/src/main/java/se/sst_55t/betterthanelectricity/block/solarpanel/TileEntitySolarPanel.java#L36

These two interfaces even supply the same method:

https://github.com/44tim44/BetterThanElectricity_1.12.2/blob/master/src/main/java/se/sst_55t/betterthanelectricity/block/solarpanel/TileEntitySolarPanel.java#L45-L51

5) Why are you using Interfaces at all? This is why capabilities exist. Forge has even supplied a capability definition for Energy.

6) This else return false is extraneous.

https://github.com/44tim44/BetterThanElectricity_1.12.2/blob/master/src/main/java/se/sst_55t/betterthanelectricity/block/solarpanel/TileEntitySolarPanel.java#L121

7) This boolean does nothing

https://github.com/44tim44/BetterThanElectricity_1.12.2/blob/master/src/main/java/se/sst_55t/betterthanelectricity/block/solarpanel/TileEntitySolarPanel.java#L42

 

1) Code is still WIP, and a lot of the code is copypasted from when I was only using batteries, and not cables. (Will be fixed)
2) See 1) The nullable is just an addon, so I don't have to pass an Itemstack to get the chargeRate.

3) See 1)
4) Because an Battery can give and take Charge, IChargeable can only take. (In the logic of my machines)
5) Because whenever I've tried to learn capabilities, I've run into troubles. I had troubles with capabilities attached to itemstacks not syncing properly between server and client. (I know I know, I should learn how to use capabilities properly, but I haven't yet. And this works in the meantime.)
6) Will fix.
7) Old code not cleaned up properly.

Posted (edited)

4) In that case, IBattery should extend IChargable or you have to interfaces IChargable and IDrainalbe (or whatever) and a battery just implements both.

 

But in any case, you should really learn to use capabilities. It really isn't that difficult. This is pretty much all you need:

https://github.com/Draco18s/ReasonableRealism/blob/master/src/main/java/com/draco18s/hardlib/api/capability/RawMechanicalPowerHandler.java

Only instead of IMechanicalPower you'll use whatever the interface is that Forge supplies for Energy.

You'd then set up some kind of object for the energy storage (look at the ItemStackHandler for inspiration on how to handle it) and then supply that object instance via getCapability.

 

You might need this, I'm not sure:

https://github.com/Draco18s/ReasonableRealism/blob/master/src/main/java/com/draco18s/hardlib/api/capability/SimpleCapabilityProvider.java

Edited by Draco18s
  • Like 1

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

Yeah, I'll try to learn capabilities.
But, I'd rather start from a point where my code is already working, so I don't run into trouble.

I do think I found the problem myself though: the variable totalChargeTime in TESolarPanel is not initialized unless an item is place in the charging slot in ContainerSolarPanel.
Which is a bit stupid, since as you said, I don't even use the passed itemstack in getItemChargeTime() to calculate totalChargeTime.

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

    • logs too big for one pastebin https://pastebin.com/ZjUGHu3u  https://pastebin.com/RqCUZf3X  https://pastebin.com/6ZPS99nD
    • You probably used jd-gui to open it, didn't you? Nothing wrong with that, I also made that mistake, except that Notch was a smart guy and he obfuscated the code. That's why you only see files called "a", "b", "c" and then a file that combines them all. As I said, use RetroMCP to deobfuscate the code so that you will 100% understand it and be able to navigate it.
    • Decompiling minecraft indev, infdev, alpha, beta or whichever legacy version is really easy. I'm not a plug, I just also got interested in modding legacy versions (Infdev to be specific). Use https://github.com/MCPHackers/RetroMCP-Java Once you install their client and the Zulu Architecture that they say they recommend (or use your own Java). I encountered some problems, so I run it with: "java -jar RetroMCP-Java-CLI.jar". You should run it in a seperate folder (not in downloads), otherwise the files and folders will go all over the place. How to use RetroMCP: Type setup (every time you want change version), copy-paste the version number from their list (they support indev), write "decompile" and done! The code will now be deobfuscated and filenames will be normal, instead of "a", "b" and "c"! Hope I helped you, but I don't expect you to reply, as this discussion is 9 years old! What a piece of history!  
    • I know that this may be a basic question, but I am very new to modding. I am trying to have it so that I can create modified Vanilla loot tables that use a custom enchantment as a condition (i.e. enchantment present = item). However, I am having trouble trying to implement this; the LootItemRandomChanceWithEnchantedBonusCondition constructor needs a Holder<Enchantment> and I am unable to use the getOrThrow() method on the custom enchantment declared in my mod's enchantments class. Here is what I have so far in the GLM:   protected void start(HolderLookup.Provider registries) { HolderLookup.RegistryLookup<Enchantment> registrylookup = registries.lookupOrThrow(Registries.ENCHANTMENT); LootItemRandomChanceWithEnchantedBonusCondition lootItemRandomChanceWithEnchantedBonusCondition = new LootItemRandomChanceWithEnchantedBonusCondition(0.0f, LevelBasedValue.perLevel(0.07f), registrylookup.getOrThrow(*enchantment here*)); this.add("nebu_from_deepslate", new AddItemModifier(new LootItemCondition[]{ LootItemBlockStatePropertyCondition.hasBlockStateProperties(Blocks.DEEPSLATE).build(), LootItemRandomChanceCondition.randomChance(0.25f).build(), lootItemRandomChanceWithEnchantedBonusCondition }, OrichalcumItems.NEBU.get())); }   Inserting Enchantments.[vanilla enchantment here] actually works but trying to declare an enchantment from my custom enchantments class as [mod enchantment class].[custom enchantment] does not work even though they are both a ResourceKey and are registered in Registries.ENCHANTMENT. Basically, how would I go about making it so that a custom enchantment declared as a ResourceKey<Enchantment> of value ResourceKey.create(Registries.ENCHANTMENT, ResourceLocation.fromNamespaceAndPath([modid], [name])), declared in a seperate enchantments class, can be used in the LootItemRandomChanceWithEnchantedBonusCondition constructor as a Holder? I can't use getOrThrow() because there is no level or block entity/entity in the start() method and it is running as datagen. It's driving me nuts.
  • Topics

×
×
  • Create New...

Important Information

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