AstralTransRocketries/LucidCube
1
0
Fork 0
Code Issues Pull Requests Packages Projects 1 Releases Wiki Activity

Added some code improvements

Browse Source
This commit is contained in:
bibo38 2015-11-06 17:11:54 +01:00
parent 0447af8bcb
commit 429f615390
2 changed files with 42 additions and 22 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

52
src/Blocks/BlockPiston.cpp
View File

@ -32,7 +32,7 @@ void cBlockPistonHandler::OnDestroyed(cChunkInterface & a_ChunkInterface, cWorld
{ {
NIBBLETYPE OldMeta = a_ChunkInterface.GetBlockMeta(a_BlockX, a_BlockY, a_BlockZ); NIBBLETYPE OldMeta = a_ChunkInterface.GetBlockMeta(a_BlockX, a_BlockY, a_BlockZ);
const Vector3i pushDir = VectorFromMetaData(OldMeta); const Vector3i pushDir = MetadataToOffset(OldMeta);
int newX = a_BlockX + pushDir.x; int newX = a_BlockX + pushDir.x;
int newY = a_BlockY + pushDir.y; int newY = a_BlockY + pushDir.y;
int newZ = a_BlockZ + pushDir.z; int newZ = a_BlockZ + pushDir.z;
@ -73,7 +73,7 @@ bool cBlockPistonHandler::GetPlacementBlockTypeMeta(
Vector3i cBlockPistonHandler::VectorFromMetaData(int a_PistonMeta) Vector3i cBlockPistonHandler::MetadataToOffset(NIBBLETYPE a_PistonMeta)
{ {
switch (a_PistonMeta & 0x07) switch (a_PistonMeta & 0x07)
{ {
@ -86,6 +86,7 @@ Vector3i cBlockPistonHandler::VectorFromMetaData(int a_PistonMeta)
default: default:
{ {
LOGWARNING("%s: invalid direction %d, ignoring", __FUNCTION__, a_PistonMeta & 0x07); LOGWARNING("%s: invalid direction %d, ignoring", __FUNCTION__, a_PistonMeta & 0x07);
ASSERT(!"Invalid direction");
return Vector3i(); return Vector3i();
} }
} }
@ -95,7 +96,8 @@ Vector3i cBlockPistonHandler::VectorFromMetaData(int a_PistonMeta)
void cBlockPistonHandler::PushBlocks(const std::unordered_set<Vector3i, VectorHasher<int>> & a_BlocksToPush, void cBlockPistonHandler::PushBlocks(
const Vector3iSet & a_BlocksToPush,
cWorld * a_World, const Vector3i & a_PushDir cWorld * a_World, const Vector3i & a_PushDir
) )
{ {
@ -110,7 +112,7 @@ void cBlockPistonHandler::PushBlocks(const std::unordered_set<Vector3i, VectorHa
// Move every block // Move every block
BLOCKTYPE moveBlock; BLOCKTYPE moveBlock;
NIBBLETYPE moveMeta; NIBBLETYPE moveMeta;
for (Vector3i & moveBlockPos : sortedBlocks) for (auto & moveBlockPos : sortedBlocks)
{ {
a_World->GetBlockTypeMeta(moveBlockPos.x, moveBlockPos.y, moveBlockPos.z, moveBlock, moveMeta); a_World->GetBlockTypeMeta(moveBlockPos.x, moveBlockPos.y, moveBlockPos.z, moveBlock, moveMeta);
a_World->SetBlock(moveBlockPos.x, moveBlockPos.y, moveBlockPos.z, E_BLOCK_AIR, 0); a_World->SetBlock(moveBlockPos.x, moveBlockPos.y, moveBlockPos.z, E_BLOCK_AIR, 0);
@ -142,11 +144,17 @@ void cBlockPistonHandler::PushBlocks(const std::unordered_set<Vector3i, VectorHa
bool cBlockPistonHandler::CanPushBlock( bool cBlockPistonHandler::CanPushBlock(
int a_BlockX, int a_BlockY, int a_BlockZ, cWorld * a_World, bool a_RequirePushable, int a_BlockX, int a_BlockY, int a_BlockZ, cWorld * a_World, bool a_RequirePushable,
std::unordered_set<Vector3i, VectorHasher<int>> & a_BlocksPushed, const Vector3i & a_PushDir Vector3iSet & a_BlocksPushed, const Vector3i & a_PushDir
) )
{ {
const static std::array<Vector3i, 6> pushingDirs = {{ Vector3i(-1, 0, 0), Vector3i(1, 0, 0), Vector3i(0, -1, 0), Vector3i(0, 1, 0), const static std::array<Vector3i, 6> pushingDirs =
Vector3i(0, 0, -1), Vector3i(0, 0, 1) }}; {
{
Vector3i(-1, 0, 0), Vector3i(1, 0, 0),
Vector3i( 0, -1, 0), Vector3i(0, 1, 0),
Vector3i( 0, 0, -1), Vector3i(0, 0, 1)
}
};
BLOCKTYPE currBlock; BLOCKTYPE currBlock;
NIBBLETYPE currMeta; NIBBLETYPE currMeta;
@ -184,9 +192,12 @@ bool cBlockPistonHandler::CanPushBlock(
if (currBlock == E_BLOCK_SLIME_BLOCK) if (currBlock == E_BLOCK_SLIME_BLOCK)
{ {
// Try to push the other directions // Try to push the other directions
for (const Vector3i & testDir : pushingDirs) for (const auto & testDir : pushingDirs)
{ {
if (!CanPushBlock(a_BlockX + testDir.x, a_BlockY + testDir.y, a_BlockZ + testDir.z, a_World, false, a_BlocksPushed, a_PushDir)) if (!CanPushBlock(
a_BlockX + testDir.x, a_BlockY + testDir.y, a_BlockZ + testDir.z,
a_World, false, a_BlocksPushed, a_PushDir)
)
{ {
// When it's not possible for a direction, then fail // When it's not possible for a direction, then fail
return false; return false;
@ -195,7 +206,10 @@ bool cBlockPistonHandler::CanPushBlock(
} }
// Try to push the block in front of this block // Try to push the block in front of this block
return CanPushBlock(a_BlockX + a_PushDir.x, a_BlockY + a_PushDir.y, a_BlockZ + a_PushDir.z, a_World, true, a_BlocksPushed, a_PushDir); return CanPushBlock(
a_BlockX + a_PushDir.x, a_BlockY + a_PushDir.y, a_BlockZ + a_PushDir.z,
a_World, true, a_BlocksPushed, a_PushDir
);
} }
@ -214,10 +228,11 @@ void cBlockPistonHandler::ExtendPiston(int a_BlockX, int a_BlockY, int a_BlockZ,
return; return;
} }
Vector3i pushDir = VectorFromMetaData(pistonMeta); Vector3i pushDir = MetadataToOffset(pistonMeta);
std::unordered_set<Vector3i, VectorHasher<int>> blocksPushed; Vector3iSet blocksPushed;
if (!CanPushBlock(a_BlockX + pushDir.x, a_BlockY + pushDir.y, a_BlockZ + pushDir.z, if (!CanPushBlock(
a_BlockX + pushDir.x, a_BlockY + pushDir.y, a_BlockZ + pushDir.z,
a_World, true, blocksPushed, pushDir) a_World, true, blocksPushed, pushDir)
) )
{ {
@ -253,7 +268,7 @@ void cBlockPistonHandler::RetractPiston(int a_BlockX, int a_BlockY, int a_BlockZ
return; return;
} }
Vector3i pushDir = VectorFromMetaData(pistonMeta); Vector3i pushDir = MetadataToOffset(pistonMeta);
// Check the extension: // Check the extension:
if (a_World->GetBlock(a_BlockX + pushDir.x, a_BlockY + pushDir.y, a_BlockZ + pushDir.z) != E_BLOCK_PISTON_EXTENSION) if (a_World->GetBlock(a_BlockX + pushDir.x, a_BlockY + pushDir.y, a_BlockZ + pushDir.z) != E_BLOCK_PISTON_EXTENSION)
@ -282,7 +297,7 @@ void cBlockPistonHandler::RetractPiston(int a_BlockX, int a_BlockY, int a_BlockZ
// Try to "push" the pulling block in the opposite direction // Try to "push" the pulling block in the opposite direction
pushDir *= -1; pushDir *= -1;
std::unordered_set<Vector3i, VectorHasher<int>> pushedBlocks; Vector3iSet pushedBlocks;
if (!CanPushBlock(a_BlockX, a_BlockY, a_BlockZ, a_World, false, pushedBlocks, pushDir)) if (!CanPushBlock(a_BlockX, a_BlockY, a_BlockZ, a_World, false, pushedBlocks, pushDir))
{ {
// Not pushable, bail out // Not pushable, bail out
@ -312,7 +327,7 @@ void cBlockPistonHeadHandler::OnDestroyedByPlayer(cChunkInterface & a_ChunkInter
{ {
NIBBLETYPE OldMeta = a_ChunkInterface.GetBlockMeta(a_BlockX, a_BlockY, a_BlockZ); NIBBLETYPE OldMeta = a_ChunkInterface.GetBlockMeta(a_BlockX, a_BlockY, a_BlockZ);
Vector3i pushDir = cBlockPistonHandler::VectorFromMetaData(OldMeta); Vector3i pushDir = cBlockPistonHandler::MetadataToOffset(OldMeta);
int newX = a_BlockX - pushDir.x; int newX = a_BlockX - pushDir.x;
int newY = a_BlockY - pushDir.y; int newY = a_BlockY - pushDir.y;
int newZ = a_BlockZ - pushDir.z; int newZ = a_BlockZ - pushDir.z;
@ -331,3 +346,8 @@ void cBlockPistonHeadHandler::OnDestroyedByPlayer(cChunkInterface & a_ChunkInter
a_WorldInterface.SpawnItemPickups(Pickups, a_BlockX + 0.5, a_BlockY + 0.5, a_BlockZ + 0.5); a_WorldInterface.SpawnItemPickups(Pickups, a_BlockX + 0.5, a_BlockY + 0.5, a_BlockZ + 0.5);
} }
} }

12
src/Blocks/BlockPiston.h
View File

@ -81,10 +81,8 @@ public:
} }
} }
/** This method converts the magic piston metadata into a direction vector. /** Converts piston block's metadata into a unit vector representing the direction in which the piston will extend. */
This vector has a length of 1 and points into the direction, in which the piston will extend. static Vector3i MetadataToOffset(NIBBLETYPE a_PistonMeta);
*/
static Vector3i VectorFromMetaData(int a_PistonMeta);
static void ExtendPiston(int a_BlockX, int a_BlockY, int a_BlockZ, cWorld * a_World); static void ExtendPiston(int a_BlockX, int a_BlockY, int a_BlockZ, cWorld * a_World);
static void RetractPiston(int a_BlockX, int a_BlockY, int a_BlockZ, cWorld * a_World); static void RetractPiston(int a_BlockX, int a_BlockY, int a_BlockZ, cWorld * a_World);
@ -97,6 +95,8 @@ public:
private: private:
typedef std::unordered_set<Vector3i, VectorHasher<int>> Vector3iSet;
/** Returns true if the piston (specified by blocktype) is a sticky piston */ /** Returns true if the piston (specified by blocktype) is a sticky piston */
static inline bool IsSticky(BLOCKTYPE a_BlockType) { return (a_BlockType == E_BLOCK_STICKY_PISTON); } static inline bool IsSticky(BLOCKTYPE a_BlockType) { return (a_BlockType == E_BLOCK_STICKY_PISTON); }
@ -151,11 +151,11 @@ private:
/** Tries to push a block and increases the pushed blocks variable. Returns true if the block is pushable */ /** Tries to push a block and increases the pushed blocks variable. Returns true if the block is pushable */
static bool CanPushBlock( static bool CanPushBlock(
int a_BlockX, int a_BlockY, int a_BlockZ, cWorld * a_World, bool a_RequirePushable, int a_BlockX, int a_BlockY, int a_BlockZ, cWorld * a_World, bool a_RequirePushable,
std::unordered_set<Vector3i, VectorHasher<int>> & a_BlocksPushed, const Vector3i & a_PushDir Vector3iSet & a_BlocksPushed, const Vector3i & a_PushDir
); );
/** Moves a list of blocks in a specific direction */ /** Moves a list of blocks in a specific direction */
static void PushBlocks(const std::unordered_set<Vector3i, VectorHasher<int>> & a_BlocksToPush, static void PushBlocks(const Vector3iSet & a_BlocksToPush,
cWorld * a_World, const Vector3i & a_PushDir cWorld * a_World, const Vector3i & a_PushDir
); );
} ; } ;