summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSTRWarrior <niels.breuker@hotmail.nl>2014-01-27 21:34:22 +0100
committerSTRWarrior <niels.breuker@hotmail.nl>2014-01-27 21:34:22 +0100
commit33ad2761a0acb79ce938ebf518a731264617c03d (patch)
tree83ed49212f661dd8c887b07a0b5888f3e2b3826d
parentVillagers: Harvesting is more rare. (diff)
downloadcuberite-33ad2761a0acb79ce938ebf518a731264617c03d.tar
cuberite-33ad2761a0acb79ce938ebf518a731264617c03d.tar.gz
cuberite-33ad2761a0acb79ce938ebf518a731264617c03d.tar.bz2
cuberite-33ad2761a0acb79ce938ebf518a731264617c03d.tar.lz
cuberite-33ad2761a0acb79ce938ebf518a731264617c03d.tar.xz
cuberite-33ad2761a0acb79ce938ebf518a731264617c03d.tar.zst
cuberite-33ad2761a0acb79ce938ebf518a731264617c03d.zip
-rw-r--r--src/Mobs/Villager.cpp86
-rw-r--r--src/Mobs/Villager.h10
2 files changed, 65 insertions, 31 deletions
diff --git a/src/Mobs/Villager.cpp b/src/Mobs/Villager.cpp
index 46346fa9f..e39d8bc1f 100644
--- a/src/Mobs/Villager.cpp
+++ b/src/Mobs/Villager.cpp
@@ -13,7 +13,7 @@
cVillager::cVillager(eVillagerType VillagerType) :
super("Villager", mtVillager, "", "", 0.6, 1.8),
m_Type(VillagerType),
- m_DidFindCrops(false),
+ m_VillagerAction(false),
m_ActionCountDown(-1)
{
}
@@ -50,33 +50,33 @@ void cVillager::Tick(float a_Dt, cChunk & a_Chunk)
{
case vtFarmer:
{
- if (m_World->GetBlock(m_CropsPos.x, m_CropsPos.y - 1, m_CropsPos.z) == E_BLOCK_FARMLAND)
- {
- m_World->SetBlock(m_CropsPos.x, m_CropsPos.y, m_CropsPos.z, E_BLOCK_CROPS, 0);
- }
+ HandleFarmerNoCountDown();
}
}
}
return;
}
- if (m_DidFindCrops && !m_bMovingToDestination)
+ if (m_VillagerAction)
{
- if ((GetPosition() - m_CropsPos).Length() < 2)
+ switch (m_Type)
{
- BLOCKTYPE CropBlock = m_World->GetBlock(m_CropsPos.x, m_CropsPos.y, m_CropsPos.z);
- if (IsBlockFarmable(CropBlock) && m_World->GetBlockMeta(m_CropsPos.x, m_CropsPos.y, m_CropsPos.z) == 0x7)
+ case vtFarmer:
{
- cBlockHandler * Handler = cBlockHandler::GetBlockHandler(CropBlock);
- Handler->DropBlock(m_World, this, m_CropsPos.x, m_CropsPos.y, m_CropsPos.z);
- m_World->SetBlock(m_CropsPos.x, m_CropsPos.y, m_CropsPos.z, E_BLOCK_AIR, 0);
- m_ActionCountDown = 20;
+ HandleFarmerAction();
}
}
- m_DidFindCrops = false;
+ m_VillagerAction = false;
}
- if (m_World->GetTickRandomNumber(100) != 0)
+ // The villager already has an special action activated.
+ if (m_VillagerAction)
+ {
+ return;
+ }
+
+ // Don't always try to do a special action. Each tick has 1% to do a special action.
+ if (m_World->GetTickRandomNumber(99) != 0)
{
return;
}
@@ -85,10 +85,7 @@ void cVillager::Tick(float a_Dt, cChunk & a_Chunk)
{
case vtFarmer:
{
- if (!m_DidFindCrops)
- {
- HandleFarmer();
- }
+ HandleFarmerAttemptSpecialAction();
}
}
}
@@ -96,16 +93,19 @@ void cVillager::Tick(float a_Dt, cChunk & a_Chunk)
-void cVillager::HandleFarmer()
+void cVillager::HandleFarmerAttemptSpecialAction()
{
cBlockArea Surrounding;
- Surrounding.Read(m_World,
- (int) GetPosX() - 5,
- (int) GetPosX() + 5,
- (int) GetPosY() - 3,
- (int) GetPosY() + 3,
- (int) GetPosZ() - 5,
- (int) GetPosZ() + 5);
+ /// Read a 11x7x11 area.
+ Surrounding.Read(
+ m_World,
+ (int) GetPosX() - 5,
+ (int) GetPosX() + 5,
+ (int) GetPosY() - 3,
+ (int) GetPosY() + 3,
+ (int) GetPosZ() - 5,
+ (int) GetPosZ() + 5
+ );
for (int I = 0; I < 5; I++)
@@ -125,7 +125,7 @@ void cVillager::HandleFarmer()
continue;
}
- m_DidFindCrops = true;
+ m_VillagerAction = true;
m_CropsPos = Vector3i((int) GetPosX() + X - 5, (int) GetPosY() + Y - 3, (int) GetPosZ() + Z - 5);
MoveToPosition(Vector3f((float) (m_CropsPos.x + 0.5), (float) m_CropsPos.y, (float) (m_CropsPos.z + 0.5)));
return;
@@ -137,6 +137,36 @@ void cVillager::HandleFarmer()
+void cVillager::HandleFarmerAction()
+{
+ if (!m_bMovingToDestination && (GetPosition() - m_CropsPos).Length() < 2)
+ {
+ BLOCKTYPE CropBlock = m_World->GetBlock(m_CropsPos.x, m_CropsPos.y, m_CropsPos.z);
+ if (IsBlockFarmable(CropBlock) && m_World->GetBlockMeta(m_CropsPos.x, m_CropsPos.y, m_CropsPos.z) == 0x7)
+ {
+ cBlockHandler * Handler = cBlockHandler::GetBlockHandler(CropBlock);
+ Handler->DropBlock(m_World, this, m_CropsPos.x, m_CropsPos.y, m_CropsPos.z);
+ m_World->SetBlock(m_CropsPos.x, m_CropsPos.y, m_CropsPos.z, E_BLOCK_AIR, 0);
+ m_ActionCountDown = 20;
+ }
+ }
+}
+
+
+
+
+void cVillager::HandleFarmerNoCountDown()
+{
+ if (m_World->GetBlock(m_CropsPos.x, m_CropsPos.y - 1, m_CropsPos.z) == E_BLOCK_FARMLAND)
+ {
+ m_World->SetBlock(m_CropsPos.x, m_CropsPos.y, m_CropsPos.z, E_BLOCK_CROPS, 0);
+ }
+}
+
+
+
+
+
bool cVillager::IsBlockFarmable(BLOCKTYPE a_BlockType)
{
switch (a_BlockType)
diff --git a/src/Mobs/Villager.h b/src/Mobs/Villager.h
index 235e1f40e..20dbada61 100644
--- a/src/Mobs/Villager.h
+++ b/src/Mobs/Villager.h
@@ -34,19 +34,23 @@ public:
virtual void Tick (float a_Dt, cChunk & a_Chunk) override;
// cVillager functions
- void HandleFarmer();
bool IsBlockFarmable(BLOCKTYPE a_BlockType);
+ // Farmer functions
+ void HandleFarmerAttemptSpecialAction();
+ void HandleFarmerAction();
+ void HandleFarmerNoCountDown();
+
// Get and set functions.
int GetVilType(void) const { return m_Type; }
Vector3i GetCropsPos(void) const { return m_CropsPos; }
- bool DidFindCrops(void) const { return m_DidFindCrops; }
+ bool DoesHaveActionActivated(void) const { return m_VillagerAction; }
private:
int m_ActionCountDown;
int m_Type;
- bool m_DidFindCrops;
+ bool m_VillagerAction;
Vector3i m_CropsPos;
} ;