[1.7.10]RF Production

[NEG2013]

How do I create a simple RF producing block

[larsgerrits]

You use the API.

 

 

If you want more help, supply more info, and show what you've tried.

 

[NEG2013]

here is my Tile Entity that I tried

 

 

package neg2013.acsension.tile_entity;

import cofh.api.energy.IEnergyHandler;
import cofh.api.energy.IEnergyProvider;
import neg2013.acsension.Acsension;
import neg2013.acsension.infrastructure.BlockSolarGenny;
import net.minecraft.client.Minecraft;
import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.inventory.IInventory;
import net.minecraft.item.ItemStack;
import net.minecraft.nbt.NBTTagCompound;
import net.minecraft.tileentity.TileEntity;
import net.minecraftforge.common.util.ForgeDirection;
import net.minecraftforge.event.entity.player.PlayerUseItemEvent.Tick;

public class TESolar extends TileEntity implements IInventory, IEnergyProvider{

ItemStack[] items;



//crusher

public TESolar(){
	this.items = new ItemStack [128];
}

//crusher







//NBT NBT NBT NBT NBT NBT NBT NBT NBT NBT

@Override
public void writeToNBT(NBTTagCompound nbt) {
	super.writeToNBT(nbt);
}

@Override
public void readFromNBT(NBTTagCompound nbt) {
	super.readFromNBT(nbt);

}

//NBT NBT NBT NBT NBT NBT NBT NBT NBT NBT 







@Override
public boolean receiveClientEvent(int id, int value) {

	return super.receiveClientEvent(id, value);
}

@Override
public void updateEntity() {

	super.updateEntity();
}








//inventory

@Override
public int getSizeInventory() {

	return items.length;
}

@Override
public ItemStack getStackInSlot(int slotIn) {

	return items[slotIn];
}

@Override
public ItemStack decrStackSize(int i, int count) {

	ItemStack itemStack =  getStackInSlot(i);
	if(itemStack != null){
		if(itemStack.stackSize >= count){
			setInventorySlotContents(i, null);
		}
		else{
			itemStack = itemStack.splitStack(count);
			this.markDirty();
		}
	}


	return itemStack;
}

@Override
public ItemStack getStackInSlotOnClosing(int i) {
	ItemStack itemStack = getStackInSlot(i);
	setInventorySlotContents(i, null);
	return itemStack;
}

@Override
public void setInventorySlotContents(int i, ItemStack itemStack) {

	items[i] = itemStack;
	if(itemStack != null && itemStack.stackSize >= getInventoryStackLimit()){
		itemStack.stackSize = getInventoryStackLimit();
	}
	this.markDirty();
}

@Override
public String getInventoryName() {

	return "Solar Genny";
}

@Override
public boolean hasCustomInventoryName() {

	return true;
}

@Override
public int getInventoryStackLimit() {

	return 64;
}

@Override
public boolean isUseableByPlayer(EntityPlayer player) {



	return true;//player.getDistanceSq(xCoord + 0.5f, yCoord + 0.5f, zCoord + 0.5f) <= 64;
}

@Override
public void openInventory() {


}

@Override
public void closeInventory() {


}

@Override
public boolean isItemValidForSlot(int i, ItemStack itemStack) {

	return itemStack.getItem() == items[i].getItem();
}


//power

@Override
public boolean canConnectEnergy(ForgeDirection from) {
	return true;
}



@Override
public int extractEnergy(ForgeDirection from, int maxExtract, boolean simulate) {
	return 16;
}

@Override
public int getMaxEnergyStored(ForgeDirection from) {
	return 16;
}

int energy;

@Override
public int getEnergyStored(ForgeDirection from) {
	if(Minecraft.getMinecraft().theWorld.isDaytime()){
		if(Minecraft.getMinecraft().theWorld.canBlockSeeTheSky(this.xCoord, this.yCoord, this.zCoord)){
			return 16;
		}

	}

	return 0;

}


}

[Draco18s]

	@Override
public int getEnergyStored(ForgeDirection from) {
	if(Minecraft.getMinecraft().theWorld.isDaytime()){
		if(Minecraft.getMinecraft().theWorld.canBlockSeeTheSky(this.xCoord, this.yCoord, this.zCoord)){
			return 16;
		}

	}

	return 0;

}

 

No. Bad modder.  No cookie.

 

DO NOT USE Minecraft.getMinecraft()!

this.worldObj

! Use it!

[NEG2013]

that did nothing

[Silly511]

Make it extend TileEnergyHandler not IEnergyHandler. Then in updateEntity() check if it can see the sky, if true, then receiveEnergy.

[Draco18s]

that did nothing

 

Not for the problem you indicated, but it means that now you won't have a problem running a dedicated server.

i.e. I fixed a problem you didn't even know you had and averted the creation of yet another "Crash: ClassNotFoundException on Dedicated Server" thread.

[NEG2013]

Ive redone it like 20 time snow so heres the class as is currently

 

package neg2013.acsension.tile_entity;

import cofh.api.energy.EnergyStorage;
import cofh.api.energy.IEnergyHandler;
import cofh.api.energy.IEnergyProvider;
import cofh.api.energy.TileEnergyHandler;
import neg2013.acsension.Acsension;
import neg2013.acsension.infrastructure.BlockSolarGenny;
import net.minecraft.client.Minecraft;
import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.inventory.IInventory;
import net.minecraft.item.ItemStack;
import net.minecraft.nbt.NBTTagCompound;
import net.minecraft.tileentity.TileEntity;
import net.minecraftforge.common.util.ForgeDirection;
import net.minecraftforge.event.entity.player.PlayerUseItemEvent.Tick;

public class TESolar extends TileEntity implements IInventory, IEnergyHandler{

ItemStack[] items;


//power

	protected EnergyStorage storage = new EnergyStorage(16);


		@Override
		public boolean canConnectEnergy(ForgeDirection from) {
			if(from == ForgeDirection.DOWN){
				return true;
			}if(from == ForgeDirection.EAST){
				return true;
			}if(from == ForgeDirection.NORTH){
				return true;
			}if(from == ForgeDirection.SOUTH){
				return true;
			}if(from == ForgeDirection.UP){
				return false;
			}if(from == ForgeDirection.WEST){
				return true;	
			}
			else{
				return true;
			}
		}



		@Override
		public int extractEnergy(ForgeDirection from, int maxExtract, boolean simulate) {

			return storage.extractEnergy(maxExtract, simulate);




			/*
			  if(from == ForgeDirection.DOWN){
				return 2;
			}if(from == ForgeDirection.EAST){
				return 2;
			}if(from == ForgeDirection.NORTH){
				return 2;
			}if(from == ForgeDirection.SOUTH){
				return 2;
			}if(from == ForgeDirection.UP){
				return 2;
			}if(from == ForgeDirection.WEST){
				return 2;	
			}
			else{
				return 2;
			}*/
		}




		@Override
		public int getMaxEnergyStored(ForgeDirection from) {
			return storage.getMaxEnergyStored();
		}

		int energy;



		@Override
		public int getEnergyStored(ForgeDirection from) {

			return storage.getEnergyStored();

		}


		@Override
		public int receiveEnergy(ForgeDirection from, int maxReceive, boolean simulate) {
			return storage.receiveEnergy(maxReceive, simulate);
		}


@Override
public void updateEntity() {

	if(this.worldObj.isDaytime()){
		if(this.worldObj.canBlockSeeTheSky(this.xCoord, this.yCoord, this.zCoord)){
			this.storage.setEnergyStored(2);



		}
	}





}

/*if(from == ForgeDirection.DOWN){
				if(this.worldObj.isDaytime()){
					if(this.worldObj.canBlockSeeTheSky(this.xCoord, this.yCoord, this.zCoord)){
						return 2;
					}
				}
			}

			if(from == ForgeDirection.EAST){
				if(this.worldObj.isDaytime()){
					if(this.worldObj.canBlockSeeTheSky(this.xCoord, this.yCoord, this.zCoord)){
						return 2;
					}
				}
			}

			if(from == ForgeDirection.NORTH){
				if(this.worldObj.isDaytime()){
					if(this.worldObj.canBlockSeeTheSky(this.xCoord, this.yCoord, this.zCoord)){
						return 2;
					}
				}
			}

			if(from == ForgeDirection.SOUTH){
				if(this.worldObj.isDaytime()){
					if(this.worldObj.canBlockSeeTheSky(this.xCoord, this.yCoord, this.zCoord)){
						return 2;
					}
				}
			}

			if(from == ForgeDirection.UP){
				if(this.worldObj.isDaytime()){
					if(this.worldObj.canBlockSeeTheSky(this.xCoord, this.yCoord, this.zCoord)){
						return 2;
					}
				}
			}

			if(from == ForgeDirection.WEST){
				if(this.worldObj.isDaytime()){
					if(this.worldObj.canBlockSeeTheSky(this.xCoord, this.yCoord, this.zCoord)){
						return 2;
					}
				}
			}


			else{
				return 0;
			}*/




//solar

public TESolar(){
	this.items = new ItemStack [128];
	this.storage.setCapacity(16);
	this.storage.setMaxExtract(4);
	this.storage.setMaxReceive(2);
	this.storage.setMaxTransfer(2);
}

//solar







//NBT NBT NBT NBT NBT NBT NBT NBT NBT NBT

@Override
public void writeToNBT(NBTTagCompound nbt) {
	super.writeToNBT(nbt);
	storage.writeToNBT(nbt);
}

@Override
public void readFromNBT(NBTTagCompound nbt) {
	super.readFromNBT(nbt);
	storage.readFromNBT(nbt);
}

//NBT NBT NBT NBT NBT NBT NBT NBT NBT NBT 







@Override
public boolean receiveClientEvent(int id, int value) {

	return super.receiveClientEvent(id, value);
}










//inventory

@Override
public int getSizeInventory() {

	return items.length;
}

@Override
public ItemStack getStackInSlot(int slotIn) {

	return items[slotIn];
}

@Override
public ItemStack decrStackSize(int i, int count) {

	ItemStack itemStack =  getStackInSlot(i);
	if(itemStack != null){
		if(itemStack.stackSize >= count){
			setInventorySlotContents(i, null);
		}
		else{
			itemStack = itemStack.splitStack(count);
			this.markDirty();
		}
	}


	return itemStack;
}

@Override
public ItemStack getStackInSlotOnClosing(int i) {
	ItemStack itemStack = getStackInSlot(i);
	setInventorySlotContents(i, null);
	return itemStack;
}

@Override
public void setInventorySlotContents(int i, ItemStack itemStack) {

	items[i] = itemStack;
	if(itemStack != null && itemStack.stackSize >= getInventoryStackLimit()){
		itemStack.stackSize = getInventoryStackLimit();
	}
	this.markDirty();
}

@Override
public String getInventoryName() {

	return "Solar Genny";
}

@Override
public boolean hasCustomInventoryName() {

	return true;
}

@Override
public int getInventoryStackLimit() {

	return 64;
}

@Override
public boolean isUseableByPlayer(EntityPlayer player) {



	return true;//player.getDistanceSq(xCoord + 0.5f, yCoord + 0.5f, zCoord + 0.5f) <= 64;
}

@Override
public void openInventory() {


}

@Override
public void closeInventory() {


}

@Override
public boolean isItemValidForSlot(int i, ItemStack itemStack) {

	return itemStack.getItem() == items[i].getItem();
}








/*

	if(Minecraft.getMinecraft().theWorld.isDaytime()){
		if(Minecraft.getMinecraft().theWorld.canBlockSeeTheSky(this.xCoord, this.yCoord, this.zCoord)){
			int tick = Acsension.blockSolarGenny.tickRate(Minecraft.getMinecraft().theWorld);
			int stored = tick * 2;

			this.energy = stored;

			Minecraft.getMinecraft().theWorld.getWorldTime();





	if(energy > getMaxEnergyStored(from)){
		return getMaxEnergyStored(from);
	}
	else{
		return energy;
	}
		}
		else{
			return energy;
		}
	}
	else{
		return energy;
	}*/

}

[shadowfacts]

Oh lord, /me facedesks so hard

 

@Override
public boolean canConnectEnergy(ForgeDirection from) {
if(from == ForgeDirection.DOWN){
	return true;
}if(from == ForgeDirection.EAST){
	return true;
}if(from == ForgeDirection.NORTH){
	return true;
}if(from == ForgeDirection.SOUTH){
	return true;
}if(from == ForgeDirection.UP){
	return false;
}if(from == ForgeDirection.WEST){
	return true;	
}
else{
	return true;
}
}

 

This mess can be simplified to one line:

 return from != ForgeDirection.UP;

 

Also in your

updateEntity

method, you should call

this.storage.receiveEnergy(2, false);

instead of

this.storage.setEnergyStored(2);

. The former adds 2 RF to the storage whereas the latter sets the storage to contain 2 RF.

[NEG2013]

I have done those corrections and nothing heppend

[Silly511]

USE TILENERGYHANDLER AND FORGET THIS MESS! It already has all the methods fill out, so all you need to do is make it add energy if it can see the sky.

[NEG2013]

yes I realize that but even with it minimized to this it still fails

 

	protected EnergyStorage storage = new EnergyStorage(16, 2, 4);


		@Override
		public boolean canConnectEnergy(ForgeDirection from) {
			if(from == ForgeDirection.UP){
				return false;
			}
			else{
				return true;
			}
		}


@Override
public void updateEntity() {

	if(this.worldObj.isDaytime()){
		if(this.worldObj.canBlockSeeTheSky(this.xCoord, this.yCoord, this.zCoord)){
			this.storage.receiveEnergy(2, false); 



		}
	}

[Silly511]

Use this:

 

public class TESolar extends TileEnergyHandler implements IInventory {

public TESolar() {
	super(16, 2, 4);
}

@Override
public boolean canConnectEnergy(ForgeDirection from) {
	return from != ForgeDirection.UP;
}

@Override
public int extractEnergy(ForgeDirection from, int maxExtract, boolean simulate) {
	return 0;
}

@Override
public void updateEntity() {
	if(this.worldObj.isDaytime())
		if(this.worldObj.canBlockSeeTheSky(this.xCoord, this.yCoord + 1, this.zCoord)){
			this.receiveEnergy(2, false);
		}
}

}

 

You can add the inventory stuff.

[NEG2013]

the constructor didn't like that super and this.recieveenergy had an error so I changed them to this

 

public TESolar() {
	this.storage.setCapacity(16);
		this.storage.setMaxReceive(2);
		this.storage.setMaxExtract(4); 
}

@Override
public boolean canConnectEnergy(ForgeDirection from) {
	return from != ForgeDirection.UP;
}

@Override
public int extractEnergy(ForgeDirection from, int maxExtract, boolean simulate) {
	return 0;
}

@Override
public void updateEntity() {
	if(this.worldObj.isDaytime())
		if(this.worldObj.canBlockSeeTheSky(this.xCoord, this.yCoord + 1, this.zCoord)){
			this.storage.receiveEnergy(2, false);
		}
}

[Silly511]

The reason that the constructor didn't work is because you didn't make it extend TileEnergyHandler. Replace your entire class with the code I gave you and then add the inventory stuff.

[NEG2013]

It still threw the error in the constructor

[Silly511]

Sorry I was thinking of the RF item class. What you had before is fine.

[NEG2013]

it still doesn't work though could it be the version of thermal/cofh I'm using or the api

[Silly511]

What do you mean by 'it doesn't work'?

[NEG2013]

I mean it doesn't produce the desired 2 rf/t at all it just does nothing

[Silly511]

I think it is producing RF it's just not outputting it. Try adding this to your entityUpdate() method:

 

if ((storage.getEnergyStored() > 0)) {
		for (int i = 0; i < 6; i++){
			TileEntity tile = worldObj.getTileEntity(xCoord + ForgeDirection.getOrientation(i).offsetX, yCoord + ForgeDirection.getOrientation(i).offsetY, zCoord + ForgeDirection.getOrientation(i).offsetZ);
			if (tile != null && tile instanceof IEnergyReceiver) {
				storage.extractEnergy(((IEnergyReceiver)tile).receiveEnergy(ForgeDirection.getOrientation(i).getOpposite(), storage.extractEnergy(storage.getMaxExtract(), true), false), false);
			}
		}
	}

 

And adding this to your class:

 

@Override
public Packet getDescriptionPacket() {
	NBTTagCompound tagCompound = new NBTTagCompound();
	writeToNBT(tagCompound);
	return new S35PacketUpdateTileEntity(this.xCoord, this.yCoord, this.zCoord, 1, tagCompound);
}

@Override
public void onDataPacket(NetworkManager net, S35PacketUpdateTileEntity pkt) {
	readFromNBT(pkt.func_148857_g());
}

[NEG2013]

didn't work and also what is with that packet stuff

 

you could try create a solar generator or any generator and see what was different

[Silly511]

Try replacing your entityUpdate with this

 

@Override
public void updateEntity() {
	if(this.worldObj.isDaytime())
		if(this.worldObj.canBlockSeeTheSky(this.xCoord, this.yCoord + 1, this.zCoord)){
			this.storage.receiveEnergy(2, false);
			this.worldObj.markBlockForUpdate(this.xCoord, this.yCoord, this.zCoord);
		}

	if ((storage.getEnergyStored() > 0)) {
		for (int i = 0; i < 6; i++){
			TileEntity tile = worldObj.getTileEntity(xCoord + ForgeDirection.getOrientation(i).offsetX, yCoord + ForgeDirection.getOrientation(i).offsetY, zCoord + ForgeDirection.getOrientation(i).offsetZ);
			if (tile != null && tile instanceof IEnergyReceiver) {
				storage.extractEnergy(((IEnergyReceiver)tile).receiveEnergy(ForgeDirection.getOrientation(i).getOpposite(), storage.extractEnergy(storage.getMaxExtract(), true), false), false);
			}
		}
	}
}

[NEG2013]

Fuck yay man omfg thanks so much that worked