[1.14.4] Trouble with Custom Recipes

[BlockyPenguin]

Hi, so, I'm trying to make my own recipe type that uses a format like this, but with as few or as many steps as you like:

{
	"type": "labkit:cauldron",
	
	"method": {
		"steps": [
			
			"one": {
				"input": {
					"item": "minecraft:dirt",
					"count": 1
				},
				
				"wait": 100,
				"waterused": 0
			},
			
			"two": {
				"input": {
					"item": "minecraft:seeds",
					"count": 1
				},
				
				"wait": 20,
				"waterused": 1
			}
			
		],
		
		"output": {
			"item": "minecraft:grass_block",
			"count": 1
		}
	}
}

I've tried following some tutorials on YouTube and adapting it to use my idea of steps, but I always hit roadblocks. Any ideas?

Thanks!

EDIT: Here's my code:
 

package ...

import ...

public class CauldronRecipe implements ICauldronRecipe {
	
	private final ResourceLocation resLoc;
	private CauldronRecipeMethod method;
	private final ItemStack output;
	
	public CauldronRecipe(ResourceLocation resLoc, CauldronRecipeMethod method, ItemStack output) {
		this.resLoc = resLoc;
		this.method = method;
		this.output = output;
	}
	
	@Override
	public boolean matches(RecipeWrapper inv, World world) {
		if(method.getSteps()[0].getInput().test(inv.getStackInSlot(0)))
			return true;
		else
			return false;
	}
	
	@Override
	public ItemStack getCraftingResult(RecipeWrapper inv) {
		return output;
	}
	
	@Override
	public ItemStack getRecipeOutput() {
		return output;
	}
	
	@Override
	public IRecipeSerializer<?> getSerializer() {
		return ModRegistry.ModRecipeSerializers.CAULDRON_RECIPE.get();
	}
	
	@Override
	public Ingredient[] getInput() {
		Ingredient[] ingredients = {};
		for(int i = 0; i < method.getSteps().length; i++) {
			ingredients[i] = method.getSteps()[i].getInput();
		}
		return ingredients;
	}

	@Override
	public NonNullList<Ingredient> getIngredients() {
		return NonNullList.from(null, getInput());
	}

	@Override
	public ResourceLocation getId() {
		return resLoc;
	}
}

 

package ...

import ...

public interface ICauldronRecipe extends IRecipe<RecipeWrapper> {
	
	ResourceLocation RECIPE_TYPE_RESOURCE_LOCATION = new ResourceLocation(LabKit.MODID, "cauldron");
	
	@Nonnull
	@Override
	default IRecipeType<?> getType() {
		return Registry.RECIPE_TYPE.getValue(RECIPE_TYPE_RESOURCE_LOCATION).get();
	}
	
	@Override
	default boolean canFit(int width, int height) {
		return false;
	}
	
	Ingredient[] getInput();
	
}

 

package ...

import ...

public class CauldronRecipeSerializer extends ForgeRegistryEntry<IRecipeSerializer<?>> implements IRecipeSerializer<CauldronRecipe> {
	
	@Override
	public CauldronRecipe read(ResourceLocation recipeResLoc, JsonObject json) {
		
		JsonObject jsonMethod = JSONUtils.getJsonObject(json, "method");
		JsonArray jsonSteps = jsonMethod.getAsJsonArray("steps");
		
		ItemStack output = ShapedRecipe.deserializeItem(JSONUtils.getJsonObject(jsonMethod, "output"));
		
		CauldronRecipeStep[] steps = getCauldronRecipeStepsFromJson(jsonSteps);
		CauldronRecipeMethod method = new CauldronRecipeMethod(output, steps);
		
		return new CauldronRecipe(recipeResLoc, method, output);
	}
	
  	// IDK how to get a CauldronRecipeMethod from the buffer, these two methods are still from the tutorial that I followed's code
	@Override
	public CauldronRecipe read(ResourceLocation recipeResLoc, PacketBuffer buffer) {
		ItemStack output = buffer.readItemStack();
		Ingredient input = Ingredient.read(buffer);
		
		return new CauldronRecipe(recipeResLoc, input, output);
	}
	
	@Override
	public void write(PacketBuffer buffer, CauldronRecipe recipe) {
		Ingredient input = recipe.getIngredients().get(0);
		input.write(buffer);
		
		buffer.writeItemStack(recipe.getRecipeOutput(), false);
	}
	
	private CauldronRecipeStep[] getCauldronRecipeStepsFromJson(JsonArray stepsInput) {
		ArrayList<CauldronRecipeStep> stepsOutArrayList = new ArrayList<CauldronRecipeStep>();
		CauldronRecipeStep[] stepsOutArray = {};
		
		for(JsonElement step : stepsInput) {
			JsonObject jsonInput = step.getAsJsonObject().getAsJsonObject("input");
			Ingredient input = Ingredient.deserialize(jsonInput);
			
			int waitTime = step.getAsJsonObject().get("wait").getAsInt();
			
			int waterUsed = step.getAsJsonObject().get("waterused").getAsInt();
			
			stepsOutArrayList.add(new CauldronRecipeStep(input, waitTime, waterUsed));
		}
		
		return stepsOutArrayList.toArray(stepsOutArray);
	}
}

 

package ...

import ...

public class CauldronRecipeMethod {
	
	private CauldronRecipeStep[] steps;
	private ItemStack output;
	
	public CauldronRecipeMethod(ItemStack output, CauldronRecipeStep... steps) {
		this.steps = steps;
		this.output = output;
	}
	
	public CauldronRecipeStep[] getSteps() {
		return steps;
	}
	
	public ItemStack getOutput() {
		return output;
	}
	
}

 

package ...

import ...

public class CauldronRecipeStep {
	
	private Ingredient input;
	private int waitTime;
	private int waterUsed;
	
	public CauldronRecipeStep(Ingredient input, int waitTime, int waterUsed) {
		this.input = input;
		this.waitTime = waitTime;
		
		if(waterUsed > 3)
			throw new JsonSyntaxException("\"waterused\" is greater than 3. Please set it to a value between 0 and 3.");
		else if(waterUsed < 0)
			throw new JsonSyntaxException("\"waterused\" is less than 0. Please set it to a value between 0 and 3.");
		else
			this.waterUsed = waterUsed;
	}

	public Ingredient getInput() {
		return input;
	}
	
	public int getWaitTime() {
		return waitTime;
	}
	
	public int getWaterUsed() {
		return waterUsed;
	}
	
}

EDIT 2: I know that "You're going about this completely the wrong way" is probably a massive understatement, but I would like to know...

Edited August 10, 2020 by BlockyPenguin
Added my code

[Draco18s]

2 hours ago, BlockyPenguin said:

extends ForgeRegistryEntry<IRecipeSerializer<?>>

Did you register this class?

[BlockyPenguin]

I did, but even so, the game wouldn't load due to the errors in my code. I guess what I probably need to know, is how to get the json data out of the PacketBuffer at 

15 hours ago, BlockyPenguin said:

@Override
public CauldronRecipe read(ResourceLocation recipeResLoc, PacketBuffer buffer) {
    ItemStack output = buffer.readItemStack();
    Ingredient input = Ingredient.read(buffer);
    return new CauldronRecipe(recipeResLoc, input, output);
}

so that I can insert it into "CauldronRecipeStep"s, and insert that into a "CauldronRecipeMethod". Is that possible?

[Alpvax]

17 hours ago, BlockyPenguin said:

[ "one": {

That is not valid json. Currently you have an array with keys.

Either it should be an array of objects "[ { ..." or it should be an object with keys of "one, two,...".

I would suggest going for the first option (especially as that is what you support in your code):

17 hours ago, BlockyPenguin said:

JsonArray jsonSteps = jsonMethod.getAsJsonArray("steps");

 

Edited August 11, 2020 by Alpvax

[BlockyPenguin]

3 hours ago, Alpvax said:

That is not valid json. Currently you have an array with keys.

Either it should be an array of objects "[ { ..." or it should be an object with keys of "one, two,...".

I would suggest going for the first option (especially as that is what you support in your code):

 

Ah ok, thanks

[BlockyPenguin]

Ok, so as it stands, I've changed some things around, and hopefully simplified the system, but I still haven't tested it, as there are still errors in my code that would stop the game from launching. Here's the problem code:

@Override
public CauldronRecipe read(ResourceLocation recipeId, PacketBuffer buffer) {
	ItemStack output = buffer.readItemStack();
	
	return new CauldronRecipe(recipeId, output, steps);
}

Here's the thing, the steps variable doesn't exist, as I don't know how to get a JSONArray from a PacketBuffer (if it's even possible). I have a method I can call which takes in a JSONArray and returns an array of CauldronRecipeSteps, which is what I need to pass in.

 

So, if it is possible, how would I get a JSONArray from a PacketBuffer, and if it's not, how else could I do this?

Thanks!

Edited August 11, 2020 by BlockyPenguin

[Draco18s]

Aren't you the one who wrote the recipe -> packetbuffer code in the first place?

Yes, yes you are.

Quote

	@Override
	public void write(PacketBuffer buffer, CauldronRecipe recipe) {
		Ingredient input = recipe.getIngredients().get(0);
		input.write(buffer);
		
		buffer.writeItemStack(recipe.getRecipeOutput(), false);
	}

 

Your buffer doesn't contain that information. The only thing you write are the ingredients and the output.

[BlockyPenguin]

7 minutes ago, Draco18s said:

Aren't you the one who wrote the recipe -> packetbuffer code in the first place?

Yes, yes you are.

Your buffer doesn't contain that information. The only thing you write are the ingredients and the output.

Ah, I get it now! Thanks! Do you know how I could write an array of CauldronRecipeSteps? buffer.writeArray or something similar doesn't seem to exist...

[Draco18s]

Of course it doesn't exist, buffer.write__ methods only write primitive data types and some helpers for ItemStacks and BlockStates (both of which can be decomposed to primitives, but Forge/Vanilla was kind enough to supply those for you).

You need to decompose your Steps down into primitives.

[BlockyPenguin]

Ok, thanks  I think I've managed to do that correctly now, as the game loads with no errors. I have no way of checking if everything is serialized/deserialized properly yet though, as I have no block to use this crafting system with. I'll make that, and post my results

[BlockyPenguin]

22 hours ago, BlockyPenguin said:

Ok, thanks  I think I've managed to do that correctly now, as the game loads with no errors. I have no way of checking if everything is serialized/deserialized properly yet though, as I have no block to use this crafting system with. I'll make that, and post my results

Ok, well, easier said than done. I've taken a look different crafting blocks in vanilla's code (mostly AbstractFunaceTileEntity, as the other blocks are done without a tile), and I keep writing code, getting stuck, then rewriting it, over and over. How would I go about making this work? I have a custom cauldron block, and I want a player to right-click it to add one of the items in the ItemStack they're holding to the tile's inventory, or if they're holding nothing then drop all the items in the inventory. Here's my code so far:

 

package ...

import ...

public class CauldronTileEntity extends TileEntity {
	
	private LazyOptional<IItemHandler> itemHandler = LazyOptional.of(this::createItemHandler);
	
	private ArrayList<CauldronRecipe> recipes = CauldronRecipe.getAllCauldronRecipes();
	private CauldronRecipe currentRecipe;
	
	public CauldronTileEntity() {
		super(ModRegistry.TileTypes.CAULDRON.get());
	}
	
	@Override
	@SuppressWarnings("unchecked")
    public void read(CompoundNBT tag) {
        CompoundNBT invTag = tag.getCompound("inv");
        itemHandler.ifPresent(h -> ((INBTSerializable<CompoundNBT>) h).deserializeNBT(invTag));
        
        super.read(tag);
    }

	@Override
	@SuppressWarnings("unchecked")
    public CompoundNBT write(CompoundNBT tag) {
        itemHandler.ifPresent(h -> {
			CompoundNBT compound = ((INBTSerializable<CompoundNBT>) h).serializeNBT();
            tag.put("inv", compound);
        });
        
        return super.write(tag);
    }

    private IItemHandler createItemHandler() {
        return new ItemStackHandler(50) {

            @Override
            protected void onContentsChanged(int slot) {
                markDirty();
            }

            @Override
            public boolean isItemValid(int slot, @Nonnull ItemStack stack) {
                return true;
            }
            
        };
    }
	
	public void handleRightClick(PlayerEntity player, Hand hand) {
		ItemStack stack = player.getHeldItem(hand);
		
		if(stack.isEmpty()) {
			popOutAllItems();
		}else {
			itemHandler.ifPresent(h -> {
				for(int i = 0; i <= h.getSlots(); i++) {
					if(h.insertItem(i, stack.split(1), true) != stack.split(1)) {
						player.setHeldItem(hand, h.insertItem(i, stack.split(1), false));
					}else {
						player.sendStatusMessage(ITextComponent.Serializer.fromJsonLenient("Cauldron Full!"), true);
					}
				}
			});
			checkRecipe();
		}
	}
	
	private void popOutAllItems() {
		itemHandler.ifPresent(h -> {
			ItemEntity[] itemEntities = {};
			for(int i = 0; i <= h.getSlots(); i++) {
				itemEntities[i] = new ItemEntity(world, pos.getX(), pos.getY(), pos.getZ(), h.getStackInSlot(i));
			}
			
			for(ItemEntity e : itemEntities) {
				world.addEntity(e);
			}
		});
	}
	
	private void checkRecipe() {
		
	}
}

 

handleRightClick is called from onBlockActivated in the Cauldron's block class.

 

As you can see, checkRecipe() is empty. I'd like it so that whenever a player changes the items in the inventory, it checks whether it can do a recipe (which is why I call it from handleRightClick). How would I go about it? I've changed the json format to be this instead:
 

{
	"type": "labkit:cauldron",

	"inputs": [
		{
			"item": "minecraft:dirt",
			"count": 1
		},
		{
			"tag": "forge:seeds",
			"count": 3
		}
	],

	"cooking_delay": 50,

	"output": {
		"item": "minecraft:grass_block",
		"count": 1
	}
}

 

the order of the inputs doesn't matter to me, I'd just like to know how to check if they equal a recipe's ingredients.
 

(Also, I've just tested running the game, and my cauldron doesn't have a tile entity... I'm using DeferredRegister, and I've got these two methods in my block's class:

@Override
public boolean hasTileEntity(BlockState state) {
	return true;
}

@Override
public TileEntity createTileEntity(BlockState state, IBlockReader world) {
	return new CauldronTileEntity();
}

which as far as i'm aware, should do the trick...)

Thanks!

[BlockyPenguin]

Ok, I figured out where I was going wrong with it not having a tile entity, so that's fixed. Still not sure how to check for recipes...

[BlockyPenguin]

...and there's also a whole bunch of errors relating to item handling  I'll make a new thread for this.

Edited August 12, 2020 by BlockyPenguin