Solved Create new Block object

I have blocks stored in a database (world, location, type and metadata) how do I create a new Block object from those? (I do not want to directly edit the world as I'm planning to pass a list of Blocks)

 

You can't construct a block by regular means.And they're not supposed to be, it likely would cause errors and such.

You want to to use getBlockAt(x, y, z) in World or use getBlock() in Location.

 

Unless you are using world and type information to be fed to a plugin/website or something in order to display them as stats etc then I don't see any need in storing that in a database. Storing a location is all you need really...

Code:java

Block block = location.getBlock();

Where 'location' represents of course a Location object in which you've created based off the location information from your database.

Once you have the block you can of course edit it according to your other information.

 

Wingzzz they are stored in database as they are not meant to always be visible to players, but still need to be saved between server restarts

but I will give location.getBlock() a chance to see if it does not modify the block in the world nope changes the block as well

Like I said they are NOT for editing the world that's why I need them to read/change data from without getting the block from the world as changing a value will change it in the world. Nether are they stored in the world so methods such as getBlockAt are useless. They will be used to pass data between the plugin, the best I can see at the moment is to create a custom Block class and use that, but I assumed Bukkit had some build in Block class that I could pre modify and ever wanted to dump into the world would be as easy as player.getWorld().replaceBlock(newBlock).

I don't see how new Block(blockId, location, etc) would create errors at all, or a method in Block that could spit out a blank block when you call it like Block.getBlockById(blockId) which would return a Block object, which is of the id Type (location being 0, 0, 0 and meta being 0)

To give an example of what I want to achieve:

Code:java

public static List<Block> getPlayerBlocks(final String playerID, final String worldID) {
List<Block> blockList = new CopyOnWriteArrayList();
Database tempDB = ConfigHandler.getTheDataHub();
if (!tempDB.isOpen() && !tempDB.open()) {
Blueprint.error("Could not work with database");
}
try {
ResultSet result = (tempDB.query("SELECT blockID, blockX, blockY, blockZ, blockMeta FROM blocks WHERE playerID = '" + playerID + "' AND world = '"+worldID+"';"));
do {
blockList.add(new Block(result.get("typeID"), result.get("blockX"), result.get("blockY"), result.get("blockZ"), (byte)result.get("blockMeta"));
} while(result);
} catch (SQLException ex) {
tempDB.close();
Blueprint.error("Couldn't deactivate player", ex);
}
tempDB.close();
return blockList;
}

P.S. Using .getBlockAt() to modify a block in the world does not seem intuitive to me to be honest (I know you can then call .setTypeId(), etc) but being able to use something .setBlockAt(newBlock)/.setBlockAt(newBlock, blockLocation) would be more intuitive

 

Max9403
Okay, so you want to construct blocks so that they may be on a 'standby' so that they do not directly effect the world until you decide to use one, in which case the block can override another block, correct?

ie: Collection/Array of 'custom' blocks or block 'templates' to be used.

(Also, I'm just sort of throwing it out there in-case it is anywhere close to what you may want as I await a reply-- Player#sendBlockChange(Location location, Material material, byte data); : fake a block change to the player via packets)

 

Max9403
You're not making any sense. You want to modify/read blocks without getting them..?

 

Wingzzz Yes that is pretty much what I'm trying to achieve just hoping I don't have to make/use a custom class for it, (this is something else from what I'm asking but couldn't get completely right ether) I tried sending a fake sendBlockChange after canceling a BlockPlaceEvent but it did not seem to have an effect (guess it was because the cancelation was send afterward an removing the block again tryed useing protcolLib for it as well, but had some issues with block 32 and try to get a FallingBlock entity to stay inside it)

The_Doctor_123 Like I said in the previous post I have the needed data for a block stored externally in a database, e.g. like a book would be to ones brain or notes written in class would be to ones textbook, what I want to create is a block object for passing data in my plugin from that external data, therefore I can not get the data from the world as it is not the data in the database, and modifying such would modify the world which is not my direct intent. In other words .getBlockAt(location) != database.getBlockDetails(); because one example would be getBlockAt(location).getType() could return AIR while database.getBlockDetails().getType() could return DIRT

 

Max9403
You could store the temporary block data in a separate world. That's the only way I can see that you'll get the Block interface.

 

Okay, so how about you create a BlockData class that will hold all the information for a block you would want.
So you could make one with BlockData blockData = new BlockData(Block block); as well as a bunch of other constructors. It would have methods like getWorld(), getLocation(), getType() etc... You could also throw in a few methods like materialize(); or something of the sort that would then actually modify the world based on the object's data.

 

Wingzzz I was hopping to avoid having to do that a well >.>

The_Doctor_123 Thanks for the info but I'm gonna stick with Wingzzz suggestion on this one, though your suggestion would indeed give me the block interface that I'm looking for but would mean loading in an other world, etc which would make the requirements of my plugin go a higher then need be so I'd much rather battle out a new class file then having to spawn worlds (because someone will find a way of "accidentally" getting into that world and (though only probably hilarious to watch) isn't exactly what I'm looking for.

Thanks for the help everybody, btw would anyone be interested in a copy of the class file that Wingzzz suggested (maybe, if I can get it coded correctly , make it part of an API)?

 

Max9403
Naturally I'm interested, if you're going to make it feel free to message me about it as I will probably sometime go about making my own BlockWrapper. It really shouldn't take long, but would be interested in the outcome none the less for the goal you've described.

 

Wingzzz I'll start working on it then, just had a good morning of sleep XD 3 a.m to 1 p.m

Here is what I have at the moment (I'm using TypeId for backward compatibility):

Code:java

package com.emberringstudios.blueprint;
 
import java.util.logging.Level;
import java.util.logging.Logger;
import org.bukkit.Material;
import org.bukkit.World;
import org.bukkit.block.Block;
 
/**
 *
 * @author Max9403 <[email][email protected][/email]>
 */
public class BlockData {
 
    private int type;
    private int locX;
    private int locY;
    private int locZ;
    private byte data;
    private World blockWorld;
 
    public BlockData(final int blockType, final int locX, final int locY, final int locZ, final byte data) {
        this.type = blockType;
        this.locX = locX;
        this.locY = locY;
        this.locZ = locZ;
        this.data = data;
    }
 
    public BlockData(final Material blockType, final int locX, final int locY, final int locZ, final byte data) {
        this.type = blockType.getId();
        this.locX = locX;
        this.locY = locY;
        this.locZ = locZ;
        this.data = data;
    }
 
    public BlockData(final Block blockData) {
        this.type = blockData.getTypeId();
        this.locX = blockData.getX();
        this.locY = blockData.getY();
        this.locZ = blockData.getZ();
        this.data = blockData.getData();
    }
 
    public BlockData(final int blockType, final int locX, final int locY, final int locZ, final byte data, final World blockWorld) {
        this.type = blockType;
        this.locX = locX;
        this.locY = locY;
        this.locZ = locZ;
        this.data = data;
        this.blockWorld = blockWorld;
    }
 
    public BlockData(final Material blockType, final int locX, final int locY, final int locZ, final byte data, final World blockWorld) {
        this.type = blockType.getId();
        this.locX = locX;
        this.locY = locY;
        this.locZ = locZ;
        this.data = data;
        this.blockWorld = blockWorld;
    }
 
    public BlockData(final Block blockData, final World blockWorld) {
        this.type = blockData.getTypeId();
        this.locX = blockData.getX();
        this.locY = blockData.getY();
        this.locZ = blockData.getZ();
        this.data = blockData.getData();
        this.blockWorld = blockWorld;
    }
 
    /**
     * @return the type
     */
    public int getType() {
        return type;
    }
 
    /**
     * @param type the type to set
     */
    public void setType(int type) {
        this.type = type;
    }
 
    /**
     * @return the locX
     */
    public int getX() {
        return locX;
    }
 
    /**
     * @param locX the locX to set
     */
    public void setX(int locX) {
        this.locX = locX;
    }
 
    /**
     * @return the locY
     */
    public int getY() {
        return locY;
    }
 
    /**
     * @param locY the locY to set
     */
    public void setY(int locY) {
        this.locY = locY;
    }
 
    /**
     * @return the locZ
     */
    public int getZ() {
        return locZ;
    }
 
    /**
     * @param locZ the locZ to set
     */
    public void setZ(int locZ) {
        this.locZ = locZ;
    }
 
    /**
     * @return the data
     */
    public byte getData() {
        return data;
    }
 
    /**
     * @param data the data to set
     */
    public void setData(byte data) {
        this.data = data;
    }
 
    /**
     * @return the blockWorld
     */
    public World getBlockWorld() {
        return blockWorld;
    }
 
    /**
     * @param blockWorld the blockWorld to set
     */
    public void setBlockWorld(World blockWorld) {
        this.blockWorld = blockWorld;
    }
 
    public void loadBlockIntoWorld(World blockWorld) {
        setBlockWorld(blockWorld);
        try {
            loadBlockIntoWorld();
        } catch (NoWorldGivenException ex) {
            Logger.getLogger("Minecraft").log(Level.SEVERE, "Something went horribly wrong, this method should not give this error", ex);
        }
    }
 
    public void loadBlockIntoWorld() throws NoWorldGivenException {
        if (blockWorld == null) {
            throw new NoWorldGivenException("Developer forgot to set a world for this block");
        }
        Block modBlock = this.blockWorld.getBlockAt(locX, locY, locZ);
        modBlock.setTypeId(type);
        modBlock.setData(data);
    }
 
    public boolean equalToBlock(Block tempBlock) {
        return tempBlock.getTypeId() == type && tempBlock.getX() == locX && tempBlock.getY() == locY && tempBlock.getZ() == locZ && tempBlock.getData() == data;
    }
}