Fixed the style problems and added some comments
This commit is contained in:
parent
5fa077f869
commit
64012bf46f
@ -99,12 +99,15 @@ void cBlockPistonHandler::PushBlocks(const std::unordered_set<Vector3i, VectorHa
|
|||||||
cWorld * a_World, const Vector3i & a_PushDir
|
cWorld * a_World, const Vector3i & a_PushDir
|
||||||
)
|
)
|
||||||
{
|
{
|
||||||
|
// Sort blocks to move the blocks first, which are farest away from the piston
|
||||||
|
// This prevents the overwriting of existing blocks
|
||||||
std::vector<Vector3i> sortedBlocks(a_BlocksToPush.begin(), a_BlocksToPush.end());
|
std::vector<Vector3i> sortedBlocks(a_BlocksToPush.begin(), a_BlocksToPush.end());
|
||||||
std::sort(sortedBlocks.begin(), sortedBlocks.end(), [a_PushDir](const Vector3i & a, const Vector3i & b)
|
std::sort(sortedBlocks.begin(), sortedBlocks.end(), [a_PushDir](const Vector3i & a, const Vector3i & b)
|
||||||
{
|
{
|
||||||
return a.Dot(a_PushDir) > b.Dot(a_PushDir);
|
return a.Dot(a_PushDir) > b.Dot(a_PushDir);
|
||||||
});
|
});
|
||||||
|
|
||||||
|
// Move every block
|
||||||
BLOCKTYPE moveBlock;
|
BLOCKTYPE moveBlock;
|
||||||
NIBBLETYPE moveMeta;
|
NIBBLETYPE moveMeta;
|
||||||
for (Vector3i & moveBlockPos : sortedBlocks)
|
for (Vector3i & moveBlockPos : sortedBlocks)
|
||||||
@ -115,6 +118,7 @@ void cBlockPistonHandler::PushBlocks(const std::unordered_set<Vector3i, VectorHa
|
|||||||
moveBlockPos += a_PushDir;
|
moveBlockPos += a_PushDir;
|
||||||
if (cBlockInfo::IsPistonBreakable(moveBlock))
|
if (cBlockInfo::IsPistonBreakable(moveBlock))
|
||||||
{
|
{
|
||||||
|
// Block is breakable, drop it
|
||||||
cBlockHandler * Handler = BlockHandler(moveBlock);
|
cBlockHandler * Handler = BlockHandler(moveBlock);
|
||||||
if (Handler->DoesDropOnUnsuitable())
|
if (Handler->DoesDropOnUnsuitable())
|
||||||
{
|
{
|
||||||
@ -126,6 +130,7 @@ void cBlockPistonHandler::PushBlocks(const std::unordered_set<Vector3i, VectorHa
|
|||||||
}
|
}
|
||||||
} else
|
} else
|
||||||
{
|
{
|
||||||
|
// Not breakable, just move it
|
||||||
a_World->SetBlock(moveBlockPos.x, moveBlockPos.y, moveBlockPos.z, moveBlock, moveMeta);
|
a_World->SetBlock(moveBlockPos.x, moveBlockPos.y, moveBlockPos.z, moveBlock, moveMeta);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
@ -161,11 +166,13 @@ bool cBlockPistonHandler::CanPushBlock(
|
|||||||
|
|
||||||
if (!CanPush(currBlock, currMeta))
|
if (!CanPush(currBlock, currMeta))
|
||||||
{
|
{
|
||||||
|
// When it's not required to push this block, don't fail
|
||||||
return !a_RequirePushable;
|
return !a_RequirePushable;
|
||||||
}
|
}
|
||||||
|
|
||||||
if (a_BlocksPushed.size() >= PISTON_MAX_PUSH_DISTANCE)
|
if (a_BlocksPushed.size() >= PISTON_MAX_PUSH_DISTANCE)
|
||||||
{
|
{
|
||||||
|
// Do not allow to push too much blocks
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -181,11 +188,13 @@ bool cBlockPistonHandler::CanPushBlock(
|
|||||||
{
|
{
|
||||||
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
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// 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);
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -221,6 +230,7 @@ void cBlockPistonHandler::ExtendPiston(int a_BlockX, int a_BlockY, int a_BlockZ,
|
|||||||
|
|
||||||
PushBlocks(blocksPushed, a_World, pushDir);
|
PushBlocks(blocksPushed, a_World, pushDir);
|
||||||
|
|
||||||
|
// Set the extension and the piston base correctly
|
||||||
a_World->SetBlock(a_BlockX, a_BlockY, a_BlockZ, pistonBlock, pistonMeta | 0x8);
|
a_World->SetBlock(a_BlockX, a_BlockY, a_BlockZ, pistonBlock, pistonMeta | 0x8);
|
||||||
a_World->SetBlock(a_BlockX + pushDir.x, a_BlockY + pushDir.y, a_BlockZ + pushDir.z,
|
a_World->SetBlock(a_BlockX + pushDir.x, a_BlockY + pushDir.y, a_BlockZ + pushDir.z,
|
||||||
E_BLOCK_PISTON_EXTENSION, pistonMeta | (IsSticky(pistonBlock) ? 8 : 0)
|
E_BLOCK_PISTON_EXTENSION, pistonMeta | (IsSticky(pistonBlock) ? 8 : 0)
|
||||||
@ -265,6 +275,7 @@ void cBlockPistonHandler::RetractPiston(int a_BlockX, int a_BlockY, int a_BlockZ
|
|||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// Get the block to pull
|
||||||
a_BlockX += 2 * pushDir.x;
|
a_BlockX += 2 * pushDir.x;
|
||||||
a_BlockY += 2 * pushDir.y;
|
a_BlockY += 2 * pushDir.y;
|
||||||
a_BlockZ += 2 * pushDir.z;
|
a_BlockZ += 2 * pushDir.z;
|
||||||
|
@ -151,6 +151,7 @@ private:
|
|||||||
std::unordered_set<Vector3i, VectorHasher<int>> & a_BlocksPushed, const Vector3i & a_PushDir
|
std::unordered_set<Vector3i, VectorHasher<int>> & a_BlocksPushed, const Vector3i & a_PushDir
|
||||||
);
|
);
|
||||||
|
|
||||||
|
/** 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 std::unordered_set<Vector3i, VectorHasher<int>> & a_BlocksToPush,
|
||||||
cWorld * a_World, const Vector3i & a_PushDir
|
cWorld * a_World, const Vector3i & a_PushDir
|
||||||
);
|
);
|
||||||
|
Loading…
Reference in New Issue
Block a user