Solved Trouble (DoubleChest)

For some reason I keep getting an error saying that the doublechest block cannot be defined as a double chest? any help?

Relevant Code

Code:java

 @SuppressWarnings("deprecation")
@EventHandler
public void trommelPlace(BlockPlaceEvent event){
Block block = event.getBlock();
Player player = event.getPlayer();
 
if(block.getType() == Material.FURNACE && player.getItemInHand().getItemMeta().getDisplayName().equals(ChatColor.AQUA + "Trommel") == true){
 
File locationsdir = new File(this.getDataFolder() + File.separator + "users" + File.separator + player.getUniqueId() + ".yml");
FileConfiguration config = YamlConfiguration.loadConfiguration(locationsdir);
 
config.set("Player", player.getName());
config.set("World", player.getWorld().getName().toString());
config.set("CoordinateX", block.getX());
config.set("CoordinateY", block.getY());
config.set("CoordinateZ", block.getZ());
try {
config.save(locationsdir);
} catch (IOException e1) {
e1.printStackTrace();
}

Line 49

Code:java

DoubleChest chest = (DoubleChest) Bukkit.getServer().getWorld("world").getBlockAt(config.getInt("CoordinateX"), config.getInt("CoordinateY"), config.getInt("CoordinateZ")).getState();

Error Code

Code:

[11:44:18 WARN]: [Trommel] Task #12 for Trommel v0.1 generated an exception
java.lang.ClassCastException: org.bukkit.craftbukkit.v1_7_R3.block.CraftChest cannot be cast to org.bukkit.block.DoubleChest
    at com.kassestral.plugins.trommel.Plugin$1.run(Plugin.java:49) ~[?:?]
    at org.bukkit.craftbukkit.v1_7_R3.scheduler.CraftTask.run(CraftTask.java:53) ~[bukkit.jar:git-Bukkit-1.7.2-R0.3-66-g43d8943-b3078jnks]
    at org.bukkit.craftbukkit.v1_7_R3.scheduler.CraftScheduler.mainThreadHeartbeat(CraftScheduler.java:345) [bukkit.jar:git-Bukkit-1.7.2-R0.3-66-g43d8943-b3078jnks]
    at net.minecraft.server.v1_7_R3.MinecraftServer.v(MinecraftServer.java:600) [bukkit.jar:git-Bukkit-1.7.2-R0.3-66-g43d8943-b3078jnks]
    at net.minecraft.server.v1_7_R3.DedicatedServer.v(DedicatedServer.java:260) [bukkit.jar:git-Bukkit-1.7.2-R0.3-66-g43d8943-b3078jnks]
    at net.minecraft.server.v1_7_R3.MinecraftServer.u(MinecraftServer.java:558) [bukkit.jar:git-Bukkit-1.7.2-R0.3-66-g43d8943-b3078jnks]
    at net.minecraft.server.v1_7_R3.MinecraftServer.run(MinecraftServer.java:469) [bukkit.jar:git-Bukkit-1.7.2-R0.3-66-g43d8943-b3078jnks]
    at net.minecraft.server.v1_7_R3.ThreadServerApplication.run(SourceFile:628) [bukkit.jar:git-Bukkit-1.7.2-R0.3-66-g43d8943-b3078jnks]
[11:44:19 INFO]: runnable executed :3
[11:44:19 WARN]: [Trommel] Task #12 for Trommel v0.1 generated an exception
java.lang.ClassCastException: org.bukkit.craftbukkit.v1_7_R3.block.CraftChest cannot be cast to org.bukkit.block.DoubleChest
    at com.kassestral.plugins.trommel.Plugin$1.run(Plugin.java:49) ~[?:?]
    at org.bukkit.craftbukkit.v1_7_R3.scheduler.CraftTask.run(CraftTask.java:53) ~[bukkit.jar:git-Bukkit-1.7.2-R0.3-66-g43d8943-b3078jnks]
    at org.bukkit.craftbukkit.v1_7_R3.scheduler.CraftScheduler.mainThreadHeartbeat(CraftScheduler.java:345) [bukkit.jar:git-Bukkit-1.7.2-R0.3-66-g43d8943-b3078jnks]
    at net.minecraft.server.v1_7_R3.MinecraftServer.v(MinecraftServer.java:600) [bukkit.jar:git-Bukkit-1.7.2-R0.3-66-g43d8943-b3078jnks]
    at net.minecraft.server.v1_7_R3.DedicatedServer.v(DedicatedServer.java:260) [bukkit.jar:git-Bukkit-1.7.2-R0.3-66-g43d8943-b3078jnks]
    at net.minecraft.server.v1_7_R3.MinecraftServer.u(MinecraftServer.java:558) [bukkit.jar:git-Bukkit-1.7.2-R0.3-66-g43d8943-b3078jnks]
    at net.minecraft.server.v1_7_R3.MinecraftServer.run(MinecraftServer.java:469) [bukkit.jar:git-Bukkit-1.7.2-R0.3-66-g43d8943-b3078jnks]
    at net.minecraft.server.v1_7_R3.ThreadServerApplication.run(SourceFile:628) [bukkit.jar:git-Bukkit-1.7.2-R0.3-66-g43d8943-b3078jnks]
[11:44:20 INFO]: runnable executed :3
[11:44:20 WARN]: [Trommel] Task #12 for Trommel v0.1 generated an exception
java.lang.ClassCastException: org.bukkit.craftbukkit.v1_7_R3.block.CraftChest cannot be cast to org.bukkit.block.DoubleChest
    at com.kassestral.plugins.trommel.Plugin$1.run(Plugin.java:49) ~[?:?]
    at org.bukkit.craftbukkit.v1_7_R3.scheduler.CraftTask.run(CraftTask.java:53) ~[bukkit.jar:git-Bukkit-1.7.2-R0.3-66-g43d8943-b3078jnks]
    at org.bukkit.craftbukkit.v1_7_R3.scheduler.CraftScheduler.mainThreadHeartbeat(CraftScheduler.java:345) [bukkit.jar:git-Bukkit-1.7.2-R0.3-66-g43d8943-b3078jnks]
    at net.minecraft.server.v1_7_R3.MinecraftServer.v(MinecraftServer.java:600) [bukkit.jar:git-Bukkit-1.7.2-R0.3-66-g43d8943-b3078jnks]
    at net.minecraft.server.v1_7_R3.DedicatedServer.v(DedicatedServer.java:260) [bukkit.jar:git-Bukkit-1.7.2-R0.3-66-g43d8943-b3078jnks]
    at net.minecraft.server.v1_7_R3.MinecraftServer.u(MinecraftServer.java:558) [bukkit.jar:git-Bukkit-1.7.2-R0.3-66-g43d8943-b3078jnks]
    at net.minecraft.server.v1_7_R3.MinecraftServer.run(MinecraftServer.java:469) [bukkit.jar:git-Bukkit-1.7.2-R0.3-66-g43d8943-b3078jnks]
    at net.minecraft.server.v1_7_R3.ThreadServerApplication.run(SourceFile:628) [bukkit.jar:git-Bukkit-1.7.2-R0.3-66-g43d8943-b3078jnks]

Help ASAP would be appreciated

KingFaris11 Necrodoom

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

Use Chest and getInventory (not getBlockInventory() because that returns only that one chest, not both).

DoubleChest is just an Inventory holder, it isn't actually the block. You can do this if you want:

Code:

Chest chest = (Chest) block.getState();
InventoryHolder iH = chest.getInventory().getHolder();
if (iH instanceof DoubleChest) {
     DoubleChest doubleChest = (DoubleChest) iH;
}

By the way, line 7 of the first relevant code you posted, you're going to get a NullPointerException most of the time as only items renamed have a display name. First check if it has one, itemMeta.hasDisplayName().

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

KingFaris11

Thanks that worked, however how can I take only 1 cobblestone from the inventory per seconds?

Current code

Code:java

    public void onEnable(){
        initialize();
        trommelItem();
        pm.registerEvents(this, this);
        if(plugin == null){
            plugin = this;
        }
 
        BukkitScheduler scheduler = Bukkit.getServer().getScheduler();
 
        scheduler.scheduleSyncRepeatingTask(this, new Runnable() {
            @Override
            public void run() {
              Bukkit.getServer().getLogger().info("runnable executed :3");
 
              for (File file : new File(plugin.getDataFolder() + File.separator + "users").listFiles()){
                  FileConfiguration config = YamlConfiguration.loadConfiguration(file);
                  Chest chest = (Chest) Bukkit.getServer().getWorld("world").getBlockAt(config.getInt("CoordinateX"), config.getInt("CoordinateY"), config.getInt("CoordinateZ")).getState();
                  InventoryHolder iH = chest.getInventory().getHolder();
 
                  if (iH instanceof DoubleChest) {
                        DoubleChest doubleChest = (DoubleChest) iH;
                        if(doubleChest.getInventory().contains(Material.COBBLESTONE)){
                            doubleChest.getInventory().remove(Material.COBBLESTONE);
                        }
                  }
              }
 
            }
        }, 0L, 20L);

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

Kassestral
doubleChest.getInventory.remove(new ItemStack(Material.COBBLESTONE, AmountYouWantToRemove));

May be spelling errors I didn't check

 

Heirteir

I tried that, it only takes out the single cobblestone stacks, anything bigger than 2 is untouched

 

Kassestral
oh yeah forgot about that you could do

Code:java

for (int x = 0; x < amountyouwanttoremove; x++){
  doubleChest.getInventory().remove(Material.COBBLESTONE);
}

Kassestral
Are you trying to remove all cobblestone?
if you are you can do

Code:java

while (inv.contains(Material.COBBLESTONE)){
  inv.remove(Material.COBBLESTONE);
}

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

Heirteir

Well I want to get the amount of cobblestone that the double chest contains, and then take 1 away per second

 

Kassestral
1 per second would be

Code:java

new BukkitRunnable(){
  public void run(){
  if (inv.contains(Material.COBBLESTONE))
      inv.remove(Material.COBBLESTONE);
  else
      this.cancel();
}
}.runTaskTimer(plugin, 0L, 20L);

where plugin is your main class that extends JavaPlugin and 0L is the delay, 20L is 1 second because it is ticks.

 

Heirteir

I know but like I said here

Code:java

    if(doubleChest.getInventory().contains(Material.COBBLESTONE)){
                            doubleChest.getInventory().remove(Material.COBBLESTONE);

this removes ALL of the cobblestone, not just one

 

Kassestral
oh then did you try .removeItem(); I can't remember if that takes a Material or an ItemStack but either way it's simple just .removeItem(Material.COBBLESTONE); or .removeItem(new ItemStack(Material.COBBLESTONE));

 

Heirteir

Its alright I made a form of method that seems to work

Code:java

 if (iH instanceof DoubleChest) {
DoubleChest doubleChest = (DoubleChest) iH;
if(doubleChest.getInventory().contains(Material.COBBLESTONE)){
doubleChest.getInventory().getItem(doubleChest.getInventory().first(Material.COBBLESTONE)).setAmount(doubleChest.getInventory().getItem(doubleChest.getInventory().first(Material.COBBLESTONE)).getAmount() - 1);
}
}

now I just need to work out how to define a different chest xD

 

I had a similar problem and I ultimately came up with the same solution for removing just one of an item. However, I found that if the ItemStack you are working on only has an amount of 1, then setAmount(0) wouldn't actually remove it. (Although I haven't tried it on 1.7.9... maybe it does now which would make this post irrelevant).

What I had to do was something like this:
(I removed some conditions that aren't relevant so there may be typos... but basically check if the amount is 1 and set the item to null if it is.)

ItemStack[] items = inv.getContents();
ItemStack tempItem = items[inv.first(Material.COBBLESTONE)];

if (tempItem != null) {
if (tempItem.getAmount() == 1)
inv.setItem(items[inv.first(Material.COBBLESTONE)], null);
else
tempItem.setAmount(tempItem.getAmount() - 1;
}

 

getAmount() -1 will leave the last item.

I suggest doing the following:

Code:java

ItemStack is = new ItemStack(get an itemstack here);
is.setAmount(1);
getInventory().remove(is);

 

adventuretc

Thank you but I realised this while testing and came up with this

Code:java

                            if(resultChest.getInventory().firstEmpty() == -1){
                                Bukkit.getServer().broadcastMessage("Chest now full");
                            }
                            else{
                                if(doubleChest.getInventory().getItem(doubleChest.getInventory().first(Material.COBBLESTONE)).getAmount() == 1){
                                    //Checking if only 1 cobblestone left
                                    doubleChest.getInventory().remove(Material.COBBLESTONE);
                                    resultChest.getInventory().addItem(new ItemStack(Material.GRAVEL, 1));
                                }
                                else{
                                    //Method for taking cobblestone
                                    doubleChest.getInventory().getItem(doubleChest.getInventory().first(Material.COBBLESTONE)).setAmount(doubleChest.getInventory().getItem(doubleChest.getInventory().first(Material.COBBLESTONE)).getAmount() - 1);
                                    resultChest.getInventory().addItem(new ItemStack(Material.GRAVEL, 1));
                                }
                            }

 

With that code if you have a stack of 1 item and a stack of 64 items, all will be removed

 

adventuretc

Ah thank you never noticed that, its now fixed and working well!

Code:java

                            if(resultChest.getInventory().firstEmpty() == -1){
                                Bukkit.getServer().broadcastMessage("Chest now full");
                            }
                            else{
                                if(doubleChest.getInventory().getItem(doubleChest.getInventory().first(Material.COBBLESTONE)).getAmount() == 1){
                                    //Checking if only 1 cobblestone left
                                    doubleChest.getInventory().remove(new ItemStack(Material.COBBLESTONE, 1));
                                    resultChest.getInventory().addItem(new ItemStack(Material.GRAVEL, 1));
                                }
                                else{
                                    //Method for taking cobblestone
                                    doubleChest.getInventory().getItem(doubleChest.getInventory().first(Material.COBBLESTONE)).setAmount(doubleChest.getInventory().getItem(doubleChest.getInventory().first(Material.COBBLESTONE)).getAmount() - 1);
                                    resultChest.getInventory().addItem(new ItemStack(Material.GRAVEL, 1));
                                }
                            }

Hey sorry I only just saw your post :3 thanks for the advice but its working fine now haha xD setting the amount to 0 still doesnt work though

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

Kassestral
Why don't you just use .removeItem(); :3

 

I'm confused - any errors/problems?

 

Heirteir

I am haha xD

Heirteir KingFaris11
http://dev.bukkit.org/bukkit-plugins/trommel/files/1-trommel_v0-5-beta/

EDIT by Moderator: merged posts, please use the edit button instead of double posting.